On 10/26/11 12:16, michael.hennerich@xxxxxxxxxx wrote:
> From: Michael Hennerich <michael.hennerich@xxxxxxxxxx>
>
> Free channels in case read fails with error.
>
> Signed-off-by: Michael Hennerich <michael.hennerich@xxxxxxxxxx>
Acked-by: Jonathan Cameron <jic23@xxxxxxxxx>
> ---
> drivers/staging/iio/adc/ad7280a.c | 3 ++-
> 1 files changed, 2 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/staging/iio/adc/ad7280a.c b/drivers/staging/iio/adc/ad7280a.c
> index 0c212e0..f45b66b 100644
> --- a/drivers/staging/iio/adc/ad7280a.c
> +++ b/drivers/staging/iio/adc/ad7280a.c
> @@ -688,7 +688,7 @@ static irqreturn_t ad7280_event_handler(int irq, void *private)
>
> ret = ad7280_read_all_channels(st, st->scan_cnt, channels);
> if (ret < 0)
> - return IRQ_HANDLED;
> + goto out;
>
> for (i = 0; i < st->scan_cnt; i++) {
> if (((channels[i] >> 23) & 0xF) <= AD7280A_CELL_VOLTAGE_6) {
> @@ -731,6 +731,7 @@ static irqreturn_t ad7280_event_handler(int irq, void *private)
> }
> }
>
> +out:
> kfree(channels);
>
> return IRQ_HANDLED;
--
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
