linux-kernel - Re: [PATCH v1 0/2] adding support for Microchip PAC193X Power Monitor

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <178ee962c5fc7ee7806475cb38527b8bdbfa8d09.camel@microchip.com>
Date:   Mon, 6 Mar 2023 14:03:52 +0000
From:   <Marius.Cristea@...rochip.com>
To:     <jic23@...nel.org>
CC:     <devicetree@...r.kernel.org>, <lars@...afoo.de>,
        <linux-iio@...r.kernel.org>, <robh+dt@...nel.org>,
        <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v1 0/2] adding support for Microchip PAC193X Power Monitor


Hi Jonathan,

  Please, see my comments below...


On Sat, 2023-02-25 at 17:11 +0000, Jonathan Cameron wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you
> know the content is safe
> 
> On Mon, 20 Feb 2023 14:32:30 +0200
> <marius.cristea@...rochip.com> wrote:
> 
> > From: Marius Cristea <marius.cristea@...rochip.com>
> > 
> > Adding support for Microchip PAC193X series of Power Monitor with
> > Accumulator chip family.
> 
> This device is at the messy boundary between IIO and HWMON. Perhaps
> call out
> the reasons you think IIO is more appropriate in this cover letter.
> + Often a good idea for these borderline parts to cc both mailing
> lists and
> maintainers.
> 
> Often the conclusion is that it is fine to have these in IIO because
> we can
> bridge to hwmon anyway with the iio-hwmon driver.
> 

Indeed the driver (the device) is at the boundary between IIO and
HWMON. I was thinking to start with a simple driver (this one that is
more apropiate to be a HWMON) and add more functionality later (like
data buffering that is quite important for example if someone wants to
profile power consumtion of the procesor itself, or a pheriperic, or a
battery)


> > 
> > Differences related to previous patch:
> > 
> > v1:
> > - first version comitted to review
> > 
> > 
> > Marius Cristea (2):
> >   dt-bindings: iio: adc: adding dt-bindings for PAC193X
> >   iio: adc: adding support for pac193x
> > 
> >  .../bindings/iio/adc/microchip,pac193x.yaml   |  122 +
> >  MAINTAINERS                                   |    7 +
> >  drivers/iio/adc/Kconfig                       |   12 +
> >  drivers/iio/adc/Makefile                      |    1 +
> >  drivers/iio/adc/pac193x.c                     | 2072
> > +++++++++++++++++
> >  5 files changed, 2214 insertions(+)
> >  create mode 100644
> > Documentation/devicetree/bindings/iio/adc/microchip,pac193x.yaml
> >  create mode 100644 drivers/iio/adc/pac193x.c
> > 
> 

Thanks,
Marius