Re: [PATCH 1/3] ACPI: move ACPI_SYSTEM_HID to acpi_drivers.h

["Prakash, Prashanth" <pprakash@codeaurora.org>]

Hi Rafael,

On 3/16/2016 7:50 PM, Rafael J. Wysocki wrote:
>  #include "internal.h"
> @@ -22,7 +24,6 @@ ACPI_MODULE_NAME("scan");
>  extern struct acpi_device *acpi_root;
>
>  #define ACPI_BUS_CLASS                 "system_bus"
> -#define ACPI_BUS_HID                   "LNXSYBUS"
> No.
>
> This is not a valid device ID and should never be used as such.
Section 5.6.6 in ACPI 6.1 is defining the notification value for the graceful
shutdown request as a ACPI device specific value(0x81) and it is targeting
the system bus(\_SB), so we took the current approach of creating a sybus
driver to handle the notifications targeting the system bus.

Let me go through the code again and see if we can avoid using the
ACPI_BUS_HID. In the meantime, if you have any specific thoughts on
how this should be handled, please let us know.

Thanks,
Prashanth