Re: [PATCH v3 1/8] dt-bindings: clock: renesas: Split out RZ/T2H CPG binding
"Lad, Prabhakar" <[email protected]> Mon, 20 Jul 2026 11:50:10 +0100
| Newsgroups | org.kernel.vger.linux-watchdog,org.kernel.vger.linux-clk,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-kernel,org.kernel.vger.linux-renesas-soc |
|---|---|
| Message-ID | <CA+V-a8tfCL58j2TvoO1VhhDAZf=8LB89R4O0SycLghYGf7Rr4w@mail.gmail.com> |
Hi Krzysztof, On Mon, Jul 20, 2026 at 6:55 AM Krzysztof Kozlowski <[email protected]> wrote: > > On Thu, Jul 16, 2026 at 01:34:27PM +0100, Prabhakar wrote: > > From: Lad Prabhakar <[email protected]> > > > > Move the RZ/T2H and RZ/N2H clock controller bindings from the generic > > renesas,cpg-mssr.yaml schema into a dedicated binding. > > > > Unlike the traditional Renesas CPG/MSSR block, the RZ/T2H and RZ/N2H > > system controller integrates additional functionality beyond clock and > > module-stop control, including reset control, clock monitor support, > > write-protection registers and other system management features. > > Nothing of above differences got reflected in the code, so I do not > understand this reasoning. Binding is exactly the same as the old one. > A syscon regmap is implemented [0] as part of the RZ/T2H CPG driver. Since the RZ/T2H clock driver uses this syscon wrapper, a function pointer is implemented to handle the RZ/T2H-specific configuration. Also, as the initial implementation used separate device tree nodes, Geert suggested splitting this binding schema off into a dedicated file [1]. [0] https://lore.kernel.org/all/[email protected]/ [1] https://lore.kernel.org/all/CAMuHMdU62updtKeBBk-vO3YP+qaWTGqHQnzKvFJbrz4J5eEPMQ@mail.gmail.com/ Cheers, Prabhakar Cheers, Prabhakar