ACK: [PATCH 0/1][SRU][OEM-5.10/H/U] UBUNTU: SAUCE: ALSA: hda/cirrus: Set Initial DMIC volume to -26 dB

[Tim Gardner]

Acked-by: Tim Gardner <tim.gardner at canonical.com>

On 6/1/21 2:24 AM, You-Sheng Yang wrote:
> BugLink: https://bugs.launchpad.net/bugs/1929803
> 
> [Impact]
> 
> On some platforms with Cirrus CS8409 audio codec, DMIC volume default
> value in kernel driver is 0, which follows no audio sound will be played
> during installation and OOBE until being explicitly configured again via
> GNOME Settings GUI or other methods.
> 
> [Fix]
> 
> Commit 527ff9550682 ("ALSA: hda/cirrus: Set Initial DMIC volume to -26
> dB") currently landed in upstream tiwai/sound.git is required to address
> this issue. It's a follow-up of commit 0e853a9c3937 ("ALSA: hda/cirrus:
> Set Initial DMIC volume for Bullseye to -26 dB"), which is applied in
> bug 1923557.
> 
> [Test Case]
> 
> Try to remove all the user space settings caches and reboot. The
> detailed steps are:
> 
>    # reboot to another system with access, e.g. livecd
>    livecd$ sudo mount /dev/nvme0n1p3 /mnt
>    livecd$ sudo rm /mnt/var/lib/alsa/asound.state
>    livecd$ sudo rm -rf /mnt/home/u/.config/pulse
>    livecd$ sudo reboot # reboot back
> 
> The original status is like:
> 
>    $ amixer sget 'Internal Mic',0
>    Simple mixer control 'Internal Mic',0
>      Capabilities: cvolume cswitch
>      Capture channels: Front Left - Front Right
>      Limits: Capture 0 - 63
>      Front Left: Capture 0 [0%] [-51.00dB] [off]
>      Front Right: Capture 0 [0%] [-51.00dB] [off]
> 
> Expect 40% in the front left/right capture volumes.
> 
>    $ amixer -c 1 sget 'Internal Mic',0
>    Simple mixer control 'Internal Mic',0
>      Capabilities: cvolume cswitch
>      Capture channels: Front Left - Front Right
>      Limits: Capture 0 - 63
>      Front Left: Capture 25 [40%] [-26.00dB] [on]
>      Front Right: Capture 25 [40%] [-26.00dB] [on]
> 
> [Where problems could occur]
> 
> This gives the default value of Cirrus CS8409 audio codec capture volume.
> It was previously found on another platform, Bullseye, and fixed in bug
> 1923557, while latterly convinced being a common bug that should be
> fixed without the quirk.
> 
> [Other Info]
> 
> By SRU we skipped a couple of refactoring changes in asound upstream
> tree, so additional changes to the original patch are required to
> backport. On v5.12 and on, say unstable, the fix can be cleanly
> cherry-picked.
> 
> Stefan Binding (1):
>    ALSA: hda/cirrus: Set Initial DMIC volume to -26 dB
> 
>   sound/pci/hda/patch_cirrus.c | 7 +++----
>   1 file changed, 3 insertions(+), 4 deletions(-)
> 

-- 
-----------
Tim Gardner
Canonical, Inc