Skip to content

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

Merged
merged 2 commits into from
May 23, 2019
Merged

Update cube h7 to 1.4.0 #10631

merged 2 commits into from
May 23, 2019

Conversation

VVESTM
Copy link
Contributor

@VVESTM VVESTM commented May 22, 2019

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

[ ] Fix
[ ] Refactor
[X] Target update
[ ] Functionality change
[ ] Docs update
[ ] Test update
[ ] Breaking change

Reviewers

@jeromecoutant

Copy link
Collaborator

@jeromecoutant jeromecoutant left a 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

@jeromecoutant
Copy link
Collaborator

Note that this STM32H7 driver version is introducing Dual Core targets, including H747

@facchinm
@MarceloSalazar
@LMESTM

@ciarmcom
Copy link
Member

@VVESTM, thank you for your changes.
@ARMmbed/mbed-os-maintainers please review.

@ciarmcom ciarmcom requested a review from a team May 22, 2019 09:00
@@ -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
Copy link
Contributor

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?

Copy link
Contributor Author

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.

@0xc0170
Copy link
Contributor

0xc0170 commented May 23, 2019

Let us know once updated, we can start CI.

VVESTM added 2 commits May 23, 2019 14:01
Keep #define USE_FULL_LL_DRIVER and define this switch in target.json

Signed-off-by: Vincent Veron <[email protected]>
@VVESTM
Copy link
Contributor Author

VVESTM commented May 23, 2019

Here is a new version removing the previous #if 1...

@0xc0170
Copy link
Contributor

0xc0170 commented May 23, 2019

CI started

@mbed-ci
Copy link

mbed-ci commented May 23, 2019

Test run: SUCCESS

Summary: 11 of 11 test jobs passed
Build number : 1
Build artifacts

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

Successfully merging this pull request may close these issues.

5 participants