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

[Rob Herring]

On Mon, Aug 03, 2026 at 02:01:18PM +0200, Geert Uytterhoeven wrote:
> Hi Rob,
> 
> On Thu, 16 Jul 2026 at 23:49, Rob Herring <robh@xxxxxxxxxx> wrote:
> > On Fri, Jul 10, 2026 at 01:02:39PM +0200, Geert Uytterhoeven wrote:
> > > On Fri, 10 Jul 2026 at 12:21, Krzysztof Kozlowski <krzk@xxxxxxxxxx> wrote:
> > > > 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@xxxxxxxxx>
> > > > > ---
> > > > > 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"
> >
> > If you can tell me the resource that a 'firmware' provider provides then
> > it is fine, but you can't.
> 
> It provides a firmware API, which is not available until the firmware
> provider is initialized.

I don't know what "a firmware API" is. What does it provide? I know what 
a clock, GPIO, reset, interrupt, etc. provider is.

Rob