ACPI / scan: set status to 0 if _STA failed
authorAaron Lu <aaron.lu@intel.com>
Thu, 19 Nov 2015 03:05:25 +0000 (11:05 +0800)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Mon, 30 Nov 2015 14:06:16 +0000 (15:06 +0100)
commite399037e06beec6d190a302ff5e4fee4aae77c50
tree5155c29dcc2abb3b8e1e3a395908ecb5906ffbe8
parent31ade3b83e1821da5fbb2f11b5b3d4ab2ec39db8
ACPI / scan: set status to 0 if _STA failed

Some ACPI node's _STA will touch operation region field, since the
evaluation of _STA in acpi_bus_type_and_status is very early, the
operation region handler is not ready yet. Instead of fail that function
and not creating the acpi_device node consequently, set status to 0 so
that later when the driver for that device is probing, it can find
the acpi_device node and proceed normally. And at that time, the
handler for the operation region is ready and its _STA evaluation will
succeed, its present status can be checked there.

Even there will be no driver using this node later, it doesn't seem
hurt to have one more acpi_device node created with status set to 0.

This happens on Microsoft Surface 3, where the SPI device node NTRG's
_STA touches GPIO fields and the SPI core driver will only enumerate SPI
devices from ACPI if the acpi_device node is 1: created; 2: _STA
indicates it's present.

Note that due to another problem in SPI driver, for NTRG to be actually
enumerated, some changes have to be made in the SPI layer, which is
addressed by Mika(not send out yet):
https://bugzilla.kernel.org/show_bug.cgi?id=104291#c23

Link: https://bugzilla.kernel.org/show_bug.cgi?id=104291
Reported-by: Bastien Nocera <bugzilla@hadess.net>
Signed-off-by: Aaron Lu <aaron.lu@intel.com>
Reviewed-by: Mika Westerberg <mika.westerberg@linux.intel.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/acpi/scan.c