linux-toradex.git/drivers/hwmon/lm87.c, branch v3.0.48 Linux kernel for Apalis and Colibri modules i2c: Drop I2C_CLIENT_INSMOD_2 to 8 2009-12-14T20:17:27+00:00 Jean Delvare khali@linux-fr.org 2009-12-14T20:17:27+00:00 e5e9f44c246fbafe723e579e9fe887677beb40e4 These macros simply declare an enum, so drivers might as well declare it themselves. This puts an end to the arbitrary limit of 8 chip types per i2c driver. Signed-off-by: Jean Delvare <khali@linux-fr.org> Tested-by: Wolfram Sang <w.sang@pengutronix.de> 
These macros simply declare an enum, so drivers might as well declare
it themselves. This puts an end to the arbitrary limit of 8 chip types
per i2c driver.

Signed-off-by: Jean Delvare <khali@linux-fr.org>
Tested-by: Wolfram Sang <w.sang@pengutronix.de>
i2c: Get rid of struct i2c_client_address_data 2009-12-14T20:17:25+00:00 Jean Delvare khali@linux-fr.org 2009-12-14T20:17:25+00:00 c3813d6af177fab19e322f3114b1f64fbcf08d71 Struct i2c_client_address_data only contains one field at this point, which makes its usefulness questionable. Get rid of it and pass simple address lists around instead. Signed-off-by: Jean Delvare <khali@linux-fr.org> Tested-by: Wolfram Sang <w.sang@pengutronix.de> 
Struct i2c_client_address_data only contains one field at this point,
which makes its usefulness questionable. Get rid of it and pass simple
address lists around instead.

Signed-off-by: Jean Delvare <khali@linux-fr.org>
Tested-by: Wolfram Sang <w.sang@pengutronix.de>
i2c: Drop the kind parameter from detect callbacks 2009-12-14T20:17:23+00:00 Jean Delvare khali@linux-fr.org 2009-12-14T20:17:23+00:00 310ec79210d754afe51e2e4a983e846b60179abd The "kind" parameter always has value -1, and nobody is using it any longer, so we can remove it. Signed-off-by: Jean Delvare <khali@linux-fr.org> Tested-by: Wolfram Sang <w.sang@pengutronix.de> 
The "kind" parameter always has value -1, and nobody is using it any
longer, so we can remove it.

Signed-off-by: Jean Delvare <khali@linux-fr.org>
Tested-by: Wolfram Sang <w.sang@pengutronix.de>
hwmon: Clean up detect functions 2009-12-09T19:35:57+00:00 Jean Delvare khali@linux-fr.org 2009-12-09T19:35:57+00:00 52df6440a29123eed912183fe785bbe174ef14b9 As kind is now hard-coded to -1, there is room for code clean-ups. Signed-off-by: Jean Delvare <khali@linux-fr.org> Acked-by: Corentin Labbe <corentin.labbe@geomatys.fr> Cc: "Mark M. Hoffman" <mhoffman@lightlink.com> Cc: Juerg Haefliger <juergh@gmail.com> Cc: Riku Voipio <riku.voipio@iki.fi> Acked-by: "Hans J. Koch" <hjk@linutronix.de> Cc: Rudolf Marek <r.marek@assembler.cz> 
As kind is now hard-coded to -1, there is room for code clean-ups.

