Re: [PATCH] hwmon (ina3221) Add mutex lock to shunt nodes

2018-11-12 Thread Nicolin Chen
On Mon, Nov 12, 2018 at 08:31:22PM -0800, Guenter Roeck wrote: > On Mon, Nov 12, 2018 at 08:23:24PM -0800, Nicolin Chen wrote: > > The shunt resistor values are used to calculate shunt voltages > > and currents. As a part of sysfs nodes, it would be better to > > get protected with the same mutex t

Re: [PATCH] hwmon (ina3221) Add mutex lock to shunt nodes

2018-11-12 Thread Guenter Roeck
On Mon, Nov 12, 2018 at 08:23:24PM -0800, Nicolin Chen wrote: > The shunt resistor values are used to calculate shunt voltages > and currents. As a part of sysfs nodes, it would be better to > get protected with the same mutex too as other sysfs ABI nodes, > although this is not very critical becau

[PATCH] hwmon (ina3221) Add mutex lock to shunt nodes

2018-11-12 Thread Nicolin Chen
The shunt resistor values are used to calculate shunt voltages and currents. As a part of sysfs nodes, it would be better to get protected with the same mutex too as other sysfs ABI nodes, although this is not very critical because the mutex was added to mainly protect register access. So this pat