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.

Sign up for GitHub

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

Jump to bottom

fix(BLE): MAX32690 DATC Hardfaulting with ICC enabled #988

Merged
merged 13 commits into from
Apr 15, 2024
Merged

fix(BLE): MAX32690 DATC Hardfaulting with ICC enabled #988

merged 13 commits into from
Apr 15, 2024

Conversation

EricB-ADI
Copy link
Contributor

@EricB-ADI EricB-ADI commented Apr 12, 2024 •
edited
Loading

Description

  • Disabled ICC altogether on MAX32690 EvKits
    Hardfaults were occuring at AppTerminalInit. Seemingly due to timing.
  • With optimizations turned off in int main(), hardfaults went away.
  • Trace level 0 and trace level 2 had no hardfaults, only trace 1.
  • Adding an instruction sync barrier caused the following to happen, which lead to believing cache is the issue.
terminal: init
terminal: init
terminal: init
terminal: init
terminal: init
terminal: init
terminal: init
terminal: init
terminal: init
terminal: init
terminal: init
terminal: init
terminal: init
terminal: init
terminal: init
terminal: init
HardFault_Handler

Disabling cache around the init code, resolved the issue. This is a band aid fix and so It will just be disabled altogether.

@github-actions github-actions bot added MAX32690 Related to the MAX32690 (ME18) BLE Related to Bluetooth labels Apr 12, 2024
@EricB-ADI
Copy link
Contributor Author

/clang-format-run

@EricB-ADI
Copy link
Contributor Author

/clang-format-run

@github-actions github-actions bot added Workflow Related to Workflow development and removed BLE Related to Bluetooth labels Apr 12, 2024
@github-actions github-actions bot added the BLE Related to Bluetooth label Apr 12, 2024
@EricB-ADI
Copy link
Contributor Author

/clang-format-run

@EricB-ADI EricB-ADI merged commit 0665fe0 into main Apr 15, 2024
@EricB-ADI EricB-ADI deleted the fix/BLE_datc_icc branch April 16, 2024 19:05
EricB-ADI pushed a commit that referenced this pull request Aug 21, 2024
Revert "fix(BLE): MAX32690 DATC Hardfaulting with ICC enabled (#988)"
970ddd7 
This reverts commit 0665fe0.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yc-adi yc-adi yc-adi approved these changes

@AbbyWolf-ADI AbbyWolf-ADI Awaiting requested review from AbbyWolf-ADI

Assignees
No one assigned
Labels
BLE Related to Bluetooth MAX32690 Related to the MAX32690 (ME18) Workflow Related to Workflow development
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@EricB-ADI @yc-adi