On Fri, Nov 09, 2012 at 05:04:09PM +0100, Rafał Miłecki wrote: > 2012/11/9 Christian König : > > On 09.11.2012 16:45, Rafał Miłecki wrote: > >> I was told it won't work on different endian devices. See > >> [RFC][PATCH] drm/radeon/hdmi: define struct for AVI infoframe > >> http://lists.freedesktop.org/archives/dri-devel/2012-May/022544.html > > > > > > Yeah, that's indeed true. And honestly adding just another implementation of > > the HDMI info frames sounds like somebody should finally sit down and > > implement it in a common drm_hdmi.c > > Agree. Any volunteers? If nobody else steps up I'm willing to put some work into this. I'm a bit reluctant to do this right away since time is already very short before the 3.8 merge window. If I promise to look at this once the Tegra DRM driver is in, would people be willing to let it slip for now? I know this is not much of an argument and I'm all in favour of a generic implementation, but I don't think endianess will be an issue in the near future on Tegra. I don't believe there are any plans to make it support big endian. Thierry