On 05/31/2018 07:56 PM, Jani Nikula wrote:
Virtualized non-PCH systems such as Broxton or Geminilake should use
PCH_NONE to indicate no PCH rather than PCH_NOP. The latter is a
specific case to indicate a PCH system without south display.

Reported-by: Colin Xu <[email protected]>
Cc: Colin Xu <[email protected]>
Reviewed-by: Ville Syrjälä <[email protected]>
Signed-off-by: Jani Nikula <[email protected]>
---
  drivers/gpu/drm/i915/i915_drv.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c
index 8f002ae22e62..c42e389a27f3 100644
--- a/drivers/gpu/drm/i915/i915_drv.c
+++ b/drivers/gpu/drm/i915/i915_drv.c
@@ -287,7 +287,7 @@ static void intel_detect_pch(struct drm_i915_private 
*dev_priv)
                                if (WARN_ON(pch_type == PCH_NONE))
                                        pch_type = PCH_NOP;
                        } else {
-                               pch_type = PCH_NOP;
+                               pch_type = PCH_NONE;
                        }
                        dev_priv->pch_type = pch_type;
                        dev_priv->pch_id = id;

Tested on BXT gvt-g and got expected behaviour.

Tested-by: Colin Xu <[email protected]>
Reviewed-by: Colin Xu <[email protected]>

--
Best Regards,
Colin Xu

_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to