Message ID | 3392385.iIbC2pHGDl@kreacher (mailing list archive) |
---|---|
State | Mainlined, archived |
Headers | show |
Series | ACPI: scan: Fixes and cleanups related to dependencies list handling | expand |
Hi, On 6/16/21 4:24 PM, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki <rafael.j.wysocki@intel.com> > > Move the invocation of acpi_attach_data() in acpi_device_add() > into a separate function. > > No intentional functional impact. > > Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> Thanks, patch looks good to me: Reviewed-by: Hans de Goede <hdegoede@redhat.com> Regards, Hans > --- > drivers/acpi/scan.c | 31 ++++++++++++++++++++----------- > 1 file changed, 20 insertions(+), 11 deletions(-) > > Index: linux-pm/drivers/acpi/scan.c > =================================================================== > --- linux-pm.orig/drivers/acpi/scan.c > +++ linux-pm/drivers/acpi/scan.c > @@ -640,23 +640,32 @@ static int acpi_device_set_name(struct a > return 0; > } > > +static int acpi_tie_acpi_dev(struct acpi_device *adev) > +{ > + acpi_handle handle = adev->handle; > + acpi_status status; > + > + if (!handle) > + return 0; > + > + status = acpi_attach_data(handle, acpi_scan_drop_device, adev); > + if (ACPI_FAILURE(status)) { > + acpi_handle_err(handle, "Unable to attach device data\n"); > + return -ENODEV; > + } > + > + return 0; > +} > + > int acpi_device_add(struct acpi_device *device, > void (*release)(struct device *)) > { > struct acpi_device_bus_id *acpi_device_bus_id; > int result; > > - if (device->handle) { > - acpi_status status; > - > - status = acpi_attach_data(device->handle, acpi_scan_drop_device, > - device); > - if (ACPI_FAILURE(status)) { > - acpi_handle_err(device->handle, > - "Unable to attach device data\n"); > - return -ENODEV; > - } > - } > + result = acpi_tie_acpi_dev(device); > + if (result) > + return result; > > /* > * Linkage > > >
Index: linux-pm/drivers/acpi/scan.c =================================================================== --- linux-pm.orig/drivers/acpi/scan.c +++ linux-pm/drivers/acpi/scan.c @@ -640,23 +640,32 @@ static int acpi_device_set_name(struct a return 0; } +static int acpi_tie_acpi_dev(struct acpi_device *adev) +{ + acpi_handle handle = adev->handle; + acpi_status status; + + if (!handle) + return 0; + + status = acpi_attach_data(handle, acpi_scan_drop_device, adev); + if (ACPI_FAILURE(status)) { + acpi_handle_err(handle, "Unable to attach device data\n"); + return -ENODEV; + } + + return 0; +} + int acpi_device_add(struct acpi_device *device, void (*release)(struct device *)) { struct acpi_device_bus_id *acpi_device_bus_id; int result; - if (device->handle) { - acpi_status status; - - status = acpi_attach_data(device->handle, acpi_scan_drop_device, - device); - if (ACPI_FAILURE(status)) { - acpi_handle_err(device->handle, - "Unable to attach device data\n"); - return -ENODEV; - } - } + result = acpi_tie_acpi_dev(device); + if (result) + return result; /* * Linkage