Re: [PATCH] leds/leds-pwm: Display error code on request_pwm failure

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

 



On Mon, Dec 16, 2013 at 5:26 PM, Bryan Wu <cooloney@xxxxxxxxx> wrote:
> On Mon, Dec 16, 2013 at 4:08 PM, Ben Gamari <bgamari.foss@xxxxxxxxx> wrote:
>> This can be useful in debugging devicetrees.
>>
>
> OK, I will merge this.
>


> Thanks,
> -Bryan
>
>> Signed-off-by: Ben Gamari <bgamari.foss@xxxxxxxxx>
>> ---
>>  drivers/leds/leds-pwm.c | 4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/leds/leds-pwm.c b/drivers/leds/leds-pwm.c
>> index b31d8e9..707a82a 100644
>> --- a/drivers/leds/leds-pwm.c
>> +++ b/drivers/leds/leds-pwm.c
>> @@ -165,8 +165,8 @@ static int led_pwm_probe(struct platform_device *pdev)
>>                         if (IS_ERR(led_dat->pwm)) {
>>                                 ret = PTR_ERR(led_dat->pwm);
>>                                 dev_err(&pdev->dev,
>> -                                       "unable to request PWM for %s\n",
>> -                                       cur_led->name);
>> +                                       "unable to request PWM for %s: error=%d\n",

Oh, hold on, this line has more than 80 characters. Please fix this
coding style error.

-Bryan

>> +                                       cur_led->name, ret);
>>                                 goto err;
>>                         }
>>
>> --
>> 1.8.3.2
>>
--
To unsubscribe from this list: send the line "unsubscribe linux-leds" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux