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

MKR boards: I2C to sercom2, SPI1 + Serial2 to sercom4 #378

Merged
merged 1 commit into from
Dec 13, 2018

Conversation

sandeepmistry
Copy link
Contributor

This pull request changes the sercom used for I2C and SPI1/Serial2, so that sercom can be used for alternative purposes.

-> I2C: from sercom 0 to 2
-> SPI: from sercom 2 to 4

We'll have to be super careful and test the changes on all MKR board types.

@Rocketct if you have some time this week please test this out.

@ArduinoBot
Copy link

✅ Build completed.

⬇️ Build URL: http://downloads.arduino.cc/PR/samd/package_samd-b200_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 MKR boards: I2C to sercom2, SPI1 + Serial2 to sercom4 #378
  5. Select one of the boards under SAMD Pull Request MKR boards: I2C to sercom2, SPI1 + Serial2 to sercom4 #378 in Tools->Board menu
  6. Compile/Upload as usual

@Rocketct
Copy link
Contributor

Rocketct commented Dec 12, 2018

@sandeepmistry tested both SPI and I2C by using the examples of each boards and a I2C sensors and all goes fine, ok for me

@agdl agdl merged commit a69fbe3 into arduino:master Dec 13, 2018
@sslupsky
Copy link

sslupsky commented May 2, 2019

What is the purpose of this change? The OP says "so that sercom can be used for alternative purposes". What "alternative purposes" are you referring to?

@sandeepmistry
Copy link
Contributor Author

What "alternative purposes" are you referring to?

Adding an additional SPI interface on pins A3 + A4 to use in conjunction with the new MKR RGB shield.

@henklaak
Copy link
Contributor

henklaak commented Dec 22, 2023

Late to the game, but the labels in the MKRWiFi1010V2.0 schematics (ABX00023-schematics.pdf) still reflect the old sercom numbers. Confused me no end (was there a board revision I had missed?), until I traced back this commit and the reasoning behind it (which helped me a lot in fact, as I needed two extra sercoms on the digital pins badly, so thanks!).

Don't know where else I should report docs issues....

Screenshot from 2023-12-22 02-23-41

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.

6 participants