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

[target] Add target for NextCoinP0 #130

Closed
wants to merge 2 commits into from
Closed

Conversation

hollanderic
Copy link
Collaborator

Support for NextCoinP0 - mainly a copied subset of dartuinoP0

@travisg
Copy link
Member

travisg commented May 4, 2016

Looks generally okay, I'd name it target/nextcoin personally. P0 vs P1 vs P2 and whatnot is probably a project level #define and/or hardware revision bits.

@travisg
Copy link
Member

travisg commented May 13, 2016

If this is getting rolled up into the demo branch, which I assume will get merged in once the demo stuff is over, then go ahead and close it here.

@nvll
Copy link
Member

nvll commented May 13, 2016

Demo branch is full of too many wips, we'll have to clean up and cherry
pick things over to master.
On May 13, 2016 8:08 AM, "Travis Geiselbrecht" notifications@github.com
wrote:

If this is getting rolled up into the demo branch, which I assume will get
merged in once the demo stuff is over, then go ahead and close it here.


You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
#130 (comment)

@travisg
Copy link
Member

travisg commented May 25, 2016

Is this going here or did you guys push it to the other repo?

@travisg
Copy link
Member

travisg commented Aug 26, 2016

What do you guys want to do with this one?

@nvll
Copy link
Member

nvll commented Aug 26, 2016

I'd vote abandon for the time being so we don't add a mostly untestable
target.

On Aug 25, 2016 5:38 PM, "Travis Geiselbrecht" notifications@github.com
wrote:

What do you guys want to do with this one?


You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#130 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAD0hUDIW2pjRQVvQmOuc1bcftgR_Ybsks5qjjVlgaJpZM4IXc50
.

@travisg
Copy link
Member

travisg commented Sep 6, 2016

going to abandon this for now.

@travisg travisg closed this Sep 6, 2016
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.

3 participants