Messages in this thread | | | Date | Thu, 27 Apr 2023 03:32:56 -0700 | Subject | Re: [PATCH] hwmon: (max597x) Add Maxim Max597x | From | Guenter Roeck <> |
| |
On 4/27/23 02:50, Naresh Solanki wrote: > Hi Guenter, > [ ... ]
>> Again, I think it would make much more sense to add hwmon support >> to the regulator driver than having a separate driver since there >> is lots of overlap. For example, the regulator driver already >> sets and monitors limits. > I agree. But the chip also has led functionality. I got review feedback to make it MFD. So when rewriting as MFD driver made separate driver based on subsystem. I'm not sure if it is ok to use hwmon subsytem in regulator driver. Will once check with maintainer on this.
LED has completely different functionality. Overlap between regulator and hwmon is substantial because the regulator code handles limits and alarms. Sure, you kind of bypassed (ignored) that by not implementing limit and alarm support in the hwmon driver, but that is really just avoiding the problem, not solving it.
Guenter
| |