also has another 'unsigned long val' warnings, I will send patch v2 to merge them together.
On 2013年04月07日 11:09, Chen Gang wrote: > > val is unsigned long which never < 0 > > Signed-off-by: Chen Gang <[email protected]> > --- > drivers/misc/tsl2550.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/drivers/misc/tsl2550.c b/drivers/misc/tsl2550.c > index 1e7bc0e..558dd20 100644 > --- a/drivers/misc/tsl2550.c > +++ b/drivers/misc/tsl2550.c > @@ -204,7 +204,7 @@ static ssize_t tsl2550_store_power_state(struct device > *dev, > unsigned long val = simple_strtoul(buf, NULL, 10); > int ret; > > - if (val < 0 || val > 1) > + if (val > 1) > return -EINVAL; > > mutex_lock(&data->update_lock); > -- Chen Gang Asianux Corporation -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

