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

Good catch. I can swear I fixed this before...

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


Well, bit 18 is called "Video DIP frequency". And the frequency can be
"Once", "Every Vsync" or "At least every other VSync", so I guess
there's no need in putting the "vsync" to the name...

Thank you,
Paulo

-- 
Paulo Zanoni
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to