[PATCH 01/16] beatnik: Move -Dxxx to configure.ac

Sebastian Huber sebastian.huber at embedded-brains.de
Fri May 18 13:25:12 UTC 2018


On 18/05/18 15:21, Joel Sherrill wrote:
>
>
> On Fri, May 18, 2018 at 12:41 AM, Sebastian Huber 
> <sebastian.huber at embedded-brains.de 
> <mailto:sebastian.huber at embedded-brains.de>> wrote:
>
>     On 16/05/18 17:09, Joel Sherrill wrote:
>
>         Updates #3425.
>         ---
>           bsps/powerpc/beatnik/config/beatnik.cfg       | 2 +-
>           c/src/lib/libbsp/powerpc/beatnik/configure.ac
>         <http://configure.ac> | 2 ++
>           2 files changed, 3 insertions(+), 1 deletion(-)
>
>         diff --git a/bsps/powerpc/beatnik/config/beatnik.cfg
>         b/bsps/powerpc/beatnik/config/beatnik.cfg
>         index 1190cad..05bea38 100644
>         --- a/bsps/powerpc/beatnik/config/beatnik.cfg
>         +++ b/bsps/powerpc/beatnik/config/beatnik.cfg
>         @@ -14,7 +14,7 @@ RTEMS_BSP_FAMILY=beatnik
>           #  This contains the compiler options necessary to select
>         the CPU model
>           #  and (hopefully) optimize for it.
>           #
>         -CPU_CFLAGS = -mcpu=7400 -D__ppc_generic
>         +CPU_CFLAGS = -mcpu=7400
>           #T. Straumann; disable sdata=eabi for now until CEXP
>         supports it -meabi -msdata=eabi
>             # optimize flag: typically -0, could use -O4 or -fast
>         diff --git a/c/src/lib/libbsp/powerpc/beatnik/configure.ac
>         <http://configure.ac>
>         b/c/src/lib/libbsp/powerpc/beatnik/configure.ac
>         <http://configure.ac>
>         index a48007c..b332aaa 100644
>         --- a/c/src/lib/libbsp/powerpc/beatnik/configure.ac
>         <http://configure.ac>
>         +++ b/c/src/lib/libbsp/powerpc/beatnik/configure.ac
>         <http://configure.ac>
>         @@ -34,6 +34,8 @@ Note that the policy can still be defined by
>         the application
>           CONFIGURE_MALLOC_BSP_SUPPORTS_SBRK this feature is removed
>           and a little memory is saved.])
>           +AC_DEFINE(__ppc_generic, 1, [PowerPC model option])
>         +
>           # Explicitly list all Makefiles here
>           AC_CONFIG_FILES([Makefile])
>
>
>     I would rather eliminate the only user of __ppc_generic in
>     bsps/powerpc/include/rtems/powerpc/powerpc.h.
>
>
> I have file another ticket #3431 to address this and assigned it to you.
>
> I am committing my changes because they eliminate use of -D on the GCC 
> command line
> and allow packages using rtems_waf to compile.

Did you check that every consumer of these defines includes <bspopts.h> 
somehow?

-- 
Sebastian Huber, embedded brains GmbH

Address : Dornierstr. 4, D-82178 Puchheim, Germany
Phone   : +49 89 189 47 41-16
Fax     : +49 89 189 47 41-09
E-Mail  : sebastian.huber at embedded-brains.de
PGP     : Public key available on request.

Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.




More information about the devel mailing list