[RFC 1/1] platform/kendryte: disable FW_PIC

[Bin Meng]

On Sat, Sep 4, 2021 at 9:56 PM Bin Meng <bmeng.cn at gmail.com> wrote:
>
> On Sat, Sep 4, 2021 at 5:49 PM Heinrich Schuchardt <xypron.glpk at gmx.de> wrote:
> >
> > Since commit bf3ef53bb7f5 ("firmware: Enable FW_PIC by default")
> > building with
> >
> > make \
> > PLATFORM=kendryte/k210 \
> > FW_PAYLOAD=y \
> > FW_PAYLOAD_OFFSET=0x40000 \
> > FW_PAYLOAD_PATH=u-boot-dtb.bin
> >
> > fails:
> >
> > riscv64-linux-gnu/bin/ld:
> > opensbi/build/platform/kendryte/k210/firmware/fw_payload.o:
> > in function `.L0 ':
> > opensbi/firmware/fw_base.S:77:
> > undefined reference to `__rel_dyn_start'
> > riscv64-linux-gnu/bin/ld:
> > opensbi/firmware/fw_base.S:78:
> > undefined reference to `__rel_dyn_end'
> > riscv64-linux-gnu/bin/ld:
> > opensbi/firmware/fw_base.S:89:
> > undefined reference to `__dyn_sym_start'
> > collect2: error: ld returned 1 exit status
> >
>
> It looks like your toolchain is broken. I can build with your command
> without any error. I am using buildroot 2020.08-1 toolchain.
>
> > With FW_PIC=n building becomes possible.
> >
> > Fixes: bf3ef53bb7f5 ("firmware: Enable FW_PIC by default")
> > Signed-off-by: Heinrich Schuchardt <xypron.glpk at gmx.de>
> > ---
> > We need a description of what FW_PIC does in and
> > a description of how to build for the K210 platform.
> > in doc/.
> > ---

And FW_PIC is already documented in docs/firmware/fw.md

Regards,
Bin