ACPI / IPMI: Cleanup some initialization codes
authorLv Zheng <lv.zheng@intel.com>
Fri, 13 Sep 2013 05:14:31 +0000 (13:14 +0800)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Mon, 30 Sep 2013 17:46:12 +0000 (19:46 +0200)
This (trivial) patch.
 1. Changes dynamic mutex initialization to static initialization.
 2. Removes one acpi_ipmi_init() variable initialization as it is not
    needed.

Signed-off-by: Lv Zheng <lv.zheng@intel.com>
Reviewed-by: Huang Ying <ying.huang@intel.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/acpi/acpi_ipmi.c

index 90d57c8..f7b6598 100644 (file)
@@ -127,6 +127,7 @@ static struct ipmi_driver_data driver_data = {
        .ipmi_hndlrs = {
                .ipmi_recv_hndl = ipmi_msg_handler,
        },
+       .ipmi_lock = __MUTEX_INITIALIZER(driver_data.ipmi_lock)
 };
 
 static struct acpi_ipmi_device *
@@ -591,13 +592,11 @@ out_msg:
 
 static int __init acpi_ipmi_init(void)
 {
-       int result = 0;
+       int result;
        acpi_status status;
 
        if (acpi_disabled)
-               return result;
-
-       mutex_init(&driver_data.ipmi_lock);
+               return 0;
 
        status = acpi_install_address_space_handler(ACPI_ROOT_OBJECT,
                                ACPI_ADR_SPACE_IPMI, &acpi_ipmi_space_handler,