Re: [PATCH] iio: remove useless irq_enabled variable in at91

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

 



On 10/13/2012 09:06 AM, Peter Meerwald wrote:
> irq_enabled is only set, but never read
> 
> Signed-off-by: Peter Meerwald <pmeerw@xxxxxxxxxx>
> Acked-by: Maxime Ripard <maxime.ripard@xxxxxxxxxxxxxxxxxx>
Added to togreg branch of iio.git  thanks.

> ---
>  drivers/iio/adc/at91_adc.c |    3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/drivers/iio/adc/at91_adc.c b/drivers/iio/adc/at91_adc.c
> index 3ed94bf..b3ba8af 100644
> --- a/drivers/iio/adc/at91_adc.c
> +++ b/drivers/iio/adc/at91_adc.c
> @@ -46,7 +46,6 @@ struct at91_adc_state {
>  	struct clk		*clk;
>  	bool			done;
>  	int			irq;
> -	bool			irq_enabled;
>  	u16			last_value;
>  	struct mutex		lock;
>  	u8			num_channels;
> @@ -85,7 +84,6 @@ static irqreturn_t at91_adc_trigger_handler(int irq, void *p)
>  	buffer->access->store_to(buffer, (u8 *)st->buffer);
>  
>  	iio_trigger_notify_done(idev->trig);
> -	st->irq_enabled = true;
>  
>  	/* Needed to ACK the DRDY interruption */
>  	at91_adc_readl(st, AT91_ADC_LCDR);
> @@ -106,7 +104,6 @@ static irqreturn_t at91_adc_eoc_trigger(int irq, void *private)
>  
>  	if (iio_buffer_enabled(idev)) {
>  		disable_irq_nosync(irq);
> -		st->irq_enabled = false;
>  		iio_trigger_poll(idev->trig, iio_get_time_ns());
>  	} else {
>  		st->last_value = at91_adc_readl(st, AT91_ADC_LCDR);
> 
--
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