[lm-sensors] [PATCH 2.6] hwmon: tag super-i/o find functions __init

Jean Delvare khali at linux-fr.org
Wed Jul 27 21:32:02 CEST 2005


Hi Greg, all

Super-I/O find functions in hardware monitoring drivers can be tagged
__init as they are only called from functions themselves tagged __init.
Two of them (smsc47b397 and w83627ehf) already do, but the other four
of them (it87, pc87360, smsc47m1 and w83627hf) did not.

This saves a few bytes of memory after the drivers are loaded, 192 in
the case of the it87 driver.

Signed-off-by: Jean Delvare <khali at linux-fr.org>

 drivers/hwmon/it87.c     |    2 +-
 drivers/hwmon/pc87360.c  |    2 +-
 drivers/hwmon/smsc47m1.c |    2 +-
 drivers/hwmon/w83627hf.c |    2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

--- linux-2.6.13-rc3.orig/drivers/hwmon/it87.c	2005-07-24 15:20:08.000000000 +0200
+++ linux-2.6.13-rc3/drivers/hwmon/it87.c	2005-07-24 15:22:39.000000000 +0200
@@ -707,7 +707,7 @@
 }
 
 /* SuperIO detection - will change isa_address if a chip is found */
-static int it87_find(int *address)
+static int __init it87_find(int *address)
 {
 	int err = -ENODEV;
 
--- linux-2.6.13-rc3.orig/drivers/hwmon/pc87360.c	2005-07-24 15:20:08.000000000 +0200
+++ linux-2.6.13-rc3/drivers/hwmon/pc87360.c	2005-07-24 15:22:39.000000000 +0200
@@ -626,7 +626,7 @@
  * Device detection, registration and update
  */
 
-static int pc87360_find(int sioaddr, u8 *devid, unsigned short *addresses)
+static int __init pc87360_find(int sioaddr, u8 *devid, unsigned short *addresses)
 {
 	u16 val;
 	int i;
--- linux-2.6.13-rc3.orig/drivers/hwmon/smsc47m1.c	2005-07-24 15:20:08.000000000 +0200
+++ linux-2.6.13-rc3/drivers/hwmon/smsc47m1.c	2005-07-24 15:22:39.000000000 +0200
@@ -345,7 +345,7 @@
 
 static DEVICE_ATTR(alarms, S_IRUGO, get_alarms, NULL);
 
-static int smsc47m1_find(unsigned short *addr)
+static int __init smsc47m1_find(unsigned short *addr)
 {
 	u8 val;
 
--- linux-2.6.13-rc3.orig/drivers/hwmon/w83627hf.c	2005-07-24 15:20:08.000000000 +0200
+++ linux-2.6.13-rc3/drivers/hwmon/w83627hf.c	2005-07-24 15:22:39.000000000 +0200
@@ -958,7 +958,7 @@
 } while (0)
 
 
-static int w83627hf_find(int sioaddr, unsigned short *addr)
+static int __init w83627hf_find(int sioaddr, unsigned short *addr)
 {
 	u16 val;
 


-- 
Jean Delvare




More information about the lm-sensors mailing list