[Guide] How to patch DSDT for working battery status

[vonian]

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.

I already did restart, so the latter one is the answer I believe.

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...

Hmmm but somehow I feel it's useless to patch ENIB, so I'll leave that in my own patch.
I don't do restart yet, but I guess you can add Lenovo ideapad 510s to be compatible to be patched with lenovo y580's patch in this case.

 

[RehabMan]

I don't do restart yet, but I guess you can add Lenovo ideapad 510s to be compatible to be patched with lenovo y580's patch in this case.

I'm pretty sure it will work, but let me know after you have tested it.

 

[vonian]

I'm pretty sure it will work, but let me know after you have tested it.

Did restart, and yes, it doesn't have a problem.

 

[RehabMan]

Did restart, and yes, it doesn't have a problem.

Thanks for confirming.

 

[jhax01]

Here's another battery patch for the repo. It's for the HP Elite X2 1012. It's a very good skylake core m tablet hack...everything works except the card reader and TB3.

 

[RehabMan]

Here's another battery patch for the repo. It's for the HP Elite X2 1012. It's a very good skylake core m tablet hack...everything works except the card reader and TB3.

This patch is wrong. B1B2 cannot be used as the target of a Store.

This patch translates:

Store(Zero, MAXC)

To:

Store(Zero, B1B2(XC00,XC01))

It is not valid. Must be broken into two separate Store ops:

Store(Zero, XC00)
Store(Zero, XC01)

 

[jhax01]

This patch is wrong. B1B2 cannot be used as the target of a Store.

This patch translates:

Store(Zero, MAXC)

To:

Store(Zero, B1B2(XC00,XC01))

It is not valid. Must be broken into two separate Store ops:

Store(Zero, XC00)
Store(Zero, XC01)

Ah, ok. Didn't seem to cause any adverse issues, or at least not any that were apparent. lol

Corrected version attached.

Hang on, bad formatting on that one, I cut and paste from the wrong thing.

 

[jhax01]

Ah, ok. Didn't seem to cause any adverse issues, or at least not any that were apparent. lol

Corrected version attached.

Hang on, bad formatting on that one, I cut and paste from the wrong thing.

Now it's corrected, though I've no doubt you will let me know if it isn't....lol

 

[RehabMan]

Now it's corrected, though I've no doubt you will let me know if it isn't....lol

The CBT field is 16-bits and is unpatched:

             CBT,    16,

It is accessed in GBTI.

 

[jhax01]

The CBT field is 16-bits and is unpatched:

             CBT,    16,

It is accessed in GBTI.

Third time is the charm....I think....lol...