Re: [PATCH] iio: imu: adis16480: clean up a condition

2019-10-07 Thread Dan Carpenter
On Mon, Oct 07, 2019 at 10:21:07AM +0100, Jonathan Cameron wrote: > On Sun, 6 Oct 2019 21:14:40 +0300 > Dan Carpenter wrote: > > > On Sun, Oct 06, 2019 at 09:51:33AM +0100, Jonathan Cameron wrote: > > > On Thu, 26 Sep 2019 14:36:30 +0300 > > > Dan Carpenter wrote: > > > > > > > On Thu, Sep 26

Re: [PATCH] iio: imu: adis16480: clean up a condition

2019-10-07 Thread Jonathan Cameron
On Sun, 6 Oct 2019 21:14:40 +0300 Dan Carpenter wrote: > On Sun, Oct 06, 2019 at 09:51:33AM +0100, Jonathan Cameron wrote: > > On Thu, 26 Sep 2019 14:36:30 +0300 > > Dan Carpenter wrote: > > > > > On Thu, Sep 26, 2019 at 11:06:39AM +, Ardelean, Alexandru wrote: > > > > On Thu, 2019-09-2

Re: [PATCH] iio: imu: adis16480: clean up a condition

2019-10-06 Thread Dan Carpenter
On Sun, Oct 06, 2019 at 09:51:33AM +0100, Jonathan Cameron wrote: > On Thu, 26 Sep 2019 14:36:30 +0300 > Dan Carpenter wrote: > > > On Thu, Sep 26, 2019 at 11:06:39AM +, Ardelean, Alexandru wrote: > > > On Thu, 2019-09-26 at 11:10 +0300, Dan Carpenter wrote: > > > > [External] > > > > > >

Re: [PATCH] iio: imu: adis16480: clean up a condition

2019-10-06 Thread Jonathan Cameron
On Thu, 26 Sep 2019 14:36:30 +0300 Dan Carpenter wrote: > On Thu, Sep 26, 2019 at 11:06:39AM +, Ardelean, Alexandru wrote: > > On Thu, 2019-09-26 at 11:10 +0300, Dan Carpenter wrote: > > > [External] > > > > > > The "t" variable is unsigned so it can't be less than zero. We really > > > a

Re: [PATCH] iio: imu: adis16480: clean up a condition

2019-09-26 Thread Dan Carpenter
On Thu, Sep 26, 2019 at 11:06:39AM +, Ardelean, Alexandru wrote: > On Thu, 2019-09-26 at 11:10 +0300, Dan Carpenter wrote: > > [External] > > > > The "t" variable is unsigned so it can't be less than zero. We really > > are just trying to prevent divide by zero bugs so just checking against >

Re: [PATCH] iio: imu: adis16480: clean up a condition

2019-09-26 Thread Ardelean, Alexandru
On Thu, 2019-09-26 at 11:10 +0300, Dan Carpenter wrote: > [External] > > The "t" variable is unsigned so it can't be less than zero. We really > are just trying to prevent divide by zero bugs so just checking against > zero is sufficient. > > Signed-off-by: Dan Carpenter > --- > drivers/iio/im

[PATCH] iio: imu: adis16480: clean up a condition

2019-09-26 Thread Dan Carpenter
The "t" variable is unsigned so it can't be less than zero. We really are just trying to prevent divide by zero bugs so just checking against zero is sufficient. Signed-off-by: Dan Carpenter --- drivers/iio/imu/adis16480.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/driv