Re: [PATCH 01/15] staging:iio:adis16400: Don't pass 0 to ilog2

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 01/16/2013 12:48 PM, Lars-Peter Clausen wrote:
> ilog2 is not defined for 0, so we need to handle the case where the requested
> frequency is larger than the base sampling rate. In this case we'll round down
> and set the sampling rate to the base sampling rate.
> 
> Signed-off-by: Lars-Peter Clausen <lars@xxxxxxxxxx>
Added whole series to togreg branch of iio.git

Also, now these are in, I have applied the sw_ring removal series that
I held at Lars-Peter's request as this series was on its way.


> ---
>  drivers/staging/iio/imu/adis16400_core.c | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/staging/iio/imu/adis16400_core.c b/drivers/staging/iio/imu/adis16400_core.c
> index 9c8f5ab..cb66225 100644
> --- a/drivers/staging/iio/imu/adis16400_core.c
> +++ b/drivers/staging/iio/imu/adis16400_core.c
> @@ -178,7 +178,11 @@ static int adis16334_set_freq(struct iio_dev *indio_dev, unsigned int freq)
>  {
>  	unsigned int t;
>  
> -	t = ilog2(8192 / (freq * 10));
> +	freq *= 10;
> +	if (freq < 8192)
> +		t = ilog2(8192 / freq);
> +	else
> +		t = 0;
>  
>  	if (t > 0x31)
>  		t = 0x31;
> 
--
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


[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux