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
Creating config files for Seeed Xiao ESP32-S3 Sense Board #9118
base: main
Are you sure you want to change the base?
Conversation
Removing config CIRCUITPY_ESPCAMERA = 1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed CIRCUITPY_ESPCAMERA = 1 not working with Seeed Xiao ESP32-S3 Sense
Added PSRAM config USB_VID = 0x303A USB_PID = 0x1001 USB_PRODUCT = "Seeed Xiao ESP32-S3 Sense" USB_MANUFACTURER = "Seeed Studio" IDF_TARGET = esp32s3 CIRCUITPY_ESP_FLASH_MODE = qio CIRCUITPY_ESP_FLASH_FREQ = 80m CIRCUITPY_ESP_FLASH_SIZE = 8MB CIRCUITPY_ESP_PSRAM_MODE = qio CIRCUITPY_ESP_PSRAM_SIZE = 8MB CIRCUITPY_ESP_PSRAM_FREQ = 80m
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Configuring PSRAM in order to give Camera suport
USB_VID = 0x303A
USB_PID = 0x1001
USB_PRODUCT = "Seeed Xiao ESP32-S3 Sense"
USB_MANUFACTURER = "Seeed Studio"
IDF_TARGET = esp32s3
CIRCUITPY_ESP_FLASH_MODE = qio
CIRCUITPY_ESP_FLASH_FREQ = 80m
CIRCUITPY_ESP_FLASH_SIZE = 8MB
CIRCUITPY_ESP_PSRAM_MODE = qio
CIRCUITPY_ESP_PSRAM_SIZE = 8MB
CIRCUITPY_ESP_PSRAM_FREQ = 80m
What is not working exactly? I had it working in 8.x. Even it there is a bug that makes it not work right now, I think it should be still enabled and the bug fixed? |
By the way, there is an old PR for that board: #8095 |
I think that the real problem is the PSRAM settings. |
Updated CIRCUITPY_ESPCAMERA = 1 to check if it works now
The pre-commit is failing on extra white space at the ends of the lines in pins.c |
How can I change that? I am using notepad ++ to change files here. |
Removing all whitespaces... Will work?
Just delete the extra spaces. |
Removing whitespaces
I think now I've done. Sorry for this. Is my first time doing this. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please get a unique PID for this board instead of using the generic one. https://github.com/espressif/usb-pids
I'm sorry, but this is what appears when I follow the steps outlined here: https://superuser.com/questions/1106247/how-can-i-get-the-vendor-id-and-product-id-for -a-usb-device I think this is something that Seeed Studio should address? |
Yes, we asked them for it on their forum, at https://forum.seeedstudio.com/t/usb-vid-pid-for-the-xiao-esp32-s3-and-xiao-esp32-s3-sense-for-circuitpython/272245/10 but either they don't understand, or they are not interested. |
I give out USB PIDs for open source software on third-party boards from https://pid.codes too. Make sure to link to your branch of CircuitPython with the changes. |
Trying to comply with Seeed Spreedsheet in wiki.
https://files.seeedstudio.com/wiki/SeeedStudio-XIAO-ESP32S3/res/XIAO_ESP32S3_Sense_Pinout.xlsx
Added camera pinout and remove unused pins.