On 22/09/16 21:44, Sandhya Bankar wrote: > Fixing belows sparse warning : > > drivers/iio/accel/mma7660.c:42:11: warning: symbol 'mma7660_nscale' > was not declared. Should it be static? > > Signed-off-by: Sandhya Bankar <bankarsandhya512@xxxxxxxxx> Sorry, I just applied an alternative (later :( ) patch that makes the same change. I'm afraid I tend to read my emails backwards in time to pick up new versions of patches first, so whilst I'd normally apply the first poster's version for this sort of patch I came across the other one and applied it first. Jonathan > --- > drivers/iio/accel/mma7660.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/iio/accel/mma7660.c b/drivers/iio/accel/mma7660.c > index 03beadf..3a40774 100644 > --- a/drivers/iio/accel/mma7660.c > +++ b/drivers/iio/accel/mma7660.c > @@ -39,7 +39,7 @@ > > #define MMA7660_SCALE_AVAIL "0.467142857" > > -const int mma7660_nscale = 467142857; > +static const int mma7660_nscale = 467142857; > > #define MMA7660_CHANNEL(reg, axis) { \ > .type = IIO_ACCEL, \ > -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html