mbox series

[v4,0/5] DTS updates for Marvell Armada CN913x platforms

Message ID 20210209134610.19904-1-kostap@marvell.com (mailing list archive)
Headers show
Series DTS updates for Marvell Armada CN913x platforms | expand

Message

Kostya Porotchkin Feb. 9, 2021, 1:46 p.m. UTC
From: Konstantin Porotchkin <kostap@marvell.com>

This patch series contains the following changes/fixes:
1. Add support for Armada CN913x Development Board topology "B"
2. Add support for Armada CN913x Reference Design boards (CRB)
3. Fixes the CP11X references in PHY binding document
4. Fixes the NAND paritioninig scheme in DTS eliminating gap between
consecutive partitions
All above changes are already intergated into Marvell official SDK sources

v2:
- extract common nodes from DB boards to separate DTSI files
- disable eth2 on CRB boards until the required phy mode support is added
- mention the switch part ID in CRB board description

v3:
- do not rename setup A DTS files for DB variants
- fix a couple of wrong include names

v4:
- Rebase on top of Linus master branch (v5.11-rc7)
- Remove HS400 timing from CRB AP SDHCI (not supported by connected eMMC)
- Add cd-gpio entry to CP0 SDHCI for SD card support
- Add a comment to CN9130-CRB patch about the required defconfig
  changes for supporting the onboard i2c expander
- Add Stefan's patch to enable 10G port on CN9130-DB platforms

Grzegorz Jaszczyk (1):
  Documentation/bindings: phy: update references to cp11x

Konstantin Porotchkin (3):
  arch/arm64/boot/dts/marvell: fix NAND partitioning scheme
  arm64: dts: cn913x: add device trees for topology B boards
  arm64: dts: add support for Marvell cn9130-crb platform

Stefan Chulski (1):
  dts: marvell: Enable 10G interface on 9130-DB board

 .../bindings/phy/phy-mvebu-comphy.txt         |  12 +-
 arch/arm64/boot/dts/marvell/Makefile          |   5 +
 arch/arm64/boot/dts/marvell/cn9130-crb-A.dts  |  38 ++
 arch/arm64/boot/dts/marvell/cn9130-crb-B.dts  |  46 ++
 arch/arm64/boot/dts/marvell/cn9130-crb.dtsi   | 221 ++++++++++
 arch/arm64/boot/dts/marvell/cn9130-db-B.dts   |  22 +
 arch/arm64/boot/dts/marvell/cn9130-db.dts     | 393 +----------------
 arch/arm64/boot/dts/marvell/cn9130-db.dtsi    | 402 ++++++++++++++++++
 arch/arm64/boot/dts/marvell/cn9131-db-B.dts   |  22 +
 arch/arm64/boot/dts/marvell/cn9131-db.dts     | 192 +--------
 arch/arm64/boot/dts/marvell/cn9131-db.dtsi    | 201 +++++++++
 arch/arm64/boot/dts/marvell/cn9132-db-B.dts   |  22 +
 arch/arm64/boot/dts/marvell/cn9132-db.dts     | 211 +--------
 arch/arm64/boot/dts/marvell/cn9132-db.dtsi    | 220 ++++++++++
 14 files changed, 1223 insertions(+), 784 deletions(-)
 create mode 100644 arch/arm64/boot/dts/marvell/cn9130-crb-A.dts
 create mode 100644 arch/arm64/boot/dts/marvell/cn9130-crb-B.dts
 create mode 100644 arch/arm64/boot/dts/marvell/cn9130-crb.dtsi
 create mode 100644 arch/arm64/boot/dts/marvell/cn9130-db-B.dts
 create mode 100644 arch/arm64/boot/dts/marvell/cn9130-db.dtsi
 create mode 100644 arch/arm64/boot/dts/marvell/cn9131-db-B.dts
 create mode 100644 arch/arm64/boot/dts/marvell/cn9131-db.dtsi
 create mode 100644 arch/arm64/boot/dts/marvell/cn9132-db-B.dts
 create mode 100644 arch/arm64/boot/dts/marvell/cn9132-db.dtsi

Comments

Gregory CLEMENT June 17, 2021, 3:40 p.m. UTC | #1
Hello,

> From: Konstantin Porotchkin <kostap@marvell.com>
>
> This patch series contains the following changes/fixes:
> 1. Add support for Armada CN913x Development Board topology "B"
> 2. Add support for Armada CN913x Reference Design boards (CRB)
> 3. Fixes the CP11X references in PHY binding document
> 4. Fixes the NAND paritioninig scheme in DTS eliminating gap between
> consecutive partitions
> All above changes are already intergated into Marvell official SDK sources
>
> v2:
> - extract common nodes from DB boards to separate DTSI files
> - disable eth2 on CRB boards until the required phy mode support is added
> - mention the switch part ID in CRB board description
>
> v3:
> - do not rename setup A DTS files for DB variants
> - fix a couple of wrong include names
>
> v4:
> - Rebase on top of Linus master branch (v5.11-rc7)
> - Remove HS400 timing from CRB AP SDHCI (not supported by connected eMMC)
> - Add cd-gpio entry to CP0 SDHCI for SD card support
> - Add a comment to CN9130-CRB patch about the required defconfig
>   changes for supporting the onboard i2c expander
> - Add Stefan's patch to enable 10G port on CN9130-DB platforms
>
> Grzegorz Jaszczyk (1):
>   Documentation/bindings: phy: update references to cp11x
>
> Konstantin Porotchkin (3):
>   arch/arm64/boot/dts/marvell: fix NAND partitioning scheme
>   arm64: dts: cn913x: add device trees for topology B boards
>   arm64: dts: add support for Marvell cn9130-crb platform
>
> Stefan Chulski (1):
>   dts: marvell: Enable 10G interface on 9130-DB board
>

I've applied patches 1 and 2, but I have issue with patch 3, there was some
changes around the phys and I am not sure of the correct way to modify
the patch.

Could you rebase it on v5.13-c1 and fix the merge issue ?

Thanks,

Gregory

>  .../bindings/phy/phy-mvebu-comphy.txt         |  12 +-
>  arch/arm64/boot/dts/marvell/Makefile          |   5 +
>  arch/arm64/boot/dts/marvell/cn9130-crb-A.dts  |  38 ++
>  arch/arm64/boot/dts/marvell/cn9130-crb-B.dts  |  46 ++
>  arch/arm64/boot/dts/marvell/cn9130-crb.dtsi   | 221 ++++++++++
>  arch/arm64/boot/dts/marvell/cn9130-db-B.dts   |  22 +
>  arch/arm64/boot/dts/marvell/cn9130-db.dts     | 393 +----------------
>  arch/arm64/boot/dts/marvell/cn9130-db.dtsi    | 402 ++++++++++++++++++
>  arch/arm64/boot/dts/marvell/cn9131-db-B.dts   |  22 +
>  arch/arm64/boot/dts/marvell/cn9131-db.dts     | 192 +--------
>  arch/arm64/boot/dts/marvell/cn9131-db.dtsi    | 201 +++++++++
>  arch/arm64/boot/dts/marvell/cn9132-db-B.dts   |  22 +
>  arch/arm64/boot/dts/marvell/cn9132-db.dts     | 211 +--------
>  arch/arm64/boot/dts/marvell/cn9132-db.dtsi    | 220 ++++++++++
>  14 files changed, 1223 insertions(+), 784 deletions(-)
>  create mode 100644 arch/arm64/boot/dts/marvell/cn9130-crb-A.dts
>  create mode 100644 arch/arm64/boot/dts/marvell/cn9130-crb-B.dts
>  create mode 100644 arch/arm64/boot/dts/marvell/cn9130-crb.dtsi
>  create mode 100644 arch/arm64/boot/dts/marvell/cn9130-db-B.dts
>  create mode 100644 arch/arm64/boot/dts/marvell/cn9130-db.dtsi
>  create mode 100644 arch/arm64/boot/dts/marvell/cn9131-db-B.dts
>  create mode 100644 arch/arm64/boot/dts/marvell/cn9131-db.dtsi
>  create mode 100644 arch/arm64/boot/dts/marvell/cn9132-db-B.dts
>  create mode 100644 arch/arm64/boot/dts/marvell/cn9132-db.dtsi
>
> -- 
> 2.17.1
>
Kostya Porotchkin July 8, 2021, 7:50 a.m. UTC | #2
Hi, Gregory,

The last series was v5. I will rebase on top of Linus 5.13-rcx and then post new v6 set.

Regards
Kosta

