Messages in this thread | | | Subject | Re: [PATCH v2 06/16] hwmon: (mr75203) fix multi-channel voltage reading | Date | Mon, 22 Aug 2022 15:37:20 +0300 | From | "Farber, Eliav" <> |
| |
On 8/18/2022 11:03 PM, Guenter Roeck wrote: > On Wed, Aug 17, 2022 at 05:43:11AM +0000, Eliav Farber wrote: >> - Fix voltage reading to support number of channels in VM IP (CH_NUM). >> - Configure the ip-polling register to enable polling for all channels. >> > > That fails to explain what is actually wrong in the current code. > Also, one fix per patch, please. I moved the configuration of the ip-polling register to a separate patch.
The problem in the current code is that it allocates in_config according to the total number of voltage monitors and not according to the total number of channels in all voltage monitors. Therefore it didn’t create enough sysfs to read all inputs. Also pvr_read_in() only tries to access the first channel in each voltage monitor. I will add this explanation to next version.
-- Thanks, Eliav
| |