From f804788da1b5d7b5761dee97dca4c1f108f1c2ab Mon Sep 17 00:00:00 2001 From: Jean Delvare Date: Sun, 4 May 2008 16:41:03 +0000 Subject: [PATCH] Don't probe I2C addresses 0x40-0x47. We only probed them for the MAX6633, which is a rare chip, and hard to identify anyway. Given that some I2C chips have been reported to live at 0x47 and to hang the bus when probed, the benefit just isn't worth the risk. git-svn-id: http://lm-sensors.org/svn/lm-sensors/trunk@5234 7894878c-1315-0410-8ee3-d5d059ff63e0 --- CHANGES | 1 + prog/detect/sensors-detect | 4 +++- 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/CHANGES b/CHANGES index 5c170a92..363d8563 100644 --- a/CHANGES +++ b/CHANGES @@ -16,6 +16,7 @@ SVN-HEAD Add National Semiconductor LM64 detection Add Asus F8000 detection Add Intel ICH10 (bus) detection + Don't probe I2C addresses 0x40-0x47 2.10.6 "Welcome Home Lina" (20080306) Library: Fix fschrc and fschmd support diff --git a/prog/detect/sensors-detect b/prog/detect/sensors-detect index c6da784a..e9b8e395 100755 --- a/prog/detect/sensors-detect +++ b/prog/detect/sensors-detect @@ -1128,7 +1128,9 @@ use vars qw(@pci_adapters_sis5595 @pci_adapters_sis645 @pci_adapters_sis96x); { name => "Maxim MAX6633/MAX6634/MAX6635", driver => "lm92", - i2c_addrs => [0x40..0x4f], + i2c_addrs => [0x48..0x4f], # The MAX6633 can also use 0x40-0x47 but we + # don't want to probe these addresses, it's + # dangerous. i2c_detect => sub { lm92_detect(2, @_); }, }, {