mbox series

[v2,0/2,SRU,Jammy] Support Cirrus audio codec

Message ID 20220706015601.791262-1-acelan.kao@canonical.com
Headers show
Series Support Cirrus audio codec | expand

Message

AceLan Kao July 6, 2022, 1:55 a.m. UTC 
From: "Chia-Lin Kao (AceLan)" <acelan.kao@canonical.com>

BugLink: https://bugs.launchpad.net/bugs/1968861

[Impact]
CS8409 needs new configuration for Odin platform

[Fix]
Cirrus provides us 2 patches to add the support which landed on
v5.19-rc1 now
f129f26f7695 ALSA: hda/cs8409: Add Speaker Playback Switch for Cyborg
22bb82264c65 ALSA: hda/cs8409: Support new Odin Variants

[Test]
Verified on the target machine

[Where problems could occur]
The first commit only affects the new added Odin platform, and the second one
do some modification for other platforms which is hard to know what may lead to.

Stefan Binding (2):
  ALSA: hda/cs8409: Support new Odin Variants
  ALSA: hda/cs8409: Add Speaker Playback Switch for Cyborg

 sound/pci/hda/patch_cs8409-tables.c | 19 +++++++
 sound/pci/hda/patch_cs8409.c        | 84 ++++++++++++++++++++++++-----
 sound/pci/hda/patch_cs8409.h        |  4 ++
 3 files changed, 93 insertions(+), 14 deletions(-)

Comments

Tim Gardner July 6, 2022, 11:22 a.m. UTC | #1 
On 7/5/22 19:55, AceLan Kao wrote:
> From: "Chia-Lin Kao (AceLan)" <acelan.kao@canonical.com>
> 
> BugLink: https://bugs.launchpad.net/bugs/1968861
> 
> [Impact]
> CS8409 needs new configuration for Odin platform
> 
> [Fix]
> Cirrus provides us 2 patches to add the support which landed on
> v5.19-rc1 now
> f129f26f7695 ALSA: hda/cs8409: Add Speaker Playback Switch for Cyborg
> 22bb82264c65 ALSA: hda/cs8409: Support new Odin Variants
> 
> [Test]
> Verified on the target machine
> 
> [Where problems could occur]
> The first commit only affects the new added Odin platform, and the second one
> do some modification for other platforms which is hard to know what may lead to.
> 
> Stefan Binding (2):
>    ALSA: hda/cs8409: Support new Odin Variants
>    ALSA: hda/cs8409: Add Speaker Playback Switch for Cyborg
> 
>   sound/pci/hda/patch_cs8409-tables.c | 19 +++++++
>   sound/pci/hda/patch_cs8409.c        | 84 ++++++++++++++++++++++++-----
>   sound/pci/hda/patch_cs8409.h        |  4 ++
>   3 files changed, 93 insertions(+), 14 deletions(-)
> 
Acked-by: Tim Gardner <tim.gardner@canonical.com>
Luke Nowakowski-Krijger July 7, 2022, 8:40 p.m. UTC | #2 
Acked-by: Luke Nowakowski-Krijger <luke.nowakowskikrijger@canonical.com>

On Tue, Jul 5, 2022 at 6:57 PM AceLan Kao <acelan.kao@canonical.com> wrote:

> From: "Chia-Lin Kao (AceLan)" <acelan.kao@canonical.com>
>
> BugLink: https://bugs.launchpad.net/bugs/1968861
>
> [Impact]
> CS8409 needs new configuration for Odin platform
>
> [Fix]
> Cirrus provides us 2 patches to add the support which landed on
> v5.19-rc1 now
> f129f26f7695 ALSA: hda/cs8409: Add Speaker Playback Switch for Cyborg
> 22bb82264c65 ALSA: hda/cs8409: Support new Odin Variants
>
> [Test]
> Verified on the target machine
>
> [Where problems could occur]
> The first commit only affects the new added Odin platform, and the second
> one
> do some modification for other platforms which is hard to know what may
> lead to.
>
> Stefan Binding (2):
>   ALSA: hda/cs8409: Support new Odin Variants
>   ALSA: hda/cs8409: Add Speaker Playback Switch for Cyborg
>
>  sound/pci/hda/patch_cs8409-tables.c | 19 +++++++
>  sound/pci/hda/patch_cs8409.c        | 84 ++++++++++++++++++++++++-----
>  sound/pci/hda/patch_cs8409.h        |  4 ++
>  3 files changed, 93 insertions(+), 14 deletions(-)
>
> --
> 2.25.1
>
>
> --
> kernel-team mailing list
> kernel-team@lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team
>
Stefan Bader July 8, 2022, 4:14 p.m. UTC | #3 
On 06.07.22 03:55, AceLan Kao wrote:
> From: "Chia-Lin Kao (AceLan)" <acelan.kao@canonical.com>
> 
> BugLink: https://bugs.launchpad.net/bugs/1968861
> 
> [Impact]
> CS8409 needs new configuration for Odin platform
> 
> [Fix]
> Cirrus provides us 2 patches to add the support which landed on
> v5.19-rc1 now
> f129f26f7695 ALSA: hda/cs8409: Add Speaker Playback Switch for Cyborg
> 22bb82264c65 ALSA: hda/cs8409: Support new Odin Variants
> 
> [Test]
> Verified on the target machine
> 
> [Where problems could occur]
> The first commit only affects the new added Odin platform, and the second one
> do some modification for other platforms which is hard to know what may lead to.
> 
> Stefan Binding (2):
>    ALSA: hda/cs8409: Support new Odin Variants
>    ALSA: hda/cs8409: Add Speaker Playback Switch for Cyborg
> 
>   sound/pci/hda/patch_cs8409-tables.c | 19 +++++++
>   sound/pci/hda/patch_cs8409.c        | 84 ++++++++++++++++++++++++-----
>   sound/pci/hda/patch_cs8409.h        |  4 ++
>   3 files changed, 93 insertions(+), 14 deletions(-)
> 

Applied to jammy:linux/master-next. Thanks.

-Stefan