Commit 249c720d authored by Joe Perches's avatar Joe Perches Committed by Matthew Garrett

acer-wmi: pr_<level> cleanups

Convert pr_warning to pr_warn.
Add some missing newlines to pr_<level> uses.
Signed-off-by: default avatarJoe Perches <joe@perches.com>
Signed-off-by: default avatarMatthew Garrett <mjg@redhat.com>
parent 3a35125f
...@@ -1081,7 +1081,7 @@ static acpi_status wmid3_get_device_status(u32 *value, u16 device) ...@@ -1081,7 +1081,7 @@ static acpi_status wmid3_get_device_status(u32 *value, u16 device)
return AE_ERROR; return AE_ERROR;
} }
if (obj->buffer.length != 8) { if (obj->buffer.length != 8) {
pr_warning("Unknown buffer length %d\n", obj->buffer.length); pr_warn("Unknown buffer length %d\n", obj->buffer.length);
kfree(obj); kfree(obj);
return AE_ERROR; return AE_ERROR;
} }
...@@ -1090,8 +1090,8 @@ static acpi_status wmid3_get_device_status(u32 *value, u16 device) ...@@ -1090,8 +1090,8 @@ static acpi_status wmid3_get_device_status(u32 *value, u16 device)
kfree(obj); kfree(obj);
if (return_value.error_code || return_value.ec_return_value) if (return_value.error_code || return_value.ec_return_value)
pr_warning("Get Device Status failed: " pr_warn("Get Device Status failed: 0x%x - 0x%x\n",
"0x%x - 0x%x\n", return_value.error_code, return_value.error_code,
return_value.ec_return_value); return_value.ec_return_value);
else else
*value = !!(return_value.devices & device); *value = !!(return_value.devices & device);
...@@ -1317,7 +1317,7 @@ static void acer_wmi_notify(u32 value, void *context) ...@@ -1317,7 +1317,7 @@ static void acer_wmi_notify(u32 value, void *context)
status = wmi_get_event_data(value, &response); status = wmi_get_event_data(value, &response);
if (status != AE_OK) { if (status != AE_OK) {
pr_warning("bad event status 0x%x\n", status); pr_warn("bad event status 0x%x\n", status);
return; return;
} }
...@@ -1326,12 +1326,12 @@ static void acer_wmi_notify(u32 value, void *context) ...@@ -1326,12 +1326,12 @@ static void acer_wmi_notify(u32 value, void *context)
if (!obj) if (!obj)
return; return;
if (obj->type != ACPI_TYPE_BUFFER) { if (obj->type != ACPI_TYPE_BUFFER) {
pr_warning("Unknown response received %d\n", obj->type); pr_warn("Unknown response received %d\n", obj->type);
kfree(obj); kfree(obj);
return; return;
} }
if (obj->buffer.length != 8) { if (obj->buffer.length != 8) {
pr_warning("Unknown buffer length %d\n", obj->buffer.length); pr_warn("Unknown buffer length %d\n", obj->buffer.length);
kfree(obj); kfree(obj);
return; return;
} }
...@@ -1356,11 +1356,11 @@ static void acer_wmi_notify(u32 value, void *context) ...@@ -1356,11 +1356,11 @@ static void acer_wmi_notify(u32 value, void *context)
} }
if (!sparse_keymap_report_event(acer_wmi_input_dev, if (!sparse_keymap_report_event(acer_wmi_input_dev,
return_value.key_num, 1, true)) return_value.key_num, 1, true))
pr_warning("Unknown key number - 0x%x\n", pr_warn("Unknown key number - 0x%x\n",
return_value.key_num); return_value.key_num);
break; break;
default: default:
pr_warning("Unknown function number - %d - %d\n", pr_warn("Unknown function number - %d - %d\n",
return_value.function, return_value.key_num); return_value.function, return_value.key_num);
break; break;
} }
...@@ -1389,7 +1389,7 @@ wmid3_set_lm_mode(struct lm_input_params *params, ...@@ -1389,7 +1389,7 @@ wmid3_set_lm_mode(struct lm_input_params *params,
return AE_ERROR; return AE_ERROR;
} }
if (obj->buffer.length != 4) { if (obj->buffer.length != 4) {
pr_warning("Unknown buffer length %d\n", obj->buffer.length); pr_warn("Unknown buffer length %d\n", obj->buffer.length);
kfree(obj); kfree(obj);
return AE_ERROR; return AE_ERROR;
} }
...@@ -1414,11 +1414,11 @@ static int acer_wmi_enable_ec_raw(void) ...@@ -1414,11 +1414,11 @@ static int acer_wmi_enable_ec_raw(void)
status = wmid3_set_lm_mode(&params, &return_value); status = wmid3_set_lm_mode(&params, &return_value);
if (return_value.error_code || return_value.ec_return_value) if (return_value.error_code || return_value.ec_return_value)
pr_warning("Enabling EC raw mode failed: " pr_warn("Enabling EC raw mode failed: 0x%x - 0x%x\n",
"0x%x - 0x%x\n", return_value.error_code, return_value.error_code,
return_value.ec_return_value); return_value.ec_return_value);
else else
pr_info("Enabled EC raw mode"); pr_info("Enabled EC raw mode\n");
return status; return status;
} }
...@@ -1437,9 +1437,9 @@ static int acer_wmi_enable_lm(void) ...@@ -1437,9 +1437,9 @@ static int acer_wmi_enable_lm(void)
status = wmid3_set_lm_mode(&params, &return_value); status = wmid3_set_lm_mode(&params, &return_value);
if (return_value.error_code || return_value.ec_return_value) if (return_value.error_code || return_value.ec_return_value)
pr_warning("Enabling Launch Manager failed: " pr_warn("Enabling Launch Manager failed: 0x%x - 0x%x\n",
"0x%x - 0x%x\n", return_value.error_code, return_value.error_code,
return_value.ec_return_value); return_value.ec_return_value);
return status; return status;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment