mbox series

[v3,0/4] ALSA: hda/cirrus: Make CS8409 driver more generic by using fixups

Message ID 20210315190716.47686-1-vitalyr@opensource.cirrus.com (mailing list archive)
Headers show
Series ALSA: hda/cirrus: Make CS8409 driver more generic by using fixups | expand

Message

Vitaly Rodionov March 15, 2021, 7:07 p.m. UTC 
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(-)

Comments

Takashi Iwai March 16, 2021, 7:22 a.m. UTC | #1 
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