[PATCH v4 v5 03/10] ARM: dts: socfpga: add Mercury+ AA1 for u-boot dts

[Chee, Tien Fong]

Hi,

> -----Original Message-----
> From: U-Boot <u-boot-bounces at lists.denx.de> On Behalf Of Lothar Rubusch
> <l.rubusch at gmail.com>
> Sent: Wednesday, November 20, 2024 6:47 AM
> To: u-boot at lists.denx.de; trini at konsulko.com; marex at denx.de;
> simon.k.r.goldschmidt at gmail.com; tien.fong.chee at intel.com;
> sumit.garg at linaro.org
> Cc: sjg at chromium.org; xypron.glpk at gmx.de; michal.simek at amd.com;
> jit.loon.lim at intel.com; barnas at google.com; l.rubusch at gmail.com
> Subject: [PATCH v4 v5 03/10] ARM: dts: socfpga: add Mercury+ AA1 for u-
> boot dts
> 
> Introduce u-boot specific device-tree files for Enclustra Mercury+ AA1 SoMs
> (Intel/arria10).
> 
> Generic device-tree fragments for linux and U-boot shall be provided in
> dts/upstream. The selection of the generic device-tree fragments depends
> on a selected boot-mode and a selected carrier board.
> 
> On Intel/Arria10 a handoff setup is needed for initialization of various clock /
> pinmux / DRAM settings only used by U-Boot provided by the current patch.
> 
> Signed-off-by: Andreas Buerkler <andreas.buerkler at enclustra.com>
> Signed-off-by: Lothar Rubusch <l.rubusch at gmail.com>
> ---
>  ..._arria10_enclustra_mercury_aa1-u-boot.dtsi |  45
> +++  .../socfpga_arria10_enclustra_mercury_aa1.dts |  45 +++
>  .../socfpga_arria10_mercury_aa1-u-boot.dtsi   |  19 ++
>  .../dts/socfpga_arria10_mercury_aa1_handoff.h | 305
> ++++++++++++++++++
>  board/enclustra/mercury_aa1/Kconfig           |  37 +++
>  board/enclustra/mercury_aa1/MAINTAINERS       |   2 +
>  6 files changed, 453 insertions(+)
>  create mode 100644 arch/arm/dts/socfpga_arria10_enclustra_mercury_aa1-
> u-boot.dtsi
>  create mode 100644
> arch/arm/dts/socfpga_arria10_enclustra_mercury_aa1.dts
>  create mode 100644 arch/arm/dts/socfpga_arria10_mercury_aa1_handoff.h
> 
> diff --git a/arch/arm/dts/socfpga_arria10_enclustra_mercury_aa1-u-
> boot.dtsi b/arch/arm/dts/socfpga_arria10_enclustra_mercury_aa1-u-
> boot.dtsi
> new file mode 100644
> index 0000000000..6e38286572
> --- /dev/null
> +++ b/arch/arm/dts/socfpga_arria10_enclustra_mercury_aa1-u-boot.dtsi
> @@ -0,0 +1,45 @@
> +// SPDX-License-Identifier: GPL-2.0+ OR MIT
> +/*
> + * Copyright (C) 2024 Lothar Rubusch <l.rubusch at gmail.com>  */
> +
> +/* Arria10 handoff (u-boot) */
> +#include "socfpga_arria10_mercury_aa1_handoff.h"
> +#include "socfpga_arria10-handoff.dtsi"
> +#include "socfpga_arria10_handoff_u-boot.dtsi"
> +#include "socfpga_arria10_mercury_aa1-u-boot.dtsi"
> +
> +/* Specific boot-mode (u-boot) */
> +#if IS_ENABLED(CONFIG_ENCLUSTRA_SDMMC) ||
> +IS_ENABLED(CONFIG_ENCLUSTRA_EMMC)
> +
> +/ {
> +	fs_loader0: fs-loader {
> +		bootph-all;
> +		compatible = "u-boot,fs-loader";
> +		phandlepart = <&mmc 1>;
> +	};
> +};
> +
> +&fpga_mgr {
> +	bootph-all;
> +	firmware-loader = <&fs_loader0>;
> +	altr,bitstream = "fpga.itb";
> +};
> +
> +#elif IS_ENABLED(CONFIG_ENCLUSTRA_QSPI)
> +
> +/ {
> +	fs_loader0: fs-loader {
> +		bootph-all;
> +		compatible = "u-boot,fs-loader";
> +		sfconfig = <0 0 50000000 3>;

The RAW read implementation for fs_loader is not upstream yet, I afraid this might not working, you want help to upstream?

[...]

Best regards,
TF