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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID:
 <DU2PR04MB85990A0AB8AF8ABFCDA4CBD995AB2@DU2PR04MB8599.eurprd04.prod.outlook.com>
Date: Thu, 25 Jul 2024 07:06:30 +0000
From: Pankaj Gupta <pankaj.gupta@....com>
To: Conor Dooley <conor@...nel.org>
CC: Jonathan Corbet <corbet@....net>, Rob Herring <robh@...nel.org>, Krzysztof
 Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>, Shawn Guo
	<shawnguo@...nel.org>, Sascha Hauer <s.hauer@...gutronix.de>, Pengutronix
 Kernel Team <kernel@...gutronix.de>, Fabio Estevam <festevam@...il.com>, Rob
 Herring <robh+dt@...nel.org>, "linux-doc@...r.kernel.org"
	<linux-doc@...r.kernel.org>, "linux-kernel@...r.kernel.org"
	<linux-kernel@...r.kernel.org>, "devicetree@...r.kernel.org"
	<devicetree@...r.kernel.org>, "imx@...ts.linux.dev" <imx@...ts.linux.dev>,
	"linux-arm-kernel@...ts.infradead.org" <linux-arm-kernel@...ts.infradead.org>
Subject: RE: [EXT] Re: [PATCH v6 2/5] dt-bindings: arm: fsl: add imx-se-fw
 binding doc



> -----Original Message-----
> From: Conor Dooley <conor@...nel.org>
> Sent: Wednesday, July 24, 2024 9:00 PM
> To: Pankaj Gupta <pankaj.gupta@....com>
> Cc: Jonathan Corbet <corbet@....net>; Rob Herring <robh@...nel.org>;
> Krzysztof Kozlowski <krzk+dt@...nel.org>; Conor Dooley
> <conor+dt@...nel.org>; Shawn Guo <shawnguo@...nel.org>; Sascha Hauer
> <s.hauer@...gutronix.de>; Pengutronix Kernel Team
> <kernel@...gutronix.de>; Fabio Estevam <festevam@...il.com>; Rob
> Herring <robh+dt@...nel.org>; linux-doc@...r.kernel.org; linux-
> kernel@...r.kernel.org; devicetree@...r.kernel.org; imx@...ts.linux.dev;
> linux-arm-kernel@...ts.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@...nel.org>
> > > Sent: Tuesday, July 23, 2024 7:38 PM
> > > To: Pankaj Gupta <pankaj.gupta@....com>
> > > Cc: Jonathan Corbet <corbet@....net>; Rob Herring <robh@...nel.org>;
> > > Krzysztof Kozlowski <krzk+dt@...nel.org>; Conor Dooley
> > > <conor+dt@...nel.org>; Shawn Guo <shawnguo@...nel.org>; Sascha
> Hauer
> > > <s.hauer@...gutronix.de>; Pengutronix Kernel Team
> > > <kernel@...gutronix.de>; Fabio Estevam <festevam@...il.com>; Rob
> > > Herring <robh+dt@...nel.org>; linux-doc@...r.kernel.org; linux-
> > > kernel@...r.kernel.org; devicetree@...r.kernel.org;
> > > imx@...ts.linux.dev; linux-arm-kernel@...ts.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@...nel.org>
> > > > > Sent: Monday, July 22, 2024 10:20 PM
> > > > > To: Pankaj Gupta <pankaj.gupta@....com>
> > > > > Cc: Jonathan Corbet <corbet@....net>; Rob Herring
> > > > > <robh@...nel.org>; Krzysztof Kozlowski <krzk+dt@...nel.org>;
> > > > > Conor Dooley <conor+dt@...nel.org>; Shawn Guo
> > > > > <shawnguo@...nel.org>; Sascha
> > > Hauer
> > > > > <s.hauer@...gutronix.de>; Pengutronix Kernel Team
> > > > > <kernel@...gutronix.de>; Fabio Estevam <festevam@...il.com>; Rob
> > > > > Herring <robh+dt@...nel.org>; linux-doc@...r.kernel.org; linux-
> > > > > kernel@...r.kernel.org; devicetree@...r.kernel.org;
> > > > > imx@...ts.linux.dev; linux-arm-kernel@...ts.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.