[Solution] Trackpad VoodooI2CHID Conflicts with ACPIBatteryManager kext

[FreeJHack]

Some users are having a conflict problem with VoodooI2CHID in polling-mode & ACPIBatteryManager (1.90.1).
Me too. (GPIO correctly pinned...)
With AC power disconnected, the cursor is "micro-stopping" and "crazy-jumping" with a cycle of one second.

Conflict explanation:
ACPIBatteryManager is polling the battery status in order to update the battery data with a cycle of one second.
When the battery is charged, ACPIBatteryManager stops polling. This is the reason why we're facing the problem during discharging only.
The first task performed by the kext during polling, it is calling the _STA method of device BAT0 to know whether the battery is present and operating.Then it calls other methods _BIF, _BST, ...

The problem is just related to _STA method.
Most probably, your BAT0 (or BAT1, check your DSDT) _STA method looks like mine:

            Method (_STA, 0, NotSerialized)  // _STA: Status
            {
                If (^^PCI0.LPCB.EC.ECOK)
                {
                    Sleep (0x012C)
                    If (^^PCI0.LPCB.EC.BAT0)
                    {
                        Return (0x1F)
                    }
                    Else
                    {
                        Return (0x0F)
                    }
                }
                Else
                {
                    Return (0x0F)
                }
            }

As you can see, a Sleep function of 300mS is called every time... No good.
This is useful during start-up only to give PC (battery) enough time to set up.
During battery discharging the ACPIBatteryManager starts polling and this sleep function generates the "glitch" in the cursor.

Solution:
Adding a variable to perform the Sleep at the first time resolved the problem.
No more crazy-jumping cursor...
Patch your _STA method as below:

            Name (DOST, Zero)
            Method (_STA, 0, NotSerialized)  // _STA: Status
            {
                If (^^PCI0.LPCB.EC.ECOK)
                {
                    If (LNot (DOST))
                    {
                        Sleep (0x012C)
                        Store (One, DOST)
                    }

                    If (^^PCI0.LPCB.EC.BAT0)
                    {
                        Return (0x1F)
                    }
                    Else
                    {
                        Return (0x0F)
                    }
                }
                Else
                {
                    Return (0x0F)
                }
            }

Note that if DOST name is already used in your ACPI, change it with another one (YSTA, WSTA, ...)
Alternatively, Sleep statement can be deleted at all.

Hope this could help someone...

 

[djlucas456]

Hello. Can you write me some steps to fix this error? I'm not so proficient at this. Thank you very much.

 

[FreeJHack]

Hello. Can you write me some steps to fix this error? I'm not so proficient at this. Thank you very much.

You have to be familiar with ACPI patching.
Carefully follow the below guide and extract the ACPI files using Clover

[Guide] Patching LAPTOP DSDT/SSDTs

Overview In order to make many OS X features work well on a laptop, you will always need a properly patched DSDT (and maybe some of the SSDTs). The purpose of this guide is to provide a foundation for proper patching of your OEM DSDT/SSDTs. Advanced users may wish to implement hotpatching via...

www.tonymacx86.com

Then, your DSDT file needs to be patched according to post #1 and copied in CLOVER/ACPI/patched folder.

 

[djlucas456]

Thanks and one more question. after repair do I delete ACPIBatteryManager.kext or keep?

 

[Feartech]

Thanks and one more question. after repair do I delete ACPIBatteryManager.kext or keep?

keep it if you want battery status to work

 

[djlucas456]

Hello. so I solved the problem with VirtualSMC