Re: [PATCH v2 05/10] dt-bindings: clock: renesas,r8a78000-cpg: Add firmware property

[Krzysztof Kozlowski <krzk@kernel.org>]

On Wed, Jul 08, 2026 at 12:16:23PM +0200, Geert Uytterhoeven wrote:
> Add a firmware property, for systems where SCMI must be used instead of
> direct hardware access.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---
> v2:
>   - Spin off from "dt-bindings: clock: Document Renesas R-Car X5H Clock
>     Pulse Generator".
> ---
>  .../devicetree/bindings/clock/renesas,r8a78000-cpg.yaml    | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/clock/renesas,r8a78000-cpg.yaml b/Documentation/devicetree/bindings/clock/renesas,r8a78000-cpg.yaml
> index 4e23bba334382346..fc499e7cf52e4f0c 100644
> --- a/Documentation/devicetree/bindings/clock/renesas,r8a78000-cpg.yaml
> +++ b/Documentation/devicetree/bindings/clock/renesas,r8a78000-cpg.yaml
> @@ -35,6 +35,12 @@ properties:
>        <dt-bindings/clock/renesas,r8a78000-cpg.h>.
>      const: 1
>  
> +  firmware:

RPi is doing similar, but I have doubts. It looks way too close to
generic "syscon" which is disallowed.

I think this should be vendor+purpose specific, e.g.:
"renesas,scmi-transport" or "renesas,scmi-firmware"

Also we have "firmware" nodes, so this might cause conflicts in dtschema
validation.  Anyway, the phandle itself is fine for me, just the naming.

I'll ping also Rob to chime in.

Best regards,
Krzysztof