From patchwork Fri Jan 11 20:31:43 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: [1/16] ACPI: Separate adding ACPI device objects from probing ACPI drivers X-Patchwork-Submitter: "Rafael J. Wysocki" X-Patchwork-Id: 211416 Message-Id: <2859994.arbXorpfaz@vostro.rjw.lan> To: Mika Westerberg Cc: ACPI Devel Maling List , Toshi Kani , Bjorn Helgaas , LKML , linux-pci@vger.kernel.org, Yinghai Lu , Myron Stowe , Yijing Wang , Jiang Liu Date: Fri, 11 Jan 2013 21:31:43 +0100 From: "Rafael J. Wysocki" List-Id: On Friday, January 11, 2013 10:00:46 PM Mika Westerberg wrote: > (Sorry to jump in late but I noticed one problem with this series while > testing). > > On Thu, Dec 20, 2012 at 02:47:47AM +0100, Rafael J. Wysocki wrote: > > +static acpi_status acpi_bus_probe_start(acpi_handle handle, u32 lvl, > > + void *context, void **not_used) > > +{ > > + struct acpi_bus_ops *ops = context; > > + acpi_status status = AE_OK; > > + struct acpi_device *device; > > + unsigned long long sta_not_used; > > + int type_not_used; > > + > > + /* > > + * Ignore errors ignored by acpi_bus_check_add() to avoid terminating > > + * namespace walks prematurely. > > + */ > > + if (acpi_bus_type_and_status(handle, &type_not_used, &sta_not_used)) > > + return AE_OK; > > + > > + if (acpi_bus_get_device(handle, &device)) > > + return AE_CTRL_DEPTH; > > + > > + if (ops->acpi_op_add) { > > + if (!acpi_match_device_ids(device, acpi_platform_device_ids)) { > > + /* This is a known good platform device. */ > > + acpi_create_platform_device(device); > > + } else if (device_attach(&device->dev)) { > > device_attach() returns 1 if it succeeds to attach device to a driver. In > that case we should continue and not return AE_CTRL_DEPTH, right? That's correct. The check is reversed, so we need the appended patch on top of linux-pm.git/acpi-scan. Thanks, Rafael --- From: Rafael J. Wysocki Subject: ACPI / scan: Fix check of device_attach() return value. Since device_attach() returns 1 on success and 0 on failure, the check against its return value in acpi_bus_device_attach() should be reveresed. Make it so. Reported-by: Mika Westerberg Signed-off-by: Rafael J. Wysocki --- drivers/acpi/scan.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Index: linux-pm/drivers/acpi/scan.c =================================================================== --- linux-pm.orig/drivers/acpi/scan.c +++ linux-pm/drivers/acpi/scan.c @@ -1598,7 +1598,7 @@ static acpi_status acpi_bus_device_attac if (!acpi_match_device_ids(device, acpi_platform_device_ids)) { /* This is a known good platform device. */ acpi_create_platform_device(device); - } else if (device_attach(&device->dev)) { + } else if (!device_attach(&device->dev)) { status = AE_CTRL_DEPTH; } return status;