Skip to content

Audio support for WCL#5397

Merged
ranj063 merged 4 commits intothesofproject:topic/sof-devfrom
ujfalusi:peter/sof/pr/wcl-support-01
Apr 30, 2025
Merged

Audio support for WCL#5397
ranj063 merged 4 commits intothesofproject:topic/sof-devfrom
ujfalusi:peter/sof/pr/wcl-support-01

Conversation

@ujfalusi
Copy link
Collaborator

PCI ID and support for Wildcat Lake audio

.default_fw_filename = {
[SOF_IPC_TYPE_4] = "sof-wcl.ri",
},
.nocodec_tplg_filename = "sof-wcl-nocodec.tplg",
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How should this be?
we use the snd_soc_acpi_intel_ptl_machines and snd_soc_acpi_intel_ptl_sdw_machines but the nocodec is using sof-wcl-
Should we duplicate the ptl mach with s/ptl/wcl applied or change the nocodec to sof-ptl as well?

WCL is in essence PTL with reduced number of cores.

Copy link
Collaborator

@kv2019i kv2019i Apr 29, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think in general WCL tplgs should be separate. Let's have nocodec separate as well. sof-ptl-nocodec.tplg only uses cores 0+1+2 now, but if this is changed at some point, it'll break WCL machines using this (albeit this is for R&D but still).
UPDATE: I'll change my mind, ptl-nocodec is ok.

@ujfalusi ujfalusi force-pushed the peter/sof/pr/wcl-support-01 branch from 171274e to 5f1bb7f Compare April 29, 2025 12:22
Add PCI id for Wildcat Lake audio.

Signed-off-by: Peter Ujfalusi <peter.ujfalusi@linux.intel.com>
Clone PTL and adjust the number of cores from 5 to 3.

Signed-off-by: Peter Ujfalusi <peter.ujfalusi@linux.intel.com>
WCL uses the same receipt as PTL and PTL-H

Signed-off-by: Peter Ujfalusi <peter.ujfalusi@linux.intel.com>
One more PCI ID.

Signed-off-by: Peter Ujfalusi <peter.ujfalusi@linux.intel.com>
@ujfalusi
Copy link
Collaborator Author

Changes since v1:

  • correct the order of PCI IDs
  • Use sof-ptl-nocodec as default nocodec topology name for now as the ptl match tables are used as well.

Copy link
Collaborator

@kv2019i kv2019i left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After some back and forth, I'm now ok with this in its current form.

.power_down_dsp = mtl_power_down_dsp,
.disable_interrupts = lnl_dsp_disable_interrupts,
.hw_ip_version = SOF_INTEL_ACE_3_0,
};
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ujfalusi this patch is good for now but yuou know what I'm thinking. We should abstract some of the fields here that are largely unmodified gen to gen. something to consider doing in the future.

@ranj063 ranj063 merged commit 15b3016 into thesofproject:topic/sof-dev Apr 30, 2025
8 of 16 checks passed
@ujfalusi ujfalusi deleted the peter/sof/pr/wcl-support-01 branch May 19, 2025 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants