RE: [EXT] Re: [PATCH v5 2/5] dt-bindings: arm: fsl: add imx-se-fw binding doc

From: Pankaj Gupta
Date: Fri Jul 19 2024 - 06:08:29 EST


Hi Conor,

> > > > +properties:
> > > > + $nodename:
> > > > + pattern: "firmware@[0-9a-f]+$"

As part of removing the reg property, it is required to remove the variable node name as well.

> > > > +
> > > > +additionalProperties: false
> > > > +
> > > > +examples:
> > > > + - |
> > > > + firmware {
Since node name "firmware" is a very generic name and broad reference.
It is a request to revisit the last suggestion to replace it with any one of the below close suggestions:

- se (Stands for secure enclave, already mentioned in the description).
- senclave (like sram: secure-ram)
- senclave-firmware
- se-fw

Kindly suggest.

Thanks.

> > > > + compatible = "fsl,imx95-se";
> > > > + mboxes = <&ele_mu0 0 0>, <&ele_mu0 1 0>;
> > > > + mbox-names = "tx", "rx";
> > > > + };
> > > > +...
> > > >
> > > > --
> > > > 2.34.1
> > > >