| Message ID | 20240909163608.80541-1-martyn.welch@collabora.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | arm64: dts: ls1028a-rdb: Add iio-hwmon compatibility table | expand |
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 > > Signed-off-by: Martyn Welch <martyn.welch@collabora.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>; > };
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@collabora.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>; > > };
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>; };
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. Signed-off-by: Martyn Welch <martyn.welch@collabora.com> --- arch/arm64/boot/dts/freescale/fsl-ls1028a-rdb.dts | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-)