Re: [lm-sensors] [PATCH 04/11] hwmon: Storage class should be before const qualifier

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

 



Hi Tobias,

On Thu, 20 May 2010 10:34:48 +0200, Tobias Klauser wrote:
> The C99 specification states in section 6.11.5:
> 
> The placement of a storage-class specifier other than at the beginning
> of the declaration specifiers in a declaration is an obsolescent
> feature.
> 
> Signed-off-by: Tobias Klauser <tklauser@xxxxxxxxxx>
> ---
>  drivers/hwmon/emc1403.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/drivers/hwmon/emc1403.c b/drivers/hwmon/emc1403.c
> index 28cca34..0e4b564 100644
> --- a/drivers/hwmon/emc1403.c
> +++ b/drivers/hwmon/emc1403.c
> @@ -308,7 +308,7 @@ static const unsigned short emc1403_address_list[] = {
>  	0x18, 0x2a, 0x4c, 0x4d, I2C_CLIENT_END
>  };
>  
> -const static struct i2c_device_id emc1403_idtable[] = {
> +static const struct i2c_device_id emc1403_idtable[] = {
>  	{ "emc1403", 0 },
>  	{ }
>  };

Good catch. I've folded your fix into the original patch, as it isn't
upstream yet.

Could this error be caught by checkpatch.pl in the future? It seems
easy enough to identify, and would save you the manual work.

-- 
Jean Delvare
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Kernel Development]     [Kernel Announce]     [Kernel Newbies]     [Linux Networking Development]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Device Mapper]

  Powered by Linux