[PATCH v3 5/7] riscv: dts: Add device tree for Microchip Icicle Kit

[Daire McNamara]

On 12/11/2020, 08:15, "Atish Patra" <atishp at atishpatra.org> wrote:

    On Tue, Nov 10, 2020 at 5:40 AM Bin Meng <bmeng.cn at gmail.com> wrote:
    >
    > On Tue, Nov 10, 2020 at 6:46 PM Padmarao Begari
    > <padmarao.begari at microchip.com> wrote:
    > >
    > > Add device tree for Microchip PolarFire SoC Icicle Kit.
    > >
    > > Signed-off-by: Padmarao Begari <padmarao.begari at microchip.com>
    > > Reviewed-by: Anup Patel <anup.patel at wdc.com>
    > > ---
    > >  arch/riscv/dts/Makefile                       |   1 +
    > >  .../dts/microchip-mpfs-icicle-kit-u-boot.dtsi |  15 +
    > >  arch/riscv/dts/microchip-mpfs-icicle-kit.dts  | 421 ++++++++++++++++++
    > >  3 files changed, 437 insertions(+)
    > >  create mode 100644 arch/riscv/dts/microchip-mpfs-icicle-kit-u-boot.dtsi
    > >  create mode 100644 arch/riscv/dts/microchip-mpfs-icicle-kit.dts
    > >
    >
    > I believe we will need to wait for Atish's Linux kernel patch to be
    > reviewed upstream before we can merge U-Boot's ?
    >

    What should we do with eMMC/SD card nodes ? As per the current fpga,
    both can not be enabled at the same time.
    The U-Boot DT seems to enable eMMC while I enabled SD card in Linux
    one as it allows more storage.

    I am assuming it is okay that both device trees can diverge a little
    until fpga bit streams are updated. Let me know if there
    is any concerns regarding that.
Yes, from our point of view, that's absolutely fine at the moment.  We'd prefer if both enabled eMMC for the time being but it's not a deal-breaker. We're working on a bit stream to unify them in any case.

    > Regards,
    > Bin



    -- 
    Regards,
    Atish