May 2026 - Sound-open-firmware - mailman.alsa-project.org

Re: Dell XPS 13 9315: *sof-audio-pci-intel-tgl* still not loaded automatically
by Paul Menzel 25 May '26

25 May '26

[Cc: Remove bouncing ranjani.sridharan(a)linux.intel.com] Dear Péter, Thank you for your quick reply. Am 22.05.26 um 15:18 schrieb Péter Ujfalusi: > On 22/05/2026 15:31, Paul Menzel wrote: >> By default there is no sound on the Dell XPS 13 9315, and the module >> *sof-audio-pci-intel-tgl* still has to be loaded manually with Linux >> 7.1-rc4. It reminds me of issue 4540 [1], but that is supposedly fixed. >> >> Please find the Linux messages attached. At 184.662140 the command below >> is executed: >> >> sudo modprobe sof-audio-pci-intel-tgl > > if manually loading the driver works and then audio also works then can > it be that the driver is on denylist? I am so stupid. Indeed, that was it. An old workaround from back then. sudo rm /etc/modprobe.d/blacklist.conf and removing the scripts loading the module later, sound still works. Sorry about the noise. […] Kind regards, Paul

1 0

Dell XPS 13 9315: *sof-audio-pci-intel-tgl* still not loaded automatically
by Paul Menzel 25 May '26

25 May '26

Dear Linux folks, By default there is no sound on the Dell XPS 13 9315, and the module *sof-audio-pci-intel-tgl* still has to be loaded manually with Linux 7.1-rc4. It reminds me of issue 4540 [1], but that is supposedly fixed. Please find the Linux messages attached. At 184.662140 the command below is executed: sudo modprobe sof-audio-pci-intel-tgl Kind regards, Paul [1]: https://github.com/thesofproject/linux/issues/4540

1 0

[PATCH] ASoC: SOF: amd: Fix error code handling in psp_send_cmd()
by Mario Limonciello 12 May '26

12 May '26

The smn_read_register() helper returns negative error codes on failure or the register value on success. When used with read_poll_timeout(), the return value is stored in the 'data' variable. Currently 'data' is declared as u32, which causes negative error codes to be cast to large positive values. This makes the condition 'data > 0' incorrectly treat errors as success. Fix by changing 'data' from u32 to int, matching the pattern used in psp_mbox_ready() which correctly handles the same helper function. Reported-by: Dan Carpenter <error27(a)gmail.com> Closes: https://lore.kernel.org/linux-sound/agGES8vWrLOrBu28@stanley.mountain/ Fixes: f120cf33d232 ("ASoC: SOF: amd: Use AMD_NODE") Signed-off-by: Mario Limonciello <mario.limonciello(a)amd.com> --- sound/soc/sof/amd/acp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/soc/sof/amd/acp.c b/sound/soc/sof/amd/acp.c index 71a18f156de23..f615b8d1c8020 100644 --- a/sound/soc/sof/amd/acp.c +++ b/sound/soc/sof/amd/acp.c @@ -223,7 +223,7 @@ static int psp_send_cmd(struct acp_dev_data *adata, int cmd) { struct snd_sof_dev *sdev = adata->dev; int ret; - u32 data; + int data; if (!cmd) return -EINVAL; -- 2.43.0

1 0