2021-03-13 11:40:59

by Vitaly Rodionov

[permalink] [raw]
Subject: [PATCH v1 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 | 506 ++++++++++++++++-------------------
1 file changed, 228 insertions(+), 278 deletions(-)

--
2.25.1


2021-03-14 08:39:40

by Takashi Iwai

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

On Sat, 13 Mar 2021 12:34:06 +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.

Is this the same content as the series you've already submitted in
[email protected] ?


thanks,

Takashi

2021-03-14 10:28:30

by Vitaly Rodionov

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

On 14/03/2021 8:36 am, Takashi Iwai wrote:
> On Sat, 13 Mar 2021 12:34:06 +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.
> Is this the same content as the series you've already submitted in
> [email protected] ?

Hi Takashi,

Yes, this is second version of same series, where we have fixed warnings
from 0-day bot.

Thanks,

Vitaly

>
>
> thanks,
>
> Takashi