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

[Thierry Reding]

On Fri, Nov 09, 2012 at 05:04:09PM +0100, RafaÅ MiÅecki wrote:
> 2012/11/9 Christian KÃnig <deathsimple@xxxxxxxxxxx>:
> > 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: pgp00000.pgp
Description: PGP signature