search

ARM-software / Cortex_DFP

CMSIS generic Arm Cortex-M device family pack
Apache License 2.0
14 stars 5 forks source link

Avoid hidden FLASH overflow in GCC linker scripts #3

Closed ghost closed 1 year ago

ghost commented 1 year ago

updated Cortex-M GCC linker scripts

JonatanAntoni commented 1 year ago

Looks good, tested with CoreValidation and seems working. Please update the config file versions of all affected linker scripts in the pack description.

ghost commented 1 year ago

Updated config file versions of affected linker scripts in the pack description

JonatanAntoni commented 1 year ago

Good point. Shall we update the Cortex-A linker scripts as well? Should they have the same file header? Otherwise one can hardly see which version was used at time of copying from the pack.

ghost commented 1 year ago

I would suggest to add file headers to the Cortex-A linker scripts and update with the latest changes. Who can review the changes and how are they tested?

JonatanAntoni commented 1 year ago

I'd hope to be able to run the CoreValidation tests to check if linker script still does the job.

ghost commented 1 year ago

I will change update the Cortex-A linker scripts accordingly.

ghost commented 1 year ago

we should postpone changing Cortex-A linker scripts.