Linux-ARM-Kernel Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Frank Li <Frank.li@oss.nxp.com>
To: Ioana Ciornei <ioana.ciornei@nxp.com>
Cc: Frank.Li@nxp.com, robh@kernel.org, krzk+dt@kernel.org,
	conor+dt@kernel.org, devicetree@vger.kernel.org,
	vladimir.oltean@nxp.com, linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org, imx@lists.linux.dev,
	linux@ew.tq-group.com
Subject: Re: [PATCH v3 00/11] arm64: dts: describe the Lynx 10G and 28G SerDes blocks for Layerscape SoCs
Date: Fri, 17 Jul 2026 10:53:23 -0500	[thread overview]
Message-ID: <alpP81Et1Z2qWEH5@SMW015318> (raw)
In-Reply-To: <20260717120635.2118610-1-ioana.ciornei@nxp.com>

On Fri, Jul 17, 2026 at 03:06:24PM +0300, Ioana Ciornei wrote:
> This patch set adds the device tree nodes for the Lynx10G SerDes blocks
> found on the LS1028A, LS1046A, LS1088A and LS2088A SoCs.
>
> The first 5 patches slowly change the LX2160A DTS support to a new
> structure which does not use any delete-node or delete-property
> directives. This is a direct result of the discussion with Frank in the
> v2 of this set:
> https://lore.kernel.org/all/akwAmzAbtg0Ejey0@SMW015318/
>
> Patches 5 and 6 transition the LX2160A and LX2162 SoC dtsi to use the
> device-specific Lynx28G SerDes compatible while the remainder of the set
> adds the SerDes block descriptions for the LS1028A, LS1046A, LS2088A and
> LS1088A SoCs.
>
> Changes in v3:
> - Changes how the LX216XA based SoC dtsi files are structured.
> - Split the enabling of serdes_1 into a separate patch
> - Split the LX2162A changes to another patch
> - Move the status property to be the last for serdes_3
> - Move the changes to the new fsl-lx2160a-serdes.dtsi file

Overall look good, let wait for some days for other reviewers.

Frank

