<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, May 18, 2018 at 12:41 AM, Sebastian Huber <span dir="ltr"><<a href="mailto:sebastian.huber@embedded-brains.de" target="_blank">sebastian.huber@embedded-brains.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On 16/05/18 17:09, Joel Sherrill wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Updates #3425.<br>
---<br>
  bsps/powerpc/beatnik/config/be<wbr>atnik.cfg       | 2 +-<br>
  c/src/lib/libbsp/powerpc/beatn<wbr>ik/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a> | 2 ++<br>
  2 files changed, 3 insertions(+), 1 deletion(-)<br>
<br>
diff --git a/bsps/powerpc/beatnik/config/<wbr>beatnik.cfg b/bsps/powerpc/beatnik/config/<wbr>beatnik.cfg<br>
index 1190cad..05bea38 100644<br>
--- a/bsps/powerpc/beatnik/config/<wbr>beatnik.cfg<br>
+++ b/bsps/powerpc/beatnik/config/<wbr>beatnik.cfg<br>
@@ -14,7 +14,7 @@ RTEMS_BSP_FAMILY=beatnik<br>
  #  This contains the compiler options necessary to select the CPU model<br>
  #  and (hopefully) optimize for it.<br>
  #<br>
-CPU_CFLAGS = -mcpu=7400 -D__ppc_generic<br>
+CPU_CFLAGS = -mcpu=7400<br>
  #T. Straumann; disable sdata=eabi for now until CEXP supports it -meabi -msdata=eabi<br>
    # optimize flag: typically -0, could use -O4 or -fast<br>
diff --git a/c/src/lib/libbsp/powerpc/bea<wbr>tnik/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a> b/c/src/lib/libbsp/powerpc/bea<wbr>tnik/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a><br>
index a48007c..b332aaa 100644<br>
--- a/c/src/lib/libbsp/powerpc/bea<wbr>tnik/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a><br>
+++ b/c/src/lib/libbsp/powerpc/bea<wbr>tnik/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a><br>
@@ -34,6 +34,8 @@ Note that the policy can still be defined by the application<br>
  CONFIGURE_MALLOC_BSP_SUPPORTS_<wbr>SBRK this feature is removed<br>
  and a little memory is saved.])<br>
  +AC_DEFINE(__ppc_generic, 1, [PowerPC model option])<br>
+<br>
  # Explicitly list all Makefiles here<br>
  AC_CONFIG_FILES([Makefile])<br>
  <br>
</blockquote>
<br></div></div>
I would rather eliminate the only user of __ppc_generic in bsps/powerpc/include/rtems/pow<wbr>erpc/powerpc.h.</blockquote><div><br></div><div>I have file another ticket #3431 to address this and assigned it to you. </div><div><br></div><div>I am committing my changes because they eliminate use of -D on the GCC command line</div><div>and allow packages using rtems_waf to compile.</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="HOEnZb"><font color="#888888"><br>
<br>
-- <br>
Sebastian Huber, embedded brains GmbH<br>
<br>
Address : Dornierstr. 4, D-82178 Puchheim, Germany<br>
Phone   : +49 89 189 47 41-16<br>
Fax     : +49 89 189 47 41-09<br>
E-Mail  : <a href="mailto:sebastian.huber@embedded-brains.de" target="_blank">sebastian.huber@embedded-brain<wbr>s.de</a><br>
PGP     : Public key available on request.<br>
<br>
Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.<br>
<br>
</font></span></blockquote></div><br></div></div>