summaryrefslogtreecommitdiff
path: root/src/arch
diff options
context:
space:
mode:
authorDuncan Laurie <dlaurie@chromium.org>2018-05-21 23:31:36 +0000
committerDuncan Laurie <dlaurie@chromium.org>2018-05-21 23:52:24 +0000
commit045cc899c90dfe483897f245bf8ac398ecb9424b (patch)
tree377845c504befc458ee6621bd4e017c8c8b4e215 /src/arch
parente4988ccf06c198b3aa311f4b6d84fd20b53a0aa4 (diff)
downloadcoreboot-045cc899c90dfe483897f245bf8ac398ecb9424b.tar.xz
Revert "acpi: device: Walk up the tree to find identifier"
This reverts commit 8ccf59a94778fb54cc08368fb58a42b64d9489f6. This wasn't meant to be submitted yet and seems to be causing issues, just as Patrick warned me.. Signed-off-by: Duncan Laurie <dlaurie@chromium.org> Change-Id: I8c4b57ba92ef4e0535e4975485188114a1084f09 Reviewed-on: https://review.coreboot.org/26452 Reviewed-by: Daniel Kurtz <djkurtz@google.com> Reviewed-by: Furquan Shaikh <furquan@google.com> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/arch')
-rw-r--r--src/arch/x86/acpi_device.c16
1 files changed, 3 insertions, 13 deletions
diff --git a/src/arch/x86/acpi_device.c b/src/arch/x86/acpi_device.c
index df2226d340..c8d313e037 100644
--- a/src/arch/x86/acpi_device.c
+++ b/src/arch/x86/acpi_device.c
@@ -55,9 +55,6 @@ static void acpi_device_fill_len(void *ptr)
/* Locate and return the ACPI name for this device */
const char *acpi_device_name(struct device *dev)
{
- struct device *pdev = dev;
- const char *name;
-
if (!dev)
return NULL;
@@ -65,16 +62,9 @@ const char *acpi_device_name(struct device *dev)
if (dev->ops->acpi_name)
return dev->ops->acpi_name(dev);
- /* Walk up the tree to find if any parent can identify this device */
- while (pdev->bus) {
- if (pdev->path.type == DEVICE_PATH_ROOT)
- break;
- if (pdev->bus->dev->ops && pdev->bus->dev->ops->acpi_name)
- name = pdev->bus->dev->ops->acpi_name(dev);
- if (name)
- return name;
- pdev = pdev->bus->dev;
- }
+ /* Check parent device in case it has a global handler */
+ if (dev->bus && dev->bus->dev->ops->acpi_name)
+ return dev->bus->dev->ops->acpi_name(dev);
return NULL;
}