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

2014-02-05 Thread Lars-Peter Clausen
On 02/05/2014 07:07 PM, Jean-Francois Moine wrote: On Wed, 05 Feb 2014 10:19:22 +0100 Lars-Peter Clausen wrote: So, in the CODEC, I don't see how I could update the parameters dictated by the EDID otherwise in changing the DAI driver parameters. The startup function is the right place. But

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

2014-02-05 Thread Jean-Francois Moine
On Wed, 05 Feb 2014 10:19:22 +0100 Lars-Peter Clausen wrote: > > So, in the CODEC, I don't see how I could update the parameters > > dictated by the EDID otherwise in changing the DAI driver parameters. > > > > The startup function is the right place. But instead of modifying the DAI > use sn

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

2014-02-05 Thread Mark Brown
On Wed, Feb 05, 2014 at 02:31:09PM +0100, Lars-Peter Clausen wrote: > On 02/05/2014 12:18 PM, Mark Brown wrote: > >...as previously requested. I know there was some discusion of broader > >moves to factor this stuff out but it'd still be good to keep it > >separated out even prior to a final non-

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

2014-02-05 Thread Lars-Peter Clausen
On 02/05/2014 12:18 PM, Mark Brown wrote: On Wed, Feb 05, 2014 at 10:19:22AM +0100, Lars-Peter Clausen wrote: [..] Bonus points for making this a generic helper function that takes a runtime and a EDID and then applies the EDID constraints on the runtime. ...as previously requested. I know t

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

2014-02-05 Thread Mark Brown
On Wed, Feb 05, 2014 at 10:19:22AM +0100, Lars-Peter Clausen wrote: > On 02/05/2014 10:11 AM, Jean-Francois Moine wrote: > >So, in the CODEC, I don't see how I could update the parameters > >dictated by the EDID otherwise in changing the DAI driver parameters. > The startup function is the right

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

2014-02-05 Thread Lars-Peter Clausen
On 02/05/2014 10:11 AM, Jean-Francois Moine wrote: On Tue, 4 Feb 2014 18:06:25 + Mark Brown wrote: On Mon, Jan 27, 2014 at 09:48:54AM +0100, Jean-Francois Moine wrote: + /* change the snd_soc_pcm_stream values of the driver */ + stream->rates = rate_mask; +

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

2014-02-05 Thread Jean-Francois Moine
On Tue, 4 Feb 2014 18:06:25 + Mark Brown wrote: > On Mon, Jan 27, 2014 at 09:48:54AM +0100, Jean-Francois Moine wrote: > > > + /* change the snd_soc_pcm_stream values of the driver */ > > + stream->rates = rate_mask; > > + stream->channels_max = max_channels; >

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

2014-02-04 Thread Mark Brown
On Mon, Jan 27, 2014 at 09:48:54AM +0100, Jean-Francois Moine wrote: > + /* change the snd_soc_pcm_stream values of the driver */ > + stream->rates = rate_mask; > + stream->channels_max = max_channels; > + stream->formats = formats; > + /* copy

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

2014-02-01 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