Re: [PATCH 2/2] ASoC: intel: ptl: Add machine driver entry for Dell XPS 14 DA14260

[Richard Fitzgerald <rf@opensource.cirrus.com>]

On 02/04/2026 1:12 am, John Meyer wrote:
> The Dell XPS 14 DA14260 (Panther Lake) uses the following SoundWire
> audio topology:
> 
>    Link 0: CS42L45 (mfr 0x01FA, part 0x4245) -- SDCA headset codec
>              ADR 0x00003001FA424501
>              Endpoints: UAJ (jack playback), SmartMic (DMIC), UAJ (jack
>              capture)
> 
>    Link 2: Two CS35L57 speaker amps (mfr 0x01FA, part 0x3557)
>              ADR 0x00023201FA355701 (left/AMP1)
>              ADR 0x00023301FA355701 (AMP2)
> 
>    Link 3: Two CS35L57 speaker amps
>              ADR 0x00033001FA355701 (right/AMP3)
>              ADR 0x00033101FA355701 (AMP4)
> 
> ADR values were obtained from the platform ACPI DSDT.
> 
> Add the necessary ADR device arrays, endpoint descriptors, link array,
> and machine table entry.  The CS42L45 uses three non-aggregated
> endpoints (jack playback/capture and DMIC) matching the SDCA functions
> advertised by the hardware.  The four CS35L57 amps use the existing
> aggregated speaker endpoint descriptors (spk_l, spk_2, spk_r, spk_3).
> 
> The topology file name follows the established PTL naming convention.
> Note: a PTL-specific topology file does not yet exist; users can
> symlink sof-lnl-cs42l43-l0-cs35l56-l23.tplg as a workaround until
> a dedicated PTL topology is submitted to linux-firmware.
> 
No. This is completely the wrong thing to do, and isn't even
necessary because these parts are already supported and working.

PTL doesn't need match entries or hardcoded tplgs.

You seem to be sending random patches that
(a) are not necessary, (b) are wrong, (c) will break things

 > Note: a PTL-specific topology file does not yet exist; users can

The PTL tplgs do exist. These are SDCA parts that use the function
topologies.