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 PIDs for the ESP32-S3-Super-Mini #181

Closed
wants to merge 1 commit into from

Conversation

justM4D
Copy link
Contributor

@justM4D justM4D commented Sep 11, 2024

Description

For a while there have been "ESP32-S3 Super Mini" Boards (with a "ESP32-S3FH4R2" chip) floating around on Aliexpress and other chinese marketplaces.
I couldn't find out who the original designer is, since all shops just use the same pictures and diagrams with different watermarks.
Apparently there are revisions with the open source logo on them, but even so I couldn't find any more infos on the board than on the shops and this chinese site.
If it's a clone, I couldn't find the original.

(They are often listed together with the C3-Super-Mini variants, which look like Waveshare clones, but they don't have a variant of the S3 Super Mini I could find anywhere)

All that is to say, I'd like to add this board to the usual places, so it can be used with Arduino and Micro-/CircuitPython without any guesswork for the users.

Since I couldn't make out a company, I put in "Aliexpress" as company name.
Maybe we could discuss a "N/A" or "Generic", "OEM", "Knockoff" or something instead :)

@Spritetm
Copy link
Member

Oh my, looks like they inadvertently added a non-applicable pull request template here. Let's see if I can fix it.

@Spritetm
Copy link
Member

Spritetm commented Sep 11, 2024

Generally, this looks good, although I would indeed use 'generic' rather than 'Aliexpress' as Aliexpress is not the manufacturer. Unfortunately, the bad issue template messed up this PR (I can't get it out of 'draft' state anymore) so I need to close it. Would you mind making it again? It should put something more applicable in the PR description if you do so, but you can also just copy-paste the text you wrote above.

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