Re: [PATCH 07/11] staging:iio:max1363: Do not return error code in interrupt handler

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

 



On 07/04/2012 06:09 PM, Lars-Peter Clausen wrote:
> The interrupt handler should only ever return one of the three irqreturn_t
> constants and not an error code. Also make sure to always call
> iio_trigger_notify_done before leaving the trigger handler.
> 
> Signed-off-by: Lars-Peter Clausen <lars@xxxxxxxxxx>

Hm, looks like something went wrong here, I actually do have a different
patch in my local tree, this one will introduce a build warning...

> ---
>  drivers/staging/iio/adc/max1363_ring.c |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/staging/iio/adc/max1363_ring.c b/drivers/staging/iio/adc/max1363_ring.c
> index b302013..bf6b0c2 100644
> --- a/drivers/staging/iio/adc/max1363_ring.c
> +++ b/drivers/staging/iio/adc/max1363_ring.c
> @@ -64,11 +64,11 @@ static irqreturn_t max1363_trigger_handler(int irq, void *p)
>  	 * no harm.
>  	 */
>  	if (numvals == 0)
> -		return IRQ_HANDLED;
> +		goto done;
>  
>  	rxbuf = kmalloc(d_size,	GFP_KERNEL);
>  	if (rxbuf == NULL)
> -		return -ENOMEM;
> +		goto done;
>  	if (st->chip_info->bits != 8)
>  		b_sent = i2c_master_recv(st->client, rxbuf, numvals*2);
>  	else

--
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