Signed-off-by: Jean Delvare <khali@linux-fr.org>
Acked-by: Corentin Labbe <corentin.labbe@geomatys.fr>
Cc: "Mark M. Hoffman" <mhoffman@lightlink.com>
Cc: Juerg Haefliger <juergh@gmail.com>
Cc: Riku Voipio <riku.voipio@iki.fi>
Acked-by: "Hans J. Koch" <hjk@linutronix.de>
Cc: Rudolf Marek <r.marek@assembler.cz>
hwmon: (lm87) Add support for configuration through platform_data 2008-10-17T15:51:12+00:00 Ben Hutchings bhutchings@solarflare.com 2008-10-17T15:51:12+00:00 47064d645bc55863c7887a7c96cde39c9a37ee5f The lm87 driver normally assumes that firmware configured the chip correctly. Since this is not always the case, alllow platform code to set the channel register value via platform_data. All other configuration registers can be changed after driver initialisation. Signed-off-by: Ben Hutchings <bhutchings@solarflare.com> Signed-off-by: Jean Delvare <khali@linux-fr.org> 
The lm87 driver normally assumes that firmware configured the chip
correctly.  Since this is not always the case, alllow platform code to
set the channel register value via platform_data.  All other
configuration registers can be changed after driver initialisation.

Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
Signed-off-by: Jean Delvare <khali@linux-fr.org>
hwmon: (lm87) Restore original configuration register on removal 2008-10-17T15:51:11+00:00 Ben Hutchings bhutchings@solarflare.com 2008-10-17T15:51:11+00:00 d2cac802feae6f0c246a9251eefc482bf5ec0f0f This means that if we have to start the monitor when probed, we also stop it on removal. Signed-off-by: Ben Hutchings <bhutchings@solarflare.com> Signed-off-by: Jean Delvare <khali@linux-fr.org> 
This means that if we have to start the monitor when probed, we also
stop it on removal.

Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
Signed-off-by: Jean Delvare <khali@linux-fr.org>
hwmon: (lm87) Fix masking of config register in lm87_init_client() 2008-10-17T15:51:11+00:00 Ben Hutchings bhutchings@solarflare.com 2008-10-17T15:51:11+00:00 49ae6cc8005f45edadf237089c31885834f2608e lm87_init_client() conditionally sets the Start bit and clears the INT#_Clear bit in the Config 1 register. The condition should be that either of these bits needs changing, but currently it checks the (self-clearing) Initialization bit instead of INT#_Clear. Fix the condition and also ensure we never set the Initialization bit. Signed-off-by: Ben Hutchings <bhutchings@solarflare.com> Signed-off-by: Jean Delvare <khali@linux-fr.org> 
lm87_init_client() conditionally sets the Start bit and clears the
INT#_Clear bit in the Config 1 register.  The condition should be that
either of these bits needs changing, but currently it checks the
(self-clearing) Initialization bit instead of INT#_Clear.

Fix the condition and also ensure we never set the Initialization bit.

Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
Signed-off-by: Jean Delvare <khali@linux-fr.org>
hwmon: (lm87) Convert to a new-style i2c driver 2008-07-16T17:30:14+00:00 Jean Delvare khali@linux-fr.org 2008-07-16T17:30:14+00:00 a888420af07b9a26b6d518328baa913fb704e950 The new-style lm87 driver implements the optional detect() callback to cover the use cases of the legacy driver. Signed-off-by: Jean Delvare <khali@linux-fr.org> Cc: Ben Hutchings <bhutchings@solarflare.com> 
The new-style lm87 driver implements the optional detect() callback
to cover the use cases of the legacy driver.

Signed-off-by: Jean Delvare <khali@linux-fr.org>
Cc: Ben Hutchings <bhutchings@solarflare.com>
hwmon: normal_i2c arrays should be const 2008-02-19T02:58:15+00:00 Mark M. Hoffman mhoffman@lightlink.com 2008-02-18T03:28:03+00:00 25e9c86d5a6d82ea45eb680fc66bf73ac5e50dff Signed-off-by: Mark M. Hoffman <mhoffman@lightlink.com> 
Signed-off-by: Mark M. Hoffman <mhoffman@lightlink.com>
hwmon: VRM is not written to registers 2008-02-08T01:39:44+00:00 Jean Delvare khali@linux-fr.org 2007-12-01T10:25:33+00:00 8f74efe81d122c071410fd74f42879ef81439fa4 What was true of reading the VRM value is also true of writing it: not being a register value, it doesn't need hardware access, so we don't need a reference to the i2c client. This allows for a minor code cleanup. As gcc appears to be smart enough to simplify the generated code by itself, this cleanup only affects the source code, the generated binaries are unchanged. Signed-off-by: Jean Delvare <khali@linux-fr.org> Signed-off-by: Mark M. Hoffman <mhoffman@lightlink.com> 
What was true of reading the VRM value is also true of writing it: not
being a register value, it doesn't need hardware access, so we don't
need a reference to the i2c client. This allows for a minor code
cleanup. As gcc appears to be smart enough to simplify the generated
code by itself, this cleanup only affects the source code, the
generated binaries are unchanged.

Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Mark M. Hoffman <mhoffman@lightlink.com>