-
Notifications
You must be signed in to change notification settings - Fork 3k
Update cube h7 to 1.4.0 #10631
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
Update cube h7 to 1.4.0 #10631
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested OK with ST CI
Note that this STM32H7 driver version is introducing Dual Core targets, including H747 |
@VVESTM, thank you for your changes. |
@@ -82,7 +82,7 @@ static const uint16_t USART_PRESCALER_TAB[] = | |||
#endif /*USE_FULL_LL_DRIVER*/ | |||
|
|||
/* Exported types ------------------------------------------------------------*/ | |||
#if defined(USE_FULL_LL_DRIVER) | |||
#if 1 //mbed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we dont rather define the macro as 1 instead of rewriting the driver files?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're right. I have done like this because it was already the case before...
I check how to clean this part.
Let us know once updated, we can start CI. |
Keep #define USE_FULL_LL_DRIVER and define this switch in target.json Signed-off-by: Vincent Veron <[email protected]>
Here is a new version removing the previous #if 1... |
CI started |
Test run: SUCCESSSummary: 11 of 11 test jobs passed |
Description
Update of the cube H7 HAL and LL. This is coming from st.com website : https://www.st.com/en/embedded-software/stm32cubeh7.html
Used version is 1.4.0
Pull request type
Reviewers
@jeromecoutant