[Devel] AcpiGetEventStatus: interface with behavior

[Andriy Gapon <avg at FreeBSD.org>]

[-- Attachment #1: Type: text/plain, Size: 1010 bytes --]


AcpiGetEventStatus has an out parameter of type ACPI_EVENT_STATUS.
My impression that values of that type should conform to bit definitions
provided near the type declaration (ACPI_EVENT_FLAG_ENABLED,
ACPI_EVENT_FLAG_SET, etc).
But it seems that currently AcpiGetEventStatus returns only values zero and one
corresponding to inactive and active status.

The linux (adaption of the) code here
http://lxr.linux.no/#linux+v3.7.1/drivers/acpi/acpica/evxfevnt.c#L313
seems to be a bit more elaborate.  It actually tries to produce a value that
conforms to the definitions by examining not only the status register but also
enable register.

It looks like AcpiHwGetGpeStatus is the model for how ACPI_EVENT_STATUS should
be populated.  I am not sure if AcpiGetEventStatus should be made compatible
with ACPI_EVENT_STATUS or if its out type should be changed to something else.

Also, it looks that the comment near ACPI_EVENT_STATUS definitions is a little
bit out of sync.

-- 
Andriy Gapon