Skip to content

Added alternate ports 44,45 to make the SWCLK and SWDIO pins available #331

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 23, 2018

Conversation

helmut64
Copy link
Contributor

@helmut64 helmut64 commented Jul 13, 2018

Added alternate ports 44,45 to make the SWCLK and SWDIO pins available
when no debugger is being used.
BTW. The presence of a debugger can be checked via:
if (DSU->STATUSB.bit.DBGPRES) ...
@cmaglie it would be great if this patch can be reviewed and merged. Thank you for your help.

when no debugger is being used.
BTW. The presence of a debugger can be checked via:
if (DSU->STATUSB.bit.DBGPRES) ...
@helmut64
Copy link
Contributor Author

@cmaglie @facchinm I hope my patch can be reviewed soon.
Also the patch #267 from @Adminius is as well of interest and hanging around for 10 month or so.

I love to fix issues and bring enhancements, however my motivation goes to zero when it takes 9 month to review 5 lines. Lets collaborate and bring this forward.

Warm regards from the sunny Hannover, Germany.
Helmut

@sandeepmistry
Copy link
Contributor

Hi @helmut64,

First, thank you for submitting this pull request!

Is the intent to use the tiny header on the Arduino Zero board or is this for another board? Are you running out of pins that are broken out on the standard Arduino form factor headers?

@helmut64
Copy link
Contributor Author

helmut64 commented Aug 31, 2018

Hi Sandeep, this patch is indeed when you run out of pins on the Arduino Zero boards, certainly you need to attach cables to the pins.

We developed also a own Arduino Zero board with LoRa and Debugger and Display Interface where we need this additional pins. Of course for our solution we check if the SWD pin is in use with a debugger and don't use it when busy. Here is our board: http://www.radioshuttle.de/en/longra-2/board/ It is a great benefit for our users that they can just specify Arduino Zero in the board manager.
Regards Helmut

@sandeepmistry
Copy link
Contributor

We developed also a own Arduino Zero board with LoRa and Debugger and Display Interface where we need this additional pins.

So for this scenario I would recommend you create your own core/variant, which depends on the official SAMD core: https://github.com/arduino/Arduino/wiki/Arduino-IDE-1.5-3rd-party-Hardware-specification#referencing-another-core-variant-or-tool

@Adminius
Copy link
Contributor

bords.txt will be overwritten after each update... and "own" core should be maintained/updated after arduino-core update...
it is a good solution for testing, but terrible for normal operation :/

@helmut64
Copy link
Contributor Author

We designed the D21 Board to be compatible with the zero board, the patch applies to the regular Arduino Zero as well to our D21 board. We have done the Zero compatibility by purpose.

I feel that the patch enabling the pins should be ok.
PS: For different MCUs and features I agree that a custom variant is ok, however we are not in the volume business that it makes much sense.

@helmut64
Copy link
Contributor Author

helmut64 commented Sep 2, 2018

On the original Arduino Zero, the pins I added are available to users on the header J100, this is a good reason to add them for users.

@cmaglie cmaglie merged commit 2ef5b78 into arduino:master Oct 23, 2018
@cmaglie cmaglie added this to the Release 1.6.20 milestone Oct 23, 2018
@helmut64
Copy link
Contributor Author

Danke Cristian.

boseji pushed a commit to go-ut/combined-ArduinoCore-samd that referenced this pull request Sep 11, 2022
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.

4 participants