On Sun, May 3, 2020 at 1:54 PM Miquel Raynal <miquel.raynal@xxxxxxxxxxx> wrote: > > The MAX7313 chip is fully compatible with the PCA9535 on its basic > functions but can also manage the intensity on each of its ports with > PWM. Each output is independent and may be tuned with 16 values (4 > bits per output). The period is always 32kHz, only the duty-cycle may > be changed. One can use any output as GPIO or PWM. Besides the messing with parameter types (int vs. bool) it should be rebased on top of Bartosz's tree. Also, it might be that we can instantiate a kind of device (MFD?) that will share same regmap between two and have naturally different drivers for GPIO and PWM. Side note: I still think this should be a function of the pin when driver will be converted to pin control. Now this change delays the conversion and better approach. But as I said before, if GPIO maintainers consider this good enough to go like this, I won't object. Some background. It's known that some pin control devices may have PWM function (limited, like blinking led or so, or full) and it would be nice to have a pin mux option which enables PWM on a requested pin. Or PWM tries to enable proper pin muxing (this sounds even better in order of sharing same API from pin control, like "pin is available for GPIO"). -- With Best Regards, Andy Shevchenko