<div dir="ltr"><div dir="ltr"><div>Hi Hesham/Sebastian,</div><div><br></div><div>Please find the attached device tree source file which I used for rtems testing on PolarFire SoC ICICLE Kit. <br></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, May 19, 2021 at 2:43 PM Hesham Almatary <<a href="mailto:hesham.almatary@cl.cam.ac.uk">hesham.almatary@cl.cam.ac.uk</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hello Somesh,<br>
<br>
This patch will break other existing BSPs that don't have<br>
"stdout-path" (in your case serial@xxxxx) node part of "/soc" node.<br>
You can either 1) Modify your DTS to have /serial node outside /soc,<br>
or 2) Add a new config for your BSP such as<br>
RISCV_ENABLE_FRDME310ARTY_SUPPORT and use it instead. I am not sure if<br>
 DTS allows having something like /soc/serial@xxxx in the stdout-path.<br></blockquote><div><span style="color:rgb(76,17,48)"><span style="color:rgb(153,0,0)">Thanks for the suggestion Hesham. We are discussing internally whether to go with a variant of the existing riscv BSP or introduce a new riscv BSP altogether</span>. </span><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
On Tue, 18 May 2021 at 20:43, Sebastian Huber<br>
<<a href="mailto:sebastian.huber@embedded-brains.de" target="_blank">sebastian.huber@embedded-brains.de</a>> wrote:<br>
><br>
> On 05/05/2021 19:55, Somesh Deshmukh wrote:<br>
> > - Parsing the sub-node should be available generic not specific to Freedom<br>
> >    Arty310 board. If we remove the Freedom Arty macro now, it will lose<br>
> >    backward compatibility.The proposed change will retain the backward<br>
> >    compatibility and also adds the necessary fix for parsing sub-node.<br>
> > ---<br>
> >   bsps/riscv/riscv/console/console-config.c | 2 +-<br>
> >   1 file changed, 1 insertion(+), 1 deletion(-)<br>
> ><br>
> > diff --git a/bsps/riscv/riscv/console/console-config.c b/bsps/riscv/riscv/console/console-config.c<br>
> > index d962a5a418..797506a31b 100644<br>
> > --- a/bsps/riscv/riscv/console/console-config.c<br>
> > +++ b/bsps/riscv/riscv/console/console-config.c<br>
> > @@ -91,7 +91,7 @@ static int riscv_get_console_node(const void *fdt)<br>
> >       stdout_path = "";<br>
> >     }<br>
> ><br>
> > -#if RISCV_ENABLE_FRDME310ARTY_SUPPORT != 0<br>
> > +#if ((RISCV_ENABLE_FRDME310ARTY_SUPPORT != 0) || (RISCV_CONSOLE_MAX_NS16550_DEVICES > 0))<br>
> >     int root;<br>
> >     int soc;<br>
> >     root = fdt_path_offset(fdt, "/");<br>
><br>
> What is the purpose of this subnode stuff here:<br>
><br>
> static int riscv_get_console_node(const void *fdt)<br>
> {<br>
>    const char *stdout_path;<br>
>    int node;<br>
><br>
>    node = fdt_path_offset(fdt, "/chosen");<br>
><br>
>    stdout_path = fdt_getprop(fdt, node, "stdout-path", NULL);<br>
>    if (stdout_path == NULL) {<br>
>      stdout_path = "";<br>
>    }<br>
><br>
> #if ((RISCV_ENABLE_FRDME310ARTY_SUPPORT != 0) ||<br>
> (RISCV_CONSOLE_MAX_NS16550_DEVICES > 0))<br>
>    int root;<br>
>    int soc;<br>
>    root = fdt_path_offset(fdt, "/");<br>
>    soc = fdt_subnode_offset(fdt, root, "soc");<br>
><br>
>    int offset=fdt_subnode_offset(fdt, soc,stdout_path);<br>
><br>
>    return offset;<br>
> #else<br>
>    return fdt_path_offset(fdt, stdout_path);<br>
> #endif<br>
> }<br>
><br>
> What is the value of stdout_path in your setup? Can you show the<br>
> relevant parts of the device tree sources?<br>
><br>
> --<br>
> embedded brains GmbH<br>
> Herr Sebastian HUBER<br>
> Dornierstr. 4<br>
> 82178 Puchheim<br>
> Germany<br>
> email: <a href="mailto:sebastian.huber@embedded-brains.de" target="_blank">sebastian.huber@embedded-brains.de</a><br>
> phone: +49-89-18 94 741 - 16<br>
> fax:   +49-89-18 94 741 - 08<br>
><br>
> Registergericht: Amtsgericht München<br>
> Registernummer: HRB 157899<br>
> Vertretungsberechtigte Geschäftsführer: Peter Rasmussen, Thomas Dörfler<br>
> Unsere Datenschutzerklärung finden Sie hier:<br>
> <a href="https://embedded-brains.de/datenschutzerklaerung/" rel="noreferrer" target="_blank">https://embedded-brains.de/datenschutzerklaerung/</a><br>
> _______________________________________________<br>
> devel mailing list<br>
> <a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
> <a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
</blockquote></div></div>