From: Ulf Hansson <ulf.hansson@linaro.org>
To: Joy Zou <joy.zou@nxp.com>
Cc: robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org,
shawnguo@kernel.org, s.hauer@pengutronix.de,
kernel@pengutronix.de, festevam@gmail.com, peng.fan@nxp.com,
richardcochran@gmail.com, catalin.marinas@arm.com,
will@kernel.org, andrew+netdev@lunn.ch, davem@davemloft.net,
edumazet@google.com, kuba@kernel.org, pabeni@redhat.com,
mcoquelin.stm32@gmail.com, alexandre.torgue@foss.st.com,
frieder.schrempf@kontron.de, primoz.fiser@norik.com,
othacehe@gnu.org, Markus.Niebel@ew.tq-group.com,
alexander.stein@ew.tq-group.com, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org, imx@lists.linux.dev,
linux-arm-kernel@lists.infradead.org, linux@ew.tq-group.com,
netdev@vger.kernel.org, linux-pm@vger.kernel.org,
linux-stm32@st-md-mailman.stormreply.com, Frank.Li@nxp.com
Subject: Re: [PATCH v8 00/11] Add i.MX91 platform support
Date: Tue, 19 Aug 2025 14:26:52 +0200 [thread overview]
Message-ID: <CAPDyKFrTu6T7VtogZdh=VnZonh81d9xRGBOHuNv5nECSBkL0SQ@mail.gmail.com> (raw)
In-Reply-To: <20250806114119.1948624-1-joy.zou@nxp.com>
On Wed, 6 Aug 2025 at 13:42, Joy Zou <joy.zou@nxp.com> wrote:
>
> The design of i.MX91 platform is very similar to i.MX93.
> Extracts the common parts in order to reuse code.
>
> The mainly difference between i.MX91 and i.MX93 is as follows:
> - i.MX91 removed some clocks and modified the names of some clocks.
> - i.MX91 only has one A core.
> - i.MX91 has different pinmux.
>
> ---
> Changes for v8:
> - add Reviewed-by tag for patch #2/3/4/5/6/7/8/9/11.
> - modify commit message for patch #10.
> - move imx91 before imx93 in Makefile for patch #6.
> - modify the commit message to keep wrap at 75 chars for patch #5.
> - Link to v7: https://lore.kernel.org/imx/20250728071438.2332382-1-joy.zou@nxp.com/
>
> Changes for v7:
> - Optimize i.MX91 num_clks hardcode with ARRAY_SIZE()for patch #10.
> - Add new patch in order to optimize i.MX93 num_clks hardcode
> with ARRAY_SIZE() for patch #9.
> - remove this unused comments for patch #6.
> - align all pinctrl value to the same column for patch #6.
> - add aliases because remove aliases from common dtsi for patch #6.
> - remove fec property eee-broken-1000t from imx91 and imx93 board dts
> for patch #6 and #7.
> - The aliases are removed from common.dtsi because the imx93.dtsi
> aliases are removed for patch #4.
> - Add new patch that move aliases from imx93.dtsi to board dts for
> patch #3.
> - These aliases aren't common, so need to drop in imx93.dtsi for patch #3.
> - Only add aliases using to imx93 board dts for patch #3.
> - patch #3 changes come from review comments:
> https://lore.kernel.org/imx/4e8f2426-92a1-4c7e-b860-0e10e8dd886c@kernel.org/
> - add clocks constraints in the if-else branch for patch #2.
> - reorder the imx93 and imx91 if-else branch for patch #2.
> - patch #2 changes come from review comments:
> https://lore.kernel.org/imx/urgfsmkl25woqy5emucfkqs52qu624po6rd532hpusg3fdnyg3@s5iwmhnfsi26/
> - add Reviewed-by tag for patch #2.
> - Link to v6: https://lore.kernel.org/imx/20250623095732.2139853-1-joy.zou@nxp.com/
>
> Changes for v6:
> - add changelog in per patch.
> - correct commit message spell for patch #1.
> - merge rename imx93.dtsi to imx91_93_common.dtsi and move i.MX93
> specific part from imx91_93_common.dtsi to imx93.dtsi for patch #3.
> - modify the commit message for patch #3.
> - restore copyright time and add modification time for common dtsi for
> patch #3.
> - remove unused map0 label in imx91_93_common.dtsi for patch #3.
> - remove tmu related node for patch #4.
> - remove unused regulators and pinctrl settings for patch #5.
> - add new modification for aliases change patch #6.
> - Link to v5: https://lore.kernel.org/imx/20250613100255.2131800-1-joy.zou@nxp.com/
>
> Changes for v5:
> - rename imx93.dtsi to imx91_93_common.dtsi.
> - move imx93 specific part from imx91_93_common.dtsi to imx93.dtsi.
> - modify the imx91.dtsi to use imx91_93_common.dtsi.
> - add new the imx93-blk-ctrl binding and driver patch for imx91 support.
> - add new net patch for imx91 support.
> - change node name codec and lsm6dsm into common name audio-codec and
> inertial-meter, and add BT compatible string for imx91 board dts.
> - Link to v4: https://lore.kernel.org/imx/20250121074017.2819285-1-joy.zou@nxp.com/
>
> Changes for v4:
> - Add one imx93 patch that add labels in imx93.dtsi
> - modify the references in imx91.dtsi
> - modify the code alignment
> - remove unused newline
> - delete the status property
> - align pad hex values
> - Link to v3: https://lore.kernel.org/imx/20241120094945.3032663-1-pengfei.li_1@nxp.com/
>
> Changes for v3:
> - Add Conor's ack on patch #1
> - format imx91-11x11-evk.dts with the dt-format tool
> - add lpi2c1 node
> - Link to v2: https://lore.kernel.org/imx/20241118051541.2621360-1-pengfei.li_1@nxp.com/
>
> Changes for v2:
> - change ddr node pmu compatible
> - remove mu1 and mu2
> - change iomux node compatible and enable 91 pinctrl
> - refine commit message for patch #2
> - change hex to lowercase in pinfunc.h
> - ordering nodes with the dt-format tool
> - Link to v1: https://lore.kernel.org/imx/20241108022703.1877171-1-pengfei.li_1@nxp.com/
>
> Joy Zou (10):
> dt-bindings: soc: imx-blk-ctrl: add i.MX91 blk-ctrl compatible
> arm64: dts: freescale: move aliases from imx93.dtsi to board dts
> arm64: dts: freescale: rename imx93.dtsi to imx91_93_common.dtsi and
> modify them
> arm64: dts: imx91: add i.MX91 dtsi support
> arm64: dts: freescale: add i.MX91 11x11 EVK basic support
> arm64: dts: imx93-11x11-evk: remove fec property eee-broken-1000t
> arm64: defconfig: enable i.MX91 pinctrl
> pmdomain: imx93-blk-ctrl: use ARRAY_SIZE() instead of hardcode number
> pmdomain: imx93-blk-ctrl: mask DSI and PXP PD domain register on
> i.MX91
> net: stmmac: imx: add i.MX91 support
>
> Pengfei Li (1):
> dt-bindings: arm: fsl: add i.MX91 11x11 evk board
>
> .../devicetree/bindings/arm/fsl.yaml | 6 +
> .../soc/imx/fsl,imx93-media-blk-ctrl.yaml | 59 +-
> arch/arm64/boot/dts/freescale/Makefile | 1 +
> .../boot/dts/freescale/imx91-11x11-evk.dts | 674 ++++++++
> arch/arm64/boot/dts/freescale/imx91-pinfunc.h | 770 +++++++++
> arch/arm64/boot/dts/freescale/imx91.dtsi | 71 +
> .../{imx93.dtsi => imx91_93_common.dtsi} | 176 +-
> .../boot/dts/freescale/imx93-11x11-evk.dts | 20 +-
> .../boot/dts/freescale/imx93-14x14-evk.dts | 15 +
> .../boot/dts/freescale/imx93-9x9-qsb.dts | 18 +
> .../dts/freescale/imx93-kontron-bl-osm-s.dts | 21 +
> .../dts/freescale/imx93-phyboard-nash.dts | 21 +
> .../dts/freescale/imx93-phyboard-segin.dts | 9 +
> .../freescale/imx93-tqma9352-mba91xxca.dts | 11 +
> .../freescale/imx93-tqma9352-mba93xxca.dts | 25 +
> .../freescale/imx93-tqma9352-mba93xxla.dts | 25 +
> .../dts/freescale/imx93-var-som-symphony.dts | 17 +
> arch/arm64/boot/dts/freescale/imx93.dtsi | 1512 ++---------------
> arch/arm64/configs/defconfig | 1 +
> .../net/ethernet/stmicro/stmmac/dwmac-imx.c | 2 +
> drivers/pmdomain/imx/imx93-blk-ctrl.c | 23 +-
> 21 files changed, 1938 insertions(+), 1539 deletions(-)
> create mode 100644 arch/arm64/boot/dts/freescale/imx91-11x11-evk.dts
> create mode 100644 arch/arm64/boot/dts/freescale/imx91-pinfunc.h
> create mode 100644 arch/arm64/boot/dts/freescale/imx91.dtsi
> copy arch/arm64/boot/dts/freescale/{imx93.dtsi => imx91_93_common.dtsi} (90%)
> rewrite arch/arm64/boot/dts/freescale/imx93.dtsi (97%)
>
Patch2, 9 and 10 applied for next on my pmdomain tree, thanks!
Note, patch2 (dt change) is also available on my immutable dt branch,
for SoC maintainers to pull-in.
Kind regards
Uffe
prev parent reply other threads:[~2025-08-19 12:27 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-08-06 11:41 [PATCH v8 00/11] Add i.MX91 platform support Joy Zou
2025-08-06 11:41 ` [PATCH v8 01/11] dt-bindings: arm: fsl: add i.MX91 11x11 evk board Joy Zou
2025-08-06 15:50 ` Frank Li
2025-08-22 8:40 ` Shawn Guo
2025-08-06 11:41 ` [PATCH v8 02/11] dt-bindings: soc: imx-blk-ctrl: add i.MX91 blk-ctrl compatible Joy Zou
2025-08-06 11:41 ` [PATCH v8 03/11] arm64: dts: freescale: move aliases from imx93.dtsi to board dts Joy Zou
2025-08-22 8:42 ` Shawn Guo
2025-08-06 11:41 ` [PATCH v8 04/11] arm64: dts: freescale: rename imx93.dtsi to imx91_93_common.dtsi and modify them Joy Zou
2025-08-06 11:41 ` [PATCH v8 05/11] arm64: dts: imx91: add i.MX91 dtsi support Joy Zou
2025-08-06 11:41 ` [PATCH v8 06/11] arm64: dts: freescale: add i.MX91 11x11 EVK basic support Joy Zou
2025-08-06 11:41 ` [PATCH v8 07/11] arm64: dts: imx93-11x11-evk: remove fec property eee-broken-1000t Joy Zou
2025-08-06 11:41 ` [PATCH v8 08/11] arm64: defconfig: enable i.MX91 pinctrl Joy Zou
2025-08-22 8:42 ` Shawn Guo
2025-08-06 11:41 ` [PATCH v8 09/11] pmdomain: imx93-blk-ctrl: use ARRAY_SIZE() instead of hardcode number Joy Zou
2025-08-06 11:41 ` [PATCH v8 10/11] pmdomain: imx93-blk-ctrl: mask DSI and PXP PD domain register on i.MX91 Joy Zou
2025-08-06 15:51 ` Frank Li
2025-08-06 11:41 ` [PATCH v8 11/11] net: stmmac: imx: add i.MX91 support Joy Zou
2025-08-06 13:48 ` Russell King (Oracle)
2025-08-19 12:26 ` Ulf Hansson [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CAPDyKFrTu6T7VtogZdh=VnZonh81d9xRGBOHuNv5nECSBkL0SQ@mail.gmail.com' \
--to=ulf.hansson@linaro.org \
--cc=Frank.Li@nxp.com \
--cc=Markus.Niebel@ew.tq-group.com \
--cc=alexander.stein@ew.tq-group.com \
--cc=alexandre.torgue@foss.st.com \
--cc=andrew+netdev@lunn.ch \
--cc=catalin.marinas@arm.com \
--cc=conor+dt@kernel.org \
--cc=davem@davemloft.net \
--cc=devicetree@vger.kernel.org \
--cc=edumazet@google.com \
--cc=festevam@gmail.com \
--cc=frieder.schrempf@kontron.de \
--cc=imx@lists.linux.dev \
--cc=joy.zou@nxp.com \
--cc=kernel@pengutronix.de \
--cc=krzk+dt@kernel.org \
--cc=kuba@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=linux-stm32@st-md-mailman.stormreply.com \
--cc=linux@ew.tq-group.com \
--cc=mcoquelin.stm32@gmail.com \
--cc=netdev@vger.kernel.org \
--cc=othacehe@gnu.org \
--cc=pabeni@redhat.com \
--cc=peng.fan@nxp.com \
--cc=primoz.fiser@norik.com \
--cc=richardcochran@gmail.com \
--cc=robh@kernel.org \
--cc=s.hauer@pengutronix.de \
--cc=shawnguo@kernel.org \
--cc=will@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).