mbox series

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

Message ID 20210313113410.90088-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 13, 2021, 11:34 a.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 | 506 ++++++++++++++++-------------------
 1 file changed, 228 insertions(+), 278 deletions(-)

Comments

Takashi Iwai March 14, 2021, 8:36 a.m. UTC | #1 
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
20210312184452.3288-1-vitalyr@opensource.cirrus.com ?


thanks,

Takashi
Vitaly Rodionov March 14, 2021, 10:18 a.m. UTC | #2 
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
> 20210312184452.3288-1-vitalyr@opensource.cirrus.com ?

Hi Takashi,

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

Thanks,

Vitaly

>
>
> thanks,
>
> Takashi