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