This is a 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@xxxxxxxxx> Reviewed-by: Huang Ying <ying.huang@xxxxxxxxx> --- drivers/acpi/acpi_ipmi.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers/acpi/acpi_ipmi.c b/drivers/acpi/acpi_ipmi.c index 7f93ffd..2d31003 100644 --- a/drivers/acpi/acpi_ipmi.c +++ b/drivers/acpi/acpi_ipmi.c @@ -128,6 +128,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 * @@ -583,12 +584,10 @@ out_msg: static int __init acpi_ipmi_init(void) { - int result = 0; + int result; if (acpi_disabled) - return result; - - mutex_init(&driver_data.ipmi_lock); + return 0; result = acpi_register_region(ACPI_ADR_SPACE_IPMI, &acpi_ipmi_space_handler, -- 1.7.10 -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html