On Fri, Dec 01, 2023 at 05:19:33PM +0100, Nuno Sá wrote: > On Fri, 2023-12-01 at 17:47 +0200, Andy Shevchenko wrote: > > On Fri, Dec 01, 2023 at 04:24:35PM +0100, Nuno Sá wrote: > > > On Fri, 2023-12-01 at 14:40 +0100, Linus Walleij wrote: ... > > > Yes, that is the only thing we have. Meaning that there is no hw setting to set > > > the > > > pins to open drain. Open drain is what they are. That is why I'm not seeing the > > > point > > > in having PIN_CONFIG_DRIVE_OPEN_DRAIN implemented. > > > > At least you have to implement error for PUSH_PULL mode and other modes, > > so from the (core) software point of view the user should be able to ask for > > anything and get an answer from the certain driver that "hey, i do support OD", > > or "hey, push-pull can't be supported with this hw". > > Yeah, that makes total sense. But I guess that the 'default' should already > -EOPNOTSUPP right? Without OP part (as internal Linux error code), yes. -- With Best Regards, Andy Shevchenko