lkml.org 
[lkml]   [2021]   [Oct]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH] ASoC: simple-card: Fill in driver name
Date
On Tuesday, 22 June 2021 10:27:09 CEST Guido Günther wrote:
> alsa-ucm groups by driver name so fill that in as well. Otherwise the
> presented information is redundant and doesn't reflect the used
> driver. We can't just use 'asoc-simple-card' since the driver name is
> restricted to 15 characters.
>
> Before:
>
> # cat /proc/asound/cards
> 0 [Devkit ]: Librem_5_Devkit - Librem 5 Devkit
> Librem 5 Devkit
> After:
>
> 0 [Devkit ]: simple-card - Librem 5 Devkit
> Librem 5 Devkit
>
> Signed-off-by: Guido Günther <agx@sigxcpu.org>
> ---
> This came out of a discussion about adding alsa-ucm profiles for the
> Librem 5 Devkit at https://github.com/alsa-project/alsa-ucm-conf/pull/102
>
> sound/soc/generic/simple-card.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/sound/soc/generic/simple-card.c
> b/sound/soc/generic/simple-card.c index 0015f534d42d..a3a7990b5cb6 100644
> --- a/sound/soc/generic/simple-card.c
> +++ b/sound/soc/generic/simple-card.c
> @@ -621,6 +621,7 @@ static int asoc_simple_probe(struct platform_device
> *pdev) card->owner = THIS_MODULE;
> card->dev = dev;
> card->probe = simple_soc_probe;
> + card->driver_name = "simple-card";
>
> li = devm_kzalloc(dev, sizeof(*li), GFP_KERNEL);
> if (!li)

Unfortunately this change broke multichannel audio on my Rock64 device running
Debian. My Rock64 is connected to my AVR (Pioneer SC-1224) via a HDMI cable.

Before/without this change, pulseaudio correctly identifies my AVR and gives me
hdmi-stereo/hdmi-surround/hdmi-surround71 output profiles via the HDMI card and
iec958-stereo/iec958-ac3-surround-51 on my SPDIF card.
After/with this change, I only have a stereo-fallback output profile on both
HDMI and SPDIF card.
I reported this to Debian at https://bugs.debian.org/995923
I have verified that reverting this commit on a self-built 5.14.10 brought back
multichannel audio.

On LibreELEC's current nightly builds (version 11) with kernel 5.14 on
a(nother) Rock64 device (also connected via HDMI cable), I hear no sound at
all. Previous nightly builds (version 11) with I _assume_ kernel < 5.14,
worked fine. As I didn't keep previous nightly builds, I can't verify that.
I reported that at https://github.com/LibreELEC/LibreELEC.tv/issues/5734
I have NOT verified whether reverting this commit would also restore audio with
LibreELEC as I'm not familiar with their build system.

This is the first time I'm reporting an issue to the upstream Linux kernel, so
I hope I did it correctly (DL-ed mbox file and did Reply-to-All). If not,
apologies and I hope you'll point me to the correct place/way.
(I don't know if others need to be notified too as it is part of 5.14 releases)

Cheers,
Diederik[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2021-10-10 09:48    [W:0.090 / U:0.088 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site