-
Notifications
You must be signed in to change notification settings - Fork 3k
Fix LPC11U35_401 baremetal build #12134
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
Conversation
Test run: SUCCESSSummary: 11 of 11 test jobs passed |
@toyowata could you please fill in the test section? This is there so that we know how this is tested or whether new tests are required. Thanks. |
@adbridge I updated the test section of this PR. Can you please check? |
], | ||
"overrides": { |
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.
Tickless doesn't work without RTOS...
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.
I needed to add tickless-from-us-ticker
to avoid runtime assertion here:
mbed-os/platform/source/mbed_os_timer.cpp
Line 55 in 6b89d16
MBED_ASSERT("OS timer not available - check MBED_CONF_TARGET_TICKLESS_FROM_US_TICKER" && false); |
Any suggestion to avoid this assertion for the non-RTOS target?
Lets pull it in the next patch. Low risk of breaking something. |
], | ||
"overrides": { | ||
"tickless-from-us-ticker" : true, | ||
"boot-stack-size" : "0x400" |
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.
@toyowata
How did we come up with 0x400
?
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.
How did we come up with 0x400?
It is from linker script file.
mbed-os/targets/TARGET_NXP/TARGET_LPC11UXX/device/TOOLCHAIN_ARM_STD/TARGET_LPC11U35_401/LPC11U35.sct
Line 4 in 9243abb
#define MBED_BOOT_STACK_SIZE 0x400 |
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.
Ah OK. Thanks.
Summary of changes
The PR fix LPC11U35_401 baremetal build error and runtime error.
This should be now fixed: ARMmbed/mbed-os-example-blinky-baremetal#15
Documentation
Pull request type
Test results
Greentea test does not work for this target (Mbed 2 only target).
So, I verified this fix as below:
I also put printf message in the main.cpp for debug and it worked fine.
Reviewers