On Mon, May 27, 2024 at 3:51 PM Binbin Zhou <zhoubinbin@xxxxxxxxxxx> wrote: > > Hi all: > > This patchset introduce a generic PWM framework driver for Loongson family. > Each PWM has one pulse width output signal and one pulse input signal to be measured. > > It can be found on Loongson-2K series cpus and Loongson LS7A bridge chips. > > Thanks. > > ------- > V4: > patch (2/2): > - Rebase on pwm/for-next; > - Addressed Uwe's review comments, thanks. > - Make use of devm_pwmchip_alloc() function; > - Add Limitations description; > - Add LOONGSON_ prefix for Loongson pwm register defines; > - Keep regs written only once; > - Rewrite duty/period calculation; > - Add dev_err_probe() in .probe(); > - Put the parameters used by PM into a separate > structure(pwm_loongson_suspend_store); > - Fix some code style. > > Link to V3: > https://lore.kernel.org/linux-pwm/cover.1713164810.git.zhoubinbin@xxxxxxxxxxx/ Hi Uwe: Gentle ping. I seem to have addressed your review comments for the v3 patchset, please review it when you have time. Thanks. Binbin > > V3: > patch (1/2): > - Add Reviewed-by tag from Krzysztof, thanks. > patch (2/2): > - Several code stlye adjustments, such as line breaks. > > Link to V2: > https://lore.kernel.org/all/cover.1712732719.git.zhoubinbin@xxxxxxxxxxx/ > > v2: > - Remove the dts-related patches and update dts at once after all > relevant drivers are complete. > patch (1/2): > - The dt-binding filename should match compatible, rename it as > loongson,ls7a-pwm.yaml; > - Update binding description; > - Add description for each pwm cell; > - Drop '#pwm-cells' from required, for pwm.yaml makes it required already. > > Link to v1: > https://lore.kernel.org/linux-pwm/cover.1711953223.git.zhoubinbin@xxxxxxxxxxx/ > > Binbin Zhou (2): > dt-bindings: pwm: Add Loongson PWM controller > pwm: Add Loongson PWM controller support > > .../bindings/pwm/loongson,ls7a-pwm.yaml | 66 ++++ > MAINTAINERS | 7 + > drivers/pwm/Kconfig | 12 + > drivers/pwm/Makefile | 1 + > drivers/pwm/pwm-loongson.c | 295 ++++++++++++++++++ > 5 files changed, 381 insertions(+) > create mode 100644 Documentation/devicetree/bindings/pwm/loongson,ls7a-pwm.yaml > create mode 100644 drivers/pwm/pwm-loongson.c > > -- > 2.43.0 >