> Subject: Re: [PATCH v4 1/4] dt-bindings: mailbox: fsl,mu: add i.MX95 > Generic/ELE/V2X MU compatible > > On Thu, Jan 25, 2024 at 01:20:03PM +0800, Peng Fan (OSS) wrote: > > From: Peng Fan <peng.fan@xxxxxxx> > > > > Add i.MX95 Generic, Secure Enclave and V2X Message Unit compatible > string. > > And some MUs has internal RAMs for SCMI shared buffer usage. > > Please restrict the SRAM child none to whatever the "some MUs" are that > actually have it. Ok, will update it in V5. Thanks, Peng. > > Thanks, > Conor. > > > > > Signed-off-by: Peng Fan <peng.fan@xxxxxxx> > > --- > > .../devicetree/bindings/mailbox/fsl,mu.yaml | 52 > +++++++++++++++++++++- > > 1 file changed, 50 insertions(+), 2 deletions(-) > > > > diff --git a/Documentation/devicetree/bindings/mailbox/fsl,mu.yaml > > b/Documentation/devicetree/bindings/mailbox/fsl,mu.yaml > > index 12e7a7d536a3..86759831b24a 100644 > > --- a/Documentation/devicetree/bindings/mailbox/fsl,mu.yaml > > +++ b/Documentation/devicetree/bindings/mailbox/fsl,mu.yaml > > @@ -29,10 +29,14 @@ properties: > > - const: fsl,imx8ulp-mu > > - const: fsl,imx8-mu-scu > > - const: fsl,imx8-mu-seco > > - - const: fsl,imx93-mu-s4 > > - const: fsl,imx8ulp-mu-s4 > > + - const: fsl,imx93-mu-s4 > > + - const: fsl,imx95-mu-ele > > + - const: fsl,imx95-mu-v2x > > - items: > > - - const: fsl,imx93-mu > > + - enum: > > + - fsl,imx93-mu > > + - fsl,imx95-mu > > - const: fsl,imx8ulp-mu > > - items: > > - enum: > > @@ -95,6 +99,19 @@ properties: > > power-domains: > > maxItems: 1 > > > > + ranges: true > > + > > + '#address-cells': > > + const: 1 > > + > > + '#size-cells': > > + const: 1 > > + > > +patternProperties: > > + "^sram@[a-f0-9]+": > > + $ref: /schemas/sram/sram.yaml# > > + unevaluatedProperties: false > > + > > required: > > - compatible > > - reg > > @@ -134,3 +151,34 @@ examples: > > interrupts = <GIC_SPI 176 IRQ_TYPE_LEVEL_HIGH>; > > #mbox-cells = <2>; > > }; > > + > > + - | > > + #include <dt-bindings/interrupt-controller/arm-gic.h> > > + > > + mu2: mailbox@445b0000 { > > + compatible = "fsl,imx95-mu", "fsl,imx8ulp-mu"; > > + reg = <0x445b0000 0x10000>; > > + ranges; > > + interrupts = <GIC_SPI 226 IRQ_TYPE_LEVEL_HIGH>; > > + #address-cells = <1>; > > + #size-cells = <1>; > > + #mbox-cells = <2>; > > + > > + sram@445b1000 { > > + compatible = "mmio-sram"; > > + reg = <0x445b1000 0x400>; > > + ranges = <0x0 0x445b1000 0x400>; > > + #address-cells = <1>; > > + #size-cells = <1>; > > + > > + scmi_buf0: scmi-sram-section@0 { > > + compatible = "arm,scmi-shmem"; > > + reg = <0x0 0x80>; > > + }; > > + > > + scmi_buf1: scmi-sram-section@80 { > > + compatible = "arm,scmi-shmem"; > > + reg = <0x80 0x80>; > > + }; > > + }; > > + }; > > > > -- > > 2.37.1 > >