Re: [PATCH v1 2/5] extcon: Return -EPROBE_DEFER when extcon device is not found

From: Chanwoo Choi
Date: Wed Nov 14 2018 - 20:17:05 EST


On 2018ë 11ì 14ì 23:04, Andy Shevchenko wrote:
> On Wed, Nov 14, 2018 at 1:17 PM Andy Shevchenko
> <andy.shevchenko@xxxxxxxxx> wrote:
>> On Wed, Nov 14, 2018 at 1:05 PM Chanwoo Choi <cw00.choi@xxxxxxxxxxx> wrote:
>
>>>> Changing NULL to -ENODEV is a trading bad to worse.
>
>> P.S. I still disagree with your arguments in relation to de facto use of an API.
>
> I spoke to colleagues of mine and they are agree that semantically
> -EPROBE_DEFER is a wrong error code from API that matches against some
> list.
> On the other hand, they agree with me that changing NULL to -ENODEV is
> a trading bad to worse.
>
> So, I withdraw mine complaints against API.
>

OK. Thanks.

--
Best Regards,
Chanwoo Choi
Samsung Electronics