Re: [PATCH v2 2/2] hwmon: ltc4282: add support for the LTC4282 chip

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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?

- Nuno Sá 





[Index of Archives]     [LM Sensors]     [Linux Sound]     [ALSA Users]     [ALSA Devel]     [Linux Audio Users]     [Linux Media]     [Kernel]     [Gimp]     [Yosemite News]     [Linux Media]

  Powered by Linux