[rtems commit] bsp/atsam: Allow to change optimization settings.

[Chris Johns]

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.Mauderer at 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