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

adding a openocd option for flashing with support for stlink v2 clones #20

Merged
merged 2 commits into from
Jan 26, 2020

Conversation

luben93
Copy link
Contributor

@luben93 luben93 commented Jan 26, 2020

No description provided.

@luben93 luben93 requested a review from JF002 January 26, 2020 15:41
CMakeLists.txt Show resolved Hide resolved
@JF002 JF002 merged commit 6128452 into InfiniTimeOrg:master Jan 26, 2020
@JF002
Copy link
Collaborator

JF002 commented Jan 27, 2020

It seems that the files interfaces/stlink.cfg and interfaces/nrf52.cfg are needed.
Do the users need to copy them in a specific folder? Or are these files custom to the project and need to be placed into the source code of the project?

@luben93
Copy link
Contributor Author

luben93 commented Jan 27, 2020

They are part of the openocd package compiled from AUR openocd-git

@JF002
Copy link
Collaborator

JF002 commented Jan 27, 2020

Do they have to be copied in the working directory of the project?

@Avamander
Copy link
Collaborator

In this case, yes? I suspect it would be nicer if a variable for OpenOCD's root could be specified, the location varies across OSs and distros.

@luben93
Copy link
Contributor Author

luben93 commented Jan 27, 2020

No, openocd has them in a its include path, mine on arch is /usr/share/openocd/scripts and that will differ between distributions

@luben93
Copy link
Contributor Author

luben93 commented Jan 27, 2020

But the cmake script is not defining the default root path so it should find the correct path as it does on my machine

@Avamander
Copy link
Collaborator

I haven't installed my build and I have to define either a full or a correct relative path to the script folder for OpenOCD to find them.

@luben93
Copy link
Contributor Author

luben93 commented Jan 27, 2020

Please feel free to submit any PR to make it easier to build with locally built openocd, or was this enough

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.

None yet

3 participants