Commit 918ee91c authored by Jean Delvare's avatar Jean Delvare Committed by Jean Delvare
Browse files

hwmon: I2C addresses are constant 



We can mark normal_i2c const. Almost all drivers do that already, so
fix the 3 remaining ones before they are used as (bad) examples for
new drivers.

Signed-off-by: default avatarJean Delvare <khali@linux-fr.org>
Cc: George Joseph <george.joseph@fairview5.com>
Reviewed-by: default avatarGuenter Roeck <guenter.roeck@ericsson.com>
parent bd5f47ec

drivers/hwmon/adt7475.c

+1 −1
Original line number Diff line number Diff line
@@ -146,7 +146,7 @@ 
#define TEMP_OFFSET_REG(idx) (REG_TEMP_OFFSET_BASE + (idx))

#define TEMP_TRANGE_REG(idx) (REG_TEMP_TRANGE_BASE + (idx))



static unsigned short normal_i2c[] = { 0x2c, 0x2d, 0x2e, I2C_CLIENT_END };

static const unsigned short normal_i2c[] = { 0x2c, 0x2d, 0x2e, I2C_CLIENT_END };



enum chips { adt7473, adt7475, adt7476, adt7490 };

drivers/hwmon/asc7621.c

+2 −2
Original line number Diff line number Diff line
@@ -28,7 +28,7 @@ 
#include <linux/mutex.h>



/* Addresses to scan */

static unsigned short normal_i2c[] = {

static const unsigned short normal_i2c[] = {

	0x2c, 0x2d, 0x2e, I2C_CLIENT_END

};
@@ -52,7 +52,7 @@ struct asc7621_chip { 
	u8 company_id;

	u8 verstep_reg;

	u8 verstep_id;

	unsigned short *addresses;

	const unsigned short *addresses;

};



static struct asc7621_chip asc7621_chips[] = {

drivers/hwmon/tmp421.c

+2 −2
Original line number Diff line number Diff line
@@ -36,7 +36,7 @@ 
#include <linux/sysfs.h>



/* Addresses to scan */

static unsigned short normal_i2c[] = { 0x2a, 0x4c, 0x4d, 0x4e, 0x4f,

static const unsigned short normal_i2c[] = { 0x2a, 0x4c, 0x4d, 0x4e, 0x4f,

					     I2C_CLIENT_END };



enum chips { tmp421, tmp422, tmp423 };

CRA Git | Maintained and supported by SUSTech CRA and CCSE