Re: [PATCH 2/2] drm: tegra: Add HDMI support

[Thierry Reding <thierry.reding-RM9K5IK7kjKj5M59NBduVrNAH6kLmebB@public.gmane.org>]

[-- Attachment #1: Type: text/plain, Size: 1178 bytes --]

On Fri, Nov 09, 2012 at 05:04:09PM +0100, Rafał Miłecki wrote:
> 2012/11/9 Christian König <deathsimple-ANTagKRnAhcb1SvskN2V4Q@public.gmane.org>:
> > 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

[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]