8000 Add Waveshare ESP32-S2-Pico by bill88t · Pull Request #6672 · adafruit/circuitpython · GitHub
[go: up one dir, main page]

Skip to content

Add Waveshare ESP32-S2-Pico #6672

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

Merged
merged 4 commits into from
Aug 4, 2022
Merged

Add Waveshare ESP32-S2-Pico #6672

merged 4 commits into from
Aug 4, 2022

Conversation

bill88t
Copy link
@bill88t bill88t commented Jul 30, 2022

Disclaimer: I am not affiliated with waveshare in any way.

All board functions are working as intended.

My only concern is the USB_VID & USB_PID.
Waveshare hasn't provided official values and I stole the ones you see from Waveshare RP2040 Zero.
I have opened a support ticket to waveshare and this issue is also tracked in the tinyuf2 port pr.

image
(Ljinux is a wanna-be operating system that runs on top of circuitpython and is a very good benchmark)

image

@bill88t
Copy link
Author
bill88t commented Jul 31, 2022
< 8000 /table>

@tannewt
Copy link
Member
tannewt commented Aug 1, 2022

Board picture
PXL_20220727_131556599

Thanks for the PR! You can probably get an Espressif PID here: https://github.com/espressif/usb-pids/blob/main/allocated-pids.txt

@tannewt tannewt added espressif applies to multiple Espressif chips board New board or update to a single board labels Aug 1, 2022
@bill88t
Copy link
Author
bill88t commented Aug 2, 2022

From the support ticket, I got the official values.
Commiting them now

@bill88t
Copy link
Author
bill88t commented Aug 3, 2022

I have no idea what is wrong with the github actions. Should changes be needed, let me know.

@bill88t bill88t requested a review from tannewt August 4, 2022 08:14
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 3707b54 into adafruit:main Aug 4, 2022
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 espressif applies to multiple Espressif chips
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
0