[PATCH v4] hwmon: Add support for Texas Instruments ADS1015

Wolfram Sang w.sang at pengutronix.de
Fri Mar 4 07:02:36 EST 2011


> > ---
> > Changes since v1:
> > - fixed/extended Documentation
> > - removed unused register definitions
> > - hardcoded PGA fullscale table size
> > - made sure patch applies against v2.6.38-rc4
> > - reordered functions to avoid forward declaration
> > - results from i2c_smbus_read_word_data() are handled correctly
> > - moved locking into ads1015_read_value()
> > - removed unnecessray clearing of bit
> > - proper error handling in ads1015_read_value()
> > - use DIV_ROUND_CLOSEST for scaling result
> > - removed detect()
> > 
> > Changes since v2:
> > - removed *all* leftovers from detect()
> > - fixed return with mutex held
> > - made sysfs representation configurable
> >   (hope this will be the reference implementation for generations to come ;)
> > 
> > Changes since v3:
> > - included linux/of.h
> > - remove linux/types.h from header file
> > - sysfs is now configured with a bitmask
> > - assume big-endian of-properties
> 
> I recommend putting the revision history *above* the '---' trim line.
> It turns out to be useful to have the revision history in the commit
> text that actually gets merged into mainline.

Is this common sense meanwhile? (That is quite many additional lines per
commit)

Regards,

   Wolfram

-- 
Pengutronix e.K.                           | Wolfram Sang                |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 197 bytes
Desc: Digital signature
URL: <http://lists.ozlabs.org/pipermail/devicetree-discuss/attachments/20110303/7545bb0a/attachment.pgp>


More information about the devicetree-discuss mailing list