On Thu, Aug 22, 2013 at 03:57:33PM +0200, Jean Delvare wrote:
> On Sun, 4 Aug 2013 06:10:57 -0700, Guenter Roeck wrote:
> > Fix the following smatch warning:
> >
> > emc6w201.c:52:26: warning: symbol 'subfeature' was not declared.
> > Should it be static?
> >
> > 'enum { } subtype' declares an enum as (global) variable which we don't want.
> >
> > Signed-off-by: Guenter Roeck <linux@xxxxxxxxxxxx>
> > ---
> > drivers/hwmon/emc6w201.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/hwmon/emc6w201.c b/drivers/hwmon/emc6w201.c
> > index 936898f..82e661e 100644
> > --- a/drivers/hwmon/emc6w201.c
> > +++ b/drivers/hwmon/emc6w201.c
> > @@ -49,7 +49,7 @@ static const unsigned short normal_i2c[] = { 0x2c, 0x2d, 0x2e, I2C_CLIENT_END };
> > #define EMC6W201_REG_TEMP_HIGH(nr) (0x57 + (nr) * 2)
> > #define EMC6W201_REG_FAN_MIN(nr) (0x62 + (nr) * 2)
> >
> > -enum { input, min, max } subfeature;
> > +enum subfeature { input, min, max };
> >
> > /*
> > * Per-device data
>
> Good catch. Applied, thanks.
>
Ok, I'll drop it from my queue.
Thanks,
Guenter
_______________________________________________
lm-sensors mailing list
lm-sensors@xxxxxxxxxxxxxx
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
