Re: [PATCH] [WAR] hwmon: (ina3221) Apply software WAR to offset shunt voltage

[Jon Hunter]

On 17/11/2022 14:47, Guenter Roeck wrote:
> On Thu, Nov 17, 2022 at 04:32:26PM +0800, Ninad Malwade wrote:
> > This is used as a software WAR to offset shunt voltage reading
>
> What is a software WAR ? Or a WAR in the first place ?
> What is the relevance of the "[WAR]" tag in the subject ?
>
> None of the definitions at https://acronyms.thefreedictionary.com/WAR
> seem to apply. I am sure it means something for you, and it seems to be
> important enough to use the term repeatedly, but please do not assume
> that others know what it means.

Indeed. I will work with Ninad to correct this.

> > from INA3221 to increase its accuracy. This patch implements a
> > previous downstream feature by reading the offset information
> > from DT and apply it to current readings.
>
> Where is the devicetree documentation ?

Yes that is also missing. Let me sync up with Ninad offline and get this 
re-worked properly.

Jon

--
nvpublic