Skip to contentSkip to conversation

hwmon: tag super-i/o find functions __init

PATCH New Jean Delvare 1 participant 1 message Jul 27, 2005

Subsystems (3)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 hwmon: tag super-i/o find functions __init

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.

 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(-)
drivers/hwmon/it87.c +1 -1
@@ -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;
 
drivers/hwmon/pc87360.c +1 -1
@@ -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;
drivers/hwmon/smsc47m1.c +1 -1
@@ -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;
 
drivers/hwmon/w83627hf.c +1 -1
@@ -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;
 
Signature · 1 line
Jean Delvare