[alsa-devel] [PATCH 4/4] ASoC: tda998x: adjust the audio hw parameters from EDID

2014-01-28 Thread Takashi Iwai
At Tue, 28 Jan 2014 11:00:51 +, Mark Brown wrote: > > On Tue, Jan 28, 2014 at 10:23:57AM +0100, Takashi Iwai wrote: > > Mark Brown wrote: > > > On Mon, Jan 27, 2014 at 08:49:15PM +, Russell King - ARM Linux wrote: > > > > > Yes, preferably as a generic ALSA helper rather than an ASoC help

[alsa-devel] [PATCH 4/4] ASoC: tda998x: adjust the audio hw parameters from EDID

2014-01-28 Thread Mark Brown
On Tue, Jan 28, 2014 at 10:23:57AM +0100, Takashi Iwai wrote: > Mark Brown wrote: > > On Mon, Jan 27, 2014 at 08:49:15PM +, Russell King - ARM Linux wrote: > > > Yes, preferably as a generic ALSA helper rather than an ASoC helper - > > > I don't see any need for this to be ASoC specific (I hav

[alsa-devel] [PATCH 4/4] ASoC: tda998x: adjust the audio hw parameters from EDID

2014-01-28 Thread Takashi Iwai
At Mon, 27 Jan 2014 20:54:37 +, Mark Brown wrote: > > On Mon, Jan 27, 2014 at 08:49:15PM +, Russell King - ARM Linux wrote: > > On Mon, Jan 27, 2014 at 08:44:41PM +, Mark Brown wrote: > > > > Can this parsing code be factored out - it (or large parts of it) should > > > be usable by o

[PATCH 4/4] ASoC: tda998x: adjust the audio hw parameters from EDID

2014-01-27 Thread Mark Brown
On Mon, Jan 27, 2014 at 08:49:15PM +, Russell King - ARM Linux wrote: > On Mon, Jan 27, 2014 at 08:44:41PM +, Mark Brown wrote: > > Can this parsing code be factored out - it (or large parts of it) should > > be usable by other HDMI devices shouldn't it? > Yes, preferably as a generic ALS

[PATCH 4/4] ASoC: tda998x: adjust the audio hw parameters from EDID

2014-01-27 Thread Russell King - ARM Linux
On Mon, Jan 27, 2014 at 08:44:41PM +, Mark Brown wrote: > On Mon, Jan 27, 2014 at 09:48:54AM +0100, Jean-Francois Moine wrote: > > > + eld_ver = eld[0] >> 3; > > + if (eld_ver != 2 && eld_ver != 31) > > + return 0; > > + > > + mnl = eld[4] & 0x1f

[PATCH 4/4] ASoC: tda998x: adjust the audio hw parameters from EDID

2014-01-27 Thread Mark Brown
On Mon, Jan 27, 2014 at 09:48:54AM +0100, Jean-Francois Moine wrote: > + eld_ver = eld[0] >> 3; > + if (eld_ver != 2 && eld_ver != 31) > + return 0; > + > + mnl = eld[4] & 0x1f; > + if (mnl > 16) > + return 0;

[PATCH 4/4] ASoC: tda998x: adjust the audio hw parameters from EDID

2014-01-27 Thread Jean-Francois Moine
The supported audio parameters are described in the EDID which is received by the HDMI transmitter from the connected screen. Use these ones to adjust the audio stream parameters. Signed-off-by: Jean-Francois Moine --- drivers/gpu/drm/i2c/tda998x_drv.c | 15 include/drm/i2c/tda998x.h