Received-SPF: pass (google.com: domain of linux-kernel+bounces-205409-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
From: Vabhav Sharma <vabhav.sharma@nxp.com>
To: Krzysztof Kozlowski <krzk@kernel.org>, Frank Li <frank.li@nxp.com>
CC: Rob Herring <robh@kernel.org>, Krzysztof Kozlowski <krzk+dt@kernel.org>,
	Conor Dooley <conor+dt@kernel.org>, Franck Lenormand
	<franck.lenormand@nxp.com>, Aisheng Dong <aisheng.dong@nxp.com>, Shawn Guo
	<shawnguo@kernel.org>, Sascha Hauer <s.hauer@pengutronix.de>, Pengutronix
 Kernel Team <kernel@pengutronix.de>, Fabio Estevam <festevam@gmail.com>, Peng
 Fan <peng.fan@nxp.com>, "devicetree@vger.kernel.org"
	<devicetree@vger.kernel.org>, "linux-kernel@vger.kernel.org"
	<linux-kernel@vger.kernel.org>, "imx@lists.linux.dev" <imx@lists.linux.dev>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>, Varun Sethi <V.Sethi@nxp.com>,
	Silvano Di Ninno <silvano.dininno@nxp.com>, Pankaj Gupta
	<pankaj.gupta@nxp.com>, Daniel Baluta <daniel.baluta@nxp.com>
Subject: RE: [EXT] Re: [PATCH 1/4] dt-bindings: firmware: secvio: Add device
 tree bindings
Thread-Topic: [EXT] Re: [PATCH 1/4] dt-bindings: firmware: secvio: Add device
 tree bindings
Thread-Index: AQHaoao2jC57TZpcF0ePF6O5Tvr8yrGOOJWAgAAusQCALYQ9YA==
Date: Fri, 7 Jun 2024 05:00:01 +0000
Message-ID:
 <AS1PR04MB9358029DB6E685ED7693047DF3FB2@AS1PR04MB9358.eurprd04.prod.outlook.com>
References: <20240509-secvio-v1-0-90fbe2baeda2@nxp.com>
 <20240509-secvio-v1-1-90fbe2baeda2@nxp.com>
 <Zjw9044yBRn9+adW@lizhi-Precision-Tower-5810>
 <52deb076-f2d0-4c61-9dac-079e3ae881b6@kernel.org>
In-Reply-To: <52deb076-f2d0-4c61-9dac-079e3ae881b6@kernel.org>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0:
 =?us-ascii?Q?0hBhVcAkyMEJiH1zZnA8eqsBl9FBqoE22XiyaJain0O85Em8VIuh0OuoMOXx?=
 =?us-ascii?Q?hkC+cA8b32iAlzqPwD1OoiGC5wfV/k3sF+GBxFjm2sGwGD2/eXuc06y+6XL9?=
 =?us-ascii?Q?i7prPc9cgcidf5oVfsvsvR9br0rIgDefVv6MQkMQxlxfDa6xFmyRTqCGC7ZQ?=
 =?us-ascii?Q?DfD49YetTnYNzHDLs2ymoiMdJBDCQOsuEbMYQWID6n/psVWu6XB5QYUqp3aY?=
 =?us-ascii?Q?AF5PCX5ZFhfCGXGo5FiaZ+Alhvqm7B+DV1R3X40tQecYqrNb9KZFS0yXnMqL?=
 =?us-ascii?Q?raUlp5l/3cL8UqVsEFzMGFTAVW4N24c4mQsZix2BSYyZiHWc3WtNg3IgZHTr?=
 =?us-ascii?Q?GNIFarWq/bx1a60nzZA4XpkohQoMDOj6LRWLbTk8T9UyIGuOcYnG9IdvIVYj?=
 =?us-ascii?Q?tcIFa9vZxlmmJXlVzhZLBtDGxfro2tAoy9c6BmYIEjChJ/au75FHUmLMM8uP?=
 =?us-ascii?Q?cRAamFtmmxBmIj+ph3rAAK8keUSX5s8jjytDsRGOn3DvNcE001Sqt6Y00WU2?=
 =?us-ascii?Q?parEjdhf4owKtX20r37yGBkxqBk2VRgXwmSX+7l5CgLxqrVzoY3cco+3aQGy?=
 =?us-ascii?Q?fVos4MrW2HH/X+vR6FiDq7bTXdRPF7JwpJPCjXxak9lahPkj/jCL9HVknblz?=
 =?us-ascii?Q?MpsBUZvUHjj32l9JDAz9i9Nqapi68/mv9QBkWwW/SB3NdHO/OivlivAOBpqv?=
 =?us-ascii?Q?RiKgvr4J1FuLtZemX6664GcbRdQqRkx6x7hXIGutTbmq3Fo+GaKyGR7xPkP2?=
 =?us-ascii?Q?QgWDP3K1cKEB79o1zvkWOfnDg2440tB7ntjTg9ePN59/TxnNyEINnKdTCKuk?=
 =?us-ascii?Q?wU/7u7Y3DJm6Ok3/OJP+mBwt5qd05q8LEfJKtKRkkTPOdpByLjlIaXdxCQyX?=
 =?us-ascii?Q?mWpUXX7Z2KUbQwc3XqOyAxUaHpCOx50m/jKr5QtXs+UyUMBN7PaQaG2kFb7y?=
 =?us-ascii?Q?jl1p0aRfsgXSeiyES16TFoTNU7F9pJNPwe7U7G2d2KE9TeIFEYi8Yuo353So?=
 =?us-ascii?Q?e9OS/UziizT/2rfWHEwzvvj82YiXFxzXL67lgrQ+XaN2622vwLysmK8ClVpG?=
 =?us-ascii?Q?UcAnF01bJ7qOxxBiZUBuQF/QLdiTod3wQVXJZEcTWXvV9qC3+Nns0Bk62wC0?=
 =?us-ascii?Q?+yQ1fYGi2Uf7JF+sNsZzRZcvHscbSjnvFD+ZUrI459FQ6zKTHJJx18Ud2Xng?=
 =?us-ascii?Q?X+aXFfOnfWzVndFf1ubR392BirFvKotC3JY1fDWiuW4fpOuuvLJIRmSYfkjy?=
 =?us-ascii?Q?wpyQlM6j3NZn+Pegt6V8USfTwgOVvZTl4UgRSSDmA3Rjs005l3WnHInwAow/?=
 =?us-ascii?Q?zVXlrpjyqO1g2fgQWb/pNzbLq3rwUtISuCyCzKFpw0jPm83+tmi/O97SW0gJ?=
 =?us-ascii?Q?KMLmo/XVxKpLBT/+8wfAHU+MmT7gyUZ5T6iQ4lSuOj4g2LBY9GPpo95TjAdE?=
 =?us-ascii?Q?3BNTNxHs8H4jh/IRXl9RcB15wXlVpYl6gKU+qA89z9XgckLRv7YMACALavtW?=
 =?us-ascii?Q?nkvNWxwHj59dvN8JBAZ/xw0nC0nv0H1POMq4LhuJccRguVtJVhzRcz6kespn?=
 =?us-ascii?Q?59XNH1I4SQT1VlZWhtPB1VtHkL9Dz+iuUojAtCSy?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk
X-Mailing-List: linux-kernel@vger.kernel.org
List-Id: <linux-kernel.vger.kernel.org>
List-Subscribe: <mailto:linux-kernel+subscribe@vger.kernel.org>
List-Unsubscribe: <mailto:linux-kernel+unsubscribe@vger.kernel.org>
MIME-Version: 1.0
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: AS1PR04MB9358.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: db2ea682-57f0-48d1-b70a-08dc86aeb042
X-MS-Exchange-CrossTenant-originalarrivaltime: 07 Jun 2024 05:00:01.9163
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: pnguPAiDNpbgfmDL3+Tga6OIHRKgK1N34X3d0UG0wbqcJ4gFIzCZiKgoe5jUJlKQeqvdtk8QSu5bb4R8EjjiAQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB9PR04MB9283


> -----Original Message-----
> From: Krzysztof Kozlowski <krzk@kernel.org>
> Sent: Thursday, May 9, 2024 11:24 AM
> To: Frank Li <frank.li@nxp.com>; Vabhav Sharma <vabhav.sharma@nxp.com>
> Cc: Rob Herring <robh@kernel.org>; Krzysztof Kozlowski
> <krzk+dt@kernel.org>; Conor Dooley <conor+dt@kernel.org>; Franck
> Lenormand <franck.lenormand@nxp.com>; Aisheng Dong
> <aisheng.dong@nxp.com>; Shawn Guo <shawnguo@kernel.org>; Sascha
> Hauer <s.hauer@pengutronix.de>; Pengutronix Kernel Team
> <kernel@pengutronix.de>; Fabio Estevam <festevam@gmail.com>; Peng Fan
> <peng.fan@nxp.com>; devicetree@vger.kernel.org; linux-
> kernel@vger.kernel.org; imx@lists.linux.dev; linux-arm-
> kernel@lists.infradead.org; Varun Sethi <V.Sethi@nxp.com>; Silvano Di Nin=
no
> <silvano.dininno@nxp.com>; Pankaj Gupta <pankaj.gupta@nxp.com>; Daniel
> Baluta <daniel.baluta@nxp.com>
> Subject: [EXT] Re: [PATCH 1/4] dt-bindings: firmware: secvio: Add device =
tree
> bindings
>=20
> Caution: This is an external email. Please take care when clicking links =
or
> opening attachments. When in doubt, report the message using the 'Report
> this email' button
>=20
>=20
> On 09/05/2024 05:06, Frank Li wrote:
> > On Thu, May 09, 2024 at 02:45:32AM +0200, Vabhav Sharma wrote:
> >> Document the secvio device tree bindings.
> >
> > reduntant sentence.
> >>
> >> The tampers are security feature available on i.MX products and
> >> managed by SNVS block.The tamper goal is to detect the variation
> >                         ^^ space here
> >
> >> of hardware or physical parameters, which can indicate an attack.
> >>
> >> The SNVS, which provides secure non-volatile storage, allows to
> >> detect some hardware attacks against the SoC.They are connected
> >                                                ^^ space here
> >> to the security-violation ports, which send an alert when an
> >> out-of-range value is detected.
> >>
> >> The "imx-secvio-sc" module is designed to report security violations
> >> and tamper triggering via SCU firmware to the user.
> >>
> >> Add the imx-scu secvio sub node and secvio sub node description.
> >>
> >> Signed-off-by: Franck LENORMAND <franck.lenormand@nxp.com>
> >> Signed-off-by: Vabhav Sharma <vabhav.sharma@nxp.com>
> >> ---
> >>  .../bindings/arm/freescale/fsl,scu-secvio.yaml     | 35
> ++++++++++++++++++++++
> >>  .../devicetree/bindings/firmware/fsl,scu.yaml      | 10 +++++++
> >>  2 files changed, 45 insertions(+)
> >>
> >> diff --git
> >> a/Documentation/devicetree/bindings/arm/freescale/fsl,scu-secvio.yaml
> >> b/Documentation/devicetree/bindings/arm/freescale/fsl,scu-secvio.yaml
> >> new file mode 100644
> >> index 000000000000..30dc1e21f903
> >> --- /dev/null
> >> +++ b/Documentation/devicetree/bindings/arm/freescale/fsl,scu-secvio.
> >> +++ yaml
> >> @@ -0,0 +1,35 @@
> >> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) %YAML 1.2
> >> +---
> >> +$id:
> >> +https://eur01.safelinks.protection.outlook.com/?url=3Dhttp%3A%2F%2Fde=
v
> >> +icetree.org%2Fschemas%2Farm%2Ffreescale%2Ffsl%2Cscu-
> secvio.yaml%23&d
> >>
> +ata=3D05%7C02%7Cvabhav.sharma%40nxp.com%7C16a07379ee384ddc18f908
> dc6fec
> >>
> +75e7%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C63850830857
> 3434788
> >>
> +%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIi
> LCJBTiI
> >>
> +6Ik1haWwiLCJXVCI6Mn0%3D%7C0%7C%7C%7C&sdata=3DMBhqXwhXIQjDb3A
> RdYJ4U5EXM
> >> +ryEy%2F9m5X6jGuNhHxo%3D&reserved=3D0
> >> +$schema:
> >> +https://eur01.safelinks.protection.outlook.com/?url=3Dhttp%3A%2F%2Fde=
v
> >> +icetree.org%2Fmeta-
> schemas%2Fcore.yaml%23&data=3D05%7C02%7Cvabhav.shar
> >>
> +ma%40nxp.com%7C16a07379ee384ddc18f908dc6fec75e7%7C686ea1d3bc2
> b4c6fa9
> >>
> +2cd99c5c301635%7C0%7C0%7C638508308573446476%7CUnknown%7CTWF
> pbGZsb3d8
> >>
> +eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3
> D%7
> >>
> +C0%7C%7C%7C&sdata=3Dm0RzUoVfr%2F2HkLlSOjhTq%2FQX3EM6ZAW7h5hQ
> Eidnc1g%3D
> >> +&reserved=3D0
> >> +
> >> +title: NXP i.MX Security Violation driver
> >
> > Violation detect driver
>=20
> Bindings are not for drivers.
This is security violation detection hardware exported through SCU firmware=
. I will detail the HW in the binding
>=20
> Best regards,
> Krzysztof