On Sat, Nov 16, 2013 at 04:00:09PM +0100, Daniel Vetter wrote: > Some BIOS just leak the forcewak bits, which we clean up. > Unfortunately this has been broken in > > commit 521198a2e7095c8c7daa8d7d3a76a110c346be6f > Author: Mika Kuoppala <mika.kuoppala@xxxxxxxxxxxxxxx> > Date: Fri Aug 23 16:52:30 2013 +0300 > > drm/i915: sanitize forcewake registers on reset > > To make this work both for resets and for BIOS takeover just add the > forcewake clearing call back to intel_uncore_early_sanitize. The call to intel_uncore_forcewake_reset() is in intel_uncore_sanitize(). Do you mind explaining the dependence that causes us to require an ealry_sanitize? There's no issue in doing so, just the change log feels inaccurate. > Cc: Mika Kuoppala <mika.kuoppala@xxxxxxxxxxxxxxx> > Cc: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > Reported-by: Jörg Otte <jrg.otte@xxxxxxxxx> > Cc: Jörg Otte <jrg.otte@xxxxxxxxx> > References: https://lkml.org/lkml/2013/11/16/40 > Cc: stable@xxxxxxxxxxxxxxx (for 3.12 only) > Signed-off-by: Daniel Vetter <daniel.vetter@xxxxxxxx> Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> -Chris -- Chris Wilson, Intel Open Source Technology Centre -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html