Skip to content

adding PixelWing ESP32-S3 support - #11786

Draft
skerr92 wants to merge 3 commits into
adafruit:mainfrom
skerr92:pixelwing-esp32s3
Draft

skerr92 wants to merge 3 commits into
adafruit:mainfrom
skerr92:pixelwing-esp32s3

Conversation

@skerr92

@skerr92 skerr92 commented Nov 7, 2026

Copy link
Copy Markdown

Tested this on my Kavrous Technologies PixelWing ESP32-S3 evaluation board. Awaiting approval for the USB PID on PID Codes for this. pidcodes/pidcodes.github.com#1361

@dhalbert dhalbert left a comment •

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This PR changed the C formatting on a huge number of files. I don't think that's what you wanted to do.

Also make translate needs to be done.

@skerr92

skerr92 commented Nov 7, 2026

Copy link
Copy Markdown
Author

This PR changed the C formatting on a huge number of files. I don't think that's what you wanted to do.

Also make translate needs to be done.

Not sure how to not get it to do that. I ran pre-commit locally which might have been responsible for the massive changes. I'll revert the changed files and hopefully only include the bare minimum for this.

@dhalbert

dhalbert commented Nov 8, 2026

Copy link
Copy Markdown
Collaborator

Also make translate needs to be done.

Not sure how to not get it to do that.

It looks like that wasn't needed, but the way to do it is to do make translate locally (after doing make check-translate, which is what pre-commit does), and then commit the change.

@dhalbert
dhalbert marked this pull request as draft November 8, 2026 14:09
@dhalbert

dhalbert commented Nov 8, 2026

Copy link
Copy Markdown
Collaborator

I made this draft until the PID is approved.

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