Skip to contentSkip to conversation

sensors-detect: Fix address ranges checked for

PATCH In review Guenter Roeck 2 participants 2 messages Feb 9, 2011

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 sensors-detect: Fix address ranges checked for

Update I2C address ranges for SMSC EMC1403, EMC1404, and EMC1423 to match information in datasheets.

--
Index: prog/detect/sensors-detect
=================================--- prog/detect/sensors-detect (revision 5915)
+++ prog/detect/sensors-detect (working copy)
@@ -1257,17 +1257,17 @@
}, {
name => "SMSC EMC1403",
driver => "emc1403",
- i2c_addrs => [0x18, 0x2a, 0x4c, 0x4d],
+ i2c_addrs => [0x18, 0x29, 0x4c, 0x4d],
i2c_detect => sub { emc1403_detect(@_, 0); },
}, {
name => "SMSC EMC1404",
driver => "to-be-written", # emc1403
- i2c_addrs => [0x18, 0x2a, 0x4c, 0x4d],
+ i2c_addrs => [0x18, 0x29, 0x4c, 0x4d],
i2c_detect => sub { emc1403_detect(@_, 1); },
}, {
name => "SMSC EMC1423",
driver => "emc1403",
- i2c_addrs => [0x18, 0x2a, 0x4c, 0x4d],
+ i2c_addrs => [0x4c],
i2c_detect => sub { emc1403_detect(@_, 3); },
}, {
name => "SMSC EMC1424",
Index: CHANGES
=================================--- CHANGES (revision 5915)
+++ CHANGES (working copy)
@@ -24,6 +24,8 @@

                   Add detection of Nuvoton NCT5571D, NCT5577D and NCT6776F
                   Add detection of SMSC EMC1002, EMC1033, EMC1046, EMC1047,
                         EMC1072, EMC1073, EMC1074, EMC1402, and EMC1424

+ Fixed address ranges checked for SMSC EMC1403, EMC1404,
+ and EMC1423.

3.2.0 (2010-10-10)
libsensors: Increase MAX_SENSORS_PER_TYPE to 24

_______________________________________________
lm-sensors mailing list
[email protected]
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

#1 lore raw

Re: [lm-sensors] [PATCH] sensors-detect: Fix address ranges checked

Quoted 32 lines · Guenter Roeck

Update I2C address ranges for SMSC EMC1403, EMC1404, and EMC1423 to match information in datasheets.

--
Index: prog/detect/sensors-detect
=================================> --- prog/detect/sensors-detect (revision 5915)
+++ prog/detect/sensors-detect (working copy)

(quoted diff) +3 -3
@@ -1257,17 +1257,17 @@
 	}, {
 		name => "SMSC EMC1403",
 		driver => "emc1403",
-		i2c_addrs => [0x18, 0x2a, 0x4c, 0x4d],
+		i2c_addrs => [0x18, 0x29, 0x4c, 0x4d],
 		i2c_detect => sub { emc1403_detect(@_, 0); },
 	}, {
 		name => "SMSC EMC1404",
 		driver => "to-be-written", # emc1403
-		i2c_addrs => [0x18, 0x2a, 0x4c, 0x4d],
+		i2c_addrs => [0x18, 0x29, 0x4c, 0x4d],
 		i2c_detect => sub { emc1403_detect(@_, 1); },
 	}, {
 		name => "SMSC EMC1423",
 		driver => "emc1403",
-		i2c_addrs => [0x18, 0x2a, 0x4c, 0x4d],
+		i2c_addrs => [0x4c],
 		i2c_detect => sub { emc1403_detect(@_, 3); },
 	}, {
 		name => "SMSC EMC1424",

Index: CHANGES
=================================> --- CHANGES (revision 5915)
+++ CHANGES (working copy)

Review line 28

Quoted 5 lines
(quoted diff) +2 -0
@@ -24,6 +24,8 @@
                   Add detection of Nuvoton NCT5571D, NCT5577D and NCT6776F
                   Add detection of SMSC EMC1002, EMC1033, EMC1046, EMC1047,
                         EMC1072, EMC1073, EMC1074, EMC1402, and EMC1424
+                  Fixed address ranges checked for SMSC EMC1403, EMC1404,
+                        and EMC1423.

You don't have to document if for the EMC1423, as the bogus version was never released.

Other that this, it looks OK, go ahead and commit.

Signature · 6 lines
Jean Delvare

_______________________________________________
lm-sensors mailing list
[email protected]
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors