All of lore.kernel.org
 help / color / mirror / Atom feed
From: Gregory CLEMENT <gregory.clement@bootlin.com>
To: 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, nadavh@marvell.com, stefanc@marvell.com,
	bpeled@marvell.com, Konstantin Porotchkin <kostap@marvell.com>
Subject: Re: [PATCH v4 0/5] DTS updates for Marvell Armada CN913x platforms
Date: Thu, 17 Jun 2021 17:40:00 +0200	[thread overview]
Message-ID: <87h7hw1ovj.fsf@BL-laptop> (raw)
In-Reply-To: <20210209134610.19904-1-kostap@marvell.com>

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
http://bootlin.com

WARNING: multiple messages have this Message-ID (diff)
From: Gregory CLEMENT <gregory.clement@bootlin.com>
To: 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, nadavh@marvell.com, stefanc@marvell.com,
	bpeled@marvell.com, Konstantin Porotchkin <kostap@marvell.com>
Subject: Re: [PATCH v4 0/5] DTS updates for Marvell Armada CN913x platforms
Date: Thu, 17 Jun 2021 17:40:00 +0200	[thread overview]
Message-ID: <87h7hw1ovj.fsf@BL-laptop> (raw)
In-Reply-To: <20210209134610.19904-1-kostap@marvell.com>

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
http://bootlin.com

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  parent reply	other threads:[~2021-06-17 15:40 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-02-09 13:46 [PATCH v4 0/5] DTS updates for Marvell Armada CN913x platforms kostap
2021-02-09 13:46 ` kostap
2021-02-09 13:46 ` [PATCH v4 1/5] Documentation/bindings: phy: update references to cp11x kostap
2021-02-09 13:46   ` kostap
2021-02-09 13:46 ` [PATCH v4 2/5] arch/arm64/boot/dts/marvell: fix NAND partitioning scheme kostap
2021-02-09 13:46   ` kostap
2021-02-09 13:46 ` [PATCH v4 3/5] arm64: dts: cn913x: add device trees for topology B boards kostap
2021-02-09 13:46   ` kostap
2021-02-09 13:46 ` [PATCH v4 4/5] dts: marvell: Enable 10G interface on 9130-DB board kostap
2021-02-09 13:46   ` kostap
2021-02-09 14:13   ` Marcin Wojtas
2021-02-09 14:13     ` Marcin Wojtas
2021-02-09 13:46 ` [PATCH v4 5/5] arm64: dts: add support for Marvell cn9130-crb platform kostap
2021-02-09 13:46   ` kostap
2021-06-17 15:40 ` Gregory CLEMENT [this message]
2021-06-17 15:40   ` [PATCH v4 0/5] DTS updates for Marvell Armada CN913x platforms Gregory CLEMENT
2021-07-08  7:50   ` [EXT] " Kostya Porotchkin
2021-07-08  7:50     ` Kostya Porotchkin

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=87h7hw1ovj.fsf@BL-laptop \
    --to=gregory.clement@bootlin.com \
    --cc=andrew@lunn.ch \
    --cc=bpeled@marvell.com \
    --cc=devicetree@vger.kernel.org \
    --cc=jaz@semihalf.com \
    --cc=kishon@ti.com \
    --cc=kostap@marvell.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=luka.kovacic@sartura.hr \
    --cc=miquel.raynal@bootlin.com \
    --cc=mw@semihalf.com \
    --cc=nadavh@marvell.com \
    --cc=robh+dt@kernel.org \
    --cc=sebastian.hesselbarth@gmail.com \
    --cc=stefanc@marvell.com \
    --cc=tmn505@gmail.com \
    --cc=vkoul@kernel.org \
    --cc=vladimir.vid@sartura.hr \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.