Ticket #2125 (closed enhancement: wontfix)

Opened 2 years ago

Last modified 2 weeks ago

contribute module for MAX1668, MAX1805, and MAX1989

Reported by: ticket Assigned to: somebody
Priority: minor Milestone:
Component: hardware Version: kernel
Keywords: Cc:

Description

I would like to contribute a module which covers the Maxim chips MAX1668, MAX1805, and MAX1989. It has been tested with a MAX1805 module as described on Madhacker.org or by Andreas Lenz

It still uses I2C_DRIVERID_ADM1021 as it was derived from adm1201.c and would need its own I2C_DRIVERID_MAX1668 assigned in i2c-id.h

It would be good if somebody with a MAX1668 or a MAX1989 could also test the module.

Attachments

max1668.c (17.3 kB) - added by ticket on 09/22/06 00:03:26.
max1668.c module source

Change History

09/22/06 00:03:26 changed by ticket

  • attachment max1668.c added.

max1668.c module source

09/22/06 10:10:09 changed by khali

  • priority changed from major to minor.
  • version set to kernel.

You probably don't need a device ID, so you can simply omit it (it is optional and almost never needed.)

This ticket system is probably not the more convenient place to review a new driver, so I suggest that you post your code to the lm-sensors mailing list.

Carolyn Smith was supposedly working on a driver for these chips: http://lists.lm-sensors.org/pipermail/lm-sensors/2003-September/004212.html But that was 3 years ago and we never heard back since. You may want to contact her still.

I just updated our Devices page to mention your work.

Can you please provide a patch to sensors-detect, adding detection support for the three new chips?

11/18/08 10:07:39 changed by khali

  • status changed from new to closed.
  • resolution set to wontfix.

Nobody seems to be interested in this -> closing.