FWTS method test reports critical failures on SM15000-XH

[Samantha Jian-Pielak]

FWTS method test reports critical failures on SM15000-XH, Intel Haswell configuration. Please let me know these should gate a certificate for this system.

The summary of the critical failures:
Critical failures: 5
 method: Detected error 'Division by zero' when evaluating '\_SB_.PCI0.LPCB.H_EC.BAT1._BTP'.
 method: Detected error 'Type' when evaluating '\_SB_.PCI0.LPCB.H_EC.BAT0._PCL'.
 method: Detected error 'Type' when evaluating '\_SB_.PCI0.LPCB.H_EC.BAT1._PCL'.
 method: Detected error 'Type' when evaluating '\_SB_.PCI0.LPCB.H_EC.BAT2._PCL'.
 method: Detected error 'Type' when evaluating '\_SB_.ADP1._PCL'.

[Alex Hung (alexhung)]

@Samantha,

Please attach acpidump (sudo acpidump > acpi.log)

[Alex Hung (alexhung)]

As SM15000-XH is a server with no battery, errors for BAT1 can be safely ignored.

As _PCL is not actually used by Linux kernel (ex. _PCL returns Power Consumer List – returns a list of devices powered by a power source.), we don't have to worry about the "\_SB_.ADP1._PCL"'s error either.

[Samantha Jian-Pielak (samantha-jian)]

acpidump

[Alex Hung (alexhung)]

In dis-assembled ASL, _PCL is as below:

Method (_PCL, 0, NotSerialized) // _PCL: Power Consumer List
{
       Return (_SB)
}

However, _PCL should returns "A variable-length Package containing a list of References to devices or buses", which means it should be something like below:

Method (_PCL, 0, NotSerialized) // _PCL: Power Consumer List
{
       Return (Package(_SB))
}

As in #3, _PCL is not really used, so it's safe to ignore it.