topology2: add sof-ptl-rt713-l3-rt1320-l1 for LG Gram 16Z90U#10760
Open
Pyohwan wants to merge 1 commit into
Open
topology2: add sof-ptl-rt713-l3-rt1320-l1 for LG Gram 16Z90U#10760Pyohwan wants to merge 1 commit into
Pyohwan wants to merge 1 commit into
Conversation
Collaborator
|
Can one of the admins verify this patch?
|
Add a new build target for the LG Gram Pro 2026 (16Z90U-KU7BK), which has: - RT713 jack/headset codec on SoundWire link 3 - A single stereo RT1320 SmartAmp on SoundWire link 1 - 2 NHLT DMICs (built-in microphone array) This combination is unique because: - Existing rt713-rt1320 boards have two RT1320 amps on different links (NUM_SDW_AMP_LINKS=2), but the LG Gram uses one stereo RT1320 chip, so NUM_SDW_AMP_LINKS=1 is required. - The RT713 ACPI does not expose a SMART_MIC function, so SDW_DMIC=0 is used and NUM_DMICS=2 enables the NHLT-attached PCH DMICs for the laptop's built-in microphone array. - Stream names use the SDW%d-Playback/Capture form (without -SimpleJack/-SmartAmp suffix) because each link has only one dailink, so the kernel machine driver does not append the dai-type suffix (append_dai_type=false when num_link_dailinks==1). Tested on Ubuntu 26.04 with kernel 7.0.0-15 and a matching kernel match-table patch. Speaker, headphone jack, headset mic, and internal DMIC all work via the UCM HiFi profile. Signed-off-by: Jang Pyohwan <vhgksl@daum.net>
Pyohwan
force-pushed
the
topology2-ptl-rt713-l3-rt1320-l1
branch
from
acd2013 to
3ac2ad4
Compare
Collaborator
|
@bardliao Can you check? We a bug open for 17Z90U model in thesofproject/linux#5750 |
bardliao
approved these changes
May 11, 2026
Collaborator
Assume that was tested. The change looks good to me though my original plan was to use function topology and it worked. See thesofproject/linux#5750 (comment). But as the kernel patch was applied by Mark, the topology change is required. |
Closed
6 tasks
Member
|
test this please |
lgirdwood
approved these changes
May 11, 2026
klarasm
pushed a commit
to klarasm/linux
that referenced
this pull request
May 12, 2026
Add a SoundWire machine table entry for the LG Gram Pro 2026
(16Z90U-KU7BK), which has an unusual configuration:
sdw:0:1:025d:1320:01 single stereo RT1320 SmartAmp on link 1
sdw:0:3:025d:0713:01 RT713 jack/headset codec on link 3
Existing rt713-rt1320 boards have two RT1320 amps on different links
("link_mask = BIT(1) | BIT(2) | BIT(3)"). The LG Gram uses a single
stereo RT1320 chip, so the new entry uses "link_mask = BIT(1) |
BIT(3)" with the existing rt1320_1_group2_adr structure, leaving the
two-channel routing to the topology.
The RT713 on this board does not expose a SMART_MIC function in
ACPI, so the .machine_check callback used by the existing entries
(snd_soc_acpi_intel_sdca_is_device_rt712_vb) would reject this
board. Drop machine_check for the new entry; speaker output and
the headset jack do not depend on the SMART_MIC presence check.
The corresponding topology source has been submitted to the SOF
project at thesofproject/sof#10760 . The
generated sof-ptl-rt713-l3-rt1320-l1-2ch.tplg and
nhlt-sof-ptl-rt713-l3-rt1320-l1.bin will follow in linux-firmware
once that lands.
Tested on Ubuntu 26.04 with kernel 7.0.0-15: speaker (RT1320
stereo), headphone jack with auto-routing, headset mic, and the
internal NHLT DMIC array all work via the UCM HiFi profile.
Signed-off-by: Jang Pyohwan <vhgksl@daum.net>
Link: https://patch.msgid.link/20260509175317.DnhjxHczQay7kkp5z6t4lg@vhgksl.daum.net
Signed-off-by: Mark Brown <broonie@kernel.org>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Add a new SOF topology build target for the LG Gram Pro 2026 (16Z90U-KU7BK), which uses an unusual SoundWire configuration:
Why a new target
Existing rt713-rt1320 boards have two RT1320 amps on different links (
NUM_SDW_AMP_LINKS=2), but the LG Gram uses one stereo RT1320 chip:So this target uses
NUM_SDW_AMP_LINKS=1and type-less stream names (SDW1-Playback/SDW3-Playback/SDW3-Capture) because each link has only one dailink, which makes the kernel machine driver skip the-SimpleJack/-SmartAmpsuffix (append_dai_type=falsewhennum_link_dailinks==1).Test plan
Dependencies
This topology requires a matching kernel SoundWire machine table entry that I will submit separately to alsa-devel:
sound/soc/intel/common/soc-acpi-intel-ptl-match.c— addslink_mask = BIT(1) | BIT(3)entry referencingsof-ptl-rt713-l3-rt1320-l1.tplgI'll also submit the built
.tplgandnhlt-*.binto linux-firmware once this lands.