Re: [PATCH v3 01/12] dt-bindings: clock: renesas,r8a78000-cpg: Add renesas,scmi-firmware property

Hi Sudeep,

On Fri, 4 Sept 2026 at 16:03, Sudeep Holla [off-list ref] wrote:

On Wed, Sep 02, 2026 at 01:29:17PM +0200, Geert Uytterhoeven wrote:

quoted

On systems using the ARM System Control and Management Interface (SCMI),
the Clock Pulse Generator hardware registers must not be accessed
directly.  Instead, the SCMI message protocol must be used for inquiry,
configuration, and control of clocks.

Why is this node needed in the devicetree at all or why is it not disabled
so that any driver in the OSPM is not probed at all.

The node is needed so the corresponding driver can bind against it.
The driver takes care of remapping DT hardware descriptions to the
abstractions offered by the SCMI firmware when SCMI must be used.

quoted

Add a "renesas,scmi-firmware" property to distinguish between systems
with and without SCMI, and to indicate which SCMI firmware interface
provider to communicate with.

I still don't see the need for this if you disable the node, so still

The node is not disabled.

NACK. Sorry if I have missed some discussions, please present the information
as part of this patch in the commit log if not the actual binding itself.

That's why we have cover letter, with links to previous versions and
discussions.
Thanks!

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

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