[Intel-gfx] [PATCH 08/12] drm/i915: break intel_infoframe_flags into _enable and _index

[Eugeni Dodonov]

On Wed, May 2, 2012 at 10:55 PM, Paulo Zanoni <przanoni at gmail.com> wrote:

> From: Paulo Zanoni <paulo.r.zanoni at intel.com>
>
> This will allow us to disable an infoframe without changing its
> frequency.
>
> Signed-off-by: Paulo Zanoni <paulo.r.zanoni at intel.com>
>

I think the subject should be 'Break intel_infoframe_flags into _enable and
_frequency', no?

And by looking at what this does, perhaps the 2nd function could be named
intel_infoframe_freq_vsync or intel_infoframe_vsync instead?

But besides that:
Reviewed-by: Eugeni Dodonov <eugeni.dodonov at intel.com>

-- 
Eugeni Dodonov
 <http://eugeni.dodonov.net/>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/intel-gfx/attachments/20120503/7954e147/attachment.html>