<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Aug 1, 2023, 15:30 Joel Sherrill <<a href="mailto:joel@rtems.org">joel@rtems.org</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"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Aug 1, 2023 at 2:52 PM Vijay Kumar Banerjee <<a href="mailto:vijay@rtems.org" target="_blank" rel="noreferrer">vijay@rtems.org</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"><div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Aug 1, 2023, 12:20 Joel Sherrill <<a href="mailto:joel@rtems.org" target="_blank" rel="noreferrer">joel@rtems.org</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"><div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Aug 1, 2023, 11:41 AM Vijay Kumar Banerjee <<a href="mailto:vijay@rtems.org" rel="noreferrer noreferrer" target="_blank">vijay@rtems.org</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">---<br>
 bsps/powerpc/mvme3100/start/bspclean.c        | 25 +++++++++++++++++++<br>
 .../bsps/powerpc/mvme3100/bspmvme3100.yml     |  1 +<br>
 2 files changed, 26 insertions(+)<br>
 create mode 100644 bsps/powerpc/mvme3100/start/bspclean.c<br>
<br>
diff --git a/bsps/powerpc/mvme3100/start/bspclean.c b/bsps/powerpc/mvme3100/start/bspclean.c<br>
new file mode 100644<br>
index 0000000000..251d47a46d<br>
--- /dev/null<br>
+++ b/bsps/powerpc/mvme3100/start/bspclean.c<br>
@@ -0,0 +1,25 @@<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">There is no copyright, licence, or Doxygen file header block.</div><div dir="auto"><br></div><div dir="auto"><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">
+#include <bsp.h><br>
+#include <bsp/bootcard.h><br>
+#include <rtems/bspIo.h><br>
+<br>
+void bsp_fatal_extension(<br>
+  rtems_fatal_source source,<br>
+  bool always_set_to_false,<br>
+  rtems_fatal_code error<br>
+)<br>
+{<br>
+  printk("fatal source: %s\n", rtems_fatal_source_text(source));<br>
+<br>
+  if (source == RTEMS_FATAL_SOURCE_EXCEPTION) {<br>
+    rtems_exception_frame_print((const rtems_exception_frame *) error);<br>
+  }<br>
+<br>
+  /* We can't go back to MotLoad since we blew it's memory area<br>
+   * and vectors. Just pull the reset line...<br>
+   */<br>
+  printk(<br>
+    "bsp_fatal_extension(): RTEMS terminated -- no way back to MotLoad "<br>
+      "so I reset the card\n"<br>
+  );<br>
+  bsp_reset();<br>
+}<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">What does Motorola PowerPC or the other similar bsps do?</div><div dir="auto"><br></div><div dir="auto">Can the behaviour be unified and shared?</div></div></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Thanks for the suggestion!</div><div dir="auto">Motorola powerpc uses bspfatal-default.c from bsps/shared. It was not working with mvme3100 because it added the bspreset-empty in the build. Just removing the empty reset makes it work. This patch is no longer needed. I'll send a smaller patch with this much neater solution using the shared defaults.</div></div></blockquote><div><br></div><div>Wow! Good to hear. Probably should check beatnik and the other mvme* ppc BSPs  as well,</div></div></div></blockquote></div></div><div dir="auto"><br></div><div dir="auto">At least two bsps (mvme5500 and beatnik) has this issue. I will test on the hardware and post patches for other boards too. </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="auto"><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto">Thanks!</div></div></blockquote><div><br></div><div>No problem. </div><div><br></div><div>--joel </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="auto"><div dir="auto"><br></div><div dir="auto"><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="auto"><div dir="auto"><br></div><div dir="auto"><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">
diff --git a/spec/build/bsps/powerpc/mvme3100/bspmvme3100.yml b/spec/build/bsps/powerpc/mvme3100/bspmvme3100.yml<br>
index 1667c1617a..dc04e4dd36 100644<br>
--- a/spec/build/bsps/powerpc/mvme3100/bspmvme3100.yml<br>
+++ b/spec/build/bsps/powerpc/mvme3100/bspmvme3100.yml<br>
@@ -64,6 +64,7 @@ source:<br>
 - bsps/powerpc/mvme3100/pci/detect_host_bridge.c<br>
 - bsps/powerpc/mvme3100/rtc/todcfg.c<br>
 - bsps/powerpc/mvme3100/start/bspstart.c<br>
+- bsps/powerpc/mvme3100/start/bspclean.c<br>
 - bsps/powerpc/mvme3100/start/misc.c<br>
 - bsps/powerpc/shared/btimer/btimer-ppc-dec.c<br>
 - bsps/powerpc/shared/cache/cache.c<br>
-- <br>
2.39.3<br>
<br>
_______________________________________________<br>
devel mailing list<br>
<a href="mailto:devel@rtems.org" rel="noreferrer noreferrer noreferrer" target="_blank">devel@rtems.org</a><br>
<a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer noreferrer noreferrer noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
</blockquote></div></div></div>
</blockquote></div></div></div>
</blockquote></div></div>
</blockquote></div></div></div>