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

[Eric Anholt <eric@anholt.net>]

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

Boris Brezillon <boris.brezillon@free-electrons.com> writes:

> 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).
>
> Regards,
>
> Boris
>
> [1]https://www.spinics.net/lists/arm-kernel/msg565182.html

The kernel driver is getting merged now.

Tested-by: Eric Anholt <eric@anholt.net>

PulseAudio on the desktop seems to work fine.  aplay at the console
fails:

aplay -v  ~/LRMonoPhase4.wav
Playing WAVE '/home/anholt/LRMonoPhase4.wav' : Signed 16 bit Little Endian, Rate 48000 Hz, Stereo
aplay: set_params:1297: Sample format non available
Available formats:
- IEC958_SUBFRAME_LE

but if you specify iec958 it works:

aplay -v -D iec958  ~/LRMonoPhase4.wav 
Playing WAVE '/home/anholt/LRMonoPhase4.wav' : Signed 16 bit Little Endian, Rate 48000 Hz, Stereo
IEC958 subframe conversion PCM (IEC958_SUBFRAME_LE)
Its setup is:
  stream       : PLAYBACK
...

but if you specify a mono file it doesn't:

anholt@rpi-open:/home/anholt% aplay -v -D iec958  /usr/share/sounds/alsa/Front_Left.wav 
Playing WAVE '/usr/share/sounds/alsa/Front_Left.wav' : Signed 16 bit Little Endian, Rate 48000 Hz, Mono
aplay: set_params:1361: Unable to install hw params:
ACCESS:  RW_INTERLEAVED
FORMAT:  S16_LE
SUBFORMAT:  STD
SAMPLE_BITS: 16
FRAME_BITS: 16
CHANNELS: 1
...

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

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