[U-Boot] [PATCH v4] arm64: mvebu: Add basic support for uDPU board

[Stefan Roese]

On 28.01.19 17:27, Vladimir Vid wrote:
> This adds initial support for micro-DPU (uDPU) board which is based on Armada-3720 SoC.
> micro-DPU is the single-port FTTdp "distribution point unit" made by Methode Electronics
> which offers complete modularity with replaceable SFP modules both for uplink and downlink
> (G.hn over twisted-pair, G.hn over coax, 1G and 2.5G Ethernet over Cat-5e cable).
> 
> On-board features:
> - 512 MiB DDR3
> - 2 x 2.5G SFP via HSGMII SERDES interface to the A3720 SoC
> - USB 2.0 Type-C connector
> - 4GB eMMC
> - ETSI TS 101548 reverse powering via twisted pair (RJ45) or coax (F Type)
> 
> Cc: Luka Perkov <luka.perkov at sartura.hr>
> Cc: Luis Torres <luis.torres at methode.com>
> Cc: Scott Roberts <scott.roberts at telus.com>
> Cc: Paul Arola <paul.arola at telus.com>
> Cc: Stefan Roese <sr at denx.de>
> Signed-off-by: Vladimir Vid <vladimir.vid at sartura.hr>
> ---
> 
> v4 changes:
> 
> - moved u-boot specific properties to armada-3720-uDPU-u-boot.dtsi
> - added additional 'u-boot,dm-pre-reloc' on sdhci1 (eMMC)
> 
> ---
>   arch/arm/dts/Makefile                       |   1 +
>   arch/arm/dts/armada-3720-uDPU-u-boot.dtsi   |  13 ++
>   arch/arm/dts/armada-3720-uDPU.dts           | 200 ++++++++++++++++++++
>   board/Marvell/mvebu_armada-37xx/MAINTAINERS |   5 +
>   configs/uDPU_defconfig                      |  94 +++++++++
>   5 files changed, 313 insertions(+)
>   create mode 100644 arch/arm/dts/armada-3720-uDPU-u-boot.dtsi
>   create mode 100644 arch/arm/dts/armada-3720-uDPU.dts
>   create mode 100644 configs/uDPU_defconfig
> 
> diff --git a/arch/arm/dts/Makefile b/arch/arm/dts/Makefile
> index 5c3225bcbf..9e1e1ad8e9 100644
> --- a/arch/arm/dts/Makefile
> +++ b/arch/arm/dts/Makefile
> @@ -95,6 +95,7 @@ dtb-$(CONFIG_ARCH_MVEBU) +=			\
>   	armada-3720-db.dtb			\
>   	armada-3720-espressobin.dtb		\
>   	armada-3720-turris-mox.dtb		\
> +	armada-3720-uDPU.dts			\
>   	armada-375-db.dtb			\
>   	armada-388-clearfog.dtb			\
>   	armada-388-gp.dtb			\
> diff --git a/arch/arm/dts/armada-3720-uDPU-u-boot.dtsi b/arch/arm/dts/armada-3720-uDPU-u-boot.dtsi
> new file mode 100644
> index 0000000000..ef178bdc86
> --- /dev/null
> +++ b/arch/arm/dts/armada-3720-uDPU-u-boot.dtsi
> @@ -0,0 +1,13 @@
> +// SPDX-License-Identifier: GPL-2.0+
> +
> +&spi0 {
> +	u-boot,dm-pre-reloc;
> +
> +	spi-flash at 0 {
> +		u-boot,dm-pre-reloc;
> +	};
> +};
> +
> +&sdhci1 {
> +	u-boot,dm-pre-reloc;
> +};

I'm wondering, why you don't need any "u-boot,dm-pre-reloc" in your
UART DT node.

Other than that:

Reviewed-by: Stefan Roese <sr at denx.de>

Thanks,
Stefan