On Fri, Jan 11, 2013 at 09:31:43PM +0100, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx> > 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. Not sure if it matters but it returns 0 if no device was bound to a driver and -ENODEV in case of error. If we only want to terminate in case of error, following might be better. } else if (device_attach(&device->dev) < 0) { -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html