Messages in this thread | | | Date | Fri, 9 Nov 2012 21:20:23 +0100 | From | Thierry Reding <> | Subject | Re: [PATCH 2/2] drm: tegra: Add HDMI support |
| |
On Fri, Nov 09, 2012 at 05:04:09PM +0100, Rafał Miłecki wrote: > 2012/11/9 Christian König <deathsimple@vodafone.de>: > > 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 [unhandled content-type:application/pgp-signature] | |