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

From: Andy Shevchenko
Date: Wed Nov 14 2018 - 09:04:42 EST


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.

--
With Best Regards,
Andy Shevchenko