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

Pin configuration support for the UPduino 3.1 (iCE40) #12

Conversation

bsiever
Copy link
Contributor

@bsiever bsiever commented Jun 23, 2024

This PR includes PCF support for the UPduino 3.1: https://tinyvision.ai/products/upduino-v3-1 .

Thanks for a great extension!

Copy link
Owner

@lushaylabs lushaylabs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @bsiever looks great, I am just doing a sanity check on all the OSs and should have this released later today. Thank you

@lushaylabs
Copy link
Owner

Thank you again for the amazing contribution, and sorry for the delays, it is my fault I got caught up with some other things, merging this now.

One small question regarding the UPduino I saw that you need to bridge a jumper to use the 12Mhz osc, if you don't use that oscillator does it use an internal clock or are you meant to connect an external clock through one of the IO pins ?

@lushaylabs lushaylabs merged commit 2a98261 into lushaylabs:main Jul 6, 2024
@bsiever
Copy link
Contributor Author

bsiever commented Jul 7, 2024

Thanks and no rush.

I'm not an UPduino expert...but yes, it would need to use an internal oscillator if the external 12MHz isn't connected (bridged or jumpered to another pin). The 12MHz is used for the FTDI. See https://upduino.readthedocs.io/en/latest/tutorials/oscillator.html .

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