2018-01-06 16:54:44

by Egil Hjelmeland

[permalink] [raw]
Subject: Re: [PATCH 0/2] net: dsa: lan9303: check error value from devm_gpiod_get_optional()

Den 13. nov. 2017 09:07, skrev Phil Reid:
> Replaces Pan Bian <[email protected]> patch
> "net: dsa: lan9303: correctly check return value of devm_gpiod_get_optional"
>
> Errors need to be prograted back from probe.
>
> Note: I have only compile tested the code as I don't have the hardware.
>
> Phil Reid (2):
> net: dsa: lan9303: make lan9303_handle_reset() a void function
> net: dsa: lan9303: check error value from devm_gpiod_get_optional()
>
> drivers/net/dsa/lan9303-core.c | 21 ++++++++++++---------
> 1 file changed, 12 insertions(+), 9 deletions(-)
>

Evidently this one did not make it through. Do you care to rebase and
try again?

Thanks
Egil


2018-01-09 08:39:09

by Phil Reid

[permalink] [raw]
Subject: Re: [PATCH 0/2] net: dsa: lan9303: check error value from devm_gpiod_get_optional()

On 7/01/2018 00:54, Egil Hjelmeland wrote:
> Den 13. nov. 2017 09:07, skrev Phil Reid:
>> Replaces Pan Bian <[email protected]> patch
>> "net: dsa: lan9303: correctly check return value of devm_gpiod_get_optional"
>>
>> Errors need to be prograted back from probe.
>>
>> Note: I have only compile tested the code as I don't have the hardware.
>>
>> Phil Reid (2):
>>    net: dsa: lan9303: make lan9303_handle_reset() a void function
>>    net: dsa: lan9303: check error value from devm_gpiod_get_optional()
>>
>>   drivers/net/dsa/lan9303-core.c | 21 ++++++++++++---------
>>   1 file changed, 12 insertions(+), 9 deletions(-)
>>
>
> Evidently this one did not make it through. Do you care to rebase and try again?
>
> Thanks
> Egil
>
>
Sure I'll give it another go.

--
Regards
Phil Reid