2022-09-20 17:48:41

by Mario Limonciello

[permalink] [raw]
Subject: [PATCH] ASoC: amd: yc: Add Lenovo Yoga Slim 7 Pro X to quirks table

Lenovo Yoga Slim 7 Pro X has an ACP DMIC that isn't specified in the
ASL or existing quirk list. Add it to the quirk table to let DMIC
work on these systems.

Link: https://bugzilla.kernel.org/show_bug.cgi?id=216299
Tested-by: Sebastian S <[email protected]>
Reporetd-and-tested-by: Travis Glenn Hansen <[email protected]>
Signed-off-by: Mario Limonciello <[email protected]>
---
This patch applies on top of the other patch introducing a quirk:
https://lore.kernel.org/alsa-devel/[email protected]/
If either of them need to change, we should roll both into a series.

sound/soc/amd/yc/acp6x-mach.c | 7 +++++++
1 file changed, 7 insertions(+)

diff --git a/sound/soc/amd/yc/acp6x-mach.c b/sound/soc/amd/yc/acp6x-mach.c
index 5eab3baf3573..2cb50d5cf1a9 100644
--- a/sound/soc/amd/yc/acp6x-mach.c
+++ b/sound/soc/amd/yc/acp6x-mach.c
@@ -171,6 +171,13 @@ static const struct dmi_system_id yc_acp_quirk_table[] = {
DMI_MATCH(DMI_PRODUCT_NAME, "21J6"),
}
},
+ {
+ .driver_data = &acp6x_card,
+ .matches = {
+ DMI_MATCH(DMI_BOARD_VENDOR, "LENOVO"),
+ DMI_MATCH(DMI_PRODUCT_NAME, "82"),
+ }
+ },
{
.driver_data = &acp6x_card,
.matches = {
--
2.34.1


2022-09-20 19:09:02

by Mark Brown

[permalink] [raw]
Subject: Re: [PATCH] ASoC: amd: yc: Add Lenovo Yoga Slim 7 Pro X to quirks table

On Tue, Sep 20, 2022 at 01:09:45PM -0500, Limonciello, Mario wrote:

> As I mentioned below the cutline it's on top of the other commit on the list
> touching the same code:

> https://lore.kernel.org/alsa-devel/[email protected]/

That looks like it's some patch which wasn't even sent to me...

Please include human readable descriptions of things like commits and
issues being discussed in e-mail in your mails, this makes them much
easier for humans to read especially when they have no internet access.
I do frequently catch up on my mail on flights or while otherwise
travelling so this is even more pressing for me than just being about
making things a bit easier to read.


Attachments:
(No filename) (724.00 B)
signature.asc (499.00 B)
Download all attachments

2022-09-20 19:12:30

by Mario Limonciello

[permalink] [raw]
Subject: Re: [PATCH] ASoC: amd: yc: Add Lenovo Yoga Slim 7 Pro X to quirks table

On 9/20/2022 13:08, Mark Brown wrote:
> On Tue, Sep 20, 2022 at 12:13:20PM -0500, Mario Limonciello wrote:
>> Lenovo Yoga Slim 7 Pro X has an ACP DMIC that isn't specified in the
>> ASL or existing quirk list. Add it to the quirk table to let DMIC
>> work on these systems.
>
> This doesn't apply against current code, please check and resend.

As I mentioned below the cutline it's on top of the other commit on the
list touching the same code:

https://lore.kernel.org/alsa-devel/[email protected]/

2022-09-20 19:15:07

by Mark Brown

[permalink] [raw]
Subject: Re: [PATCH] ASoC: amd: yc: Add Lenovo Yoga Slim 7 Pro X to quirks table

On Tue, Sep 20, 2022 at 12:13:20PM -0500, Mario Limonciello wrote:
> Lenovo Yoga Slim 7 Pro X has an ACP DMIC that isn't specified in the
> ASL or existing quirk list. Add it to the quirk table to let DMIC
> work on these systems.

This doesn't apply against current code, please check and resend.


Attachments:
(No filename) (305.00 B)
signature.asc (499.00 B)
Download all attachments

2022-09-20 19:37:48

by Mark Brown

[permalink] [raw]
Subject: Re: [PATCH] ASoC: amd: yc: Add Lenovo Yoga Slim 7 Pro X to quirks table

On Tue, Sep 20, 2022 at 01:56:34PM -0500, Limonciello, Mario wrote:

> I guess the author forgot to include you in To or CC. Would you like me to
> just re-send the combination of both of them as a v2 (including the first on
> on behalf of Xiaoyan Li)?

Yes, please.


Attachments:
(No filename) (275.00 B)
signature.asc (499.00 B)
Download all attachments

2022-09-20 19:47:05

by Mario Limonciello

[permalink] [raw]
Subject: Re: [PATCH] ASoC: amd: yc: Add Lenovo Yoga Slim 7 Pro X to quirks table

On 9/20/2022 13:50, Mark Brown wrote:
> On Tue, Sep 20, 2022 at 01:09:45PM -0500, Limonciello, Mario wrote:
>
>> As I mentioned below the cutline it's on top of the other commit on the list
>> touching the same code:
>
>> https://lore.kernel.org/alsa-devel/[email protected]/
>
> That looks like it's some patch which wasn't even sent to me...
>
> Please include human readable descriptions of things like commits and
> issues being discussed in e-mail in your mails, this makes them much
> easier for humans to read especially when they have no internet access.
> I do frequently catch up on my mail on flights or while otherwise
> travelling so this is even more pressing for me than just being about
> making things a bit easier to read.


I guess the author forgot to include you in To or CC. Would you like me
to just re-send the combination of both of them as a v2 (including the
first on on behalf of Xiaoyan Li)?

2022-09-21 10:20:37

by Mark Brown

[permalink] [raw]
Subject: Re: [PATCH] ASoC: amd: yc: Add Lenovo Yoga Slim 7 Pro X to quirks table

On Tue, 20 Sep 2022 12:13:20 -0500, Mario Limonciello wrote:
> Lenovo Yoga Slim 7 Pro X has an ACP DMIC that isn't specified in the
> ASL or existing quirk list. Add it to the quirk table to let DMIC
> work on these systems.
>
>

Applied to

broonie/sound.git for-next

Thanks!

[1/1] ASoC: amd: yc: Add Lenovo Yoga Slim 7 Pro X to quirks table
commit: 2232b2dd8cd4f1e6d554b2c3f6899ce36f791b67

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark