commit e0bb1f956f75c84500278ab4cc482f833acc413f
author David Brown <david.brown@linaro.org> Tue Oct 01 15:57:01 2019 -0600
committer David Brown <davidb@davidb.org> Wed Oct 02 10:17:22 2019 -0700
tree cbe09cf17e6f1d38ded88645c764e1346b81edd7
parent 510fddb8e06d76e2442b2a4603d3e1cbefe28be4

Change BOOT_MAX_ALIGN to #define

BOOT_MAX_ALIGN is defined as

    extern const uint32_t BOOT_MAX_ALIGN;

and is assigned a value in a single file.  This causes extra work when
this is used as the size of a local variable in a function.

The value was made a constant in order for the simulator to be able to
access the value.  Instead of making it a "real" constant, keep it as a
define, unifying the value of FLASH_MAX_ALIGN and this one, and provide
an accessor function for the test code to be able to access this value.

This causes a minor improvement in the code generated in
`boot_write_status`, but more importantly, eliminates a VLA from the
code, which increases the possible compilers supported by MCUboot.

Signed-off-by: David Brown <david.brown@linaro.org>