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 12/1/23 08:24, Andy Shevchenko wrote:
On Fri, Dec 01, 2023 at 08:04:12AM -0800, Guenter Roeck wrote:
On 12/1/23 07:47, 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".


It seems to me that this is heading towards a mfd driver. I don't feel comfortable
with all that gpio specific code in the hwmon subsystem.

Maybe I should request that all hwmon chips with gpio support must be implemented
as mfd drivers. I'll have to think about that.

Or auxiliary bus?


I don't know. All I can see it that it is getting way too complicated for a hwmon
driver with embedded gpio support.

Guenter





[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux