On Fri, Sep 29, 2023 at 11:36:46AM +0100, Jon Hunter wrote: > The current INA3221 driver always sums the shunt voltage for all enabled > channels regardless of the shunt-resistor used for each channel. Summing > the shunt-voltage for channels is only meaningful if the shunt resistor > is the same for each channel. This series adds device-tree support to > allow which channels are summed in device-tree. > > Changes since V4: > - Moved dt-binding comment added in V4 from patch #2 to patch #1. > > Changes since V3: > - Added missing descriptions for new structure members that was reported > by the kernel-test-bot. > - Added comment in the ina3221 dt-binding doc example to explain why we > need to explicitly disable channels. > - Added more commentary in the commit message for the new DT property > to explain that this property does not change the behaviour of the > driver unless it is populated. > > Changes since V2: > - Added note to binding-doc to indicate that input channels must be > explicitly disabled. > - Corrected ordering of properties in the binding-doc > - Updated license for the binding-doc to be dual licensed. > - Changed newly added property from 'summation-bypass' to > summation-disable'. > - Documented type for the new 'summation-disable' property. > - Corrected spelling and comments as per the feedback received. > - Used debugfs instead of sysfs for exposing the 'summation-disable' > status for each input channel. > - Populated missing instances for the ina3221 device for Tegra234 > boards. > - Populated ina219 device for the NVIDIA IGX board (not strictly > related to this series but related to populating all > power-sensors for Tegra234 boards) > > Changes since V1: > - Added yaml conversion patch for binding-doc > - Added binding-doc documentation patch for new property > - Added patch to populate ina3221 devices for Tegra234. > > Jon Hunter (2): > dt-bindings: hwmon: ina3221: Add ti,summation-disable > arm64: tegra: Add power-sensors for Tegra234 boards > > Ninad Malwade (2): > dt-bindings: hwmon: ina3221: Convert to json-schema > hwmon: ina3221: Add support for channel summation disable Jean, Guenter, do you mind if I pick up patches 1, 2 and 4 into the Tegra tree? It's usually convenient to keep the DT bindings and DT additions in the same tree for validation. Thanks, Thierry
Attachment:
signature.asc
Description: PGP signature
