On 12/10/17 12:00 am, Sebastian Huber wrote: > Module: rtems > Branch: master > Commit: 16fcd56a42dfdef514350ab6297137e2c863afa1 > Changeset: > http://git.rtems.org/rtems/commit/?id=16fcd56a42dfdef514350ab6297137e2c863afa1 > > Author: Christian Mauderer <christian.maude...@embedded-brains.de> > Date: Wed Oct 11 15:38:14 2017 +0200 > > bsp/atsam: Allow to change optimization settings. > > Allow to overwrite the optimization settings during configuration. > > --- > > c/src/lib/libbsp/arm/atsam/make/custom/atsamv.cfg | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/c/src/lib/libbsp/arm/atsam/make/custom/atsamv.cfg > b/c/src/lib/libbsp/arm/atsam/make/custom/atsamv.cfg > index 06892f9..0a2437f 100644 > --- a/c/src/lib/libbsp/arm/atsam/make/custom/atsamv.cfg > +++ b/c/src/lib/libbsp/arm/atsam/make/custom/atsamv.cfg > @@ -4,7 +4,6 @@ RTEMS_CPU = arm > > CPU_CFLAGS = -mthumb -mcpu=cortex-m7 -mfpu=fpv5-d16 -mfloat-abi=hard > > -CFLAGS_OPTIMIZE_V = -O2 -g > -CFLAGS_OPTIMIZE_V += -ffunction-sections -fdata-sections > +CFLAGS_OPTIMIZE_V ?= -O2 -g -ffunction-sections -fdata-sections
This is a really nice change. Should all the BSP's be changed? I see a 104 instances in the code and the change could be scripted. Being inconsistent on something that could be consistent across all BSPs is confusing to all users. Chris _______________________________________________ devel mailing list devel@rtems.org http://lists.rtems.org/mailman/listinfo/devel
