Re: [RESEND PATCH] conf/cards: add VC4-HDMI card

[Eric Anholt <eric@anholt.net>]

[-- Attachment #1.1: Type: text/plain, Size: 1752 bytes --]

Takashi Iwai <tiwai@suse.de> writes:

> On Thu, 06 Apr 2017 21:46:29 +0200,
> Eric Anholt wrote:
>> 
>> Takashi Iwai <tiwai@suse.de> writes:
>> 
>> > On Thu, 02 Mar 2017 11:49:33 +0100,
>> > Boris Brezillon wrote:
>> >> 
>> >> Add a conf file for the VC4-HDMI sound card.
>> >> 
>> >> Signed-off-by: Boris Brezillon <boris.brezillon@free-electrons.com>
>> >> ---
>> >> Hi,
>> >> 
>> >> Sorry for the noise, but I didn't send this patch to the alsa-lib
>> >> maintainer on my first attempt.
>> >> 
>> >> This patch is adding a card config file for the audio sound card whose
>> >> driver has been submitted here [1] (not accepted yet).
>> >> 
>> >> Since I am a total newbie to the alsa world, I'd like to get some
>> >> feedback on this patch.
>> >> 
>> >> Also, the card only supports 2 to 8 channels, and I wonder if we should
>> >> add a plug element to support mono streams, and where this element
>> >> should be added (after or before the iec958 element).
>> >
>> > Applied, thanks.
>> 
>> Thanks!
>> 
>> Any recommendations on how to restructure this so that things like
>> 'aplay' from the console also work by default?
>
> Well, once after you have your card config in
> /usr/share/alsa/cards/vc4-hdmi.conf, aplay should work as is.
>
> If not, check /proc/asounds output.  The entry looks like
>  0 [ID            ]: DRIVER - SHORTNAME
>                      LONGNAME
>
> and in your case, "DRIVER" must be "vc4-hdmi", as same as the config
> file name.

It doesn't work as is -- the default ends up not having iec958
conversion, so nothing will play.  You can do aplay -D iec958, which
works but only as long as you've got stereo input (it seems plug can't
do mono-to-stereo on iec958 data)

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 832 bytes --]

[-- Attachment #2: Type: text/plain, Size: 0 bytes --]