From: Jon Hunter <jonathanh@nvidia.com>
To: Guenter Roeck <linux@roeck-us.net>, Ninad Malwade <nmalwade@nvidia.com>
Cc: jdelvare@suse.com, robh+dt@kernel.org,
krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org,
thierry.reding@gmail.com, linux-hwmon@vger.kernel.org,
devicetree@vger.kernel.org, linux-tegra@vger.kernel.org
Subject: Re: [PATCH V2 0/4] hwmon: ina3221: Add selective summation support
Date: Tue, 29 Aug 2023 13:52:44 +0100 [thread overview]
Message-ID: <b6ca9be7-49b2-f72e-b51f-b778786e0646@nvidia.com> (raw)
In-Reply-To: <482ac044-e163-478c-8e67-5f03d7dc7820@roeck-us.net>
On 25/08/2023 18:11, Guenter Roeck wrote:
> On Sat, Aug 26, 2023 at 12:42:45AM +0800, Ninad Malwade 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.
>>
>
> V2, but no change log. I am not going to review this series.
Sorry about that the changelog is ...
Changes since V1:
- Added patch to convert dt-binding to json
- Added patch to add new dt property for bypassing/disabling summation
support for a channel
- Added patch to populate ina3221 devices for Tegra234 boards
- Updated summation bypass patch to display summation status via sysfs
Jon
--
nvpublic
next prev parent reply other threads:[~2023-08-29 12:53 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-08-25 16:42 [PATCH V2 0/4] hwmon: ina3221: Add selective summation support Ninad Malwade
2023-08-25 16:42 ` [PATCH V2 1/4] dt-bindings: hwmon: ina3221: Convert to json-schema Ninad Malwade
2023-08-26 8:53 ` Krzysztof Kozlowski
2023-08-29 12:46 ` Jon Hunter
2023-08-29 17:16 ` Krzysztof Kozlowski
2023-09-01 16:34 ` Jon Hunter
2023-09-01 16:44 ` Guenter Roeck
2023-09-05 12:21 ` Jon Hunter
2023-08-25 16:42 ` [PATCH V2 2/4] dt-bindings: hwmon: ina3221: Add summation-bypass Ninad Malwade
2023-08-26 8:56 ` Krzysztof Kozlowski
2023-08-29 12:48 ` Jon Hunter
2023-08-29 17:19 ` Krzysztof Kozlowski
2023-08-25 16:42 ` [PATCH V2 3/4] hwmon: ina3221: add support for summation channel control Ninad Malwade
2023-08-25 21:08 ` kernel test robot
2023-08-29 13:27 ` Guenter Roeck
2023-08-25 16:42 ` [PATCH V2 4/4] arm64: tegra: Populate ina3221 for Tegra234 boards Ninad Malwade
2023-08-25 17:11 ` [PATCH V2 0/4] hwmon: ina3221: Add selective summation support Guenter Roeck
2023-08-29 12:52 ` Jon Hunter [this message]
2023-08-29 13:02 ` Guenter Roeck
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=b6ca9be7-49b2-f72e-b51f-b778786e0646@nvidia.com \
--to=jonathanh@nvidia.com \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=jdelvare@suse.com \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=linux-hwmon@vger.kernel.org \
--cc=linux-tegra@vger.kernel.org \
--cc=linux@roeck-us.net \
--cc=nmalwade@nvidia.com \
--cc=robh+dt@kernel.org \
--cc=thierry.reding@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).