Re: [PATCH] drm/i915: Sanitize shared dpll state

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Jul 17, 2013 at 06:55:04AM +0200, Daniel Vetter wrote:
> There seems to be no limit to the amount of gunk the firmware can
> leave behind. Some platforms leave pch dplls on which are not in
> active use at all. The example in the bug report is a Apple Macbook
> Pro.
> 
> Note that this escape scrunity of the hw state checker until we've
> tried to use this enabled, but unused pll since we did only check for
> the inverse case of a in-used, but disabled pll.
> 
> v2: Add a WARN in the pll state checker which would have caught this
> case.
> 
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=66952
> Reported-and-tested-by: shui yangwei <yangweix.shui@xxxxxxxxx>
> Signed-off-by: Daniel Vetter <daniel.vetter@xxxxxxxx>

Plonk it in intel_sanitize_plls(), and preferably move all the sanitze
encoder/crtc/pll into intel_sanitize_display() (in a later patch), so that
intel_modeset_setup_hw_state() is not quite so broken up.

Other than that minor request,
Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux