lkml.org 
[lkml]   [2013]   [Jan]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 2/2] ACPI / scan: Clean up acpi_bus_get_parent()
2013/01/31 7:04, Rafael J. Wysocki wrote:
> From: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
>
> Make acpi_bus_get_parent() more straightforward and remove an
> unnecessary local variable ret from it.
>
> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
> ---

Acked-by: Yasuaki Ishimatsu <isimatu.yasuaki@jp.fujitsu.com>

> drivers/acpi/scan.c | 16 +++++-----------
> 1 file changed, 5 insertions(+), 11 deletions(-)
>
> Index: test/drivers/acpi/scan.c
> ===================================================================
> --- test.orig/drivers/acpi/scan.c
> +++ test/drivers/acpi/scan.c
> @@ -871,29 +871,23 @@ EXPORT_SYMBOL(acpi_bus_unregister_driver
> -------------------------------------------------------------------------- */
> static struct acpi_device *acpi_bus_get_parent(acpi_handle handle)
> {
> + struct acpi_device *device = NULL;
> acpi_status status;
> - int ret;
> - struct acpi_device *device;
>
> /*
> * Fixed hardware devices do not appear in the namespace and do not
> * have handles, but we fabricate acpi_devices for them, so we have
> * to deal with them specially.
> */
> - if (handle == NULL)
> + if (!handle)
> return acpi_root;
>
> do {
> status = acpi_get_parent(handle, &handle);
> - if (status == AE_NULL_ENTRY)
> - return NULL;
> if (ACPI_FAILURE(status))
> - return acpi_root;
> -
> - ret = acpi_bus_get_device(handle, &device);
> - if (ret == 0)
> - return device;
> - } while (1);
> + return status == AE_NULL_ENTRY ? NULL : acpi_root;
> + } while (acpi_bus_get_device(handle, &device));
> + return device;
> }
>
> acpi_status
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>




\
 
 \ /
  Last update: 2013-01-31 05:21    [W:0.110 / U:0.908 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site