-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Add new features to Toshiba's TMPM4G9 #10633
Conversation
@ganesh-ramachandran, thank you for your changes. |
@ganesh-ramachandran How we can fix ARMC5 ? It is not supported anymore, should not be used for new targets. Otherwise this PR looks fine, it can go to CI. |
@0xc0170 Updated ARMC5 to ARM and rebased. |
|
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.
just one line and ready for CI
@0xc0170 No, we are trying with `mbed export -i uvision' |
@0xc0170 |
@0xc0170 possible to start CI? |
Hi @0xc0170 possible to start CI? |
Starting now! |
Test run: SUCCESSSummary: 11 of 11 test jobs passed |
|
||
void rtc_free(void) | ||
{ | ||
if (flag) { // Check status of RTC peripheral driver is ENABLE or DISABLE |
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 believe I left the msg here yesterday, this flag - will it work after target resets? shouldn't it access registers?
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 believe I left the msg here yesterday, this flag - will it work after target resets? shouldn't it access registers?
@0xc0170, Similar comments were given in other Toshiba targets.
RTC status register is not in Toshiba MCUs. However, verified with RTC initialized status.
CI started |
Test run: SUCCESSSummary: 11 of 11 test jobs passed |
Description
Add new features to Toshiba's TMPM4G9
Tested with all 3 toolchains and attached test logs:
M4G9_ARM_TestResults.txt
M4G9_GCC_ARM_TestResults.txt
M4G9_IAR_TestResults.txt
Pull request type
Reviewers
Release Notes