Skip to contentSkip to conversation

Fix bugs in the new w83627ehf driver

PATCH New Jean Delvare 2 participants 2 messages May 19, 2005

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 Fix bugs in the new w83627ehf driver

Hi Greg,

These are the fixes for the bugs you spotted in my new w83627ehf driver:
1* w83627ehf_detect should be declared static.
2* Explicit division by 0.

I'm very sorry for introducing them in the first place, especially the second one. I'm glad you spotted them, thanks a lot.

Index: linux-2.6.12-rc3/drivers/i2c/chips/w83627ehf.c
=================================--- linux-2.6.12-rc3.orig/drivers/i2c/chips/w83627ehf.c 2005-04-25 22:36:11.000000000 +0200
+++ linux-2.6.12-rc3/drivers/i2c/chips/w83627ehf.c 2005-05-02 23:02:07.000000000 +0200
@@ -450,7 +450,7 @@
data->fan_min[nr] = 1;
new_div = 0; /* 1 = (1 << 0) */
dev_warn(dev, "fan%u low limit %u above maximum %u, set to "
- "maximum\n", nr + 1, val, fan_from_reg(1, 0));
+ "maximum\n", nr + 1, val, fan_from_reg(1, 1));
} else {
/* Automatically pick the best divider, i.e. the one such
that the min limit will correspond to a register value
@@ -659,7 +659,7 @@
}
}

-int w83627ehf_detect(struct i2c_adapter *adapter, int address, int kind) +static int w83627ehf_detect(struct i2c_adapter *adapter, int address, int kind)
{
struct i2c_client *client;
struct w83627ehf_data *data;

Signature · 1 line
Jean Delvare
Applied
#1 lore raw

PATCH 1/1 Fix bugs in the new w83627ehf driver

Quoted 10 lines · Jean Delvare

Hi Greg,

These are the fixes for the bugs you spotted in my new w83627ehf driver:
1* w83627ehf_detect should be declared static.
2* Explicit division by 0.

I'm very sorry for introducing them in the first place, especially the second one. I'm glad you spotted them, thanks a lot.

Applied, thanks.

greg k-h