linux-kernel - Re: arm64: dts: ls1028a-rdb: Add iio-hwmon compatibility table

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] [day] [month] [year] [list]

Message-ID: <d161352f2bdf7da1b6d299818c6c665ad48ab665.camel@collabora.com>
Date: Tue, 10 Sep 2024 09:19:41 +0100
From: Martyn Welch <martyn.welch@...labora.com>
To: Frank Li <Frank.li@....com>
Cc: Shawn Guo <shawnguo@...nel.org>, Rob Herring <robh@...nel.org>, 
 Krzysztof Kozlowski	 <krzk+dt@...nel.org>, Conor Dooley
 <conor+dt@...nel.org>, kernel@...labora.com, 
	linux-arm-kernel@...ts.infradead.org, devicetree@...r.kernel.org, 
	linux-kernel@...r.kernel.org
Subject: Re: arm64: dts: ls1028a-rdb: Add iio-hwmon compatibility table

On Mon, 2024-09-09 at 14:17 -0400, Frank Li wrote:
> On Mon, Sep 09, 2024 at 05:36:07PM +0100, Martyn Welch wrote:
> > The ls1028ardb uses the TI ina220 power monitor. This device has an
> > IIO and
> > hwmon drivers, but can't be used with both at the same time.
> 
> > To enable us
> > to utilise the iio framework, but also access power information via
> > the
> > hwmon framework, add an iio-hwmon map. This allows the IIO driver
> > to be
> > used for the ina220, with the iio-hwmon driver mapping the data
> > from the
> > IIO framework into hwmon.
> 
> Can you simply said:
> 
> Add an iio-hwmon map to utilise the iio framework and access power
> information via the hwmon framework at the same time.
> 
> Frank
> 

Sure

Martyn

> > Signed-off-by: Martyn Welch <martyn.welch@...labora.com>
> > ---
> >  arch/arm64/boot/dts/freescale/fsl-ls1028a-rdb.dts | 9 ++++++++-
> >  1 file changed, 8 insertions(+), 1 deletion(-)
> > 
> > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1028a-rdb.dts
> > b/arch/arm64/boot/dts/freescale/fsl-ls1028a-rdb.dts
> > index ecd2c1e..b062de5 100644
> > --- a/arch/arm64/boot/dts/freescale/fsl-ls1028a-rdb.dts
> > +++ b/arch/arm64/boot/dts/freescale/fsl-ls1028a-rdb.dts
> > @@ -94,6 +94,12 @@ simple-audio-card,codec {
> >  			system-clock-frequency = <25000000>;
> >  		};
> >  	};
> > +
> > +	iio-hwmon {
> > +		compatible = "iio-hwmon";
> > +		io-channels = <&ina220 0>, <&ina220 1>,
> > +			<&ina220 2>, <&ina220 3>;
> > +	};
> >  };
> > 
> >  &can0 {
> > @@ -222,8 +228,9 @@ i2c@2 {
> >  			#size-cells = <0>;
> >  			reg = <0x02>;
> > 
> > -			current-monitor@40 {
> > +			ina220: current-monitor@40 {
> >  				compatible = "ti,ina220";
> > +				#io-channel-cells = <1>;
> >  				reg = <0x40>;
> >  				shunt-resistor = <500>;
> >  			};