[patch 2.6.23-rc6] lm75: new-style i2c binding, cleanup

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> > Updates and cleanups for lm75:
> > 
> >  - Update the LM75 driver to handle new-style I2C driver binding
> >    (following the driver model).
> > 
> >  - Use 12 bit resolution (1/16 degree C) on chips that support it,
> >    instead of 9 bit (1/2 degree C);  needs new conversion routines.
> > 
> >  - Grow the list of compatible chips; update Kconfig accordingly.
> > 
> >  - On driver disconnect, restore the original device config (which
> >    usually means shutdown).
> > 
> >  - Some code cleanups:  a handful of whitespace fixes, and clearly
> >    marking the various driver components (sysfs/hwmon attributes,
> >    driver core support, legacy stuff, register access, module glue).
>
> This is crying for at least two separate patches, and ideally more like
> 3 or 4.

I was fairly sure someone was going to say that.  I take it that
there's no real technical comment on the changes though?


> Additionally, I can't apply your patch to my tree. Presumably you are
> missing recent patches that went into Mark's hwmon-testing tree and
> which affect the lm75 driver:

The location of that tree seems to be hidden.  I did find a GIT
tree of Mark's, but it didn't have any lm75 changes.


> hwmon: Allow writing of negative trigger temperatures
> http://lm-sensors.org/kernel?p=kernel/mhoffman/hwmon-2.6.git;a=commit;h=a1efde3fd7b3d94c942b1a675da992c8a2f1f04b

I saw a mention of this patch in the list archives, but didn't see
the patch anywhere.  That'd be a trivial tweak to the patch.


> hwmon: Convert from class_device to device
> http://lm-sensors.org/kernel?p=kernel/mhoffman/hwmon-2.6.git;a=commit;h=f89523a27def89ddc74d094a0910970f6a1c6f43

Didn't notice see that one.


> If you split your patch and make sure that the result applies on top of
> the above patches, I'll be happy to review and test your work.

I'll split it out a bit then.


> > The way to kick in higher sample resolution on a given board is to
> > use the new style binding to pass in the chip type.
> > 
> > One temporary omission:  IRQ support, or for the TI chips SMBALERT#
> > notifications.
>
> -- 
> Jean Delvare
>




[Index of Archives]     [Linux Kernel]     [Linux Hardware Monitoring]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux