Re: [PATCH] staging: IIO: DAC: Delete useless label error_ret inad5791.c

From: Jonathan Cameron
Date: Fri Aug 12 2011 - 10:13:49 EST


On 08/12/11 09:51, Jonathan Cameron wrote:
> On 08/06/11 19:09, stufever@xxxxxxxxx wrote:
>> From: Wang Shaoyan <wangshaoyan.pt@xxxxxxxxxx>
>>
>> drivers/staging/iio/dac/ad5791.c: In function 'ad5791_probe':
>> drivers/staging/iio/dac/ad5791.c:384: warning: label 'error_ret' defined but not used
>>
>> The label error_ret is defined, but never used, so delete it to avoid annoying warning.
> As with other one, this is fixed in iio-blue.git, but I'm happy to see either version merge.
I'm dozing. This is the third fix for this one. Greg, pick up which ever one you want!
>>
>> Signed-off-by: Wang Shaoyan <wangshaoyan.pt@xxxxxxxxxx>
> Acked-by: Jonathan Cameron <jic23@xxxxxxxxx>
>> ---
>> drivers/staging/iio/dac/ad5791.c | 1 -
>> 1 files changed, 0 insertions(+), 1 deletions(-)
>>
>> diff --git a/drivers/staging/iio/dac/ad5791.c b/drivers/staging/iio/dac/ad5791.c
>> index 64770d2..5b00b70 100644
>> --- a/drivers/staging/iio/dac/ad5791.c
>> +++ b/drivers/staging/iio/dac/ad5791.c
>> @@ -381,7 +381,6 @@ error_put_reg_pos:
>> if (!IS_ERR(reg_vdd))
>> regulator_put(reg_vdd);
>>
>> -error_ret:
>> return ret;
>> }
>>
>
> --
> 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
>

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/