[Bug 2129952] Re: Support for Realtek audio solution ALC3329B+ALC1708B on new Dell PTL platform

[Matthew Ruffell]

Hi Bin-Li and Kai-Chuan,

For questing:

The patch that adds support for the "Macro" is okay, because it is used for
direct device enablement for realtek chipsets. 

However, d/p/lp2130313/0007-ucm2-use-new-SetLED-macro-to-hide-the-implementation.patch
is refactoring existing code to use this Macro with no supposed change in
functionality or bug fix. It has a large regression risk due to modifying 
the file "ucm2/Intel/sof-hda-dsp/sof-hda-dsp.conf", which affects ALL Intel
chipsets, as well as a few other specific chipsets in other files.

Now, we decided to accept this because from what we can tell, this was suppose
to have landed in -devel during the QQ development cycle, and it exists
in noble, oracular, plucky, and now, resolute. It has been in noble onward since
May 2025, and is largely well tested by the community now. We searched the bug
trackers, and no one really complained about change in behaviour or anything.

We did notice the upgrade regression bug 2130874 which this patchset
fixes.

Ideally this patch should not have been accepted into N/O/P, and it was also
missed during SRU review the first time by separate people, so its easy for this
to slip through the cracks.

But in the future, this is a refactor type commit, and does not directly impact
users, so please only include these sorts of fixes if they directly fix or
impact users in some way.

We have accepted questing.

Thanks,
Matthew and ROAF.

-- 
You received this bug notification because you are a member of Ubuntu
Sponsors, which is subscribed to the bug report.
https://bugs.launchpad.net/bugs/2129952

Title:
  Support for Realtek audio solution ALC3329B+ALC1708B on new Dell PTL
  platform

Status in HWE Next:
  New
Status in alsa-ucm-conf package in Ubuntu:
  Fix Released
Status in linux package in Ubuntu:
  New
Status in linux-firmware package in Ubuntu:
  Fix Released
Status in linux-oem-6.17 package in Ubuntu:
  Invalid
Status in alsa-ucm-conf source package in Noble:
  Incomplete
Status in linux source package in Noble:
  Invalid
Status in linux-firmware source package in Noble:
  Fix Released
Status in linux-oem-6.17 source package in Noble:
  Fix Released
Status in alsa-ucm-conf source package in Questing:
  Fix Committed
Status in linux source package in Questing:
  Invalid
Status in linux-firmware source package in Questing:
  Fix Released
Status in linux-oem-6.17 source package in Questing:
  Invalid
Status in alsa-ucm-conf source package in Resolute:
  Fix Released
Status in linux source package in Resolute:
  New
Status in linux-firmware source package in Resolute:
  Fix Released
Status in linux-oem-6.17 source package in Resolute:
  Invalid

Bug description:
  SRU Justification:

  [Impact]
  New Dell platform with Realtek codec ALC3329B+ALC1708B powered by Intel PantherLake will have no basic audio functions w/o Soundwire Driver Support.

  ================ Linux Kernel ========================

  [Fix]
  Backport new Intel soundwire drivers of the following
  ASoC: sof-function-topology-lib: escalate the log when missing function topoplogy - kernel/git/broonie/sound.git - ASoC
  ASoC: SOF: Intel: use sof_sdw as default SDW machine driver - kernel/git/broonie/sound.git - ASoC
  ASoC: SOF: add platform name into sof_intel_dsp_desc - kernel/git/broonie/sound.git - ASoC
  ASoC: soc_sdw_utils: export asoc_sdw_get_dai_type - kernel/git/broonie/sound.git - ASoC
  ASoC: Intel: export sof_sdw_get_tplg_files - kernel/git/broonie/sound.git - ASoC
  ASoC: soc_sdw_utils: add name_prefix to asoc_sdw_codec_info struct - kernel/git/broonie/sound.git - ASoC
  ASoC: soc-acpi: make some variables of acpi adr and link adr non-const - kernel/git/broonie/sound.git - ASoC
  ASoC: SOF: Don't print the monolithic topology name if function topology may be used - kernel/git/broonie/sound.git - ASoC
  ASoC: SOF: don't check the existence of dummy topology - kernel/git/broonie/sound.git - ASoC

  and ALC3329B+ALC1708B from Realtek

  ASoC: rt722: add settings for rt722VB - kernel/git/broonie/sound.git - ASoC
  ASoC: rt1321: Add RT1321 amplifier support - kernel/git/broonie/sound.git - ASoC
  ASoC: rt712: avoid skipping the blind write - kernel/git/broonie/sound.git - ASoC
  ASoC: sdw_utils: add rt1321 part id to codec_info_list - Patchwork
  ASoC: sdw_utils: add name_prefix for rt1321 part id - kernel/git/broonie/sound.git - ASoC
  ASoC: SOF: Intel: add hyphen between name and index to amp name_prefix - kernel/git/broonie/sound.git
  ASoC: SOF: don't check the existence of dummy topology - kernel/git/broonie/sound.git - ASoC

  =============== linux-firmware ========================

  [Fix]
  Backport the patch firmware from Realtek
  realtek: rt1321: Add patch firmware of MCU - kernel/git/firmware/linux-firmware.git - Repository of firmware blobs for use with the Linux kernel

  =============== alsa-ucm-conf ========================

  [Fix]
  Backport the patch 
  https://github.com/alsa-project/alsa-ucm-conf/commit/2740b6c157c048deb950fd6e5792204568019034
  https://github.com/alsa-project/alsa-ucm-conf/commit/6b2f0c14672bd64b295e8900de3a7282f94f1de3
  https://github.com/alsa-project/alsa-ucm-conf/commit/b4e9b243d0de6cea31432670530136f6f47af6a7
  https://github.com/alsa-project/alsa-ucm-conf/commit/a3e1786a2025663ef9da74c9f7f2d0ecf876c0e2 
   
  and 

  PR (https://github.com/alsa-project/alsa-ucm-conf/pull/628
  [github.com])

   from upstream

  [Test Case]
  1. Boot up the machine of new Dell Intel PantherLake platform with Realtek ALC3329B+ALC1708B.
  2. Open settings->Sound->Output Device and make sure it's not Dummy audio devices
  3. Click test icon for basic audio output functions
  4. Use `arecord` to record and play the recorded audio file with `aplay` for basic speaker/microphone functions.

  [Where problems could occur]
  It adds the support for new Realtek codec+amplifier on PTL platform. No regression expected at the current stage.

To manage notifications about this bug go to:
https://bugs.launchpad.net/hwe-next/+bug/2129952/+subscriptions