[PATCH 1/3] iio: Add ADM1177 Hot Swap Controller and Digital Power Monitor driver

Hennerich, Michael Michael.Hennerich at analog.com
Wed Nov 13 08:10:50 UTC 2019



> -----Original Message-----
> From: Guenter Roeck <groeck7 at gmail.com> On Behalf Of Guenter Roeck
> Sent: Dienstag, 12. November 2019 20:18
> To: Jonathan Cameron <jonathan.cameron at huawei.com>
> Cc: Bia, Beniamin <Beniamin.Bia at analog.com>; jic23 at kernel.org;
> lars at metafoo.de; Hennerich, Michael <Michael.Hennerich at analog.com>;
> pmeerw at pmeerw.net; gregkh at linuxfoundation.org; linux-
> iio at vger.kernel.org; devel at driverdev.osuosl.org; linux-
> kernel at vger.kernel.org; mark.rutland at arm.com; robh+dt at kernel.org;
> devicetree at vger.kernel.org; paulmck at linux.ibm.com;
> mchehab+samsung at kernel.org; linus.walleij at linaro.org;
> nicolas.ferre at microchip.com; biabeniamin at outlook.com; Jean Delvare
> <jdelvare at suse.com>
> Subject: Re: [PATCH 1/3] iio: Add ADM1177 Hot Swap Controller and Digital
> Power Monitor driver
> 
> On Tue, Nov 12, 2019 at 05:37:57PM +0000, Jonathan Cameron wrote:
> > On Tue, 12 Nov 2019 17:35:50 +0200
> > Beniamin Bia <beniamin.bia at analog.com> wrote:
> >
> > > From: Michael Hennerich <michael.hennerich at analog.com>
> > >
> > > ADM1177 is a Hot Swap Controller and Digital Power Monitor with Soft
> > > Start Pin.
> > >
> > > Datasheet:
> > > Link:
> > > https://www.analog.com/media/en/technical-documentation/data-
> sheets/
> > > ADM1177.pdf
> > >
> > > Signed-off-by: Michael Hennerich <michael.hennerich at analog.com>
> > > Co-developed-by: Beniamin Bia <beniamin.bia at analog.com>
> > > Signed-off-by: Beniamin Bia <beniamin.bia at analog.com>
> >
> > Hi Beniamin,
> >
> > A couple immediate thoughts.
> >
> > 1. That cc list has some rather non obvious people on it.  Unless something
> >    fairly surprising is going on, probably better to cut it back a bit.
> >
> > 2. Why IIO?  Not entirely obvious to me.  From first glance this is definitely
> >    doing hardware monitoring.  If there is a reason there should be a clear
> >    statement here on why.
> >
> 
> I don't see why this is implemented as iio driver. I think it should be
> implemented as hardware monitoring driver.

Totally agree that this driver could have been implemented as HWMON driver.
Well we use this device as USB supply monitor on our embedded portably kits, to detect low VBUS or excess current draw. 

ADALM-PLUTO and ADALM2000:
https://www.analog.com/en/design-center/evaluation-hardware-and-software/evaluation-boards-kits/adalm-pluto.html

https://www.analog.com/en/design-center/evaluation-hardware-and-software/evaluation-boards-kits/ADALM2000.html

The only connectivity to the host PC is via IIO/libiio USB Gadget FS and Ethernet backends.

We recommend people to read the IIO attributes whenever they report an issue.
Unless libiio supports directly HWMON or HWMON adds an IIO bridge we would prefer this driver being exposed as IIO device, since HWMON users still can use the IIO/HWMON bridge.

-Michael



More information about the devel mailing list