[PATCH v4] hwmon: Add support for Texas Instruments ADS1015
Grant Likely
grant.likely at secretlab.ca
Fri Mar 4 08:15:55 EST 2011
Wolfram Sang <w.sang at pengutronix.de> wrote:
>> > ---
>> > 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)
Absolutely. Commit text is cheap and useful. I say the more detail the better.
g.
--
Sent from my Android phone with K-9 Mail. Please excuse my brevity.
More information about the devicetree-discuss
mailing list