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

2012-11-12 Thread Daniel Vetter
On Mon, Nov 12, 2012 at 8:24 AM, Thierry Reding wrote: > Actually what I had in mind was a packed binary representation of > infoframes as specified by HDMI 1.3a (I don't have access to 1.4, but I > would think it doesn't differ in this respect) in section 5.3 and 5.3.5 > more specifically.

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

2012-11-12 Thread Daniel Vetter
On Mon, Nov 12, 2012 at 8:24 AM, Thierry Reding thierry.red...@avionic-design.de wrote: Actually what I had in mind was a packed binary representation of infoframes as specified by HDMI 1.3a (I don't have access to 1.4, but I would think it doesn't differ in this respect) in section 5.3 and

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

2012-11-11 Thread Thierry Reding
On Sun, Nov 11, 2012 at 03:46:44PM +0100, Daniel Vetter wrote: > On Sat, Nov 10, 2012 at 10:01:18PM +0100, Thierry Reding wrote: > > On Fri, Nov 09, 2012 at 05:00:54PM +0100, Christian König wrote: > > > On 09.11.2012 16:45, Rafał Miłecki wrote: > > > >2012/11/9 Thierry Reding : > > > >>+/* all

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

2012-11-11 Thread Daniel Vetter
On Sat, Nov 10, 2012 at 10:01:18PM +0100, Thierry Reding wrote: > On Fri, Nov 09, 2012 at 05:00:54PM +0100, Christian König wrote: > > On 09.11.2012 16:45, Rafał Miłecki wrote: > > >2012/11/9 Thierry Reding : > > >>+/* all fields little endian */ > > >>+struct hdmi_audio_infoframe { > > >>+

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

2012-11-11 Thread Daniel Vetter
On Sat, Nov 10, 2012 at 10:01:18PM +0100, Thierry Reding wrote: On Fri, Nov 09, 2012 at 05:00:54PM +0100, Christian König wrote: On 09.11.2012 16:45, Rafał Miłecki wrote: 2012/11/9 Thierry Reding thierry.red...@avionic-design.de: +/* all fields little endian */ +struct

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

2012-11-11 Thread Thierry Reding
On Sun, Nov 11, 2012 at 03:46:44PM +0100, Daniel Vetter wrote: On Sat, Nov 10, 2012 at 10:01:18PM +0100, Thierry Reding wrote: On Fri, Nov 09, 2012 at 05:00:54PM +0100, Christian König wrote: On 09.11.2012 16:45, Rafał Miłecki wrote: 2012/11/9 Thierry Reding

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

2012-11-10 Thread Thierry Reding
On Sat, Nov 10, 2012 at 10:01:18PM +0100, Thierry Reding wrote: > On Fri, Nov 09, 2012 at 05:00:54PM +0100, Christian König wrote: > > On 09.11.2012 16:45, Rafał Miłecki wrote: > > >2012/11/9 Thierry Reding : > > >>+/* all fields little endian */ > > >>+struct hdmi_audio_infoframe { > > >>+

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

2012-11-10 Thread Thierry Reding
On Fri, Nov 09, 2012 at 05:00:54PM +0100, Christian König wrote: > On 09.11.2012 16:45, Rafał Miłecki wrote: > >2012/11/9 Thierry Reding : > >>+/* all fields little endian */ > >>+struct hdmi_audio_infoframe { > >>+ /* PB0 */ > >>+ u8 csum; > >>+ > >>+ /* PB1 */ > >>+

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

2012-11-10 Thread Thierry Reding
On Fri, Nov 09, 2012 at 05:00:54PM +0100, Christian König wrote: On 09.11.2012 16:45, Rafał Miłecki wrote: 2012/11/9 Thierry Reding thierry.red...@avionic-design.de: +/* all fields little endian */ +struct hdmi_audio_infoframe { + /* PB0 */ + u8 csum; + + /* PB1 */ +

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

2012-11-10 Thread Thierry Reding
On Sat, Nov 10, 2012 at 10:01:18PM +0100, Thierry Reding wrote: On Fri, Nov 09, 2012 at 05:00:54PM +0100, Christian König wrote: On 09.11.2012 16:45, Rafał Miłecki wrote: 2012/11/9 Thierry Reding thierry.red...@avionic-design.de: +/* all fields little endian */ +struct

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

2012-11-09 Thread Thierry Reding
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 > >>

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

2012-11-09 Thread Rafał Miłecki
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

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

2012-11-09 Thread Christian König
On 09.11.2012 16:45, Rafał Miłecki wrote: 2012/11/9 Thierry Reding : +/* all fields little endian */ +struct hdmi_audio_infoframe { + /* PB0 */ + u8 csum; + + /* PB1 */ + unsigned cc:3; /* channel count */ + unsigned res1:1; + unsigned ct:4; /* coding type */

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

2012-11-09 Thread Rafał Miłecki
2012/11/9 Thierry Reding : > +/* all fields little endian */ > +struct hdmi_audio_infoframe { > + /* PB0 */ > + u8 csum; > + > + /* PB1 */ > + unsigned cc:3; /* channel count */ > + unsigned res1:1; > + unsigned ct:4; /* coding type */ > + > + /* PB2 */ >

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

2012-11-09 Thread Rafał Miłecki
2012/11/9 Thierry Reding thierry.red...@avionic-design.de: +/* all fields little endian */ +struct hdmi_audio_infoframe { + /* PB0 */ + u8 csum; + + /* PB1 */ + unsigned cc:3; /* channel count */ + unsigned res1:1; + unsigned ct:4; /* coding type */ +

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

2012-11-09 Thread Christian König
On 09.11.2012 16:45, Rafał Miłecki wrote: 2012/11/9 Thierry Reding thierry.red...@avionic-design.de: +/* all fields little endian */ +struct hdmi_audio_infoframe { + /* PB0 */ + u8 csum; + + /* PB1 */ + unsigned cc:3; /* channel count */ + unsigned res1:1; +

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

2012-11-09 Thread Rafał Miłecki
2012/11/9 Christian König deathsim...@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,

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

2012-11-09 Thread Thierry Reding
On Fri, Nov 09, 2012 at 05:04:09PM +0100, Rafał Miłecki wrote: 2012/11/9 Christian König deathsim...@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