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 the 16MB WeAct Studio Pico #7208

Merged
merged 2 commits into from
Nov 16, 2022

Conversation

domdfcoding
Copy link

Follow up to #6411, and based on Fabian's work there.

I only have the 16MB board, so as I can't test them I haven't added the 4 and 8MB versions.

@tannewt
Copy link
Member

tannewt commented Nov 15, 2022

You'll want to add an entry here to share the USB VID/PID with the other version: https://github.com/adafruit/circuitpython/blob/main/tools/ci_check_duplicate_usb_vid_pid.py#L33

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.

Thank you!

@tannewt tannewt merged commit 6689b9a into adafruit:main Nov 16, 2022
@domdfcoding
Copy link
Author

Presumably I also need to make a PR at https://github.com/adafruit/circuitpython-org for this board variant?

@tannewt
Copy link
Member

tannewt commented Nov 17, 2022

Presumably I also need to make a PR at https://github.com/adafruit/circuitpython-org for this board variant?

Yes please!

@prplz
Copy link

prplz commented Nov 18, 2022

Is this needed? The rp2040 port auto detects flash size so it should work fine with the 2mb definition

@tannewt
Copy link
Member

tannewt commented Nov 18, 2022

@prplz Maybe not. I think its ok to have though. Explicit tends to be better.

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.

None yet

3 participants