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

2012-11-14 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-14 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-14 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-14 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-14 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

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 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