<div dir="ltr"><div class="gmail_default" style="font-size:small">Just to provide more context,</div><div class="gmail_default" style="font-size:small">When the CONFIGURE_APPLICATION_NEEDS_CONSOLE_DRIVER option is used</div><div class="gmail_default" style="font-size:small">and no --console option is provided, the console driver defaults to PL011.</div><div class="gmail_default" style="font-size:small">In raspberry pi 3 and other models whose primary UART is not PL011, we get no output.</div><div class="gmail_default" style="font-size:small">This patch fixes that by linking the primary UART to the console device.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Thanks,</div><div class="gmail_default" style="font-size:small">Niteesh</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, May 1, 2021 at 7:05 PM G S Niteesh Babu <<a href="mailto:niteesh.gs@gmail.com">niteesh.gs@gmail.com</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">When no console argument is given, the driver defaults to pl011<br>
this results in no output in case of Rpi3 whose primary uart is<br>
miniuart.<br>
This patch fixes that by defaulting to the primary uart when no<br>
console option is provided.<br>
---<br>
 bsps/arm/raspberrypi/console/console-config.c | 12 +++++++++---<br>
 1 file changed, 9 insertions(+), 3 deletions(-)<br>
<br>
diff --git a/bsps/arm/raspberrypi/console/console-config.c b/bsps/arm/raspberrypi/console/console-config.c<br>
index 6b8eb80aa4..bd3a8d34c2 100644<br>
--- a/bsps/arm/raspberrypi/console/console-config.c<br>
+++ b/bsps/arm/raspberrypi/console/console-config.c<br>
@@ -165,10 +165,16 @@ static void console_select( void )<br>
     }<br>
   }else {<br>
     /**<br>
-     * If no command line option was given, default to PL011.<br>
+     * If no console option was given we default to the primary uarts.<br>
+     * The initialization of the uart's and BSP_output_char is already done<br>
+     * in the uart_probe function called before this. So now we can safely<br>
+     * compare BSP_output_char.<br>
      */<br>
-    BSP_output_char = output_char_pl011;<br>
-    link(PL011, CONSOLE_DEVICE_NAME);<br>
+    if (BSP_output_char == output_char_pl011) {<br>
+      link(PL011, CONSOLE_DEVICE_NAME);<br>
+    }else {<br>
+      link(MINIUART, CONSOLE_DEVICE_NAME);<br>
+    }<br>
   }<br>
 }<br>
<br>
-- <br>
2.17.1<br>
<br>
</blockquote></div>