On Wed, 2017-12-20 at 09:24 +0000, Chris Wilson wrote:
> Quoting Dhinakaran Pandiyan (2017-12-20 04:35:20)
> > Commit 77affa31722b ("drm/i915/psr: Fix compiler warnings for
> > hsw_psr_disable()") swapped status and control registers while fixing
> > indentation. The _ctl at the end of the status register name must have to
> > led to this.
>
> My bad. https://www.mrc-cbu.cam.ac.uk/people/matt.davis/cmabridge/
Yeah, took me several rounds of checking to ensure I changed it
correctly.
> So what does STATUS_CTL control? That seems to be a bad name, does bspec
> really call it that?
So I looked it up, it doesn't. I'll send a patch soon.
-DK
>
> References: https://www.mrc-cbu.cam.ac.uk/people/matt.davis/cmabridge/
> > Cc: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
> > Cc: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx>
> > Signed-off-by: Dhinakaran Pandiyan <dhinakaran.pandiyan@xxxxxxxxx>
> Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
> -Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
