Re: [RFC PATCH] Trivial changes to pacify the checkpatch

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

 



Hi Jean,

> -----Original Message-----
> From: Jean Delvare [mailto:khali@xxxxxxxxxxxx]
> Sent: Friday, August 27, 2010 6:15 PM
> To: Datta, Shubhrajyoti
> Cc: lm-sensors@xxxxxxxxxxxxxx
> Subject: Re:  [RFC PATCH] Trivial changes to pacify the
> checkpatch
> 
> Hi Shubhrajyoti,
> 
> On Thu, 26 Aug 2010 19:03:17 +0530, Shubhrajyoti D wrote:
> > Some trivial changes to pacify the checkpatch so that
> > real errors are highlighted.
> >
> > Signed-off-by: Shubhrajyoti D <shubhrajyoti@xxxxxx>
> > ---
> >  drivers/hwmon/lm75.c |   40 ++++++++++++++++++++--------------------
> >  1 files changed, 20 insertions(+), 20 deletions(-)
> >
> > diff --git a/drivers/hwmon/lm75.c b/drivers/hwmon/lm75.c
> > index d579290..97b3a89 100644
> > --- a/drivers/hwmon/lm75.c
> > +++ b/drivers/hwmon/lm75.c
> > @@ -1,21 +1,21 @@
> >  /*
> > -    lm75.c - Part of lm_sensors, Linux kernel modules for hardware
> > -             monitoring
> > -    Copyright (c) 1998, 1999  Frodo Looijaard <frodol@xxxxxx>
> > -
> > -    This program is free software; you can redistribute it and/or
> modify
> > -    it under the terms of the GNU General Public License as published
> by
> > -    the Free Software Foundation; either version 2 of the License, or
> > -    (at your option) any later version.
> > -
> > -    This program is distributed in the hope that it will be useful,
> > -    but WITHOUT ANY WARRANTY; without even the implied warranty of
> > -    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> > -    GNU General Public License for more details.
> > -
> > -    You should have received a copy of the GNU General Public License
> > -    along with this program; if not, write to the Free Software
> > -    Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
> > +	lm75.c - Part of lm_sensors, Linux kernel modules for hardware
> > +			 monitoring
> > +	Copyright (c) 1998, 1999  Frodo Looijaard <frodol@xxxxxx>
> > +
> > +	This program is free software; you can redistribute it and/or modify
> > +	it under the terms of the GNU General Public License as published by
> > +	the Free Software Foundation; either version 2 of the License, or
> > +	(at your option) any later version.
> > +
> > +	This program is distributed in the hope that it will be useful,
> > +	but WITHOUT ANY WARRANTY; without even the implied warranty of
> > +	MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> > +	GNU General Public License for more details.
> > +
> > +	You should have received a copy of the GNU General Public License
> > +	along with this program; if not, write to the Free Software
> > +	Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
> >  */
> >
> >  #include <linux/module.h>
> > @@ -339,9 +339,9 @@ static struct i2c_driver lm75_driver = {
> >
> >  /* register access */
> >
> > -/* All registers are word-sized, except for the configuration register.
> > -   LM75 uses a high-byte first convention, which is exactly opposite to
> > -   the SMBus standard. */
> > +/*	All registers are word-sized, except for the configuration register.
> > +	LM75 uses a high-byte first convention, which is exactly opposite to
> > +	the SMBus standard. */
> >  static int lm75_read_value(struct i2c_client *client, u8 reg)
> >  {
> >  	int value;
> 
> That's not the right fix, sorry. The right way would be to add stars:
> 
> /*
>  * All registers are word-sized, except for the configuration register.
>  * LM75 uses a high-byte first convention, which is exactly opposite to
>  * the SMBus standard.
>  */
> 
> Same for the first comment.
Agree will fix it.
> 
> --
> Jean Delvare

_______________________________________________
lm-sensors mailing list
lm-sensors@xxxxxxxxxxxxxx
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors


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

  Powered by Linux