> -----Original Message-----
> From: Gregory CLEMENT <gregory.clement@bootlin.com>
> Sent: Thursday, June 17, 2021 18:40
> To: Kostya Porotchkin <kostap@marvell.com>; linux-kernel@vger.kernel.org;
> devicetree@vger.kernel.org; linux-arm-kernel@lists.infradead.org
> Cc: vladimir.vid@sartura.hr; tmn505@gmail.com; luka.kovacic@sartura.hr;
> sebastian.hesselbarth@gmail.com; andrew@lunn.ch; robh+dt@kernel.org;
> vkoul@kernel.org; kishon@ti.com; miquel.raynal@bootlin.com;
> mw@semihalf.com; jaz@semihalf.com; Nadav Haklai
> <nadavh@marvell.com>; Stefan Chulski <stefanc@marvell.com>; Ben Peled
> <bpeled@marvell.com>; Kostya Porotchkin <kostap@marvell.com>
> Subject: [EXT] Re: [PATCH v4 0/5] DTS updates for Marvell Armada CN913x
> platforms
> 
> External Email
> 
> ----------------------------------------------------------------------
> Hello,
> 
> > From: Konstantin Porotchkin <kostap@marvell.com>
> >
> > This patch series contains the following changes/fixes:
> > 1. Add support for Armada CN913x Development Board topology "B"
> > 2. Add support for Armada CN913x Reference Design boards (CRB) 3.
> > Fixes the CP11X references in PHY binding document 4. Fixes the NAND
> > paritioninig scheme in DTS eliminating gap between consecutive
> > partitions All above changes are already intergated into Marvell
> > official SDK sources
> >
> > v2:
> > - extract common nodes from DB boards to separate DTSI files
> > - disable eth2 on CRB boards until the required phy mode support is
> > added
> > - mention the switch part ID in CRB board description
> >
> > v3:
> > - do not rename setup A DTS files for DB variants
> > - fix a couple of wrong include names
> >
> > v4:
> > - Rebase on top of Linus master branch (v5.11-rc7)
> > - Remove HS400 timing from CRB AP SDHCI (not supported by connected
> > eMMC)
> > - Add cd-gpio entry to CP0 SDHCI for SD card support
> > - Add a comment to CN9130-CRB patch about the required defconfig
> >   changes for supporting the onboard i2c expander
> > - Add Stefan's patch to enable 10G port on CN9130-DB platforms
> >
> > Grzegorz Jaszczyk (1):
> >   Documentation/bindings: phy: update references to cp11x
> >
> > Konstantin Porotchkin (3):
> >   arch/arm64/boot/dts/marvell: fix NAND partitioning scheme
> >   arm64: dts: cn913x: add device trees for topology B boards
> >   arm64: dts: add support for Marvell cn9130-crb platform
> >
> > Stefan Chulski (1):
> >   dts: marvell: Enable 10G interface on 9130-DB board
> >
> 
> I've applied patches 1 and 2, but I have issue with patch 3, there was some
> changes around the phys and I am not sure of the correct way to modify the
> patch.
> 
> Could you rebase it on v5.13-c1 and fix the merge issue ?
> 
> Thanks,
> 
> Gregory
> 
> >  .../bindings/phy/phy-mvebu-comphy.txt         |  12 +-
> >  arch/arm64/boot/dts/marvell/Makefile          |   5 +
> >  arch/arm64/boot/dts/marvell/cn9130-crb-A.dts  |  38 ++
> > arch/arm64/boot/dts/marvell/cn9130-crb-B.dts  |  46 ++
> >  arch/arm64/boot/dts/marvell/cn9130-crb.dtsi   | 221 ++++++++++
> >  arch/arm64/boot/dts/marvell/cn9130-db-B.dts   |  22 +
> >  arch/arm64/boot/dts/marvell/cn9130-db.dts     | 393 +----------------
> >  arch/arm64/boot/dts/marvell/cn9130-db.dtsi    | 402 ++++++++++++++++++
> >  arch/arm64/boot/dts/marvell/cn9131-db-B.dts   |  22 +
> >  arch/arm64/boot/dts/marvell/cn9131-db.dts     | 192 +--------
> >  arch/arm64/boot/dts/marvell/cn9131-db.dtsi    | 201 +++++++++
> >  arch/arm64/boot/dts/marvell/cn9132-db-B.dts   |  22 +
> >  arch/arm64/boot/dts/marvell/cn9132-db.dts     | 211 +--------
> >  arch/arm64/boot/dts/marvell/cn9132-db.dtsi    | 220 ++++++++++
> >  14 files changed, 1223 insertions(+), 784 deletions(-)  create mode
> > 100644 arch/arm64/boot/dts/marvell/cn9130-crb-A.dts
> >  create mode 100644 arch/arm64/boot/dts/marvell/cn9130-crb-B.dts
> >  create mode 100644 arch/arm64/boot/dts/marvell/cn9130-crb.dtsi
> >  create mode 100644 arch/arm64/boot/dts/marvell/cn9130-db-B.dts
> >  create mode 100644 arch/arm64/boot/dts/marvell/cn9130-db.dtsi
> >  create mode 100644 arch/arm64/boot/dts/marvell/cn9131-db-B.dts
> >  create mode 100644 arch/arm64/boot/dts/marvell/cn9131-db.dtsi
> >  create mode 100644 arch/arm64/boot/dts/marvell/cn9132-db-B.dts
> >  create mode 100644 arch/arm64/boot/dts/marvell/cn9132-db.dtsi
> >
> > --
> > 2.17.1
> >
> 
> --
> Gregory Clement, Bootlin
> Embedded Linux and Kernel engineering
> https://urldefense.proofpoint.com/v2/url?u=http-
> 3A__bootlin.com&d=DwIBAg&c=nKjWec2b6R0mOyPaz7xtfQ&r=-
> N9sN4p5NSr0JGQoQ_2UCOgAqajG99W1EbSOww0WU8o&m=yb2JOcVicfAZRe
> thZJoDE-
> wbwtupb3tk_D7VGKCNsqc&s=4oskkLywYJujfGQ4XpFfP6n4ybkC0722K_8qH3xz
> bK8&e=