diff options
author | Zhao Yakui <yakui.zhao@intel.com> | 2010-07-12 23:36:08 -0400 |
---|---|---|
committer | Len Brown <len.brown@intel.com> | 2010-07-26 22:32:13 -0400 |
commit | 108029ff84fcad8f9199d2d2e2583ae2747d45a4 (patch) | |
tree | 9638986e377a77820b1ad30c0a72d2120b3f6f7b | |
parent | dbbe4649d683577de1063dbd9f6da7d4b8de2fed (diff) |
ACPI: Add the check of ADR flag in course of finding ACPI handle for PCI device
The _ADR object is used to provide OSPM with the address of one device on its
parent bus. In course of finding ACPI handle for the corresponding PCI device,
we will firstly evaluate the _ADR object and then compare the two addresses to
see whether it is the target ACPI device. But for one PCI device(0000:00:00.0)
under the PCI root bridge, the corresponding address will be constructed as
zero.In such case maybe the ACPI device without _ADR object will be misdetected
and then be used to create the relationship between PCI device and ACPI device.
https://bugzilla.kernel.org/show_bug.cgi?id=16422
Signed-off-by: Zhao Yakui <yakui.zhao@intel.com>
Signed-off-by: Len Brown <len.brown@intel.com>
-rw-r--r-- | drivers/acpi/glue.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/acpi/glue.c b/drivers/acpi/glue.c index 4af6301601e..78b0164c35b 100644 --- a/drivers/acpi/glue.c +++ b/drivers/acpi/glue.c | |||
@@ -100,7 +100,8 @@ do_acpi_find_child(acpi_handle handle, u32 lvl, void *context, void **rv) | |||
100 | 100 | ||
101 | status = acpi_get_object_info(handle, &info); | 101 | status = acpi_get_object_info(handle, &info); |
102 | if (ACPI_SUCCESS(status)) { | 102 | if (ACPI_SUCCESS(status)) { |
103 | if (info->address == find->address) | 103 | if ((info->address == find->address) |
104 | && (info->valid & ACPI_VALID_ADR)) | ||
104 | find->handle = handle; | 105 | find->handle = handle; |
105 | kfree(info); | 106 | kfree(info); |
106 | } | 107 | } |