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