Hi Jean,
On Mon, 2011-11-21 at 15:29 -0500, Jean Delvare wrote:
> On Mon, 21 Nov 2011 08:52:45 -0800, Guenter Roeck wrote:
> > LM96163 is an enhanced version of LM63 with improved PWM resolution. Add chip
> > detection code as well as support for improved PWM resolution if the chip is
> > configured to use it.
> >
> > Signed-off-by: Guenter Roeck <guenter.roeck@xxxxxxxxxxxx>
> > ---
> > Documentation/hwmon/lm63 | 8 ++++++++
> > drivers/hwmon/Kconfig | 4 ++--
> > drivers/hwmon/lm63.c | 41 ++++++++++++++++++++++++++++++++++++-----
> > 3 files changed, 46 insertions(+), 7 deletions(-)
> > (...)
> > @@ -605,6 +619,23 @@ static void lm63_init_client(struct i2c_client *client)
> > if (data->pwm1_freq == 0)
> > data->pwm1_freq = 1;
> >
> > + /*
> > + * For LM96163, check if high resolution PWM is enabled.
> > + * Also, check if unsigned temperature format is enabled
> > + * and display a warning message if it is.
> > + */
> > + if (data->kind == lm96163) {
> > + u8 config_enhanced
> > + = i2c_smbus_read_byte_data(client,
> > + LM96163_REG_CONFIG_ENHANCED);
> > + if ((config_enhanced & 0x10)
> > + && !(data->config_fan & 0x08) && data->pwm1_freq == 8)
> > + data->pwm_highres = true;
> > + if (config_enhanced & 0x08)
> > + dev_warn(&client->dev,
> > + "Unsigned format for High and Crit setpoints enabled but not supported by driver");
>
> Line too long and missing trailing newline. I fixed both, and queued
> the patch for kernel 3.3, thank you. Care to update wiki/Devices?
>
I thought long lines are now accepted and recommended for strings, to
avoid breaking a string into multiple lines. No idea where the missing
newline comes from - checkpatch didn't report anything, and the patch
seems to be ok when I look at it.
I updated the wiki a minute ago.
Thanks,
Guenter
_______________________________________________
lm-sensors mailing list
lm-sensors@xxxxxxxxxxxxxx
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
