| 1 |
LM-Sensors TODO list |
|---|
| 2 |
Contact us if you have comments or wish to help. |
|---|
| 3 |
------------------------------------------------ |
|---|
| 4 |
|
|---|
| 5 |
I2C CODE CHANGES |
|---|
| 6 |
================ |
|---|
| 7 |
|
|---|
| 8 |
See TODO in the i2c package. |
|---|
| 9 |
|
|---|
| 10 |
|
|---|
| 11 |
KERNEL MODULES |
|---|
| 12 |
============== |
|---|
| 13 |
|
|---|
| 14 |
* 2.4 / 2.5 cleanup |
|---|
| 15 |
See adm1021, lm75, i2c-amd756, and i2c-amd8111 in kernel 2.5.54 for examples |
|---|
| 16 |
- Remove inc_use and dec_use in adapters and chips... |
|---|
| 17 |
Modules shouldn't adjust their own refcounts. |
|---|
| 18 |
Do we need to move this elsewhere or just remove? |
|---|
| 19 |
- Convert PCI drivers to use pci_driver structs, |
|---|
| 20 |
registration, etc. |
|---|
| 21 |
- Use module_init and module_exit |
|---|
| 22 |
- Check return value of request_region(), don't use check_region() |
|---|
| 23 |
- C99 initializers |
|---|
| 24 |
- Remove excess #includes |
|---|
| 25 |
- interrupts/locking |
|---|
| 26 |
- Remove #ifdef MODULE, THIS_MODULE, MODULE_LICENSE, etc. |
|---|
| 27 |
- Remove EXPORT_NO_SYMBOLS |
|---|
| 28 |
- Remove other 2.1/2.2 #ifdefs |
|---|
| 29 |
* Handle -1 returns from i2c layer rather than just delivering 0xFF to |
|---|
| 30 |
/proc and libsensors; report errors through /proc 'alarms' or |
|---|
| 31 |
new 'fail' entry? |
|---|
| 32 |
* Add string function in sensors.o |
|---|
| 33 |
* ALL: cleanup_module is void; check also that cleaning up is done |
|---|
| 34 |
properly, now we know that the module will be unloaded whatever we |
|---|
| 35 |
do. |
|---|
| 36 |
* ALL chip drivers: add a readonly insmod option so we don't cause APM/ACPI |
|---|
| 37 |
to go insane. |
|---|
| 38 |
* gl518sm: using iterate==2 causes a tread to be started. this thread remains |
|---|
| 39 |
even after setting iterate==1 and/or removing the gl518sm module |
|---|
| 40 |
<koenig@uranus.tat.physik.uni-tuebingen.de> |
|---|
| 41 |
* LM78 detection: Tom Webster has proven that the reset bit in the |
|---|
| 42 |
id register can be one (strange!) |
|---|
| 43 |
* maxilife.c: Introduce new insmod variables |
|---|
| 44 |
* maxilife.c: Round REG_TO_VID correctly |
|---|
| 45 |
* icspll.c: Rewrite. The current implementation simply can't work at all. |
|---|
| 46 |
It needs i2c-level access (too bad for SMBus-only adapters). |
|---|
| 47 |
* w83781d: Some features are still unsupported for the W8378[23]. |
|---|
| 48 |
Also, alarms seem to give strange results |
|---|
| 49 |
sometimes, and there are some other minor problems, as indicated by |
|---|
| 50 |
Jonathan Bradshaw <jonathan@NrgUp.Com> |
|---|
| 51 |
* Support 10-bit addresses. At this moment, they are supported nowhere, except |
|---|
| 52 |
in Simon Vogl's i2c modules. |
|---|
| 53 |
* Better general locking, or at least a once-over to check no race-conditions |
|---|
| 54 |
remain. This is part of the SMP-safeness, and can better be done at once. |
|---|
| 55 |
* adm9240: check whether the current voltage computations are correct. |
|---|
| 56 |
Probably not, as they are different from the datasheet specifications. |
|---|
| 57 |
Also check for the supported dallas chip. |
|---|
| 58 |
* lm80: Check how OS alarms work. At page 17 of the sheet, it tells |
|---|
| 59 |
something completely different from the description at page 25. |
|---|
| 60 |
* gl518sm: Assume that new values are close to old values, so start with |
|---|
| 61 |
trying to use a small range near the old values (already partially done) |
|---|
| 62 |
* lm77: Add detection in lm75.c and sensors-detect to distinguish from |
|---|
| 63 |
an lm75. Add support for the two new registers. |
|---|
| 64 |
* Add MTP006F chip: http://www.myson.com/Pcd/MTP006/Mtp006.pdf |
|---|
| 65 |
gfiala@s.netic.de (Guido Fiala) has one on his mainboard |
|---|
| 66 |
|
|---|
| 67 |
|
|---|
| 68 |
LIBRARY |
|---|
| 69 |
======= |
|---|
| 70 |
|
|---|
| 71 |
* reload does not work (Bison/Flex problem?). |
|---|
| 72 |
* Some library routines are pretty inefficient right now. |
|---|
| 73 |
* Library should be split in more separate files, for better linking |
|---|
| 74 |
behaviour. |
|---|
| 75 |
* Some adm9240 labels start with a digit; change this, it leads to confusing |
|---|
| 76 |
syntax in the conf file (they have to be quoted) |
|---|
| 77 |
|
|---|
| 78 |
|
|---|
| 79 |
PROGRAMS |
|---|
| 80 |
======== |
|---|
| 81 |
|
|---|
| 82 |
* Sensors program needs to print a + before lm75 temperatures |
|---|
| 83 |
* Sensors program needs to be extended for many new chips |
|---|
| 84 |
|
|---|
| 85 |
|
|---|
| 86 |
OTHERS |
|---|
| 87 |
====== |
|---|
| 88 |
|
|---|
| 89 |
* Examine watchdog-4.3 or later of meskes@debian.org (Michael Meskes) |
|---|
| 90 |
tsx-11.mit.edu /pub/linux/sources/sbin or |
|---|
| 91 |
sunsite.unc.edu /pub/linux/system/daemons/watchdog |
|---|
| 92 |
|
|---|