On Wed, Nov 21, 2012 at 5:08 PM, Daniel Vetter daniel@ffwll.ch wrote:
On Wed, Nov 21, 2012 at 4:47 PM, Thierry Reding thierry.reding@avionic-design.de wrote:
Oh great, so I copied that table for nothing. Thanks for Cc'ing, I can reuse that in the HDMI infoframe series.
Wrt the infoframe series, I think it'd be awesome if you could convert i915 and radeon (iirc the existing drivers with the "best" avi infoframe support) over to the new code. This gives some nice validation, both by testing on actual hw and that the interface is sane, since it'll be used by 2-3 different drivers then.
I think the best way is to pick the infoframe implementation you best like from one of these, move it into the helper, then improve it until you're happy. And then convert over 1-2 other drivers. At least that's been my approach for the recent dp helper refactoring.
And if you could use that opportunity to integrate the kerneldoc for edid/eld and the new infoframe code into the drm docbook, that would be rather awesome ;-) -Daniel