[PATCH] sensors-detect: Drop MAX6650/51 detection

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Drop MAX6650/51 detection. It's unreliable, and that's not the kind
of chip I expect to be seen in a PC. People who have this chip know
that they do, they don't need sensors-detect to tell them.
---
I plan to drop detection from the max6650 driver too, for the same
reason.

 prog/detect/sensors-detect |   33 ---------------------------------
 1 file changed, 33 deletions(-)

--- lm-sensors.orig/prog/detect/sensors-detect	2011-05-05 22:46:59.000000000 +0200
+++ lm-sensors/prog/detect/sensors-detect	2011-05-05 22:51:11.000000000 +0200
@@ -848,11 +848,6 @@ use vars qw(@i2c_adapter_names);
 		i2c_addrs => [0x2c, 0x2e, 0x2f],
 		i2c_detect => sub { max6639_detect(@_); },
 	}, {
-		name => "Maxim MAX6650/MAX6651",
-		driver => "max6650",
-		i2c_addrs => [0x1b, 0x1f, 0x48, 0x4b],
-		i2c_detect => sub { max6650_detect(@_); },
-	}, {
 		name => "Maxim MAX6655/MAX6656",
 		driver => "max6655",
 		i2c_addrs => [0x18..0x1a, 0x29..0x2b, 0x4c..0x4e],
@@ -5776,34 +5771,6 @@ sub max6639_detect
 	return 6;
 }
 
-# The max6650 has no device ID register. However, a few registers have
-# spare bits, which are documented as being always zero on read. We read
-# all of these registers check the spare bits. Any non-zero means this
-# is not a max6650/1.
-#
-# The always zero bits are:
-#   configuration byte register (0x02) - top 2 bits
-#   gpio status register (0x14) - top 3 bits
-#   alarm enable register (0x08) - top 3 bits
-#   alarm status register (0x0A) - top 3 bits
-#   tachometer count time register (0x16) - top 6 bits
-# Additionally, not all values are possible for lower 3 bits of
-# the configuration register.
-sub max6650_detect
-{
-	my ($file, $addr) = @_;
-
-	my $conf = i2c_smbus_read_byte_data($file, 0x02);
-
-	return if i2c_smbus_read_byte_data($file, 0x16) & 0xFC;
-	return if i2c_smbus_read_byte_data($file, 0x0A) & 0xE0;
-	return if i2c_smbus_read_byte_data($file, 0x08) & 0xE0;
-	return if i2c_smbus_read_byte_data($file, 0x14) & 0xE0;
-	return if ($conf & 0xC0) or ($conf & 0x07) > 4;
-
-	return 2;
-}
-
 sub max6655_detect
 {
 	my ($file, $addr) = @_;


-- 
Jean Delvare

_______________________________________________
lm-sensors mailing list
lm-sensors@xxxxxxxxxxxxxx
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors


[Index of Archives]     [Linux Kernel]     [Linux Hardware Monitoring]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux