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 support for colorlight-i9+ board #350

Closed
wants to merge 3 commits into from

Conversation

infphyny
Copy link
Contributor

Pull request for adding colorlight i9+ via an ftdi 2232. I didn't change that much in Makefile and board.hpp. Maybe I did a mistake somewhere. Let me know and I will change the pull request.

@trabucayre
Copy link
Owner

Seems your branch is out of date, with some conflict.
colorlight i9+ has no onboard jtag interface no?

@infphyny
Copy link
Contributor Author

Ok forgot to update the cloned repository on my side. colorlight-i9+ carrier board have no onboard jtag interface. Can only upload via ch347t that need an out of tree openocd. The module have accessible jtag pins.

@trabucayre
Copy link
Owner

Hi,
Could you rebase your fork. It's to reduce history to only spiOverJtag update + colorlight addition.
Usually when a board as no onboard jtag interface cable name is left empty: user has to provides explicitly this information.

If you prefer I can fix and apply the patch manually. As you want

Thanks!

@infphyny
Copy link
Contributor Author

You can apply the patch manually. I agree with you, better leave cable name empty.
Thanks.

@trabucayre
Copy link
Owner

Applied! Thanks @infphyny

@trabucayre trabucayre closed this Jun 30, 2023
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