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

[Lothar Rubusch]

On Wed, Jan 22, 2025 at 5:35 AM Chee, Tien Fong
<tien.fong.chee at intel.com> wrote:
>
> 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?
>

Same answer here, as in my other reply. I appreciate your answer and
bringing it up again. If this continues, for sure there will be
technical issues. In this case I'll keep you in the loop. Thanx

> [...]
>
> Best regards,
> TF