Hi Bin, >-----Original Message----- >From: Bin Meng <bmeng...@gmail.com> >Sent: 02 May 2020 18:26 >To: Pragnesh Patel <pragnesh.pa...@sifive.com> >Cc: U-Boot Mailing List <u-boot@lists.denx.de>; Atish Patra ><atish.pa...@wdc.com>; Palmer Dabbelt <palmerdabb...@google.com>; Paul >Walmsley <paul.walms...@sifive.com>; Jagan Teki ><ja...@amarulasolutions.com>; Troy Benjegerdes ><troy.benjeger...@sifive.com>; Anup Patel <anup.pa...@wdc.com>; Sagar >Kadam <sagar.ka...@sifive.com>; Rick Chen <r...@andestech.com> >Subject: Re: [PATCH v7 21/22] doc: sifive: fu540: Add description for OpenSBI >generic platform > >[External Email] Do not click links or attachments unless you recognize the >sender and know the content is safe > >Hi Pragnesh, > >On Sat, May 2, 2020 at 6:09 PM Pragnesh Patel <pragnesh.pa...@sifive.com> >wrote: >> >> OpenSBI generic platform support provides platform specific >> functionality based on the FDT passed by previous booting stage. >> >> depends on OpenSBI commit: >> platform: Add generic FDT based platform support >> (sha1: f1aa9e54e00006ae70aeac638d5b75093520f65d) >> >> Signed-off-by: Pragnesh Patel <pragnesh.pa...@sifive.com> >> --- >> doc/board/sifive/fu540.rst | 19 +++++++++++++------ >> 1 file changed, 13 insertions(+), 6 deletions(-) >> >> diff --git a/doc/board/sifive/fu540.rst b/doc/board/sifive/fu540.rst >> index 610ba87074..1c3d4740f3 100644 >> --- a/doc/board/sifive/fu540.rst >> +++ b/doc/board/sifive/fu540.rst >> @@ -22,14 +22,21 @@ The support for following drivers are already >enabled: >> >> TODO: >> >> -1. U-Boot expects the serial console device entry to be present under >/chosen >> - DT node. Without a serial console U-Boot will panic. Example: >> +1. The DTB passed by U-Boot SPL or FSBL to OpenSBI should have CLINT >> + DT node and the "/chosen" DT node should have "stdout-path" DT >> + property. >> >> .. code-block:: none >> >> - chosen { >> - stdout-path = "/soc/serial@10010000:115200"; >> - }; >> + chosen { >> + stdout-path = "/soc/serial@10010000:115200"; >> + }; >> + >> + clint@2000000 { >> + compatible = "riscv,clint0"; >> + interrupts-extended = <&cpu0_intc 3 &cpu0_intc 7 &cpu1_intc 3 >&cpu1_intc 7 &cpu2_intc 3 &cpu2_intc 7 &cpu3_intc 3 &cpu3_intc 7 >&cpu4_intc 3 &cpu4_intc 7>; >> + reg = <0x0 0x2000000 0x0 0xc0000>; >> + }; > >Can we update the U-Boot dtsi to include the changes?
Can you please clarify which dtsi fu540-c000-u-boot.dtsi or fu540-c000.dtsi ? I have already added clint DT node in "arch/riscv/dts/fu540-c000-u-boot.dtsi" > >> >> Building >> -------- >> @@ -58,7 +65,7 @@ firmware. We need to compile OpenSBI with below >command: >> >> .. code-block:: none >> >> -make PLATFORM=sifive/fu540 FW_PAYLOAD_PATH=<path to u-boot- >dtb.bin> >> + make PLATFORM=generic FW_PAYLOAD_PATH=<path to u-boot- >dtb.bin> >> >> More detailed description of steps required to build FW_PAYLOAD >> firmware is beyond the scope of this document. Please refer OpenSBI >documenation. > >Regards, >Bin