Re: [PATCH] using kfifo_in_spinlocked instead of separate code.

[Jonathan Cameron <jic23@xxxxxxxxxx>]

On 03/04/2013 11:27 PM, Ge GAO wrote:
> From: Ge Gao <ggao@xxxxxxxxxxxxxx>
I added your sign off given it should have been here.
Please make sure to run checkpatch.pl against all patches
as it catches silly mistakes like this.

Applied to togreg branch of iio.git
> 
> ---
>  drivers/iio/imu/inv_mpu6050/inv_mpu_ring.c |    5 ++---
>  1 files changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/iio/imu/inv_mpu6050/inv_mpu_ring.c b/drivers/iio/imu/inv_mpu6050/inv_mpu_ring.c
> index 331781f..7da0832 100644
> --- a/drivers/iio/imu/inv_mpu6050/inv_mpu_ring.c
> +++ b/drivers/iio/imu/inv_mpu6050/inv_mpu_ring.c
> @@ -105,9 +105,8 @@ irqreturn_t inv_mpu6050_irq_handler(int irq, void *p)
>  	s64 timestamp;
>  
>  	timestamp = iio_get_time_ns();
> -	spin_lock(&st->time_stamp_lock);
> -	kfifo_in(&st->timestamps, &timestamp, 1);
> -	spin_unlock(&st->time_stamp_lock);
> +	kfifo_in_spinlocked(&st->timestamps, &timestamp, 1,
> +				&st->time_stamp_lock);
>  
>  	return IRQ_WAKE_THREAD;
>  }
> 
--
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