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

Adds support to choose between JLINK and BMP as programmers #7

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

EarthLord
Copy link

The targets of flash, flash_softdevice and erase work based
on what the variable DEBUGGER is set as: JLINK or BMP.
Needs arm-none-eabi-gdb in the path for BMP to work.

The targets of flash, flash_softdevice and erase work based
on what the variable DEBUGGER is set as: JLINK or BMP.
@electronut
Copy link
Collaborator

electronut commented Jul 28, 2017

You seem to have hardcoded the USB port:

/dev/ttyBmpGdb

That will not work on Windows and Linux. Instead, it should be a make variable.

Eg.:

$make BMP_PORT=COM5 flash

Also I think we need a check on DEBUGGER - if it's already from cmd line, it should take precedence. eg.:

$make DEBUGGER=BMP BMP_PORT=COM5 flash

@EarthLord
Copy link
Author

What is this 'Windows' that you speak of? :P

OK, I'll see what I can do for making it compatible with both the OS. Also there is an issue with the path of nrfjprog. I'll fix that too.

@electronut
Copy link
Collaborator

Windows - holds inmates. :-)

This commit removes the hardcoding of the BMP port to /dev/ttyBmpGdb
so that people using other OS apart from Linux can set it appropriately.
@EarthLord
Copy link
Author

Ok, I've made the changes. Untested in Windows and Mac though.

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.

2 participants