Re: [PATCH 4/9] drm/i915/bdw: Use centralized rc6 info print

Chris Wilson
2014-02-14T20:41:19Z

On Fri, Feb 14, 2014 at 12:34:22PM -0800, Ben Widawsky wrote:
> On Tue, Feb 11, 2014 at 05:12:17PM +0100, Daniel Vetter wrote:
> > - 6/9 lacks review ...
> >
>
> 6/9 can be skipped for this series, though I think it's a reasonable
> patch. Maybe Chris or Jesse has an opinion on 6/9?

Yes, programming random values into the hardware is just bonghits. And
then people come along complaining how the idiotic programming guide is
sacrosanct and must be adherred to...

Otoh, changing the value of GEN6_RC_VIDEO_FREQ is worrying as we never
reprogram it nor have I come across any instructions as to what it does.
-Chris

--
Chris Wilson, Intel Open Source Technology Centre