Re: [PATCH v3 0/6] R-Car X5H CPG/MDLC support

Hi Ulf,

On Mon, 10 Aug 2026 at 16:50, Geert Uytterhoeven <[email protected]> wrote:
> On Mon, 10 Aug 2026 at 16:18, Ulf Hansson <[email protected]> wrote:
> > On Wed, Aug 5, 2026 at 5:21 PM Geert Uytterhoeven
> > <[email protected]> wrote:
> > > This patch series adds minimal support for the Renesas R-Car X5H Clock
> > > Pulse Generator (CPG) and Module Controller (MDLC), to aim for replacing
> > > the dummy clocks in r8a78000.dtsi with proper hardware description.
> > > These two components are closely related, in the sense that they are
> > > both supposed to be accessed through the ARM System Control and
> > > Management Interface (SCMI) on the Ironhide development board.
> > >
> > > This series consists of three parts:
> > >   1. Patches 1-2 add DT bindings and DT binding definitions,
> > >   2. Patches 3-4 add minimal CPG and MDLC drivers,
> > >   3. Patches 5-6 update the DTS.
> > > Note that this series depends on v7.2-rc2, due to the use of
> > > <linux/device-id/of.h>.
> > >
> > > Changes compared to v2[1]:
> > >   - Add Acked-by, Reviewed-by,
> > >   - Add fixed-rate clocks to kerneldoc of r8a78000_cpg_priv.map,
> > >   - Use devm_clk_hw_register_fixed_rate(), drop unregister_fixed_clks(),
> > >   - Drop fill_clk_map(), as it is not really needed until SCMI remapping
> > >     support is added later,
> > >   - Replace <linux/mod_devicetable.h> by more specific
> > >     <linux/device-id/of.h> in v7.2-rc2,
> > >   - Add a comment to document only always-on domains are supported,
> > >   - Consistently use /* sentinel */ for zero sentinels,
> > >   - Drop trailing commas from sentinels,
> > >   - Sort MDLC nodes (alphabetically, by label).
> > >
> > > Major changes compared to v1[2] (more detail in the individual patches):
> > >   - Add default support not using SCMI,
> > >   - Drop everything related to SCMI.
> > >
> > > Note that this series does not use or rely on SCMI in any way.
> > > SCMI remapping support will be added in a subsequent series (old version
> > > at [3]).
> > > Full hardware support without using SCMI could be added, too, if anyone
> > > would care about that use case.
> > >
> > > For testing, I have pushed this series plus additional work (including
> > > a rebased and updated version of [3]) to [4].
> > >
> > > While I know it is getting late in the cycle, this work (especially the
> > > DT bindings) are a dependency for about any R-Car X5H progress, so it
> > > would be good to have it included in v7.3.
> > > Hence I plan to queue the clock patches (1, 3) in renesas-clk for v7.3
> > > (Stephen still hasn't merged my earlier PR for v7.3 anyway).
> > > Ulf: It would be great if you could take the pmdomain patches (2, 4).
> >
> > Patch 2 applied for next, but I left patch4 for now as it didn't apply
>
> Thank you!
>
> > cleanly (drivers/soc/renesas/Kconfig had a conflict), thanks!
>
> I guess your tree still has the duplicate "default ARCH_RENESAS"?
> Patch 4 just adds a single line there, and the conflict should be easy
> to fix.
>
> If you want to send me a new patch, please let me know.

I have sent a v4 against pmdomain/next:
https://lore.kernel.org/all/d4bd0cc7f26c1bcdc260edb994b4971cb5dd3a10.1786438424.git.geert+renesas@glider.be/

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds