From: Alex Elder <elder@riscstar.com>
To: mturquette@baylibre.com, sboyd@kernel.org, robh@kernel.org,
krzk+dt@kernel.org, conor+dt@kernel.org
Cc: p.zabel@pengutronix.de, dlan@gentoo.org, heylenay@4d2.org,
guodong@riscstar.com, paul.walmsley@sifive.com,
palmer@dabbelt.com, aou@eecs.berkeley.edu,
spacemit@lists.linux.dev, devicetree@vger.kernel.org,
linux-clk@vger.kernel.org, linux-riscv@lists.infradead.org,
linux-kernel@vger.kernel.org
Subject: [PATCH v3 0/7] clk: spacemit: add K1 reset support
Date: Wed, 9 Apr 2025 16:17:33 -0500 [thread overview]
Message-ID: <20250409211741.1171584-1-elder@riscstar.com> (raw)
This series adds reset controller support for the SpacemiT K1 SoC.
As before, this version is built upon the clock controller driver that
Haylen Chu has out for review (currently at v6):
https://lore.kernel.org/lkml/20250401172434.6774-1-heylenay@4d2.org/
All of these patches are available here:
https://github.com/riscstar/linux/tree/outgoing/reset-v3
Between version 2 and version 3 there was no feedback, however:
- Haylen posted v6 of the clock series, and it included some changes
that affected the logic in this reset code.
- I was informed that defining CCU nodes without any clocks led to
warnings about "clocks" being a required property when running
"make dtbs_check". For that reason, I made clock properties
optional for reset-only CCU nodes.
- This code is now based on v6.15-rc1, which includes a few commits
that were listed as dependencies previously.
Here is version 2 of this series.
https://lore.kernel.org/lkml/20250328210233.1077035-1-elder@riscstar.com/
Between version 1 and version 2:
- Added Rob's Reviewed-by tag on the first patch
- Renamed the of_match_data data type (and one or two other symbols) to
use "spacemit" rather than "k1".
- Replaced the abbreviated "rst" or "RST" in names of newly-defined
sympols with "reset" or "RESET" respectively.
- Eliminated rcdev_to_controller(), which was only used once.
- Changed a function that unsafely did a read/modify/write of a register
to use regmap_update_bits() instead as suggested by Haylen.
- Eliminated a null check for a pointer known to be non-null.
- Reordered the assignment of reset controller device fields.
- Added a "sentinel" comment as requested by Yixun.
- Updated to be based on Linux v6.14 final.
Here is the first version of this series.
https://lore.kernel.org/lkml/20250321151831.623575-1-elder@riscstar.com/
-Alex
Alex Elder (7):
dt-bindings: soc: spacemit: define spacemit,k1-ccu resets
clk: spacemit: rename spacemit_ccu_data fields
clk: spacemit: add reset controller support
clk: spacemit: define existing syscon resets
clk: spacemit: make clocks optional
clk: spacemit: define new syscons with only resets
riscv: dts: spacemit: add reset support for the K1 SoC
.../soc/spacemit/spacemit,k1-syscon.yaml | 29 +-
arch/riscv/boot/dts/spacemit/k1.dtsi | 18 +
drivers/clk/spacemit/ccu-k1.c | 340 ++++++++++++++++--
.../dt-bindings/clock/spacemit,k1-syscon.h | 128 +++++++
4 files changed, 488 insertions(+), 27 deletions(-)
--
2.45.2
next reply other threads:[~2025-04-09 21:17 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-04-09 21:17 Alex Elder [this message]
2025-04-09 21:17 ` [PATCH v3 1/7] dt-bindings: soc: spacemit: define spacemit,k1-ccu resets Alex Elder
2025-04-10 6:25 ` Krzysztof Kozlowski
2025-04-09 21:17 ` [PATCH v3 2/7] clk: spacemit: rename spacemit_ccu_data fields Alex Elder
2025-04-09 21:17 ` [PATCH v3 3/7] clk: spacemit: add reset controller support Alex Elder
2025-04-09 21:17 ` [PATCH v3 4/7] clk: spacemit: define existing syscon resets Alex Elder
2025-04-09 21:17 ` [PATCH v3 5/7] clk: spacemit: make clocks optional Alex Elder
2025-04-09 21:17 ` [PATCH v3 6/7] clk: spacemit: define new syscons with only resets Alex Elder
2025-04-09 21:17 ` [PATCH v3 7/7] riscv: dts: spacemit: add reset support for the K1 SoC Alex Elder
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=20250409211741.1171584-1-elder@riscstar.com \
--to=elder@riscstar.com \
--cc=aou@eecs.berkeley.edu \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=dlan@gentoo.org \
--cc=guodong@riscstar.com \
--cc=heylenay@4d2.org \
--cc=krzk+dt@kernel.org \
--cc=linux-clk@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-riscv@lists.infradead.org \
--cc=mturquette@baylibre.com \
--cc=p.zabel@pengutronix.de \
--cc=palmer@dabbelt.com \
--cc=paul.walmsley@sifive.com \
--cc=robh@kernel.org \
--cc=sboyd@kernel.org \
--cc=spacemit@lists.linux.dev \
/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