https://bugzilla.kernel.org/show_bug.cgi?id=106941

--- Comment #18 from Lv Zheng <[email protected]> ---
2 ACPI LID device related facts here I should emphasized.

1. No "LID open" event on this platform:
If we search LID status change notification in the AML, we only get 1 instance
here:
    Line 11619:                     Notify (LID, 0x80)
The whole source code is:
            Method (_E4C, 0, Serialized)  // _Exx: Edge-Triggered GPE
            {
                If (LEqual (HELD, One))
                {
                    Store (One, ^^LID.LIDB)
                }
                Else
                {
                    Store (Zero, ^^LID.LIDB)
                    Notify (LID, 0x80)
                }

                Notify (^^PCI0.SPI1.NTRG, One)
            }
The BIOS only generate "LID close" event for ACPI LID device and never
generates "LID open" event on this platform.

2. _LID control method only returns cached value.
And for the _LID control method, it only contains cached value that is updated
here in _E4C:
    Line 11551:             Name (LIDB, Zero)
    Line 11554:                 Return (LIDB)
    Line 11614:                     Store (One, ^^LID.LIDB)
    Line 11618:                     Store (Zero, ^^LID.LIDB)

The whole source code is:
            Name (LIDB, Zero)
            Method (_LID, 0, NotSerialized)  // _LID: Lid Status
            {
                Return (LIDB)
            }

If the ACPI LID device can work with Windows, does this imply that "Windows"
(defacto standard) is exactly working as what I've imagind?

So it seems systemd need to implement "HandleLidSwitch=CloseOnly" for all such
platforms?
It doesn't seem to be a good idea to work it around in ACPI LID device driver.

Thanks and best regards
-Lv

-- 
You are receiving this mail because:
You are watching the assignee of the bug.

------------------------------------------------------------------------------
Presto, an open source distributed SQL query engine for big data, initially
developed by Facebook, enables you to easily query your data on Hadoop in a 
more interactive manner. Teradata is also now providing full enterprise
support for Presto. Download a free open source copy now.
http://pubads.g.doubleclick.net/gampad/clk?id=250295911&iu=/4140
_______________________________________________
acpi-bugzilla mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla

Reply via email to