Battery percentage stuck

[RehabMan]

Thanks a lot again and is this a good patch?

No. ECOK should be getting set in _REG.
Something else is wrong and this fix is just covering up the real problem.

 

[Shreyasv2108]

No. ECOK should be getting set in _REG.
Something else is wrong and this fix is just covering up the real problem.

How do I fix that

 

[RehabMan]

How do I fix that

I made some suggestions that were never followed up on in this thread.

 

[Shreyasv2108]

Could you please send the consolidated one again
Thanks

 

[RehabMan]

Could you please send the consolidated one again
Thanks

You can read the thread.
I'm not interested in repeating advice already given.

 

[Shreyasv2108]

You can read the thread.
I'm not interested in repeating advice already given.

Isbthe thread the one i sent as the place from where I got the solution?

 

[RehabMan]

Isbthe thread the one i sent as the place from where I got the solution?

No idea what you're asking.

Hint: ACPIDebug.kext can be used to instrument code in ACPI. You will need to be familiar with ACPI programming and the concept of debug traces.

 

[Shreyasv2108]

No idea what you're asking.

Hint: ACPIDebug.kext can be used to instrument code in ACPI. You will need to be familiar with ACPI programming and the concept of debug traces.

Is it this thread?

After a combination of using ACPIDebug, studying a working DSDT for a machine similar to mine and harnessing my sheer obsessiveness/stubbornness, I managed to find the offending bit of code that is getting the battery stuck.

Method (SMRD, 4, NotSerialized)
    {
        If (LNot (ECOK))
            {
                Return (0xFF)
            }
....

So I changed this to the following...

Method (SMRD, 4, NotSerialized)
    {
        If (LEqual (ECOK, One))
            {
                Return (0xFF)
            }
....

... and now my issue is fixed! Battery and AC are working correctly, battery status no longer gets stuck and correct percentage is reported in both macOS and Windows upon every reboot

In light of this, I'd like to mark this as resolved, but first I'm hoping you can answer a couple of questions for me:

1) What exactly does this change do to fix the status?
2) Is the edit I've made correct/safe as it stands or would you suggest implementing it another way?

 

[RehabMan]

Is it this thread?

Yes.