[alsa-devel] [PATCH RFC v2 10/13] sound/core: add DRM ELD helper

2015-05-08 Thread Jyri Sarha
On 05/08/15 16:27, Russell King - ARM Linux wrote: > On Fri, May 08, 2015 at 04:16:08PM +0300, Jyri Sarha wrote: >> On 2015-04-02 12:22, Russell King wrote: ... >> For what I can see, you are setting the cross dependency between >> the sample rate and channels incorrectly. > > There is a

[alsa-devel] [PATCH RFC v2 10/13] sound/core: add DRM ELD helper

2015-05-08 Thread Jyri Sarha
On 2015-04-02 12:22, Russell King wrote: > Add a helper for the EDID like data structure, which is typically > passed > from a HDMI adapter to its associated audio driver. This informs the > audio driver of the capabilities of the attached HDMI sink. > > Signed-off-by: Russell King

[alsa-devel] [PATCH RFC v2 10/13] sound/core: add DRM ELD helper

2015-05-08 Thread Russell King - ARM Linux
On Fri, May 08, 2015 at 04:16:08PM +0300, Jyri Sarha wrote: > On 2015-04-02 12:22, Russell King wrote: > >+static const unsigned int eld_rates[] = { > >+32000, > >+44100, > >+48000, > >+88200, > >+96000, > >+176400, > >+192000, > >+}; > >+ > > For what I can see, you

[alsa-devel] [PATCH RFC v2 10/13] sound/core: add DRM ELD helper

2015-05-08 Thread Jyri Sarha
On 2015-04-05 20:26, Russell King - ARM Linux wrote: > On Sun, Apr 05, 2015 at 06:46:13PM +0200, Takashi Iwai wrote: >> At Sun, 5 Apr 2015 17:20:34 +0100, >> Russell King - ARM Linux wrote: >> > Since (afaik) ALSA has a lack of support for dynamic reconfiguration >> > according to the attached

[alsa-devel] [PATCH RFC v2 10/13] sound/core: add DRM ELD helper

2015-05-08 Thread Russell King - ARM Linux
On Fri, May 08, 2015 at 01:56:02PM +0300, Jyri Sarha wrote: > On 2015-04-05 20:26, Russell King - ARM Linux wrote: > >On Sun, Apr 05, 2015 at 06:46:13PM +0200, Takashi Iwai wrote: > >>At Sun, 5 Apr 2015 17:20:34 +0100, > >>Russell King - ARM Linux wrote: > >>> Since (afaik) ALSA has a lack of

[alsa-devel] [PATCH RFC v2 10/13] sound/core: add DRM ELD helper

2015-05-07 Thread Lars-Peter Clausen
On 05/07/2015 12:41 PM, Russell King - ARM Linux wrote: [...] >>> What I'm concerned about is that when the ALSA parameter refining >>> starts, we start with (eg) 2-8 channels, 32-192kHz. Given that, >>> if we invoke the channel restriction before the rate restriction, >>> we would end up

[PATCH RFC v2 10/13] sound/core: add DRM ELD helper

2015-05-07 Thread Russell King - ARM Linux
On Wed, May 06, 2015 at 08:02:52PM +0300, Anssi Hannula wrote: > 05.04.2015, 20:26, Russell King - ARM Linux kirjoitti: > > On Sun, Apr 05, 2015 at 06:46:13PM +0200, Takashi Iwai wrote: > >> > >> One another question: don't we need to deal with the sample bits in > >> sad[2]? > > > > It should,

[PATCH RFC v2 10/13] sound/core: add DRM ELD helper

2015-05-06 Thread Anssi Hannula
05.04.2015, 20:26, Russell King - ARM Linux kirjoitti: > On Sun, Apr 05, 2015 at 06:46:13PM +0200, Takashi Iwai wrote: >> >> One another question: don't we need to deal with the sample bits in >> sad[2]? > > It should, but I'm very wary about doing that without seeing more > examples of real

[alsa-devel] [PATCH RFC v2 10/13] sound/core: add DRM ELD helper

2015-05-06 Thread Liam Girdwood
On Tue, 2015-05-05 at 23:35 +0100, Mark Brown wrote: > On Sun, Apr 05, 2015 at 05:57:09PM +0200, Takashi Iwai wrote: > > At Thu, 02 Apr 2015 10:22:06 +0100, > > Russell King wrote: > > > > +config SND_PCM_ELD > > > + bool > > > I don't mind much adding this one, but a new Kconfig is always a > >

[PATCH RFC v2 10/13] sound/core: add DRM ELD helper

2015-05-06 Thread Mark Brown
On Sun, Apr 05, 2015 at 05:57:09PM +0200, Takashi Iwai wrote: > At Thu, 02 Apr 2015 10:22:06 +0100, > Russell King wrote: > > +config SND_PCM_ELD > > + bool > I don't mind much adding this one, but a new Kconfig is always a > question. I'd like to hear other's opinion, too. One point in

[PATCH RFC v2 10/13] sound/core: add DRM ELD helper

2015-04-05 Thread Takashi Iwai
At Sun, 5 Apr 2015 17:20:34 +0100, Russell King - ARM Linux wrote: > > On Sun, Apr 05, 2015 at 05:57:09PM +0200, Takashi Iwai wrote: > > > diff --git a/include/sound/pcm_drm_eld.h b/include/sound/pcm_drm_eld.h > > > new file mode 100644 > > > index ..93357b25d2e2 > > > --- /dev/null >

[PATCH RFC v2 10/13] sound/core: add DRM ELD helper

2015-04-05 Thread Russell King - ARM Linux
On Sun, Apr 05, 2015 at 06:46:13PM +0200, Takashi Iwai wrote: > At Sun, 5 Apr 2015 17:20:34 +0100, > Russell King - ARM Linux wrote: > > Since (afaik) ALSA has a lack of support for dynamic reconfiguration > > according to the attached device changing, the best we can do without > > a huge amount

[PATCH RFC v2 10/13] sound/core: add DRM ELD helper

2015-04-05 Thread Takashi Iwai
At Thu, 02 Apr 2015 10:22:06 +0100, Russell King wrote: > > Add a helper for the EDID like data structure, which is typically passed > from a HDMI adapter to its associated audio driver. This informs the > audio driver of the capabilities of the attached HDMI sink. > > Signed-off-by: Russell

[PATCH RFC v2 10/13] sound/core: add DRM ELD helper

2015-04-05 Thread Russell King - ARM Linux
On Sun, Apr 05, 2015 at 05:57:09PM +0200, Takashi Iwai wrote: > > diff --git a/include/sound/pcm_drm_eld.h b/include/sound/pcm_drm_eld.h > > new file mode 100644 > > index ..93357b25d2e2 > > --- /dev/null > > +++ b/include/sound/pcm_drm_eld.h > > @@ -0,0 +1,6 @@ > > +#ifndef

[PATCH RFC v2 10/13] sound/core: add DRM ELD helper

2015-04-02 Thread Russell King
Add a helper for the EDID like data structure, which is typically passed from a HDMI adapter to its associated audio driver. This informs the audio driver of the capabilities of the attached HDMI sink. Signed-off-by: Russell King --- include/sound/pcm_drm_eld.h