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

Wrong comment for VECT_TAB_OFFSET #2

Closed
carlo-dev-git opened this issue Dec 15, 2021 · 7 comments
Closed

Wrong comment for VECT_TAB_OFFSET #2

carlo-dev-git opened this issue Dec 15, 2021 · 7 comments
Assignees
Labels
bug Something isn't working cmsis CMSIS-related issue or pull-request. internal bug tracker Issue confirmed and reported into a ticket in the internal bug tracking system.
Milestone

Comments

@ALABSTM
Copy link
Collaborator

ALABSTM commented Jan 5, 2022

Hi @carlo-dev-git,

Thank you for having reported. Indeed, there is an inconsistence between the different comments. This will be reported to our development teams.

With regards,

PS: Best wishes for the new year 2022.

@ALABSTM
Copy link
Collaborator

ALABSTM commented Jan 5, 2022

ST Internal Reference: 120332

@ALABSTM ALABSTM added the documentation Improvements or additions to documentation label Jan 5, 2022
@ALABSTM ALABSTM self-assigned this Jan 5, 2022
@ALABSTM ALABSTM added the internal bug tracker Issue confirmed and reported into a ticket in the internal bug tracking system. label Jan 7, 2022
@ALABSTM ALABSTM added this to the v1.11.0 milestone Feb 15, 2022
@carlo-dev-git
Copy link
Author

Hi ALABSTM,

I just updated the STM32Cube STM32H7 MCU Package to v1.10.0 but this ticket is not fixed because it was scheduled to v1.11.0. From stm32cube-mcu-cmsis-dashboard the ticket status is set To release, so could you just tell me what is the correct offset (0x200 or 0x300)?
It is very important for me.

Best regards.

@ALABSTM
Copy link
Collaborator

ALABSTM commented Mar 14, 2022

Hi @carlo-dev-git,

My bad. I mistook the milestone which is v1.10.2 and not v1.11.0. Actually, the typo has been corrected as you can see from the code snippet below. The correct offset value is 0x300. The fix has been published on both this repo and STM32CubeH7.

https://github.com/STMicroelectronics/cmsis_device_h7/blob/a95954cd368b94e60d3dc42b8005a0460a18d7cb/Source/Templates/system_stm32h7xx.c#L98-L99

With regards,

@ALABSTM ALABSTM closed this as completed Mar 14, 2022
@carlo-dev-git
Copy link
Author

@carlo-dev-git
Copy link
Author

Hi ALABSTM,
please check PM0253 STM32F7 Series and STM32H7 Series Cortex®-M7 processor programming manual (Rev 5)

4.3.4 Vector table offset register
The table alignment requirements mean that bits [8:0] of the table offset are always zero.

0x300 -> 001100000000b

@ALABSTM
Copy link
Collaborator

ALABSTM commented Apr 11, 2022

Hi @carlo-dev-git,

I sent your question back to our development teams and they confirm your doubts. Actually, the VTO should be a multiple of 0x400 for both single- and dual-core part-numbers.

We do apologize for this inconvenience. I will reopen this issue pending the fix publication.

With regards,

@ALABSTM ALABSTM reopened this Apr 11, 2022
@HBOSTM HBOSTM added the cmsis CMSIS-related issue or pull-request. label Feb 21, 2023
@TOUNSTM TOUNSTM closed this as completed Feb 23, 2023
@TOUNSTM TOUNSTM modified the milestone: v1.10.2 Feb 23, 2023
@ALABSTM ALABSTM added bug Something isn't working and removed documentation Improvements or additions to documentation labels Feb 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working cmsis CMSIS-related issue or pull-request. internal bug tracker Issue confirmed and reported into a ticket in the internal bug tracking system.
Projects
None yet
Development

No branches or pull requests

4 participants