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 Wed, Nov 29, 2023 at 05:21:28PM +0100, Nuno Sá wrote:
> On Wed, 2023-11-29 at 18:18 +0200, Andy Shevchenko wrote:
> > On Wed, Nov 29, 2023 at 05:08:41PM +0100, Nuno Sá wrote:
> > > On Wed, 2023-11-29 at 15:49 +0100, Linus Walleij wrote:
> > > > On Fri, Nov 24, 2023 at 3:18 PM Nuno Sa via B4 Relay
> > > > <devnull+nuno.sa.analog.com@xxxxxxxxxx> wrote:

...

> > > > Other than that it looks fine.
> > > 
> > > Cool, I actually thought that having the direction + get/set stuff would be
> > > weird given the fact that we can only PULL_LOW or HIGH_Z the pins.
> > 
> > Oh, then it's probably should be a pin config involved.
> 
> Well, that was what I stated in the cover [1]. Please give a look and see if what I'm
> saying in there makes any sense.

I could tell only WRT GPIO/pin control. And I think what you would benefit from
is the pin configuration and "pin control as basic GPIO" (note, latter is not
in the upstream yet, but was submitted for review, interesting idea nevertheless).

> [1]: https://lore.kernel.org/linux-hwmon/20231110151905.1659873-1-nuno.sa@xxxxxxxxxx/

-- 
With Best Regards,
Andy Shevchenko






[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux