Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dummy Output - Missing tplg: sof-adl-rt711-2ch.tplg - OMEN by HP Gaming Laptop 16t-k000 - Debian Testing #4611

Closed
Kavey78 opened this issue Sep 30, 2023 · 7 comments

Comments

@Kavey78
Copy link

Kavey78 commented Sep 30, 2023

This seems very similar to issues like this one #4135 but I can't seem to find my device/model laptop sound card listed anywhere, so please forgive me if this has already been addressed. Below is dmesg output and other details I hope will provide enough information.

dmesg.txt

inxi.txt

lspci.txt

@plbossart
Copy link
Member

@Kavey78 can you add this file
sof-dyndbg.conf.txt as /etc/modprobe.d/sof-dyndbg.conf and re-attach the dmesg log. Also attach the log of 'alsa-info'. Thanks!

@Kavey78
Copy link
Author

Kavey78 commented Oct 4, 2023

Here is the output you requested.

alsa-info.txt
dmesg-sof-dyndb.txt

@Kavey78 Kavey78 changed the title Dummy Output - Missing tplg: sof-adl-rt711-2ch.tplg - OMEN by HP Gaming Laptop 16t-k000 - Debian Tesing Dummy Output - Missing tplg: sof-adl-rt711-2ch.tplg - OMEN by HP Gaming Laptop 16t-k000 - Debian Testing Oct 4, 2023
@plbossart
Copy link
Member

plbossart commented Oct 4, 2023

You have an RT1316 amplifier on link 3

sdw:3:025d:1316:01:0 

so it's clearly wrong to use the rt711-2ch topology.

the upstream code selects this topology in sound/soc/intel/common/oc-acpi-intel-adl-match.c

	{
		.link_mask = 0x9, /* 2 active links required */
		.links = adl_sdw_rt711_link0_rt1316_link3,
		.drv_name = "sof_sdw",
		.sof_tplg_filename = "sof-adl-rt711-l0-rt1316-l3.tplg",
	},

@plbossart
Copy link
Member

links 0 and 3 are enabled by ACPI

[    3.391186] snd_intel_sdw_acpi:sdw_intel_scan_controller: acpi PRP00001:00: ACPI reports 4 SDW Link devices
[    3.391190] snd_intel_sdw_acpi:sdw_intel_scan_controller: acpi PRP00001:00: Link 1 not selected in firmware
[    3.391193] snd_intel_sdw_acpi:sdw_intel_scan_controller: acpi PRP00001:00: Link 2 not selected in firmware

@plbossart
Copy link
Member

It's probably a DMI match issue, we use

{
		.matches = {
			DMI_MATCH(DMI_SYS_VENDOR, "HP"),
			DMI_MATCH(DMI_PRODUCT_NAME, "OMEN by HP Gaming Laptop 16-k0xxx"),
		},
		.driver_data = (void *)hp_omen_16,
	},

and you have

Product Name:      OMEN by HP Gaming Laptop 16t-k000

plbossart added a commit to plbossart/sound that referenced this issue Oct 4, 2023
New platforms have a slightly different DMI product name, remove
trailing 'xxx' to handle all cases

Closes: thesofproject#4611
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
plbossart added a commit to plbossart/sound that referenced this issue Oct 4, 2023
New platforms have a slightly different DMI product name, remove
trailing 'xxx' to handle all cases

Closes: thesofproject#4611
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
@plbossart
Copy link
Member

can you please try the patches in #4615 @Kavey78 ?

plbossart added a commit to plbossart/sound that referenced this issue Oct 4, 2023
New platforms have a slightly different DMI product name, remove
trailing characters/digits to handle all cases

Closes: thesofproject#4611
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
plbossart added a commit to plbossart/sound that referenced this issue Oct 4, 2023
New platforms have a slightly different DMI product name, remove
trailing characters/digits to handle all cases.

Closes: thesofproject#4611
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
@Kavey78
Copy link
Author

Kavey78 commented Oct 6, 2023

