linux-toradex.git/drivers/hwmon/gl520sm.c, branch v3.2.56

hwmon: Use i2c_smbus_{read,write}_word_swapped

2011-11-04T11:00:47+00:00

Make use of the new i2c_smbus_{read,write}_word_swapped functions.
This makes the driver code more compact and readable. It also ensures
proper error handling.

Signed-off-by: Jean Delvare 
Acked-by: Jonathan Cameron 
Acked-by: Guenter Roeck 
Cc: Dirk Eibach 
Cc: "Mark M. Hoffman" 
Cc: Guillaume Ligneul

treewide: fix potentially dangerous trailing ';' in #defined values/expressions

2011-07-21T12:10:00+00:00

All these are instances of
  #define NAME value;
or
  #define NAME(params_opt) value;

These of course fail to build when used in contexts like
  if(foo $OP NAME)
  while(bar $OP NAME)
and may silently generate the wrong code in contexts such as
  foo = NAME + 1;    /* foo = value; + 1; */
  bar = NAME - 1;    /* bar = value; - 1; */
  baz = NAME & quux; /* baz = value; & quux; */

Reported on comp.lang.c,
Message-ID: 
Initial analysis of the dangers provided by Keith Thompson in that thread.

There are many more instances of more complicated macros having unnecessary
trailing semicolons, but this pile seems to be all of the cases of simple
values suffering from the problem. (Thus things that are likely to be found
in one of the contexts above, more complicated ones aren't.)

Signed-off-by: Phil Carmody 
Signed-off-by: Jiri Kosina

i2c: Drop I2C_CLIENT_INSMOD_1

2009-12-14T20:17:26+00:00

This macro simply declares an enum, so drivers might as well declare
it themselves.

Signed-off-by: Jean Delvare 
Tested-by: Wolfram Sang

i2c: Get rid of struct i2c_client_address_data

2009-12-14T20:17:25+00:00

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 
Tested-by: Wolfram Sang

i2c: Drop the kind parameter from detect callbacks

2009-12-14T20:17:23+00:00

The "kind" parameter always has value -1, and nobody is using it any
longer, so we can remove it.

Signed-off-by: Jean Delvare 
Tested-by: Wolfram Sang

hwmon: Clean up detect functions

2009-12-09T19:35:57+00:00

As kind is now hard-coded to -1, there is room for code clean-ups.

Signed-off-by: Jean Delvare 
Acked-by: Corentin Labbe 
Cc: "Mark M. Hoffman" 
Cc: Juerg Haefliger 
Cc: Riku Voipio 
Acked-by: "Hans J. Koch" 
Cc: Rudolf Marek

hwmon: (gl520sm) Convert to a new-style i2c driver

2008-07-16T17:30:13+00:00

The new-style gl520sm driver implements the optional detect() callback
to cover the use cases of the legacy driver.

Signed-off-by: Jean Delvare 
Cc: Maarten Deprez

hwmon: normal_i2c arrays should be const

2008-02-19T02:58:15+00:00

Signed-off-by: Mark M. Hoffman

hwmon: (gl520sm) Add individual alarm and beep files

2008-02-08T01:39:42+00:00

libsensors 3.0 needs these.

Signed-off-by: Jean Delvare 
Signed-off-by: Mark M. Hoffman

hwmon: (gl520sm) De-macro the sysfs callbacks

2008-02-08T01:39:42+00:00

Use standard dynamic sysfs callbacks instead of macro-generated
wrappers. This makes the code more readable, and the binary smaller
(by about 11%).

Signed-off-by: Jean Delvare 
Signed-off-by: Mark M. Hoffman