From 045cc899c90dfe483897f245bf8ac398ecb9424b Mon Sep 17 00:00:00 2001 From: Duncan Laurie Date: Mon, 21 May 2018 23:31:36 +0000 Subject: 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 Change-Id: I8c4b57ba92ef4e0535e4975485188114a1084f09 Reviewed-on: https://review.coreboot.org/26452 Reviewed-by: Daniel Kurtz Reviewed-by: Furquan Shaikh Tested-by: build bot (Jenkins) --- src/arch/x86/acpi_device.c | 16 +++------------- 1 file 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; } -- cgit v1.2.3