Re: [PATCH] hwmon: ina238: Add support for shunt voltage scaling

[Guenter Roeck <linux@roeck-us.net>]

On 1/20/25 17:18, Potin Lai wrote:
> On Tue, Jan 21, 2025 at 12:38 AM Guenter Roeck <linux@roeck-us.net> wrote:
>>
>> On 1/20/25 08:23, Potin Lai wrote:
>>> The INA238 sensor reports shunt voltage with microvolt precision.
>>> However, the hwmon driver currently exposes this value only in
>>> millivolts via `in0_input`, which results in a loss of precision for
>>> readings within the range of ±1 mV.
>>>
>>> This patch introduces an `in0_scale` attribute to provide the scaling
>>> factor applied to the shunt voltage reading. By exposing this attribute,
>>> users can accurately interpret the in0_input values in microvolts,
>>> preserving the sensor's full precision.
>>>
>>> Signed-off-by: Potin Lai <potin.lai.pt@gmail.com>
>>
>> Sorry, this is an ABI violation and thus a no-go. in0_input is and will
>> always be reported in mV.
>>
>> Guenter
>>
> 
> Hi Guenter,
> In our use case, the shunt voltage is less than 1 mv most of the time.
> I would like to get your advice on the correct way of getting
> microvolt reading from the driver? Thank you.
> 

Given that V = I * R, assuming you know the shunt resistor value,
it is an easy calculation which does not require any code changes.

Possible alternatives:
- Create a debugfs file to display the voltage in uV.
   See drivers/hwmon/isl28022.c for an example.
- Implement a hwmon -> iio bridge and use IIO to report
   the voltage.

Having said that, do you have an actual use case ? What is the point
of knowing the voltage across the shunt resistor ? "Because the chip
reports it" is not a use case. Literally _every_ current sensor chip
ultimately reports the shunt resistor voltage, after all (typically
as current; calculation see above). Commit 63fb21afc1f5 ("hwmon:
(ina2xx) Use shunt voltage to calculate current") is a case in point.

Guenter