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

STM32U5 HAL update, NUCLEO_U575ZI upload method file #184

Merged
merged 3 commits into from
Sep 24, 2023

Conversation

multiplemonomials
Copy link
Collaborator

@multiplemonomials multiplemonomials commented Sep 23, 2023

Summary of changes

This MR updates the STM32U5 HAL to a newer version. This fixes a number of bugs, e.g. with a watchdog test failure and with DMA causing hardfaults when used. I cherry picked this MR from Mbed upstream and tested it locally.

I also took the chance to add an upload method config file for the U575ZI and update the one for the U585.

Impact of changes

None

Migration actions required

None

Documentation

None


Pull request type

[X] Patch update (Bug fix / Target update / Docs update / Test update / Refactor)
[] Feature update (New feature / Functionality change / New API)
[] Major update (Breaking change E.g. Return code change / API behaviour change)

Test results

[] No Tests required for this change (E.g docs only update)
[] Covered by existing mbed-os tests (Greentea or Unittest)
[X] Tests / results supplied as part of this PR

I finally have myself an STM32U5 board to play with! So, I ran the greentea tests against this MR, and, somewhat surprisingly, they all passed, except for hal-sleep-manager. After doing some digging, it turns out that I broke this test a while back by moving the sleep_manager_can_deep_sleep_test_check() call to before where the LP ticker time is set. This caused a race condition where setting the LP ticker time generated an interrupt, and that interrupt might not fire until after the processor goes to sleep, waking it up immediately.

With the fix in this MR, this test should be working again, and all Greentea tests pass (though, USB is not supported on STM32U5 yet...)


Reviewers


@multiplemonomials multiplemonomials changed the title STM32U5 HAL update STM32U5 HAL update, NUCLEO_U575ZI upload method file Sep 23, 2023
@multiplemonomials multiplemonomials merged commit 162acab into master Sep 24, 2023
9 checks passed
@multiplemonomials multiplemonomials deleted the dev/u5-hal-update branch September 24, 2023 16:54
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

Successfully merging this pull request may close these issues.

3 participants