On Wed, 15 Mar 2023, Andy Shevchenko wrote:

+Cc: Nicolas

On Tue, Mar 14, 2023 at 12:46:48PM -0700, Richard Cochran wrote:

quoted

On Tue, Mar 14, 2023 at 12:47:03PM +0200, Andy Shevchenko wrote:

quoted

The semantics of the above is similar to gpiod_get_optional() and since NULL
is a valid return in such cases, the PTP has to handle this transparently to
the user. Otherwise it's badly designed API which has to be fixed.

Does it now?  Whatever.

quoted

TL;DR: If I'm mistaken, I would like to know why.

git log.  git blame.

Get to know the tools of trade.

So, the culprit seems the commit d1cbfd771ce8 ("ptp_clock: Allow for it
to be optional") which did it half way.

Now I would like to know why the good idea got bad implementation.

Nicolas?

I'd be happy to help but as presented I simply don't know what you're 
talking about. Please give me more context.


Nicolas