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

Fix bug Operator new[] out of memory #10616

Merged
merged 1 commit into from May 23, 2019

Conversation

Projects
None yet
4 participants
@ThunderSoft123
Copy link
Contributor

commented May 20, 2019

Description

Modifying API gpio_init for fix bug when the NC's input value is NC, and changing stack size for fix bug which error is Operator new[] out of memory(error=0x8001011F).

Pull request type

[x] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[ ] Docs update
[ ] Test update
[ ] Breaking change

Reviewers

Release Notes

@0xc0170 0xc0170 added the needs: work label May 20, 2019

@ThunderSoft123 ThunderSoft123 force-pushed the ThunderSoft123:master branch 2 times, most recently from 87edb34 to fff9672 May 20, 2019

@0xc0170 0xc0170 changed the title Modifiying file gpio_api.c for board TT_M3HQ and TT_M4G9 when pin's value equal NC. TT_M3HQ and TT_M4G9 gpio NC fix May 20, 2019

@ThunderSoft123 ThunderSoft123 force-pushed the ThunderSoft123:master branch from fff9672 to c3c0555 May 21, 2019

@kjbracey-arm

This comment has been minimized.

Copy link
Contributor

commented May 21, 2019

NC fix is already present in #10489. There's another unconnected GPIO fix here though.

@0xc0170

This comment has been minimized.

Copy link
Member

commented May 21, 2019

NC fix is already present in #10489. There's another unconnected GPIO fix here though.

@ThunderSoft123 Can you align this PR? 10489 will land soon

@ThunderSoft123

This comment has been minimized.

Copy link
Contributor Author

commented May 21, 2019

@0xc0170 Thank you for support. Do i need to keep the PR to fix bug with Operator new[] out of memory?

@0xc0170

This comment has been minimized.

Copy link
Member

commented May 21, 2019

@0xc0170 Thank you for support. Do i need to keep the PR to fix bug with Operator new[] out of memory?

Yes, lets update it to only contain this bugfix.

panyz0725@thundersoft.com

@ThunderSoft123 ThunderSoft123 force-pushed the ThunderSoft123:master branch from c3c0555 to 6aeb048 May 22, 2019

@ThunderSoft123 ThunderSoft123 changed the title TT_M3HQ and TT_M4G9 gpio NC fix Fix bug Operator new[] out of memory May 22, 2019

@0xc0170

This comment has been minimized.

Copy link
Member

commented May 23, 2019

CI started

@mbed-ci

This comment has been minimized.

Copy link

commented May 23, 2019

Test run: SUCCESS

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

@0xc0170 0xc0170 added ready for merge and removed needs: CI labels May 23, 2019

@0xc0170 0xc0170 merged commit cfb1079 into ARMmbed:master May 23, 2019

26 checks passed

continuous-integration/jenkins/pr-head This commit looks good
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
jenkins-ci/build-ARM Success
Details
jenkins-ci/build-GCC_ARM Success
Details
jenkins-ci/build-IAR Success
Details
jenkins-ci/cloud-client-test Success
Details
jenkins-ci/dynamic-memory-usage Success
Details
jenkins-ci/exporter Success
Details
jenkins-ci/greentea-test Success
Details
jenkins-ci/mbed2-build-ARM Success
Details
jenkins-ci/mbed2-build-GCC_ARM Success
Details
jenkins-ci/mbed2-build-IAR Success
Details
jenkins-ci/unittests Success
Details
travis-ci/astyle Success!
Details
travis-ci/docs Success!
Details
travis-ci/doxy-spellcheck Success!
Details
travis-ci/events Success! Runtime is 8694 cycles.
Details
travis-ci/gitattributestest Success!
Details
travis-ci/include_check Success!
Details
travis-ci/licence_check Success!
Details
travis-ci/littlefs Success! Code size is 8448B.
Details
travis-ci/psa-autogen Success!
Details
travis-ci/tools-py2.7 Success!
Details
travis-ci/tools-py3.5 Success!
Details
travis-ci/tools-py3.6 Success!
Details
travis-ci/tools-py3.7 Success!
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.