Commit 9f6d4b0c authored by Ben Collins's avatar Ben Collins Committed by Linus Torvalds

[PATCH] therm_adt746x: Quiet fan speed change messages

Only output the messages about fan speed changes with a verbose=1 module
param.
Signed-off-by: default avatarFabio M. Di Nitto <fabbione@ubuntu.com>
Signed-off-by: default avatarBen Collins <bcollins@ubuntu.com>
Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent 683e2cc6
...@@ -52,6 +52,7 @@ static char *sensor_location[3] = {NULL, NULL, NULL}; ...@@ -52,6 +52,7 @@ static char *sensor_location[3] = {NULL, NULL, NULL};
static int limit_adjust = 0; static int limit_adjust = 0;
static int fan_speed = -1; static int fan_speed = -1;
static int verbose = 0;
MODULE_AUTHOR("Colin Leroy <colin@colino.net>"); MODULE_AUTHOR("Colin Leroy <colin@colino.net>");
MODULE_DESCRIPTION("Driver for ADT746x thermostat in iBook G4 and " MODULE_DESCRIPTION("Driver for ADT746x thermostat in iBook G4 and "
...@@ -66,6 +67,10 @@ module_param(fan_speed, int, 0644); ...@@ -66,6 +67,10 @@ module_param(fan_speed, int, 0644);
MODULE_PARM_DESC(fan_speed,"Specify starting fan speed (0-255) " MODULE_PARM_DESC(fan_speed,"Specify starting fan speed (0-255) "
"(default 64)"); "(default 64)");
module_param(verbose, bool, 0);
MODULE_PARM_DESC(verbose,"Verbose log operations "
"(default 0)");
struct thermostat { struct thermostat {
struct i2c_client clt; struct i2c_client clt;
u8 temps[3]; u8 temps[3];
...@@ -149,13 +154,13 @@ detach_thermostat(struct i2c_adapter *adapter) ...@@ -149,13 +154,13 @@ detach_thermostat(struct i2c_adapter *adapter)
if (thread_therm != NULL) { if (thread_therm != NULL) {
kthread_stop(thread_therm); kthread_stop(thread_therm);
} }
printk(KERN_INFO "adt746x: Putting max temperatures back from " printk(KERN_INFO "adt746x: Putting max temperatures back from "
"%d, %d, %d to %d, %d, %d\n", "%d, %d, %d to %d, %d, %d\n",
th->limits[0], th->limits[1], th->limits[2], th->limits[0], th->limits[1], th->limits[2],
th->initial_limits[0], th->initial_limits[1], th->initial_limits[0], th->initial_limits[1],
th->initial_limits[2]); th->initial_limits[2]);
for (i = 0; i < 3; i++) for (i = 0; i < 3; i++)
write_reg(th, LIMIT_REG[i], th->initial_limits[i]); write_reg(th, LIMIT_REG[i], th->initial_limits[i]);
...@@ -212,12 +217,14 @@ static void write_fan_speed(struct thermostat *th, int speed, int fan) ...@@ -212,12 +217,14 @@ static void write_fan_speed(struct thermostat *th, int speed, int fan)
return; return;
if (th->last_speed[fan] != speed) { if (th->last_speed[fan] != speed) {
if (speed == -1) if (verbose) {
printk(KERN_DEBUG "adt746x: Setting speed to automatic " if (speed == -1)
"for %s fan.\n", sensor_location[fan+1]); printk(KERN_DEBUG "adt746x: Setting speed to automatic "
else "for %s fan.\n", sensor_location[fan+1]);
printk(KERN_DEBUG "adt746x: Setting speed to %d " else
"for %s fan.\n", speed, sensor_location[fan+1]); printk(KERN_DEBUG "adt746x: Setting speed to %d "
"for %s fan.\n", speed, sensor_location[fan+1]);
}
} else } else
return; return;
...@@ -298,10 +305,11 @@ static void update_fans_speed (struct thermostat *th) ...@@ -298,10 +305,11 @@ static void update_fans_speed (struct thermostat *th)
if (new_speed > 255) if (new_speed > 255)
new_speed = 255; new_speed = 255;
printk(KERN_DEBUG "adt746x: setting fans speed to %d " if (verbose)
"(limit exceeded by %d on %s) \n", printk(KERN_DEBUG "adt746x: Setting fans speed to %d "
new_speed, var, "(limit exceeded by %d on %s) \n",
sensor_location[fan_number+1]); new_speed, var,
sensor_location[fan_number+1]);
write_both_fan_speed(th, new_speed); write_both_fan_speed(th, new_speed);
th->last_var[fan_number] = var; th->last_var[fan_number] = var;
} else if (var < -2) { } else if (var < -2) {
...@@ -309,8 +317,9 @@ static void update_fans_speed (struct thermostat *th) ...@@ -309,8 +317,9 @@ static void update_fans_speed (struct thermostat *th)
* so cold (lastvar >= -1) */ * so cold (lastvar >= -1) */
if (i == 2 && lastvar < -1) { if (i == 2 && lastvar < -1) {
if (th->last_speed[fan_number] != 0) if (th->last_speed[fan_number] != 0)
printk(KERN_DEBUG "adt746x: Stopping " if (verbose)
"fans.\n"); printk(KERN_DEBUG "adt746x: Stopping "
"fans.\n");
write_both_fan_speed(th, 0); write_both_fan_speed(th, 0);
} }
} }
...@@ -406,7 +415,7 @@ static int attach_one_thermostat(struct i2c_adapter *adapter, int addr, ...@@ -406,7 +415,7 @@ static int attach_one_thermostat(struct i2c_adapter *adapter, int addr,
th->initial_limits[i] = read_reg(th, LIMIT_REG[i]); th->initial_limits[i] = read_reg(th, LIMIT_REG[i]);
set_limit(th, i); set_limit(th, i);
} }
printk(KERN_INFO "adt746x: Lowering max temperatures from %d, %d, %d" printk(KERN_INFO "adt746x: Lowering max temperatures from %d, %d, %d"
" to %d, %d, %d\n", " to %d, %d, %d\n",
th->initial_limits[0], th->initial_limits[1], th->initial_limits[0], th->initial_limits[1],
......
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