[PATCH v11 00/18] RISC-V SiFive FU540 support SPL

[Bin Meng]

Hi Sean,

On Wed, May 20, 2020 at 2:38 PM Sean Anderson <seanga2 at gmail.com> wrote:
>
> On 5/20/20 2:32 AM, Bin Meng wrote:
> > Hi Sean,
> >
> > On Wed, May 20, 2020 at 2:06 PM Sean Anderson <seanga2 at gmail.com> wrote:
> >>
> >> On 5/19/20 11:07 PM, Rick Chen wrote:
> >>> Hi Bin
> >>>
> >>>> -----Original Message-----
> >>>> From: Bin Meng [mailto:bmeng.cn at gmail.com]
> >>>> Sent: Tuesday, May 19, 2020 4:44 PM
> >>>> To: Pragnesh Patel; Rick Jian-Zhi Chen(陳建志)
> >>>> Subject: Re: [PATCH v11 00/18] RISC-V SiFive FU540 support SPL
> >>>>
> >>>> Hi Rick,
> >>>>
> >>>> On Tue, May 19, 2020 at 3:04 PM Pragnesh Patel <pragnesh.patel at sifive.com> wrote:
> >>>>>
> >>>>> This series add support for SPL to FU540. U-Boot SPL can boot from
> >>>>> L2 LIM (0x0800_0000) and jump to OpenSBI(FW_DYNAMIC firmware) and
> >>>>> U-Boot proper from MMC devices.
> >>>>>
> >>>>> This series depends on:
> >>>>> [1] https://patchwork.ozlabs.org/patch/1281853
> >>>>> [2] https://patchwork.ozlabs.org/patch/1281852
> >>>>>
> >>>>> All these together is available for testing here [3] [3]
> >>>>> https://github.com/pragnesh26992/u-boot/tree/spl
> >>>>>
> >>>>> How to test this patch:
> >>>>> 1) Go to OpenSBI-dir : make PLATFORM=generic FW_DYNAMIC=y
> >>>>> 2) export
> >>>>> OPENSBI=<path/to/opensbi/build/platform/generic/firmware/fw_dynamic.bi
> >>>>> n>
> >>>>> 3) Change to u-boot-dir
> >>>>> 4) make sifive_fu540_defconfig
> >>>>> 5) make all
> >>>>> 6) Format the SD card (make sure the disk has GPT, otherwise use gdisk
> >>>>> to switch)
> >>>>>
> >>>>>         # sudo sgdisk --clear \
> >>>>>         > --set-alignment=2 \
> >>>>>         > --new=1:34:2081 --change-name=1:loader1 --typecode=1:5B193300-FC78-40CD-8002-E86C45580B47 \
> >>>>>         > --new=2:2082:10273 --change-name=2:loader2 --typecode=2:2E54B353-1271-4842-806F-E436D6AF6985 \
> >>>>>         > --new=3:10274: --change-name=3:rootfs --typecode=3:0FC63DAF-8483-4772-8E79-3D69D8477DE4 \
> >>>>>         > /dev/sda
> >>>>>
> >>>>> 7) sudo dd if=spl/u-boot-spl.bin of=/dev/sda seek=34
> >>>>> 8) sudo dd if=u-boot.itb of=/dev/sda seek=2082
> >>>>>
> >>>>> Changes in v11:
> >>>>> - Remove TPL related code and OF_PLATDATA from FU540
> >>>>>   DDR driver (drivers/ram/sifive/fu540_ddr.c)
> >>>>> - Update FU540 doc (doc/board/sifive/fu540.rst)
> >>>>>   Remove unnecessary print
> >>>>
> >>>> Could we get this v11 applied as soon as possible for v2020.07?
> >>>
> >>> No problem, if everything is OK, I will applied ASAP.
> >>> But Jagan seem have some responses, please check about it.
> >>>
> >>>>
> >>>>> This series depends on:
> >>>>> [1] https://patchwork.ozlabs.org/patch/1281853
> >>>>> [2] https://patchwork.ozlabs.org/patch/1281852
> >>>>
> >>>> Looks this series "riscv: Add Sipeed Maix support" was not applied neither ?
> >>>
> >>> Yes, the reason is that the CI verification of v10 of this series
> >>> "riscv: Add Sipeed Maix support" still fail.
> >>> Please check the discussion of [v10,20/21] doc: riscv: Add
> >>> documentation for Sipeed Maix Bit
> >>>
> >>> https://patchwork.ozlabs.org/project/uboot/patch/20200503024637.327733-21-seanga2@gmail.com/
> >>>
> >>> That is why I still not pull it yet.
> >>>
> >>> Thanks,
> >>> Rick
> >>
> >> I think I have fixed the bug. However, travis is currently failing
> >> during apt with
> >>
> >> E: Failed to fetch https://apt.llvm.org/bionic/dists/llvm-toolchain-bionic-10/main/binary-amd64/Packages.gz  File has unexpected size (8765 != 8774). Mirror sync in progress? [IP: 151.101.186.49 443]
> >>    Release file created at: Tue, 19 May 2020 21:55:10 +0000
> >
> > Travis is not stable. Could you please try Azure?
>
> I've never used azure for CI before. Is there a guide to using it with
> u-boot anywhere? Perhaps this [1]? Will I need to pay for server time?

No, it's free for open source projects. You can search for "Azure
pipelines" in the GitHub market place, and follow the instructions
step by step.

>
> [1] https://docs.microsoft.com/en-us/azure/devops/pipelines/repos/github
>
> In any case, it appears that the repo has gotten fixed. The build [2]
> passes for SPL targets (which were the only failures last time).
>
> [2] https://travis-ci.org/github/Forty-Bot/u-boot/builds/689112153
>

Regards,
Bin