Re: [PATCH V4 0/4] hwmon: ina3221: Add selective summation support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On 29/09/2023 11:27, 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 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.


Ignore this version. I messed up which patch where I added the new comment for the dt-binding doc.

Jon

--
nvpublic



[Index of Archives]     [ARM Kernel]     [Linux ARM]     [Linux ARM MSM]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux