mbox series

[SRU,j,v3,00/17] alsa/sdw: add sdw audio machine driver for several ADL machines

Message ID 20220105081313.17903-1-hui.wang@canonical.com
Headers show
Series alsa/sdw: add sdw audio machine driver for several ADL machines | expand

Message

Hui Wang Jan. 5, 2022, 8:12 a.m. UTC
BugLink: https://bugs.launchpad.net/bugs/1951563

In the v3:
add linux-next in the patch reference for 0010~0017-xxx.patch

In the v2:
Rebase the patchset to the latest master-next of jammy kernel.

[Impact]
In the Dell oem project, we need to enable the sdw audio on several
ADL machines, without these patches, the audio can't work at all.

[Fix]
Backport some patches from upstream, this adds the audio machine
drivers for those adl platforms.

[Test]
Booting the patched kernel, open the gnome-sound-setting, we could
see the speaker and microphone, plug headset, the headphone and
headset-mic all work well.

[Where problems could occur]
This add some machine drivers for sdw audio, if it could introduce
regression, it will be on the Dell previous generation machines which
has sdw audio, it could make the audio on those machines not work. But
this possibility is very low, we already tested the patches on those
machines.

Bard Liao (8):
  ASoC: intel: sof_sdw: return the original error number
  ASoC: intel: sof_sdw: rename be_index/link_id to link_index
  ASoC: intel: sof_sdw: Use a fixed DAI link id for AMP
  ASoC: intel: sof_sdw: move DMIC link id overwrite to
    create_sdw_dailink
  ASoC: intel: sof_sdw: remove SOF_RT715_DAI_ID_FIX quirk
  ASoC: intel: sof_sdw: remove sof_sdw_mic_codec_mockup_init
  ASoC: intel: sof_sdw: remove get_next_be_id
  ASoC: intel: sof_sdw: add link adr order check

Gongjun Song (9):
  ASoC: Intel: sof_sdw: Add support for SKU 0AF3 product
  ASoC: Intel: soc-acpi: add SKU 0AF3 SoundWire configuration
  ASoC: Intel: sof_sdw: Add support for SKU 0B00 and 0B01 products
  ASoC: Intel: sof_sdw: Add support for SKU 0B11 product
  ASoC: Intel: sof_sdw: Add support for SKU 0B13 product
  ASoC: Intel: soc-acpi: add SKU 0B13 SoundWire configuration
  ASoC: Intel: sof_sdw: Add support for SKU 0B29 product
  ASoC: Intel: soc-acpi: add SKU 0B29 SoundWire configuration
  ASoC: Intel: sof_sdw: Add support for SKU 0B12 product

 sound/soc/intel/boards/sof_sdw.c              | 209 ++++++++++++------
 sound/soc/intel/boards/sof_sdw_common.h       |   7 +-
 sound/soc/intel/boards/sof_sdw_rt715.c        |   7 -
 sound/soc/intel/boards/sof_sdw_rt715_sdca.c   |   7 -
 .../intel/common/soc-acpi-intel-adl-match.c   | 105 +++++++++
 5 files changed, 248 insertions(+), 87 deletions(-)

Comments

Tim Gardner Jan. 5, 2022, 12:28 p.m. UTC | #1
Acked-by: Tim Gardner <tim.gardner@canonical.com>

On 1/5/22 1:12 AM, Hui Wang wrote:
> BugLink: https://bugs.launchpad.net/bugs/1951563
> 
> In the v3:
> add linux-next in the patch reference for 0010~0017-xxx.patch
> 
> In the v2:
> Rebase the patchset to the latest master-next of jammy kernel.
> 
> [Impact]
> In the Dell oem project, we need to enable the sdw audio on several
> ADL machines, without these patches, the audio can't work at all.
> 
> [Fix]
> Backport some patches from upstream, this adds the audio machine
> drivers for those adl platforms.
> 
> [Test]
> Booting the patched kernel, open the gnome-sound-setting, we could
> see the speaker and microphone, plug headset, the headphone and
> headset-mic all work well.
> 
> [Where problems could occur]
> This add some machine drivers for sdw audio, if it could introduce
> regression, it will be on the Dell previous generation machines which
> has sdw audio, it could make the audio on those machines not work. But
> this possibility is very low, we already tested the patches on those
> machines.
> 
> Bard Liao (8):
>    ASoC: intel: sof_sdw: return the original error number
>    ASoC: intel: sof_sdw: rename be_index/link_id to link_index
>    ASoC: intel: sof_sdw: Use a fixed DAI link id for AMP
>    ASoC: intel: sof_sdw: move DMIC link id overwrite to
>      create_sdw_dailink
>    ASoC: intel: sof_sdw: remove SOF_RT715_DAI_ID_FIX quirk
>    ASoC: intel: sof_sdw: remove sof_sdw_mic_codec_mockup_init
>    ASoC: intel: sof_sdw: remove get_next_be_id
>    ASoC: intel: sof_sdw: add link adr order check
> 
> Gongjun Song (9):
>    ASoC: Intel: sof_sdw: Add support for SKU 0AF3 product
>    ASoC: Intel: soc-acpi: add SKU 0AF3 SoundWire configuration
>    ASoC: Intel: sof_sdw: Add support for SKU 0B00 and 0B01 products
>    ASoC: Intel: sof_sdw: Add support for SKU 0B11 product
>    ASoC: Intel: sof_sdw: Add support for SKU 0B13 product
>    ASoC: Intel: soc-acpi: add SKU 0B13 SoundWire configuration
>    ASoC: Intel: sof_sdw: Add support for SKU 0B29 product
>    ASoC: Intel: soc-acpi: add SKU 0B29 SoundWire configuration
>    ASoC: Intel: sof_sdw: Add support for SKU 0B12 product
> 
>   sound/soc/intel/boards/sof_sdw.c              | 209 ++++++++++++------
>   sound/soc/intel/boards/sof_sdw_common.h       |   7 +-
>   sound/soc/intel/boards/sof_sdw_rt715.c        |   7 -
>   sound/soc/intel/boards/sof_sdw_rt715_sdca.c   |   7 -
>   .../intel/common/soc-acpi-intel-adl-match.c   | 105 +++++++++
>   5 files changed, 248 insertions(+), 87 deletions(-)
>
Andrea Righi Jan. 14, 2022, 8:06 a.m. UTC | #2
On Wed, Jan 05, 2022 at 04:12:56PM +0800, Hui Wang wrote:
> BugLink: https://bugs.launchpad.net/bugs/1951563
> 
> In the v3:
> add linux-next in the patch reference for 0010~0017-xxx.patch
> 
> In the v2:
> Rebase the patchset to the latest master-next of jammy kernel.
> 
> [Impact]
> In the Dell oem project, we need to enable the sdw audio on several
> ADL machines, without these patches, the audio can't work at all.
> 
> [Fix]
> Backport some patches from upstream, this adds the audio machine
> drivers for those adl platforms.
> 
> [Test]
> Booting the patched kernel, open the gnome-sound-setting, we could
> see the speaker and microphone, plug headset, the headphone and
> headset-mic all work well.
> 
> [Where problems could occur]
> This add some machine drivers for sdw audio, if it could introduce
> regression, it will be on the Dell previous generation machines which
> has sdw audio, it could make the audio on those machines not work. But
> this possibility is very low, we already tested the patches on those
> machines.

Applied to jammy/linux, thanks!

-Andrea