Comment 149 for bug 1958019

(In reply to DavidLenovo from comment #135)
> (In reply to wave from comment #131)
> > (In reply to chenyh570 from comment #129)
> > > (In reply to DavidLenovo from comment #120)
> > > > Hi !
> > > > A man have find the solution for the Yoga 7 - Yoga 7i - 14ITL5 &
> 14ITL05
> > &
> > > > 15ITL05 & 15ITL5 :)
>
> >
> > First of all allow me to remark that this is exactly the solution that I
> > talked about in comment #97 and which I tried to explain to you
> specifically
> > in more detail in comment #107; in fact I wrote the archwiki article
> > referred to in the comment.
> >
> > Now your problem seems to be related to permissions; perhaps you are
> running
> > the whole command or padsp as sudo?
>
> Sorry :( to don't mention you.
> Thanks a lot :), effectively you see the problem firstly and you try to put
> all of us in the right way.
> To be honest my knowledge is to limited to patch me alone. I had need the
> patch "key in hand", found after (put in my tutorial).
> I think other people, need the patch "key in hand", with more simple
> tutorial.
>
> I think your have lot of skill, and me (and probably other) just want simple
> solution (and can't do hard modification).
>
> I'm not sure to understand, your are the writer of the tutorial :
> https://wiki.archlinux.org/title/Lenovo_Yoga_7i ?
>
> I think we are lot of people, to don't have lot of knowledge, and need very
> simple procedure. And for me I see lot of people very skilled, like you, to
> be documentation not enough easy for us.
> This is why I make this tutorial, with almost all commands.
> For example the patch for the Yoga 7i (on Arch Wiki), don't have the
> beginning of the file :
> --- dsdt.dsl~ 2018-04-26 09:35:29.501055509 -0600
> +++ dsdt.dsl 2018-04-26 09:36:23.769729028 -0600
> So it can't work as is.
> I had to take time to understand, and find why it don't work, I never do it
> before.
>
> Thanks another time :)

Don't worry, I didn't mean to criticize you at all :)
Quite the contrary, I know that my instructions were very terse (in that I did not duplicate instructions that are available elsewhere, which would be the wrong thing to do on the archwiki anyway), and I'm happy that you went through it, figured it out, and published your findings in an accessible way. You even cited my tutorial from archwiki so everything is perfect.

Thanks also for pointing out that the patch does not quite work with the patch command linked in the article; it works if you put the patch into a file like acpi.patch and then run
patch dsdt.dsl acpi.patch
(specifying both the file to be patched as well as the patch file). But I'll change that in the article. Nice that you figured it out though!