Thank you this worked perfectly. As soon as I applied the patch to my kernel installed and rebooted the sound card immediately started working. Sorry I didn't respond sooner, but I had a lot of issues getting the custom kernel installed properly. Eventually worked it out.

@Kavey78 Kavey78 closed this as completed Oct 6, 2023
plbossart added a commit that referenced this issue Oct 11, 2023
New platforms have a slightly different DMI product name, remove
trailing characters/digits to handle all cases

Closes: #4611
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
plbossart added a commit that referenced this issue Oct 11, 2023
New platforms have a slightly different DMI product name, remove
trailing characters/digits to handle all cases.

Closes: #4611
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
plbossart added a commit that referenced this issue Oct 12, 2023
New platforms have a slightly different DMI product name, remove
trailing characters/digits to handle all cases.

Closes: #4611
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: Rander Wang <rander.wang@intel.com>
Reviewed-by: Bard Liao <yung-chuan.liao@linux.intel.com>
plbossart added a commit that referenced this issue Oct 12, 2023
New platforms have a slightly different DMI product name, remove
trailing characters/digits to handle all cases

Closes: #4611
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: Rander Wang <rander.wang@intel.com>
Reviewed-by: Bard Liao <yung-chuan.liao@linux.intel.com>
intel-lab-lkp pushed a commit to intel-lab-lkp/linux that referenced this issue Oct 17, 2023
New platforms have a slightly different DMI product name, remove
trailing characters/digits to handle all cases.

Closes: thesofproject#4611
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: Rander Wang <rander.wang@intel.com>
Reviewed-by: Bard Liao <yung-chuan.liao@linux.intel.com>
akiyks pushed a commit to akiyks/linux that referenced this issue Oct 17, 2023
New platforms have a slightly different DMI product name, remove
trailing characters/digits to handle all cases.

Closes: thesofproject#4611
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: Rander Wang <rander.wang@intel.com>
Reviewed-by: Bard Liao <yung-chuan.liao@linux.intel.com>
Link: https://lore.kernel.org/r/20231012190826.142619-2-pierre-louis.bossart@linux.intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>
akiyks pushed a commit to akiyks/linux that referenced this issue Oct 17, 2023
New platforms have a slightly different DMI product name, remove
trailing characters/digits to handle all cases

Closes: thesofproject#4611
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: Rander Wang <rander.wang@intel.com>
Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>
Link: https://lore.kernel.org/r/20231013010833.114271-1-yung-chuan.liao@linux.intel.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
mj22226 pushed a commit to mj22226/linux that referenced this issue Nov 24, 2023
[ Upstream commit 4ea2b6d ]

New platforms have a slightly different DMI product name, remove
trailing characters/digits to handle all cases

Closes: thesofproject#4611
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: Rander Wang <rander.wang@intel.com>
Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>
Link: https://lore.kernel.org/r/20231013010833.114271-1-yung-chuan.liao@linux.intel.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
mj22226 pushed a commit to mj22226/linux that referenced this issue Nov 26, 2023
[ Upstream commit 4ea2b6d ]

New platforms have a slightly different DMI product name, remove
trailing characters/digits to handle all cases

Closes: thesofproject#4611
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: Rander Wang <rander.wang@intel.com>
Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>
Link: https://lore.kernel.org/r/20231013010833.114271-1-yung-chuan.liao@linux.intel.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Kaz205 pushed a commit to Kaz205/linux that referenced this issue Nov 26, 2023
[ Upstream commit 4ea2b6d ]

New platforms have a slightly different DMI product name, remove
trailing characters/digits to handle all cases

Closes: thesofproject#4611
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: Rander Wang <rander.wang@intel.com>
Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>
Link: https://lore.kernel.org/r/20231013010833.114271-1-yung-chuan.liao@linux.intel.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Kaz205 pushed a commit to Kaz205/linux that referenced this issue Nov 27, 2023
[ Upstream commit 4ea2b6d ]

New platforms have a slightly different DMI product name, remove
trailing characters/digits to handle all cases

