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