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? Regards, Bin