diff options
author | Paulo Zanoni <paulo.r.zanoni@intel.com> | 2016-09-22 17:00:29 -0400 |
---|---|---|
committer | Jani Nikula <jani.nikula@intel.com> | 2016-10-10 09:06:37 -0400 |
commit | 17777d61f4a87d7b6d5585e8fdffa83773c594e7 (patch) | |
tree | 31118ba9f122692c02caf1b42c526a66c476cfae | |
parent | 6e7fdb873d6255ca3c999dd5c6c18962a769ed3e (diff) |
drm/i915/kbl: KBL also needs to run the SAGV code
According to BSpec, it's the "core CPUs" that need the code, which
means SKL and KBL, but not BXT.
I don't have a KBL to test this patch on it.
v2: Only SKL should have I915_SAGV_NOT_CONTROLLED.
Cc: stable@vger.kernel.org
Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1474578035-424-4-git-send-email-paulo.r.zanoni@intel.com
(cherry picked from commit 6e3100ec21e7c774a0fc01e36a1e0739530c2f71)
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
-rw-r--r-- | drivers/gpu/drm/i915/intel_pm.c | 14 |
1 files changed, 10 insertions, 4 deletions
diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c index 1058729843fb..93ae27408e49 100644 --- a/drivers/gpu/drm/i915/intel_pm.c +++ b/drivers/gpu/drm/i915/intel_pm.c | |||
@@ -2880,8 +2880,14 @@ skl_wm_plane_id(const struct intel_plane *plane) | |||
2880 | static bool | 2880 | static bool |
2881 | intel_has_sagv(struct drm_i915_private *dev_priv) | 2881 | intel_has_sagv(struct drm_i915_private *dev_priv) |
2882 | { | 2882 | { |
2883 | return IS_SKYLAKE(dev_priv) && | 2883 | if (IS_KABYLAKE(dev_priv)) |
2884 | dev_priv->sagv_status != I915_SAGV_NOT_CONTROLLED; | 2884 | return true; |
2885 | |||
2886 | if (IS_SKYLAKE(dev_priv) && | ||
2887 | dev_priv->sagv_status != I915_SAGV_NOT_CONTROLLED) | ||
2888 | return true; | ||
2889 | |||
2890 | return false; | ||
2885 | } | 2891 | } |
2886 | 2892 | ||
2887 | /* | 2893 | /* |
@@ -2919,7 +2925,7 @@ intel_enable_sagv(struct drm_i915_private *dev_priv) | |||
2919 | * Some skl systems, pre-release machines in particular, | 2925 | * Some skl systems, pre-release machines in particular, |
2920 | * don't actually have an SAGV. | 2926 | * don't actually have an SAGV. |
2921 | */ | 2927 | */ |
2922 | if (ret == -ENXIO) { | 2928 | if (IS_SKYLAKE(dev_priv) && ret == -ENXIO) { |
2923 | DRM_DEBUG_DRIVER("No SAGV found on system, ignoring\n"); | 2929 | DRM_DEBUG_DRIVER("No SAGV found on system, ignoring\n"); |
2924 | dev_priv->sagv_status = I915_SAGV_NOT_CONTROLLED; | 2930 | dev_priv->sagv_status = I915_SAGV_NOT_CONTROLLED; |
2925 | return 0; | 2931 | return 0; |
@@ -2973,7 +2979,7 @@ intel_disable_sagv(struct drm_i915_private *dev_priv) | |||
2973 | * Some skl systems, pre-release machines in particular, | 2979 | * Some skl systems, pre-release machines in particular, |
2974 | * don't actually have an SAGV. | 2980 | * don't actually have an SAGV. |
2975 | */ | 2981 | */ |
2976 | if (result == -ENXIO) { | 2982 | if (IS_SKYLAKE(dev_priv) && result == -ENXIO) { |
2977 | DRM_DEBUG_DRIVER("No SAGV found on system, ignoring\n"); | 2983 | DRM_DEBUG_DRIVER("No SAGV found on system, ignoring\n"); |
2978 | dev_priv->sagv_status = I915_SAGV_NOT_CONTROLLED; | 2984 | dev_priv->sagv_status = I915_SAGV_NOT_CONTROLLED; |
2979 | return 0; | 2985 | return 0; |