On Fri, 2013-12-20 at 15:09 -0200, Paulo Zanoni wrote: > From: Paulo Zanoni <paulo.r.zanoni@xxxxxxxxx> > > We're iterating over the CPU transcoders, so check for the correct > power domain. > > This fixes many "unclaimed register" error messages. > > This can be reproduced by the IGT test mentioned below, but we still > get a FAIL when we run it. > > Testcase: igt/kms_lip/flip-vs-panning-vs-hang > Signed-off-by: Paulo Zanoni <paulo.r.zanoni@xxxxxxxxx> > --- > drivers/gpu/drm/i915/intel_display.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > > I didn't bisect to check, but it really really looks like a regression from > "drm/i915: add intel_display_power_enabled_sw() for use in atomic ctx". Yep, that was an overlook on my part:/ The fix looks ok: Reviewed-by: Imre Deak <imre.deak@xxxxxxxxx> > > > > diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c > index 4d1357a..4d4b4bc 100644 > --- a/drivers/gpu/drm/i915/intel_display.c > +++ b/drivers/gpu/drm/i915/intel_display.c > @@ -11459,7 +11459,8 @@ intel_display_capture_error_state(struct drm_device *dev) > enum transcoder cpu_transcoder = transcoders[i]; > > error->transcoder[i].power_domain_on = > - intel_display_power_enabled_sw(dev, POWER_DOMAIN_PIPE(i)); > + intel_display_power_enabled_sw(dev, > + POWER_DOMAIN_TRANSCODER(cpu_transcoder)); > if (!error->transcoder[i].power_domain_on) > continue; >
Attachment:
signature.asc
Description: This is a digitally signed message part
_______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
