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

[Pankaj Gupta]

> -----Original Message-----
> From: Conor Dooley <conor at kernel.org>
> Sent: Wednesday, July 24, 2024 9:00 PM
> To: Pankaj Gupta <pankaj.gupta at nxp.com>
> Cc: Jonathan Corbet <corbet at lwn.net>; Rob Herring <robh at kernel.org>;
> Krzysztof Kozlowski <krzk+dt at kernel.org>; Conor Dooley
> <conor+dt at kernel.org>; Shawn Guo <shawnguo at kernel.org>; Sascha Hauer
> <s.hauer at pengutronix.de>; Pengutronix Kernel Team
> <kernel at pengutronix.de>; Fabio Estevam <festevam at gmail.com>; Rob
> Herring <robh+dt at kernel.org>; linux-doc at vger.kernel.org; linux-
> kernel at vger.kernel.org; devicetree at vger.kernel.org; imx at lists.linux.dev;
> linux-arm-kernel at lists.infradead.org
> Subject: Re: [EXT] Re: [PATCH v6 2/5] dt-bindings: arm: fsl: add imx-se-fw
> binding doc
> 
> On Wed, Jul 24, 2024 at 11:02:21AM +0000, Pankaj Gupta wrote:
> >
> >
> > > -----Original Message-----
> > > From: Conor Dooley <conor at kernel.org>
> > > Sent: Tuesday, July 23, 2024 7:38 PM
> > > To: Pankaj Gupta <pankaj.gupta at nxp.com>
> > > Cc: Jonathan Corbet <corbet at lwn.net>; Rob Herring <robh at kernel.org>;
> > > Krzysztof Kozlowski <krzk+dt at kernel.org>; Conor Dooley
> > > <conor+dt at kernel.org>; Shawn Guo <shawnguo at kernel.org>; Sascha
> Hauer
> > > <s.hauer at pengutronix.de>; Pengutronix Kernel Team
> > > <kernel at pengutronix.de>; Fabio Estevam <festevam at gmail.com>; Rob
> > > Herring <robh+dt at kernel.org>; linux-doc at vger.kernel.org; linux-
> > > kernel at vger.kernel.org; devicetree at vger.kernel.org;
> > > imx at lists.linux.dev; linux-arm-kernel at lists.infradead.org
> > > Subject: Re: [EXT] Re: [PATCH v6 2/5] dt-bindings: arm: fsl: add
> > > imx-se-fw binding doc
> 
> Please fix this ^
> 
> > > On Tue, Jul 23, 2024 at 09:28:31AM +0000, Pankaj Gupta wrote:
> > > >
> > > > > -----Original Message-----
> > > > > From: Conor Dooley <conor at kernel.org>
> > > > > Sent: Monday, July 22, 2024 10:20 PM
> > > > > To: Pankaj Gupta <pankaj.gupta at nxp.com>
> > > > > Cc: Jonathan Corbet <corbet at lwn.net>; Rob Herring
> > > > > <robh at kernel.org>; Krzysztof Kozlowski <krzk+dt at kernel.org>;
> > > > > Conor Dooley <conor+dt at kernel.org>; Shawn Guo
> > > > > <shawnguo at kernel.org>; Sascha
> > > Hauer
> > > > > <s.hauer at pengutronix.de>; Pengutronix Kernel Team
> > > > > <kernel at pengutronix.de>; Fabio Estevam <festevam at gmail.com>; Rob
> > > > > Herring <robh+dt at kernel.org>; linux-doc at vger.kernel.org; linux-
> > > > > kernel at vger.kernel.org; devicetree at vger.kernel.org;
> > > > > imx at lists.linux.dev; linux-arm-kernel at lists.infradead.org
> > > > > Subject: [EXT] Re: [PATCH v6 2/5] dt-bindings: arm: fsl: add
> > > > > imx-se-fw binding doc
> > >
> > > Please fix this ^
> > >
> > > > >
> > > > > On Mon, Jul 22, 2024 at 10:21:37AM +0530, Pankaj Gupta wrote:
> > In case of imx8ulp, there is a single node.
> > Having a same node name for both parent and child, is bit strange.
> > firmware {
> > 	firmware {
> > 	};
> > };
> >
> > Request you to allow to re-evaluate this point.
> 
> I dunno, it's all firmware so I don't really get why it is so strange!
> Can you remind me again why it is inside a parent node to begin with?

Three type of security firmware(s): HSM, V2X-SHE, V2X-HSM, are running at the cores dedicated to the each different secure-enclave hardware IP(s).
Each firmware receives the message to act and response back with the completed act.
This message exchanges happens through the Message-Unit hardware interface. 
There could be multiple MU for multiple security firmware, that would be used for respective message exchanges.

This node defines the details of each such MU interface.

Reason to put under firmware:
	Since this node specifies interface details between kernel and firmware, it was put under parent "firmware {".
	I am not sure if this reason is correct enough to begin with.

Thanks for allowing to revisit.

I will make the change to whatever you finalize now. Thanks.