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-0001.htm>
