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

M487: Fix crash on WDT reset from power-down #12557

Merged

Conversation

ccli8
Copy link
Contributor

@ccli8 ccli8 commented Mar 3, 2020

Summary of changes

This PR tries to fix issue with WDT reset from power-down mode.

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)
[x] Covered by existing mbed-os tests (Greentea or Unittest)
[] Tests / results supplied as part of this PR

This is to guarantee SRAM bank2, not initialized yet, isn't used for stack by function preamble code at the very start.
When UART interrupt enabled and WDT reset from power-down mode, in the next
cycle, UART interrupt keeps breaking in and cannot block unless via NVIC. To
get around it, we deliberately make up a signal of WDT wake-up from power-down
mode in the start of boot proces when WDT reset is detected.
@ciarmcom ciarmcom requested review from Ronny-Liu and a team March 3, 2020 08:00
@ciarmcom
Copy link
Member

ciarmcom commented Mar 3, 2020

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

@0xc0170
Copy link
Contributor

0xc0170 commented Mar 3, 2020

CI started

@mergify mergify bot added needs: CI and removed needs: review labels Mar 3, 2020
@mbed-ci
Copy link

mbed-ci commented Mar 3, 2020

Test run: SUCCESS

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

@0xc0170 0xc0170 merged commit 4f07086 into ARMmbed:master Mar 3, 2020
@cyliangtw cyliangtw deleted the nuvoton_m487_wdt_reset_powerdown branch March 9, 2023 05:31
@cyliangtw cyliangtw restored the nuvoton_m487_wdt_reset_powerdown branch March 9, 2023 05:31
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.

None yet

4 participants