On Wed, May 2, 2012 at 10:55 PM, Paulo Zanoni <[email protected]> wrote:
> From: Paulo Zanoni <[email protected]> > > This will allow us to disable an infoframe without changing its > frequency. > > Signed-off-by: Paulo Zanoni <[email protected]> > 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 <[email protected]> -- Eugeni Dodonov <http://eugeni.dodonov.net/>
_______________________________________________ Intel-gfx mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
