Bug 578527
| Summary: | sensors-detect fails to detect | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Juha Tuomala <tuju> |
| Component: | lm_sensors | Assignee: | Nikola Pajkovsky <npajkovs> |
| Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
| Severity: | medium | Docs Contact: | |
| Priority: | low | ||
| Version: | 12 | CC: | dhoward, hdegoede, jdelvare, npajkovs, pknirsch |
| Target Milestone: | --- | ||
| Target Release: | --- | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | lm_sensors-3.1.2-2.fc13 | Doc Type: | Bug Fix |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2010-04-21 02:16:47 UTC | Type: | --- |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
Upstream commit for reference: http://www.lm-sensors.org/changeset/5832 Thanks for report. Commit is 3 hours old :) I will add this today into lm_sensors. When build will be ready I will ask you if you can test it. build: http://koji.fedoraproject.org/koji/taskinfo?taskID=2087223 Could you try it, please? Driver `lm85':
* Bus `SMBus2 AMD8111 adapter at 1440'
Busdriver `i2c_amd8111', I2C address 0x2e
Chip `Analog Devices ADT7467 or ADT7468' (confidence: 7)
Driver `k8temp' (autoloaded):
* Chip `AMD K8 thermal sensors' (confidence: 9)
Do you want to overwrite /etc/sysconfig/lm_sensors? (YES/no):
Starting lm_sensors: loading module lm85 [ OK ]
Unloading i2c-dev... OK
Appears to work as should, thanks.
btw, it would be nice, if this data: http://mysite.verizon.net/pchardwarelinks/elec.htm#amd would be somehow available in configuration. That could be added into xml file and somehow automatically matched from /proc/cpuinfo and mapped to configs. lm_sensors-3.1.2-2.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/lm_sensors-3.1.2-2.fc12 lm_sensors-3.1.2-2.fc13 has been submitted as an update for Fedora 13. http://admin.fedoraproject.org/updates/lm_sensors-3.1.2-2.fc13 lm_sensors-3.1.2-2.fc13 has been pushed to the Fedora 13 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update lm_sensors'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/lm_sensors-3.1.2-2.fc13 lm_sensors-3.1.2-2.fc12 has been pushed to the Fedora 12 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update lm_sensors'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/lm_sensors-3.1.2-2.fc12 lm_sensors-3.1.2-2.fc12 has been pushed to the Fedora 12 stable repository. If problems still persist, please make note of it in this bug report. lm_sensors-3.1.2-2.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report. |
Description of problem: Systems with "Analog Devices ADT7467 or ADT7468" system monitoring chip fail to detect it and lm_sensors package is practically useless. Version-Release number of selected component (if applicable): lm_sensors-3.1.2-1.fc12.x86_64 How reproducible: always. Steps to Reproduce: 1. run sensors-detect 2. it doesn't find any monitoring chips. 3. Expected results: detecting correct chip and able to run pwmconfig, fancontrol etc. Additional info: Someone already added support for these chips into lm85 driver, so it's enough that this gets loaded into kernel. This can be fixed in place by changing the driver name from /usr/sbin/sensors-detect script: name => "Analog Devices ADT7467 or ADT7468", driver => "lm85", i2c_addrs => [0x2e], i2c_detect => sub { adt7467_detect(@_, 0); }, original version has 'to-be-written' name there. The fix has already been done in upstream SCM and opensuse includes it.