search

xuwd1 / lenovo-legion-slim7i-gen7-knowledges

Everything we know about Lenovo slim7i gen7 (y9000x 2022)
14 stars 0 forks source link

Regarding sound on Slim 7i IAH7 #2

Open pierrehebert opened 5 months ago

pierrehebert commented 5 months ago

Hi,

I own this machine and I have been following https://bugzilla.kernel.org/show_bug.cgi?id=208555 and https://bugzilla.kernel.org/show_bug.cgi?id=216194 for a possible solution for the speaker issue. This is with great hope that I discovered your wiki page regarding the Slim 7i IAH7 (https://github.com/xuwd1/lenovo-legion-slim7i-gen7-knowledges/wiki/How-to-solve-the-speaker-problem). Unfortunately I couldn't get your patches to work. Recent kernels include some parts of your patch (serial-multi-instantiate) but the cs35l41 driver has diverged a bit. I could nevertheless adapt it for kernel 6.7+ but unfortunately this wasn't enough to get sound out of the speakers. To be honest this was a bit disappointing and I didn't look further. By chance, would you have an updated version of your patch for recent kernels (6.7 or 6.8) ? Cheers,

Pierre.

xuwd1 commented 5 months ago

Hi Pierre,

I am sorry to hear that, but I'm afraid that I couldn't be of help since I do not own this machine anymore. I'm really quite disappointed with it and have already sold this machine. I did noticed that some changes were made on the cs35l41 driver in kernel 6.7+, which is exactly the "cleaner solution" Cirrus people talked about. I did have tried to adapt my kernel patches to those changes and quite sadly I also could not make them work. After several lengthy kernel compilations, I eventually decided to give up.

Sorry.

xuwd1 commented 5 months ago

By the way, I think that maybe a DSDT patch would worth a try?

Hi Pierre,

I am sorry to hear that, but I'm afraid that I couldn't be of help since I do not own this machine anymore. I'm really quite disappointed with it and have already sold this machine. I did noticed that some changes were made on the cs35l41 driver in kernel 6.7+, which is exactly the "cleaner solution" Cirrus people talked about. I did have tried to adapt my kernel patches to those changes and quite sadly I also could not make them work. After several lengthy kernel compilations, I eventually decided to give up.

Sorry.

pierrehebert commented 5 months ago

Thank you for taking the time to answer :-) Well, I'm not sure I will ever ear sound from the speakers of this device then... This is annoying, but I can live with that. Other than that this machine fits my need and I'm a bit tired of debugging these days. I know about nothing about the DSDT. I think I understand how it is parts of the issue however, and you are probably right in that this would probably be the right path to follow. Thank you for your efforts anyway !

liujixings commented 4 months ago

You can try this https://patchwork.kernel.org/project/alsa-devel/list/?submitter=212831

pierrehebert commented 4 months ago

It works! Incredible! The volume is quite low but having something out from the speakers is enough to make me happy.

Many thanks for the link :-)