Skip to content

Launch libc initialization after hardware setup #174

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

Merged
merged 1 commit into from
Oct 12, 2016

Conversation

cmaglie
Copy link
Member

@cmaglie cmaglie commented Oct 11, 2016

This pachs allows C++ global constructors to run after hardware initialization. This helps some libraries that setups hardware in class constructor to work properly.

See also #169

/cc @totalgee

This pachs allows C++ global constructors to run after hardware
initialization. This helps some libraries that setups hardware
in class constructor to work properly.

See also arduino#169
@ArduinoBot
Copy link

✅ Build completed.

⬇️ Build URL: http://downloads.arduino.cc/PR/samd/package_samd-b105_index.json

ℹ️ To test this build:

  1. Open the Preferences of the Arduino IDE.
  2. Add the Build URL above in the Additional Boards Manager URLs field, and click OK.
  3. Open the Boards Manager (menu Tools->Board->Board Manager...)
  4. Install Arduino SAMD core - Pull Request Launch libc initialization after hardware setup #174
  5. Select one of the boards under SAMD Pull Request Launch libc initialization after hardware setup #174 in Tools->Board menu
  6. Compile/Upload as usual

@totalgee
Copy link

Wonderful, thanks! I just tried with this code locally on top of 1.6.7 with the original version of Q2HX711 (which calls pinMode() to setup an OUTPUT in a global constructor) and it does indeed work now. (ref #169)

@cmaglie cmaglie merged commit ce89e7e into arduino:master Oct 12, 2016
@cmaglie cmaglie deleted the libc-init-fix branch October 12, 2016 14:28
boseji pushed a commit to go-ut/combined-ArduinoCore-samd that referenced this pull request May 30, 2020
Enable TC6 & TC7 for __SAMD21J18A__
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.

3 participants