Re: [PATCH] ASoC: amd: yc: Fix non-functional mic on Lenovo Yoga Slim 7 Pro 14ARH7 82UU

[Markus Meier]

Am Dienstag, 3. Oktober 2023, 12:24:56 CEST schrieben Sie:
> On 02.10.23 23:04, Markus Meier wrote:
> > I own a  Lenovo Yoga Slim 7 Pro 14ARH7 where the microphone is not showing
> > up. I added the system to the quirk table and the mic is working
> > correctly (but with very low volume) now.
> > 
> > Here's an except from dmidecode:
> > System Information
> > 
> >         Manufacturer: LENOVO
> >         Product Name: 82UU
> >         Version: Yoga Slim 7 Pro 14ARH7
> >         SKU Number: LENOVO_MT_82UU_BU_idea_FM_Yoga Slim 7 Pro 14ARH7
> >         Family: Yoga Slim 7 Pro 14ARH7
> > 
> > Please add this patch to the kernel and add it to the stable kernels too.
> > I
> > guess the microphone should have worked before. Seems to be a regression
> > introduced by c008323fe361bd62a43d9fb29737dacd5c067fb7
> > 
> > [-- Attachment #2: yoga-slim-7-pro-82UU-microphone.patch --]
> > [-- Type: text/x-patch, Size: 451 bytes --]
> > 
> > --- a/sound/soc/amd/yc/acp6x-mach.c	2023-09-29 19:31:03.753786095 
+0200
> > +++ b/sound/soc/amd/yc/acp6x-mach.c	2023-09-29 19:32:06.414276453 
+0200
> > @@ -217,6 +217,13 @@
> > 
> >  		.driver_data = &acp6x_card,
> >  		.matches = {
> >  		
> >  			DMI_MATCH(DMI_BOARD_VENDOR, "LENOVO"),
> > 
> > +			DMI_MATCH(DMI_PRODUCT_NAME, "82UU"),
> > +		}
> > +	},
> > +	{
> > +		.driver_data = &acp6x_card,
> > +		.matches = {
> > +			DMI_MATCH(DMI_BOARD_VENDOR, "LENOVO"),
> > 
> >  			DMI_MATCH(DMI_PRODUCT_NAME, "82V2"),
> >  		
> >  		}
> >  	
> >  	},
> 
> I CCed a bunch of lists and people to make sure everyone is aware of
> that patch, as it otherwise might fall through the cracks.
> 
> Markus, quick question: do you have (a) an interest in contributing a
> change to Linux or (b) did you just post this because Mario asked you to
> do so in https://bugzilla.kernel.org/show_bug.cgi?id=217063#c25 ?

Nope I don't plan to become some kind of kernel developer. I just want to use 
my notebook without any issues and want to help others who want to use this 
device with Linux too. I just followed Mario's suggestions. I thought it would 
be easier for everyone when I create a patch, try if it works and then submit 
it to a bugtracker to get the bug fixed.

> If it's (a) then you might want to take a closer look at
> https://www.kernel.org/doc/html/latest/process/submitting-patches.html
> , as your patch lacks a few important things before it can be applied
> (most notably a Signed-off-by tag)
> 
> If it's (b) I wonder if it would be easier for everyone involved if some
> kernel developer could create a patch own their own, as that likely is
> way easier to review and thus likely gets the problem resolved a lot faster.

Please feel free to create a patch based on mine, when it's easier and faster 
that way. I don't really care as long as the microphone is working in one of 
the next kernel releases. 

Thanks for your help Thorsten. I really enjoyed your Heise Kernel-Log series 
and I really miss those series. Was always a pleasure to read that.

Kind regards
Markus

Attachment: signature.asc
Description: This is a digitally signed message part.