Commit 43e6fad1 authored by Ville Syrjälä's avatar Ville Syrjälä

drm/i915: Introduce HAS_SAGV()

Introuce a HAS_SAGV() macro to answer the question whether
the platform in general supports SAGV. intel_has_sagv() will
keep on giving us the more specific answer whether the current
device supports SAGV or not.
Signed-off-by: default avatarVille Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20230131002127.29305-2-ville.syrjala@linux.intel.comReviewed-by: default avatarJani Nikula <jani.nikula@intel.com>
parent c580c2d2
...@@ -64,7 +64,7 @@ static bool skl_needs_memory_bw_wa(struct drm_i915_private *i915) ...@@ -64,7 +64,7 @@ static bool skl_needs_memory_bw_wa(struct drm_i915_private *i915)
static bool static bool
intel_has_sagv(struct drm_i915_private *i915) intel_has_sagv(struct drm_i915_private *i915)
{ {
return DISPLAY_VER(i915) >= 9 && !IS_LP(i915) && return HAS_SAGV(i915) &&
i915->display.sagv.status != I915_SAGV_NOT_CONTROLLED; i915->display.sagv.status != I915_SAGV_NOT_CONTROLLED;
} }
...@@ -92,7 +92,7 @@ intel_sagv_block_time(struct drm_i915_private *i915) ...@@ -92,7 +92,7 @@ intel_sagv_block_time(struct drm_i915_private *i915)
return val; return val;
} else if (DISPLAY_VER(i915) == 11) { } else if (DISPLAY_VER(i915) == 11) {
return 10; return 10;
} else if (DISPLAY_VER(i915) == 9 && !IS_LP(i915)) { } else if (HAS_SAGV(i915)) {
return 30; return 30;
} else { } else {
return 0; return 0;
...@@ -101,7 +101,7 @@ intel_sagv_block_time(struct drm_i915_private *i915) ...@@ -101,7 +101,7 @@ intel_sagv_block_time(struct drm_i915_private *i915)
static void intel_sagv_init(struct drm_i915_private *i915) static void intel_sagv_init(struct drm_i915_private *i915)
{ {
if (!intel_has_sagv(i915)) if (!HAS_SAGV(i915))
i915->display.sagv.status = I915_SAGV_NOT_CONTROLLED; i915->display.sagv.status = I915_SAGV_NOT_CONTROLLED;
/* /*
......
...@@ -860,6 +860,7 @@ IS_SUBPLATFORM(const struct drm_i915_private *i915, ...@@ -860,6 +860,7 @@ IS_SUBPLATFORM(const struct drm_i915_private *i915,
#define HAS_64K_PAGES(dev_priv) (INTEL_INFO(dev_priv)->has_64k_pages) #define HAS_64K_PAGES(dev_priv) (INTEL_INFO(dev_priv)->has_64k_pages)
#define HAS_IPC(dev_priv) (INTEL_INFO(dev_priv)->display.has_ipc) #define HAS_IPC(dev_priv) (INTEL_INFO(dev_priv)->display.has_ipc)
#define HAS_SAGV(dev_priv) (DISPLAY_VER(dev_priv) >= 9 && !IS_LP(dev_priv))
#define HAS_REGION(i915, i) (RUNTIME_INFO(i915)->memory_regions & (i)) #define HAS_REGION(i915, i) (RUNTIME_INFO(i915)->memory_regions & (i))
#define HAS_LMEM(i915) HAS_REGION(i915, REGION_LMEM) #define HAS_LMEM(i915) HAS_REGION(i915, REGION_LMEM)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment