On Sat, May 2, 2020 at 8:57 PM Bin Meng <bmeng...@gmail.com> wrote: > > Hi Pragnesh, > > On Sat, May 2, 2020 at 11:22 PM Pragnesh Patel > <pragnesh.pa...@sifive.com> wrote: > > > > > > Hi Bin, > > > > >-----Original Message----- > > >From: Bin Meng <bmeng...@gmail.com> > > >Sent: 02 May 2020 20:47 > > >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 10:30 PM Pragnesh Patel <pragnesh.pa...@sifive.com> > > >wrote: > > >> > > >> 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" > > > > > >I thought we did not add this. So maybe we can remove this information > > >completely from the doc? > > > > OpenSBI expect a "chosen DT node and clint DT node" from prior stage > > bootloader DTB, So i think > > it's fine to add clint as a reference here. > > > > Any suggestions are welcome. > > > > Maybe such information should be put in the OpenSBI generic platform > doc, but not U-Boot doc ??
Sure, I will add more details to OpenSBI generic platform documentation. Regards, Anup