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. |
Open
6 tasks
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
4 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.