[lm-sensors] [RFC PATCH] Trivial changes to pacify the checkpatch

[Jean Delvare]

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 at ti.com>
> ---
>  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 at dds.nl>
> -
> -    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 at dds.nl>
> +
> +	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.

-- 
Jean Delvare