>
> Changes in v2:
> - Enable serdes_1 on all board DTs that has consumers for it.
> - Use the proper name for serdes_3 in fsl-lx2162a.dtsi.
> - Remove paragraph from commit message which mentioned some consumer
> changes that are no longer needed nor part of the commit.
> - Change the size of the SerDes region to 0x2000
>
> Ioana Ciornei (8):
>   arm64: dts: lx2160a: explicitly enable serdes_1 node
>   arm64: dts: lx2160a: extract the PCIe nodes into a separate dtsi
>   arm64: dts: lx2160a: extract the SerDes nodes into a separate dtsi
>   arm64: dts: lx2160a: extract the PCIe from LX2160A Rev2 into a
>     separate file
>   arm64: dts: lx2160a: split the dtsi to avoid delete-property
>   arm64: dts: lx2160a: transition to device-specific SerDes compatible
>     strings
>   arm64: dts: lx2162a: transition to device-specific SerDes compatible
>     strings
>   arm64: dts: ls1088a: describe the Lynx 10G SerDes blocks
>
> Vladimir Oltean (3):
>   arm64: dts: ls1028a: describe the Lynx 10G SerDes
>   arm64: dts: ls1046a: describe the Lynx 10G SerDes blocks
>   arm64: dts: ls208xa: describe the Lynx 10G SerDes blocks
>
>  .../arm64/boot/dts/freescale/fsl-ls1028a.dtsi |  29 ++
>  .../arm64/boot/dts/freescale/fsl-ls1046a.dtsi |  60 ++++
>  .../arm64/boot/dts/freescale/fsl-ls1088a.dtsi |  58 ++++
>  .../arm64/boot/dts/freescale/fsl-ls208xa.dtsi |  98 +++++++
>  .../dts/freescale/fsl-lx2160a-bluebox3.dts    |   2 +-
>  .../dts/freescale/fsl-lx2160a-clearfog-cx.dts |   2 +-
>  .../freescale/fsl-lx2160a-clearfog-itx.dtsi   |   4 +
>  .../dts/freescale/fsl-lx2160a-half-twins.dts  |   4 +
>  .../dts/freescale/fsl-lx2160a-honeycomb.dts   |   2 +-
>  .../dts/freescale/fsl-lx2160a-pcie-mbvl.dtsi  | 176 ++++++++++++
>  .../boot/dts/freescale/fsl-lx2160a-rdb.dts    |   4 +
>  .../boot/dts/freescale/fsl-lx2160a-rev1.dtsi  |   9 +
>  .../boot/dts/freescale/fsl-lx2160a-rev2.dtsi  | 188 +------------
>  .../dts/freescale/fsl-lx2160a-serdes.dtsi     | 167 ++++++++++++
>  .../dts/freescale/fsl-lx2160a-tqmlx2160a.dtsi |   2 +-
>  .../dts/freescale/fsl-lx2162a-clearfog.dts    |   6 +-
>  .../boot/dts/freescale/fsl-lx2162a-qds.dts    |   2 +-
>  .../dts/freescale/fsl-lx2162a-serdes.dtsi     |  86 ++++++
>  .../arm64/boot/dts/freescale/fsl-lx2162a.dtsi |   9 +
>  .../{fsl-lx2160a.dtsi => fsl-lx216x.dtsi}     | 181 ------------
>  .../dts/freescale/fsl-lx216xa-pcie-gen3.dtsi  | 257 ++++++++++++++++++
>  21 files changed, 974 insertions(+), 372 deletions(-)
>  create mode 100644 arch/arm64/boot/dts/freescale/fsl-lx2160a-pcie-mbvl.dtsi
>  create mode 100644 arch/arm64/boot/dts/freescale/fsl-lx2160a-rev1.dtsi
>  create mode 100644 arch/arm64/boot/dts/freescale/fsl-lx2160a-serdes.dtsi
>  create mode 100644 arch/arm64/boot/dts/freescale/fsl-lx2162a-serdes.dtsi
>  create mode 100644 arch/arm64/boot/dts/freescale/fsl-lx2162a.dtsi
>  rename arch/arm64/boot/dts/freescale/{fsl-lx2160a.dtsi => fsl-lx216x.dtsi} (86%)
>  create mode 100644 arch/arm64/boot/dts/freescale/fsl-lx216xa-pcie-gen3.dtsi
>
> --
> 2.25.1
>
>


      parent reply	other threads:[~2026-07-17 15:53 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-07-17 12:06 [PATCH v3 00/11] arm64: dts: describe the Lynx 10G and 28G SerDes blocks for Layerscape SoCs Ioana Ciornei
2026-07-17 12:06 ` [PATCH v3 01/11] arm64: dts: lx2160a: explicitly enable serdes_1 node Ioana Ciornei
2026-07-17 12:06 ` [PATCH v3 02/11] arm64: dts: lx2160a: extract the PCIe nodes into a separate dtsi Ioana Ciornei
2026-07-17 12:06 ` [PATCH v3 03/11] arm64: dts: lx2160a: extract the SerDes " Ioana Ciornei
2026-07-17 12:06 ` [PATCH v3 04/11] arm64: dts: lx2160a: extract the PCIe from LX2160A Rev2 into a separate file Ioana Ciornei
2026-07-17 12:06 ` [PATCH v3 05/11] arm64: dts: lx2160a: split the dtsi to avoid delete-property Ioana Ciornei
2026-07-17 12:06 ` [PATCH v3 06/11] arm64: dts: lx2160a: transition to device-specific SerDes compatible strings Ioana Ciornei
2026-07-17 12:06 ` [PATCH v3 07/11] arm64: dts: lx2162a: " Ioana Ciornei
2026-07-17 12:06 ` [PATCH v3 08/11] arm64: dts: ls1028a: describe the Lynx 10G SerDes Ioana Ciornei
2026-07-17 12:06 ` [PATCH v3 09/11] arm64: dts: ls1046a: describe the Lynx 10G SerDes blocks Ioana Ciornei
2026-07-17 12:06 ` [PATCH v3 10/11] arm64: dts: ls208xa: " Ioana Ciornei
2026-07-17 12:06 ` [PATCH v3 11/11] arm64: dts: ls1088a: " Ioana Ciornei
2026-07-17 15:53 ` Frank Li [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=alpP81Et1Z2qWEH5@SMW015318 \
    --to=frank.li@oss.nxp.com \
    --cc=Frank.Li@nxp.com \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=imx@lists.linux.dev \
    --cc=ioana.ciornei@nxp.com \
    --cc=krzk+dt@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@ew.tq-group.com \
    --cc=robh@kernel.org \
    --cc=vladimir.oltean@nxp.com \
    /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