[Guide] How to patch DSDT for working battery status

[xavier2670]

As per guide, only errors matter.

where can i find the vivobook patch

 

[RehabMan]

where can i find the vivobook patch

It is in the laptop patch repo.

Read the guide: https://www.tonymacx86.com/threads/guide-patching-laptop-dsdt-ssdts.152573/

 

[xavier2670]

It is in the laptop patch repo.

Read the guide: https://www.tonymacx86.com/threads/guide-patching-laptop-dsdt-ssdts.152573/

can't find it in Laptop-dsdt-patch-master

 

[RehabMan]

can't find it in Laptop-dsdt-patch-master

It will be obvious after you add the repo to MaciASL->Preferences->Sources. Read the guide and related READMEs carefully.

 

[00.Archer]

No.



User error.

Thanks, it was indeed that my /Extra folder was a mess.
Problem was solved by using Clover with a proper configured config.plist.

 

[xavier2670]

It will be obvious after you add the repo to MaciASL->Preferences->Sources. Read the guide and related READMEs carefully.

thank you so much

 

[vonian]

Hi, RehabMan. Here is the patch for Lenovo IdeaPad 510s.
If you don't mind, leave my name to anon like what I did in the txt please. I prefer not to credit myself.
Thanks~

 

[RehabMan]

Hi, RehabMan. Here is the patch for Lenovo IdeaPad 510s.
If you don't mind, leave my name to anon like what I did in the txt please. I prefer not to credit myself.
Thanks~

This patch is incomplete.
The patch for "Lenovo y580" is complete. Please test.

 

[vonian]

This patch is incomplete.
The patch for "Lenovo y580" is complete. Please test.

Hmm the funny thing is it works for me. That's why I posted that post.
What happens when it's incomplete?
Edit: I checked again, why 'ENIB' need to be patched? I can see for 'ERIB' but why also 'ENIB'. Don't you in #1 you said "If we search the DSDT for 'BMN0' we find only this declaration, so it is not accessed. We can ignore this one."?

 

[RehabMan]

Hmm the funny thing is it works for me. That's why I posted that post.
What happens when it's incomplete?

The code that accesses ERIB was either not reached in your test, or is not related to battery.
Still, no reason to include yet another duplicate patch, especially when the existing patch is more complete.

Edit: I checked again, why 'ENIB' need to be patched? I can see for 'ERIB' but why also 'ENIB'. Don't you in #1 you said "If we search the DSDT for 'BMN0' we find only this declaration, so it is not accessed. We can ignore this one."?

ENIB, technically, does not need patching as it is not accessed.
It doesn't hurt though.
For some, patching the field is a helpful way to insure that a future DSDT that comes along doesn't access it...