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

template file system_stm32g4xx.c should support userdefined location of vectortable #41

Open
marcgendron-dyze opened this issue Aug 9, 2023 · 2 comments
Assignees
Labels
cmsis CMSIS-related issue or pull-request. enhancement New feature or request internal bug tracker Issue confirmed and reported into a ticket in the internal bug tracking system spotted before customer Spotted and fixed internally before being pointed out by users but not published yet

Comments

@marcgendron-dyze
Copy link

Caution
The Issues are strictly limited for the reporting of problem encountered with the software provided in this project.
For any other problem related to the STM32 product, the performance, the hardware characteristics and boards, the tools the environment in general, please post a topic in the ST Community/STM32 MCUs forum.

Describe the set-up

  • The board (either ST RPN reference or your custom board).
  • IDE or at least the compiler and its version.

Describe the bug
Same as STMicroelectronics/STM32CubeF4#30
`A clear and concise description of what the bug is.
template file system_stm32g4xx.c should support userdefined location of vectortable
using a custom bootloader requires to move the default location of the vector table.
in the file system_stm32g4xx.c that is copied by the program generator is the needed settings for the move of the vector table.
But the user can not control this behaviour by compiler command arguments
in the defines section at the beginning of the file line 92ff
the lines:
/* #define VECT_TAB_SRAM /
#define VECT_TAB_OFFSET 0x00 /!< Vector Table base offset field.
This value must be a multiple of 0x200. /
/*****************************************************************************/

have to be changed to
/* #define VECT_TAB_SRAM /
#if !defined(VECT_TAB_OFFSET)
#define VECT_TAB_OFFSET 0x00 /!< Vector Table base offset field.
This value must be a multiple of 0x200. /
#endif
/*****************************************************************************/

So the user can control the define from compiler command arguments.
This allows targets for bootloader or plain memory layout.

All the existing code will not break.`

How To Reproduce

  1. Indicate the global behavior of your application project.

  2. The modules that you suspect to be the cause of the problem (Driver, BSP, MW ...).

  3. The use case that generates the problem.

  4. How we can reproduce the problem.

Additional context
If you have a first analysis or patch correction, thank you to share your proposal.

Screenshots
If applicable, add screenshots to help explain your problem.

@ALABSTM ALABSTM self-assigned this Aug 11, 2023
@ALABSTM ALABSTM added enhancement New feature or request cmsis CMSIS-related issue or pull-request. labels Aug 11, 2023
@ALABSTM ALABSTM added this to To do in stm32cube-mcu-fw-dashboard via automation Aug 11, 2023
@ALABSTM
Copy link
Contributor

ALABSTM commented Aug 18, 2023

Hi @marcgendron-dyze,

Thank you for this report. We are already aware of this point and are working on a global update. We will let you know once a fix is published. We cannot share a date for the moment. Thank you for your comprehension.

With regards,

@ALABSTM ALABSTM moved this from To do to Analyzed in stm32cube-mcu-fw-dashboard Aug 18, 2023
@ALABSTM ALABSTM added the spotted before customer Spotted and fixed internally before being pointed out by users but not published yet label Aug 18, 2023
@ALABSTM ALABSTM added the internal bug tracker Issue confirmed and reported into a ticket in the internal bug tracking system label Nov 29, 2023
@ALABSTM
Copy link
Contributor

ALABSTM commented Nov 29, 2023

ST Internal Reference: 116475

@ALABSTM ALABSTM moved this from Analyzed to In progress in stm32cube-mcu-fw-dashboard Nov 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cmsis CMSIS-related issue or pull-request. enhancement New feature or request internal bug tracker Issue confirmed and reported into a ticket in the internal bug tracking system spotted before customer Spotted and fixed internally before being pointed out by users but not published yet
Projects
Development

No branches or pull requests

2 participants