diff options
author | Jean Delvare <khali@linux-fr.org> | 2009-01-07 10:37:32 -0500 |
---|---|---|
committer | Jean Delvare <khali@linux-fr.org> | 2009-01-07 10:37:32 -0500 |
commit | 3aed198c35567e5a721f52c0bde23167867e6af6 (patch) | |
tree | 13c17bf66e033c55bff3b8efa8380480118cb3b8 /drivers/hwmon/w83791d.c | |
parent | 6e34b187bc216fc632769fb8b906d3a29ccd8f14 (diff) |
hwmon: Don't overuse I2C_CLIENT_MODULE_PARM
I2C_CLIENT_MODULE_PARM is overkill for force_subclients. We really
only use 4 out of the 48 slots, so we're better defining a custom
variable instead. This change saves 92 bytes of data for each of the
five drivers affected.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Cc: Wolfgang Grandegger <wg@grandegger.com>
Acked-by: Marc Hulsman <m.hulsman@tudelft.nl>
Cc: Mark M. Hoffman <mhoffman@lightlink.com>
Diffstat (limited to 'drivers/hwmon/w83791d.c')
-rw-r--r-- | drivers/hwmon/w83791d.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/drivers/hwmon/w83791d.c b/drivers/hwmon/w83791d.c index 5768def8a4f2..97851c5ba3a3 100644 --- a/drivers/hwmon/w83791d.c +++ b/drivers/hwmon/w83791d.c | |||
@@ -53,7 +53,10 @@ static const unsigned short normal_i2c[] = { 0x2c, 0x2d, 0x2e, 0x2f, | |||
53 | 53 | ||
54 | /* Insmod parameters */ | 54 | /* Insmod parameters */ |
55 | I2C_CLIENT_INSMOD_1(w83791d); | 55 | I2C_CLIENT_INSMOD_1(w83791d); |
56 | I2C_CLIENT_MODULE_PARM(force_subclients, "List of subclient addresses: " | 56 | |
57 | static unsigned short force_subclients[4]; | ||
58 | module_param_array(force_subclients, short, NULL, 0); | ||
59 | MODULE_PARM_DESC(force_subclients, "List of subclient addresses: " | ||
57 | "{bus, clientaddr, subclientaddr1, subclientaddr2}"); | 60 | "{bus, clientaddr, subclientaddr1, subclientaddr2}"); |
58 | 61 | ||
59 | static int reset; | 62 | static int reset; |