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

Creating config files for Seeed Xiao ESP32-S3 Sense Board #9118

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

djairjr
Copy link

@djairjr djairjr commented Mar 29, 2024

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.

Copy link
Author

@djairjr djairjr left a 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
Copy link
Author

@djairjr djairjr left a 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

@deshipu
Copy link

deshipu commented Mar 29, 2024

Removed CIRCUITPY_ESPCAMERA = 1 not working with Seeed Xiao ESP32-S3 Sense

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?

@deshipu
Copy link

deshipu commented Mar 29, 2024

By the way, there is an old PR for that board: #8095

@djairjr
Copy link
Author

djairjr commented Mar 29, 2024

I think that the real problem is the PSRAM settings.
Now appears to be working. I will add CIRCUITPY_ESPCAMERA = 1 again and see if it works;

Updated CIRCUITPY_ESPCAMERA = 1 to check if it works now
@deshipu
Copy link

deshipu commented Mar 29, 2024

The pre-commit is failing on extra white space at the ends of the lines in pins.c

@djairjr
Copy link
Author

djairjr commented Mar 29, 2024

How can I change that? I am using notepad ++ to change files here.

Removing all whitespaces...
Will work?
@deshipu
Copy link

deshipu commented Mar 29, 2024

Just delete the extra spaces.
You can see where they are if you select all text.

@djairjr
Copy link
Author

djairjr commented Mar 29, 2024

I think now I've done. Sorry for this. Is my first time doing this.

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.

Please get a unique PID for this board instead of using the generic one. https://github.com/espressif/usb-pids

@djairjr
Copy link
Author

djairjr commented Apr 2, 2024

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?

@deshipu
Copy link

deshipu commented Apr 2, 2024

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.

@tannewt
Copy link
Member

tannewt commented Apr 3, 2024

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.

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