Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Incorrect gcc_arm linker script name for stm32f407xg #14866

Closed
byq77 opened this issue Jul 2, 2021 · 3 comments · Fixed by #14898
Closed

Incorrect gcc_arm linker script name for stm32f407xg #14866

byq77 opened this issue Jul 2, 2021 · 3 comments · Fixed by #14898

Comments

@byq77
Copy link
Contributor

byq77 commented Jul 2, 2021

Description of defect

So I have a custom board that incorporates the stm32f407xG mcu. I was able to successfully configure a custom target using Mbed Os 6.12 and CLI2. I followed this guide. However, I found that the linker script name for this model is incorrect for some reason. It is in uppercase, but the CMakeLists.txt in the mbed-os/targets/TARGET_STM/TARGET_STM32F4/TARGET_STM32F407xG/ directory reference to the name in lowercase. It looks like the lowercase is also the naming convention you use. Converting STM32F407XG.ld to stm32f407xg.ld fixed the issue.

Target(s) affected by this defect ?

custom_target.json -> stm32f407xG

Toolchain(s) (name and version) displaying this defect ?

GCC_ARM

What version of Mbed-os are you using (tag or sha) ?

mbed-os-6.12.0

What version(s) of tools are you using. List all that apply (E.g. mbed-cli)

mbed CLI2

How is this defect reproduced ?

The problem is obvious, because the stm32f407xg linker script file name dosen't match the name in the line:

set(LINKER_FILE TOOLCHAIN_GCC_ARM/stm32f407xg.ld)

@mbedmain
Copy link

mbedmain commented Jul 2, 2021

@byq77 thank you for raising this issue.Please take a look at the following comments:

How can we reproduce your issue?

NOTE: If there are fields which are not applicable then please just add 'n/a' or 'None'. This indicates to us that at least all the fields have been considered.
Please update the issue header with the missing information.

@jeromecoutant
Copy link
Collaborator

jeromecoutant commented Jul 7, 2021

It looks like the lowercase is also the naming convention you use. Converting STM32F407XG.ld to stm32f407xg.ld fixed the issue.

Hi

I confirm. Please raise a pull request, we will be happy to integrate it asap.

Thx

@byq77
Copy link
Contributor Author

byq77 commented Jul 8, 2021

#14898

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants