Commit 2ee9a0db authored by Kefeng Wang's avatar Kefeng Wang Committed by Petr Mladek

macintosh: Use pr_warn instead of pr_warning

As said in commit f2c2cbcc ("powerpc: Use pr_warn instead of
pr_warning"), removing pr_warning so all logging messages use a
consistent <prefix>_warn style. Let's do it.

Link: http://lkml.kernel.org/r/20191018031850.48498-15-wangkefeng.wang@huawei.com
To: linux-kernel@vger.kernel.org
Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Cc: linuxppc-dev@lists.ozlabs.org
Signed-off-by: default avatarKefeng Wang <wangkefeng.wang@huawei.com>
Reviewed-by: default avatarSergey Senozhatsky <sergey.senozhatsky@gmail.com>
Signed-off-by: default avatarPetr Mladek <pmladek@suse.com>
parent 257daba4
...@@ -468,9 +468,7 @@ static void wf_fcu_lookup_fans(struct wf_fcu_priv *pv) ...@@ -468,9 +468,7 @@ static void wf_fcu_lookup_fans(struct wf_fcu_priv *pv)
else else
id = ((*reg) - 0x30) / 2; id = ((*reg) - 0x30) / 2;
if (id > 7) { if (id > 7) {
pr_warning("wf_fcu: Can't parse " pr_warn("wf_fcu: Can't parse fan ID in device-tree for %pOF\n", np);
"fan ID in device-tree for %pOF\n",
np);
break; break;
} }
wf_fcu_add_fan(pv, name, type, id); wf_fcu_add_fan(pv, name, type, id);
......
...@@ -124,8 +124,8 @@ static int wf_lm87_probe(struct i2c_client *client, ...@@ -124,8 +124,8 @@ static int wf_lm87_probe(struct i2c_client *client,
} }
} }
if (!name) { if (!name) {
pr_warning("wf_lm87: Unsupported sensor %pOF\n", pr_warn("wf_lm87: Unsupported sensor %pOF\n",
client->dev.of_node); client->dev.of_node);
return -ENODEV; return -ENODEV;
} }
......
...@@ -285,8 +285,8 @@ static void cpu_fans_tick_split(void) ...@@ -285,8 +285,8 @@ static void cpu_fans_tick_split(void)
/* Apply result directly to exhaust fan */ /* Apply result directly to exhaust fan */
err = wf_control_set(cpu_rear_fans[cpu], sp->target); err = wf_control_set(cpu_rear_fans[cpu], sp->target);
if (err) { if (err) {
pr_warning("wf_pm72: Fan %s reports error %d\n", pr_warn("wf_pm72: Fan %s reports error %d\n",
cpu_rear_fans[cpu]->name, err); cpu_rear_fans[cpu]->name, err);
failure_state |= FAILURE_FAN; failure_state |= FAILURE_FAN;
break; break;
} }
...@@ -296,8 +296,8 @@ static void cpu_fans_tick_split(void) ...@@ -296,8 +296,8 @@ static void cpu_fans_tick_split(void)
DBG_LOTS(" CPU%d: intake = %d RPM\n", cpu, intake); DBG_LOTS(" CPU%d: intake = %d RPM\n", cpu, intake);
err = wf_control_set(cpu_front_fans[cpu], intake); err = wf_control_set(cpu_front_fans[cpu], intake);
if (err) { if (err) {
pr_warning("wf_pm72: Fan %s reports error %d\n", pr_warn("wf_pm72: Fan %s reports error %d\n",
cpu_front_fans[cpu]->name, err); cpu_front_fans[cpu]->name, err);
failure_state |= FAILURE_FAN; failure_state |= FAILURE_FAN;
break; break;
} }
...@@ -367,22 +367,22 @@ static void cpu_fans_tick_combined(void) ...@@ -367,22 +367,22 @@ static void cpu_fans_tick_combined(void)
for (cpu = 0; cpu < nr_chips; cpu++) { for (cpu = 0; cpu < nr_chips; cpu++) {
err = wf_control_set(cpu_rear_fans[cpu], sp->target); err = wf_control_set(cpu_rear_fans[cpu], sp->target);
if (err) { if (err) {
pr_warning("wf_pm72: Fan %s reports error %d\n", pr_warn("wf_pm72: Fan %s reports error %d\n",
cpu_rear_fans[cpu]->name, err); cpu_rear_fans[cpu]->name, err);
failure_state |= FAILURE_FAN; failure_state |= FAILURE_FAN;
} }
err = wf_control_set(cpu_front_fans[cpu], intake); err = wf_control_set(cpu_front_fans[cpu], intake);
if (err) { if (err) {
pr_warning("wf_pm72: Fan %s reports error %d\n", pr_warn("wf_pm72: Fan %s reports error %d\n",
cpu_front_fans[cpu]->name, err); cpu_front_fans[cpu]->name, err);
failure_state |= FAILURE_FAN; failure_state |= FAILURE_FAN;
} }
err = 0; err = 0;
if (cpu_pumps[cpu]) if (cpu_pumps[cpu])
err = wf_control_set(cpu_pumps[cpu], pump); err = wf_control_set(cpu_pumps[cpu], pump);
if (err) { if (err) {
pr_warning("wf_pm72: Pump %s reports error %d\n", pr_warn("wf_pm72: Pump %s reports error %d\n",
cpu_pumps[cpu]->name, err); cpu_pumps[cpu]->name, err);
failure_state |= FAILURE_FAN; failure_state |= FAILURE_FAN;
} }
} }
...@@ -561,7 +561,7 @@ static void drives_fan_tick(void) ...@@ -561,7 +561,7 @@ static void drives_fan_tick(void)
err = wf_sensor_get(drives_temp, &temp); err = wf_sensor_get(drives_temp, &temp);
if (err) { if (err) {
pr_warning("wf_pm72: drive bay temp sensor error %d\n", err); pr_warn("wf_pm72: drive bay temp sensor error %d\n", err);
failure_state |= FAILURE_SENSOR; failure_state |= FAILURE_SENSOR;
wf_control_set_max(drives_fan); wf_control_set_max(drives_fan);
return; return;
......
...@@ -281,8 +281,8 @@ static void cpu_fans_tick(void) ...@@ -281,8 +281,8 @@ static void cpu_fans_tick(void)
for (i = 0; i < 3; i++) { for (i = 0; i < 3; i++) {
err = wf_control_set(cpu_fans[cpu][i], speed); err = wf_control_set(cpu_fans[cpu][i], speed);
if (err) { if (err) {
pr_warning("wf_rm31: Fan %s reports error %d\n", pr_warn("wf_rm31: Fan %s reports error %d\n",
cpu_fans[cpu][i]->name, err); cpu_fans[cpu][i]->name, err);
failure_state |= FAILURE_FAN; failure_state |= FAILURE_FAN;
} }
} }
...@@ -465,7 +465,7 @@ static void slots_fan_tick(void) ...@@ -465,7 +465,7 @@ static void slots_fan_tick(void)
err = wf_sensor_get(slots_temp, &temp); err = wf_sensor_get(slots_temp, &temp);
if (err) { if (err) {
pr_warning("wf_rm31: slots temp sensor error %d\n", err); pr_warn("wf_rm31: slots temp sensor error %d\n", err);
failure_state |= FAILURE_SENSOR; failure_state |= FAILURE_SENSOR;
wf_control_set_max(slots_fan); wf_control_set_max(slots_fan);
return; return;
......
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