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

Add ArduinoCore-API integration #560

Merged
merged 20 commits into from
Nov 25, 2020
Merged

Conversation

giulcioffi
Copy link
Contributor

@giulcioffi giulcioffi commented Sep 10, 2020

The proposed modifications to ArduinoCore-samd would allow the integration with ArduinoCore-API core.
Related CI workflows are proposed in #567 .

@giulcioffi giulcioffi changed the title Add ArduinoCore-API integration + workflow for CI Add ArduinoCore-API integration Oct 15, 2020
@aentinger
Copy link
Contributor

aentinger commented Nov 23, 2020

Tested successfully with Arduino IoT Cloud for

  • MKR 1000 (Requires WiFi101:v0.16.1) ✔️
  • MKR WiFi 1010 ✔️
  • Nano 33 IoT ✔️
  • MKR WAN 1310 ✔️
  • MKR GSM 1400 ⚠️

Tested successfully in another way:

  • Arduino Zero ✔️
  • MKR NB 1500 ✔️
  • MKR VIDOR 4000 ✔️

Not tested for:

  • MKR Zero
  • MKR FOX 1200
  • MKR WAN 1300
  • Arduino Circuit Express
  • Arduino M0 Pro ❌ (Retired)
  • Arduino M0
  • Arduino Tian

facchinm and others added 20 commits November 25, 2020 15:29
The new class PluggableUSBModule in ArduinoCore-API does not implement the fuction
void handleEndpoint(int ep)
which was implemented in the previous version of the same class. This function now needs to be called as
proper of SerialUSB and not of PluggableUSB().
…is different from the ones provided via global constant 'DEFAULT_SPI_SETTINGS' because otherwise SPI won't be configured within the 'config' method called within 'begin'
@aentinger aentinger merged commit 25fea52 into arduino:master Nov 25, 2020
@giulcioffi giulcioffi deleted the namespaced_api branch November 26, 2020 08:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants