<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Jan 3, 2020, 8:53 PM Niteesh <<a href="mailto:gsnb.gn@gmail.com" target="_blank" rel="noreferrer">gsnb.gn@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div dir="ltr">On Fri, Jan 3, 2020 at 7:32 PM Vijay Kumar Banerjee <<a href="mailto:vijaykumar9597@gmail.com" rel="noreferrer noreferrer" target="_blank">vijaykumar9597@gmail.com</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div>Hi Niteesh,</div><div>A few formatting suggestions:</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
diff --git a/bsps/arm/raspberrypi/console/console-config.c b/bsps/arm/raspberrypi/console/console-config.c<br>
index d2186c918b..4000fd1c50 100644<br>
--- a/bsps/arm/raspberrypi/console/console-config.c<br>
+++ b/bsps/arm/raspberrypi/console/console-config.c<br>
@@ -19,50 +19,132 @@<br>
  */<br>
<br>
 #include <rtems/bspIo.h><br>
-<br>
-#include <libchip/serial.h><br></blockquote><div>You removed it here and added it below</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
+#include <rtems/console.h><br>
+#include <rtems/sysinit.h><br>
<br>
 #include <bspopts.h><br>
-#include <bsp/irq.h><br>
-#include <bsp/usart.h><br></blockquote><div>same here </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
+#include <bsp.h><br>
 #include <bsp/raspberrypi.h><br>
+#include <bsp/usart.h><br></blockquote><div>added here ^ </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
+#include <bsp/arm-pl011.h><br>
 #include <bsp/fbcons.h><br>
+#include <bsp/console-termios.h><br>
+#include <bsp/fdt.h><br>
+#include <bsp/fatal.h><br>
+<br>
+#include <libfdt.h><br>
+#include <libchip/serial.h> </blockquote><div>here</div></div></div></blockquote><div>Is there any particular reason for doing it, or is it just a preference?</div></div></div></blockquote></div></div><div dir="auto">For RTEMS, I don't see any preference mentioned in the docs for the order or includes:</div><div dir="auto"><a href="https://docs.rtems.org/branches/master/eng/coding-conventions.html">https://docs.rtems.org/branches/master/eng/coding-conventions.html</a><br></div><div dir="auto"><br></div><div dir="auto">In libbsd, however, the FreeBSD style guide is followed which has a preference for the order of header includes:</div><div dir="auto"><a href="https://www.freebsd.org/cgi/man.cgi?query=style&apropos=0&sektion=9">https://www.freebsd.org/cgi/man.cgi?query=style&apropos=0&sektion=9</a><br></div><div dir="auto"><br></div><div dir="auto">Apart from that, this is a redundant change. Is there a reason to change the order?</div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div>This is just a small change and I would suggest you wait for at least a day</div><div>before sending a v2 of the patch so that you can add changes suggested</div><div>by other people (if any). Congrats on getting it working ;-)</div></div></div></blockquote><div>Thank you. </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div>Best,</div><div>Vijay </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
+<br>
+#define UART0     "/dev/ttyS0"<br>
+#define FBCONS    "/dev/fbcons"<br>
+<br>
</blockquote></div></div>
</blockquote></div></div>
</blockquote></div></div></div>