From: Binbin Zhou <zhoubinbin@loongson.cn>
To: "Binbin Zhou" <zhoubb.aaron@gmail.com>,
"Huacai Chen" <chenhuacai@loongson.cn>,
"Uwe Kleine-König" <ukleinek@kernel.org>,
"Rob Herring" <robh+dt@kernel.org>,
"Krzysztof Kozlowski" <krzysztof.kozlowski+dt@linaro.org>,
"Conor Dooley" <conor+dt@kernel.org>,
"Juxin Gao" <gaojuxin@loongson.cn>
Cc: Huacai Chen <chenhuacai@kernel.org>,
linux-pwm@vger.kernel.org, devicetree@vger.kernel.org,
Xuerui Wang <kernel@xen0n.name>,
loongarch@lists.linux.dev, Binbin Zhou <zhoubinbin@loongson.cn>
Subject: [PATCH v8 0/2] pwm: Introduce pwm driver for the Loongson family chips
Date: Tue, 10 Dec 2024 20:37:04 +0800 [thread overview]
Message-ID: <cover.1733823417.git.zhoubinbin@loongson.cn> (raw)
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset=a, Size: 3475 bytes --]
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.
-------
V8:
patch (2/2):
- Rebase on pwm/for-next;
- Drop inappropriate comments in “Limitations”;
- Drop HZ_PER_KHZ for readability;
- NANOHZ_PER_HZ -> NSEC_PER_SEC;
- Rewrite the clk fetch section to look more flexible and not have to
care about ACPI or DT;
- Add explicit initialization of the CTRL register in probe().
Link to V7:
https://lore.kernel.org/all/cover.1729583747.git.zhoubinbin@loongson.cn/
V7:
Thanks for Sean's advice.
patch (2/2):
- Set chip->atomic to keep pwm_apply_atomic() can be used with the pwm.
- Test with CONFIG_PWM_DEBUG and CONFIG_DEBUG_ATOMIC_SLEEP enabled.
Link to V6:
https://lore.kernel.org/all/cover.1728463622.git.zhoubinbin@loongson.cn/
V6:
patch (2/2):
- Rebase on pwm/for-next;
- Add Reference Manual;
- Shortcut if !pwm->state.enabled;
- When state->enabled is true, unconditionally execute
pwm_loongson_set_polarity() to avoid that the polarity register is
not set correctly.
Link to V5:
https://lore.kernel.org/all/cover.1720516327.git.zhoubinbin@loongson.cn/
V5:
patch (2/2):
- Rebase on pwm/for-next;
- Test with PWM_DEBUG enabled.
- In pwm_loongson_apply(), the pwm state is determined before the pwm
polarity, avoid test failures when PWM_DEBUG is enabled;
- Added DIV64_U64_ROUND_UP in pwm_loongson_get_state() to avoid
precision loss and to avoid test failures when PWM_DEBUG is enabled.
Link to V4:
https://lore.kernel.org/all/cover.1716795485.git.zhoubinbin@loongson.cn/
V4:
patch (2/2):
- Rebase on pwm/for-next;
- Addressed Uwe's review comments:
- 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();
- Fix some code style.
Link to V3:
https://lore.kernel.org/linux-pwm/cover.1713164810.git.zhoubinbin@loongson.cn/
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@loongson.cn/
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@loongson.cn/
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 | 296 ++++++++++++++++++
5 files changed, 382 insertions(+)
create mode 100644 Documentation/devicetree/bindings/pwm/loongson,ls7a-pwm.yaml
create mode 100644 drivers/pwm/pwm-loongson.c
base-commit: 483082d78a092a3c1f343a76a2edb196069b4092
--
2.43.5
next reply other threads:[~2024-12-10 12:37 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-12-10 12:37 Binbin Zhou [this message]
2024-12-10 12:37 ` [PATCH v8 1/2] dt-bindings: pwm: Add Loongson PWM controller Binbin Zhou
2024-12-10 12:37 ` [PATCH v8 2/2] pwm: Add Loongson PWM controller support Binbin Zhou
2025-02-10 18:26 ` Uwe Kleine-König
2025-02-11 8:02 ` Binbin Zhou
2025-02-11 17:36 ` Uwe Kleine-König
2025-02-12 7:51 ` Binbin Zhou
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=cover.1733823417.git.zhoubinbin@loongson.cn \
--to=zhoubinbin@loongson.cn \
--cc=chenhuacai@kernel.org \
--cc=chenhuacai@loongson.cn \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=gaojuxin@loongson.cn \
--cc=kernel@xen0n.name \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=linux-pwm@vger.kernel.org \
--cc=loongarch@lists.linux.dev \
--cc=robh+dt@kernel.org \
--cc=ukleinek@kernel.org \
--cc=zhoubb.aaron@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).