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

F4 HAL does not allow overriding of VECT_TAB_OFFSET #36

Open
rogerclarkmelbourne opened this issue Aug 28, 2017 · 2 comments
Open

F4 HAL does not allow overriding of VECT_TAB_OFFSET #36

rogerclarkmelbourne opened this issue Aug 28, 2017 · 2 comments

Comments

@rogerclarkmelbourne
Copy link
Contributor

I'm not sure the best way to handle this, but the F4 does not allow VECT_TAB_OFFSET to be overridden

https://github.com/danieleff/STM32GENERIC/blob/master/STM32/system/STM32F4/HAL_Src/system_stm32f4xx.c#L111

Unlike the F1, which has a "guard" indef
https://github.com/danieleff/STM32GENERIC/blob/master/STM32/system/STM32F1/HAL_Src/system_stm32f1xx.c#L113-L116

Is the best fix for this, simply for me to manually update file , to add the #ifdef and generate a PR for this update ?

@danieleff
Copy link
Owner

Yes, #ifndef VECT_TAB_OFFSET is fine.
Originally the F1 did not have it either.

@rogerclarkmelbourne
Copy link
Contributor Author

OK
I'll fork your repo and do a PR

BTW. I'm working on a F4 bootloader with @victorpv .(well Victor has done most of the work)
Hence why I need to change the VECT_TAB_OFFSET

I've managed to make it work, by creating a new upload type in your boards.txt etc

But at the moment I have to manually issue the dfu command to upload the compiled sketch

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

No branches or pull requests

2 participants