Re: [PATCH 3/3] hwmon: (adm9240) Convert to devm_hwmon_device_register_with_info API

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 5/12/21 2:54 PM, Chris Packham wrote:
Hi Guenter,

On 11/03/21 8:33 pm, Guenter Roeck wrote:
Also use regmap for register caching. This change reduces code and
data size by more than 40%.

While at it, fixed some warnings reported by checkpatch.

Cc: Chris Packham <Chris.Packham@xxxxxxxxxxxxxxxxxxx>
Signed-off-by: Guenter Roeck <linux@xxxxxxxxxxxx>

I've just been informed by our QA team that it looks like the
configuration of limits (e.g. by writing to sysfs) has been broken.
Probably by this change. I'm just starting to dig into it now but I
though I'd give you a heads up.


Thanks for the heads up.

It looks like voltage maximum writes use the wrong register,
ADM9240_REG_IN instead of ADM9240_REG_IN_MAX.
Odd, I'd have assumed that my module test code catches that.
I'll have to check why it doesn't.

Anyway, anything more specific ?

Thanks,
Guenter



[Index of Archives]     [LM Sensors]     [Linux Sound]     [ALSA Users]     [ALSA Devel]     [Linux Audio Users]     [Linux Media]     [Kernel]     [Gimp]     [Yosemite News]     [Linux Media]

  Powered by Linux