| 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 | |
|---|
