2021-03-15 23:55:41

by Vitaly Rodionov

[permalink] [raw]
Subject: [PATCH v3 0/4] ALSA: hda/cirrus: Make CS8409 driver more generic by using fixups

This series of patches will address comments by Pierre-Louis Bossart,
cleans up patch_cirrus.c source, reducing checkpatch.pl warnings from 19 to 0,
fixing an issue reported by Canonical: BugLink: https://bugs.launchpad.net/bugs/1918378,
and makes the CS8409 patch more generic by using fixups.

Stefan Binding (4):
ALSA: hda/cirrus: Add error handling into CS8409 I2C functions
ALSA: hda/cirrus: Cleanup patch_cirrus.c code.
ALSA: hda/cirrus: Fix CS42L42 Headset Mic volume control name
ALSA: hda/cirrus: Make CS8409 driver more generic by using fixups.

sound/pci/hda/patch_cirrus.c | 525 ++++++++++++++++-------------------
1 file changed, 242 insertions(+), 283 deletions(-)

--
2.25.1


2021-03-16 09:08:49

by Takashi Iwai

[permalink] [raw]
Subject: Re: [PATCH v3 0/4] ALSA: hda/cirrus: Make CS8409 driver more generic by using fixups

On Mon, 15 Mar 2021 20:07:12 +0100,
Vitaly Rodionov wrote:
>
> This series of patches will address comments by Pierre-Louis Bossart,
> cleans up patch_cirrus.c source, reducing checkpatch.pl warnings from 19 to 0,
> fixing an issue reported by Canonical: BugLink: https://bugs.launchpad.net/bugs/1918378,
> and makes the CS8409 patch more generic by using fixups.
>
> Stefan Binding (4):
> ALSA: hda/cirrus: Add error handling into CS8409 I2C functions
> ALSA: hda/cirrus: Cleanup patch_cirrus.c code.
> ALSA: hda/cirrus: Fix CS42L42 Headset Mic volume control name
> ALSA: hda/cirrus: Make CS8409 driver more generic by using fixups.

Now applied all four patches.

The patch 4 had the old style fallthrough comment and I corrected with
the fallthrough macro locally.


thanks,

Takashi