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 Board bluemicro840 #5114

Merged
merged 6 commits into from
Aug 11, 2021
Merged

Conversation

jpconstantineau
Copy link

Adding BlueMicro840 as board.

Used pre-existing/pre-approved VID/PID from openmoko.
https://github.com/openmoko/openmoko-usb-oui/blob/master/usb_product_ids.psv

What PID should be used? one specific for the board (bootloader) or a circuitpython-specific one?

Thanks

@tannewt
Copy link
Member

tannewt commented Aug 9, 2021

What PID should be used? one specific for the board (bootloader) or a circuitpython-specific one?

We generally recommend a unique PID for CircuitPython but it's really up to you. I would pick the application pid 0x6161 over the bootloader one though. Our concern is that the PID is unique amongst CP boards.

@jpconstantineau
Copy link
Author

Thanks. I'll update the PID to the app one.

@tannewt tannewt self-requested a review August 10, 2021 18:53
@tannewt tannewt added the board New board or update to a single board label Aug 10, 2021
Copy link
Member

@tannewt tannewt left a comment

Choose a reason for hiding this comment

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

Looks good! Thanks!

@ajs256
Copy link

ajs256 commented Aug 10, 2021

It looks like the CI failures were due to the outages this morning. It wouldn't hurt to poke them again, but this should be safe to merge.

@jepler jepler merged commit 1e53cf4 into adafruit:main Aug 11, 2021
@jpconstantineau jpconstantineau deleted the board-bluemicro840 branch December 10, 2021 05:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
board New board or update to a single board
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants