Hi Guenter,
On Thu, 9 Sep 2010 06:25:49 -0700, Guenter Roeck wrote:
> Signed-off-by: Guenter Roeck <guenter.roeck@xxxxxxxxxxxx>
> ---
> Documentation/hwmon/lm90 | 13 ++++++-----
> drivers/hwmon/lm90.c | 51 ++++++++++++++++++++++++++++++++++-----------
> 2 files changed, 45 insertions(+), 19 deletions(-)
> (...)
> @@ -94,7 +94,7 @@ static const unsigned short normal_i2c[] = {
> 0x18, 0x19, 0x1a, 0x29, 0x2a, 0x2b, 0x4c, 0x4d, 0x4e, I2C_CLIENT_END };
>
> enum chips { lm90, adm1032, lm99, lm86, max6657, adt7461, max6680, max6646,
> - w83l771 };
> + w83l771, max6659 };
As the numbers are internal only, it might make sense to group max6657
and max6659 together for better readability.
No other comment, other than the patch ordering (or merging) issue
already pointed out in patch 5/7.
--
Jean Delvare
_______________________________________________
lm-sensors mailing list
lm-sensors@xxxxxxxxxxxxxx
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