Closes: thesofproject#4611
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: Rander Wang <rander.wang@intel.com>
Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>
Link: https://lore.kernel.org/r/20231013010833.114271-1-yung-chuan.liao@linux.intel.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
mj22226 pushed a commit to mj22226/linux that referenced this issue Nov 28, 2023
[ Upstream commit 4ea2b6d ]

New platforms have a slightly different DMI product name, remove
trailing characters/digits to handle all cases

Closes: thesofproject#4611
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: Rander Wang <rander.wang@intel.com>
Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>
Link: https://lore.kernel.org/r/20231013010833.114271-1-yung-chuan.liao@linux.intel.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
sj-aws pushed a commit to amazonlinux/linux that referenced this issue Nov 28, 2023
[ Upstream commit 4ea2b6d ]

New platforms have a slightly different DMI product name, remove
trailing characters/digits to handle all cases

Closes: thesofproject/linux#4611
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: Rander Wang <rander.wang@intel.com>
Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>
Link: https://lore.kernel.org/r/20231013010833.114271-1-yung-chuan.liao@linux.intel.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Whissi pushed a commit to Whissi/linux-stable that referenced this issue Nov 28, 2023
[ Upstream commit 4ea2b6d ]

New platforms have a slightly different DMI product name, remove
trailing characters/digits to handle all cases

Closes: thesofproject/linux#4611
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: Rander Wang <rander.wang@intel.com>
Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>
Link: https://lore.kernel.org/r/20231013010833.114271-1-yung-chuan.liao@linux.intel.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Whissi pushed a commit to Whissi/linux-stable that referenced this issue Nov 28, 2023
[ Upstream commit 4ea2b6d ]

New platforms have a slightly different DMI product name, remove
trailing characters/digits to handle all cases

Closes: thesofproject/linux#4611
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: Rander Wang <rander.wang@intel.com>
Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>
Link: https://lore.kernel.org/r/20231013010833.114271-1-yung-chuan.liao@linux.intel.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
wanghao75 pushed a commit to openeuler-mirror/kernel that referenced this issue Dec 4, 2023
stable inclusion
from stable-v6.6.3
commit e69ec2352087d9b6f5bcf31dbd4c14028b74671a
category: bugfix
bugzilla: https://gitee.com/openeuler/kernel/issues/I8LBQP

Reference: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=e69ec2352087d9b6f5bcf31dbd4c14028b74671a

--------------------------------

[ Upstream commit 4ea2b6d3128ea4d502c4015df0dc16b7d1070954 ]

New platforms have a slightly different DMI product name, remove
trailing characters/digits to handle all cases

Closes: thesofproject/linux#4611
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: Rander Wang <rander.wang@intel.com>
Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>
Link: https://lore.kernel.org/r/20231013010833.114271-1-yung-chuan.liao@linux.intel.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Signed-off-by: Zheng Zengkai <zhengzengkai@huawei.com>
johnny-mnemonic pushed a commit to linux-ia64/linux-stable-rc that referenced this issue Dec 31, 2023
[ Upstream commit 4ea2b6d ]

New platforms have a slightly different DMI product name, remove
trailing characters/digits to handle all cases

Closes: thesofproject/linux#4611
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: Rander Wang <rander.wang@intel.com>
Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>
Link: https://lore.kernel.org/r/20231013010833.114271-1-yung-chuan.liao@linux.intel.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
yifanli-intel pushed a commit to yifanli-intel/deepin-kernel that referenced this issue Feb 26, 2024
commit a5172ef ("ASoC: Intel: sof_sdw: update HP Omen match") upstream.

New platforms have a slightly different DMI product name, remove
trailing characters/digits to handle all cases.

deepin-Intel-SIG: commit a5172ef ("ASoC: Intel: sof_sdw: update HP Omen match").

Closes: thesofproject/linux#4611
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: Rander Wang <rander.wang@intel.com>
Reviewed-by: Bard Liao <yung-chuan.liao@linux.intel.com>
Link: https://lore.kernel.org/r/20231012190826.142619-2-pierre-louis.bossart@linux.intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>
[ Yifan2 Li: amend commit log ]
Signed-off-by: Yifan2 Li <yifan2.li@intel.com>
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

No branches or pull requests

2 participants