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 MachXO2 Breakout board #214

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

neuschaefer
Copy link
Contributor

@neuschaefer neuschaefer commented Feb 18, 2023

Features (✓=tested):

  • LEDs
  • toolchain_program()
  • GPIO connectors
  • UART via FTDI

@cr1901
Copy link
Contributor

cr1901 commented Jul 22, 2023

I was planning on doing this, you already done so :D. Thanks for saving me the effort!

It looks good to me, I would've done it much the same way. I have a few questions:

  • Since the UART shares pins with connections J3, would it be better to not provide the UART resource by default (i.e. as a MachXO2_7000HE_BreakoutPlatform class variable? That way, the user can choose to either use the pins as GPIO, or convert it to UART via add_resources if they've done the appropriate mod.
  • I believe the user can solder a crystal X2 if they desire (of a frequency of their own choice). Idk offhand if add_resources handles Clocks though/if it requires special treatment. Should that be added as a MachXO2_7000HE_BreakoutPlatform class variable too?

@neuschaefer
Copy link
Contributor Author

Since the UART shares pins with connections J3, would it be better to not provide the UART resource by default (i.e. as a MachXO2_7000HE_BreakoutPlatform class variable? That way, the user can choose to either use the pins as GPIO, or convert it to UART via add_resources if they've done the appropriate mod.

That sounds good, I'll do it.

I believe the user can solder a crystal X2 if they desire (of a frequency of their own choice). Idk offhand if add_resources handles Clocks though/if it requires special treatment. Should that be added as a MachXO2_7000HE_BreakoutPlatform class variable too?

I don't know how this would work, so I'll leave it up to you to make an additional pull request.

@neuschaefer
Copy link
Contributor Author

@cr1901 I've split the UART resource as you've suggested, but I can' currently test this. I'd appreciate you testing it.

@cr1901
Copy link
Contributor

cr1901 commented Aug 26, 2023

@neuschaefer I sincerely apologize for the delay... I finally tested locally. Your UARTResource split works perfectly :).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants