netdev - RE: [EXT] Re: [PATCH v9 2/6] arm64: dts: freescale: rename imx93.dtsi to imx91_93

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:
 <AS4PR04MB9386E069DA4BA0877C951F53E13BA@AS4PR04MB9386.eurprd04.prod.outlook.com>
Date: Thu, 28 Aug 2025 01:51:40 +0000
From: Joy Zou <joy.zou@....com>
To: Alexander Stein <alexander.stein@...tq-group.com>, "robh@...nel.org"
	<robh@...nel.org>, "krzk+dt@...nel.org" <krzk+dt@...nel.org>,
	"conor+dt@...nel.org" <conor+dt@...nel.org>, "shawnguo@...nel.org"
	<shawnguo@...nel.org>, "s.hauer@...gutronix.de" <s.hauer@...gutronix.de>,
	"kernel@...gutronix.de" <kernel@...gutronix.de>, "festevam@...il.com"
	<festevam@...il.com>, "richardcochran@...il.com" <richardcochran@...il.com>,
	"andrew+netdev@...n.ch" <andrew+netdev@...n.ch>, "davem@...emloft.net"
	<davem@...emloft.net>, "edumazet@...gle.com" <edumazet@...gle.com>,
	"kuba@...nel.org" <kuba@...nel.org>, "pabeni@...hat.com" <pabeni@...hat.com>,
	"mcoquelin.stm32@...il.com" <mcoquelin.stm32@...il.com>,
	"alexandre.torgue@...s.st.com" <alexandre.torgue@...s.st.com>,
	"frieder.schrempf@...tron.de" <frieder.schrempf@...tron.de>,
	"primoz.fiser@...ik.com" <primoz.fiser@...ik.com>, "othacehe@....org"
	<othacehe@....org>, "Markus.Niebel@...tq-group.com"
	<Markus.Niebel@...tq-group.com>
CC: "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"imx@...ts.linux.dev" <imx@...ts.linux.dev>,
	"linux-arm-kernel@...ts.infradead.org"
	<linux-arm-kernel@...ts.infradead.org>, "linux@...tq-group.com"
	<linux@...tq-group.com>, "netdev@...r.kernel.org" <netdev@...r.kernel.org>,
	"linux-pm@...r.kernel.org" <linux-pm@...r.kernel.org>,
	"linux-stm32@...md-mailman.stormreply.com"
	<linux-stm32@...md-mailman.stormreply.com>, Frank Li <frank.li@....com>
Subject: RE: [EXT] Re: [PATCH v9 2/6] arm64: dts: freescale: rename imx93.dtsi
 to imx91_93_common.dtsi and modify them

Hi Alexande，
> -----Original Message-----
> From: Alexander Stein <alexander.stein@...tq-group.com>
> Sent: 2025年8月27日 16:33
> To: robh@...nel.org; krzk+dt@...nel.org; conor+dt@...nel.org;
> shawnguo@...nel.org; s.hauer@...gutronix.de; kernel@...gutronix.de;
> festevam@...il.com; richardcochran@...il.com; andrew+netdev@...n.ch;
> davem@...emloft.net; edumazet@...gle.com; kuba@...nel.org;
> pabeni@...hat.com; mcoquelin.stm32@...il.com;
> alexandre.torgue@...s.st.com; frieder.schrempf@...tron.de;
> primoz.fiser@...ik.com; othacehe@....org;
> Markus.Niebel@...tq-group.com; Joy Zou <joy.zou@....com>
> Cc: devicetree@...r.kernel.org; linux-kernel@...r.kernel.org;
> imx@...ts.linux.dev; linux-arm-kernel@...ts.infradead.org;
> linux@...tq-group.com; netdev@...r.kernel.org; linux-pm@...r.kernel.org;
> linux-stm32@...md-mailman.stormreply.com; Frank Li <frank.li@....com>
> Subject: [EXT] Re: [PATCH v9 2/6] arm64: dts: freescale: rename imx93.dtsi to
> imx91_93_common.dtsi and modify them
> 
> 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
> 
> 
> Hi,
> 
> Am Montag, 25. August 2025, 11:12:19 CEST schrieb Joy Zou:
> > The design of i.MX91 platform is very similar to i.MX93 and only
> > some small differences.
> >
> > If the imx91.dtsi include the imx93.dtsi, each add to imx93.dtsi
> > requires an remove in imx91.dtsi for this unique to i.MX93, e.g. NPU.
> > The i.MX91 isn't the i.MX93 subset, if the imx93.dtsi include the
> > imx91.dtsi, the same problem will occur.
> >
> > Common + delta is better than common - delta, so add
> imx91_93_common.dtsi
> > for i.MX91 and i.MX93, then the imx93.dtsi and imx91.dtsi will include the
> > imx91_93_common.dtsi.
> >
> > Rename imx93.dtsi to imx91_93_common.dtsi and move i.MX93 specific
> > part from imx91_93_common.dtsi to imx93.dtsi.
> >
> > Reviewed-by: Frank Li <Frank.Li@....com>
> > Signed-off-by: Joy Zou <joy.zou@....com>
> > ---
> > Changes for v7:
> > 1.The aliases are removed from common.dtsi because the imx93.dtsi aliases
> >   are removed.
> >
> > Changes for v6:
> > 1. merge rename imx93.dtsi to imx91_93_common.dtsi and move i.MX93
> >    specific part from imx91_93_common.dtsi to imx93.dtsi patch.
> > 2. restore copyright time and add modification time.
> > 3. remove unused map0 label in imx91_93_common.dtsi.
> > ---
> >  .../{imx93.dtsi => imx91_93_common.dtsi}      |  140 +-
> >  arch/arm64/boot/dts/freescale/imx93.dtsi      | 1484 ++---------------
> >  2 files changed, 163 insertions(+), 1461 deletions(-)
> >  copy arch/arm64/boot/dts/freescale/{imx93.dtsi =>
> imx91_93_common.dtsi} (91%)
> >  rewrite arch/arm64/boot/dts/freescale/imx93.dtsi (97%)
> 
> It's not shown in the diff, but can you add a 'soc' phandle for 'soc@0' node?
> So it can be referenced by imx91 or imx93 individually.
Add it when needed to reduce difference and facilitate review.
BR
Joy Zou
> 
> Best regards
> Alexander
>