Re: [PATCH] iio: accel: mma8452: define unsigned return values where appropriate

2016-11-27 Thread Jonathan Cameron
On 21/11/16 19:53, Martin Kepplinger wrote: > smatch warned: > sval_binop_signed: invalid divide LLONG_MIN/-1 > > and this fixes it. It's actually good to have, in order to avoid accidental > checking for negative return values here. > > Signed-off-by: Martin Kepplinger

Re: [PATCH] iio: accel: mma8452: define unsigned return values where appropriate

2016-11-27 Thread Jonathan Cameron
On 21/11/16 19:53, Martin Kepplinger wrote: > smatch warned: > sval_binop_signed: invalid divide LLONG_MIN/-1 > > and this fixes it. It's actually good to have, in order to avoid accidental > checking for negative return values here. > > Signed-off-by: Martin Kepplinger I'm assuming the

[PATCH] iio: accel: mma8452: define unsigned return values where appropriate

2016-11-21 Thread Martin Kepplinger
smatch warned: sval_binop_signed: invalid divide LLONG_MIN/-1 and this fixes it. It's actually good to have, in order to avoid accidental checking for negative return values here. Signed-off-by: Martin Kepplinger --- drivers/iio/accel/mma8452.c | 4 ++-- 1 file

[PATCH] iio: accel: mma8452: define unsigned return values where appropriate

2016-11-21 Thread Martin Kepplinger
smatch warned: sval_binop_signed: invalid divide LLONG_MIN/-1 and this fixes it. It's actually good to have, in order to avoid accidental checking for negative return values here. Signed-off-by: Martin Kepplinger --- drivers/iio/accel/mma8452.c | 4 ++-- 1 file changed, 2