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

boards: Extend XIAO ESPS3 board to support for XIAO ESP32S3 Sense #60148

Open
kartben opened this issue Jul 7, 2023 · 3 comments
Open

boards: Extend XIAO ESPS3 board to support for XIAO ESP32S3 Sense #60148

kartben opened this issue Jul 7, 2023 · 3 comments
Assignees
Labels
Enhancement Changes/Updates/Additions to existing features Good first issue Good for a first time contributor to take platform: ESP32 Espressif ESP32

Comments

@kartben
Copy link
Collaborator

kartben commented Jul 7, 2023

Is this request related to a missing driver support for a particular hardware platform, SoC or board? Please describe.

Seeed Studio XIAO ESP32S3 was recently added (PR #59732). As this board also has a "Sense" variant adding an OV2640 camera sensor, a microphone, and an SD card reader, it would be nice to have it supported as well.

Describe why you are asking for this support?

Having an affordable board with built-in camera supported upstream can only be a good thing, right? Plus, this is a good opportunity for someone to make their first contribution to the project as a lot of the heavy lifting was already done for the XIAO ESP32S3.

Additional context

/cc @Maxwelltoo

@kartben kartben added Good first issue Good for a first time contributor to take platform: ESP32 Espressif ESP32 Hardware Support labels Jul 7, 2023
@Maxwelltoo
Copy link
Contributor

This is also what I want to do. After trying, the microphone and SD card works, but I believe the camera driver in Zephyr still needs an update from Espressif (the driver in esp-idf is good, but it's based on freeRTOS). So just wait...

@kartben
Copy link
Collaborator Author

kartben commented Jul 11, 2023

This is also what I want to do. After trying, the microphone and SD card works, [...]

Nice!

but I believe the camera driver in Zephyr still needs an update from Espressif (the driver in esp-idf is good, but it's based on freeRTOS). So just wait...

Could you be more specific? Is Zephyr's OV2640 driver not working correctly on ESP32?

@rftafas
Copy link

rftafas commented Jul 14, 2023

This sensor has 2 interfaces: control and image. IIRC, the driver on Zephyr is the control interface. Camera interface is not supported for any ESP32 device yet. It is correct, this needs to be implemented.

Even though people can reuse/port the code from https://github.com/espressif/esp32-camera for the lower parts of the driver, I don't consider this good for 'first issue'.

@henrikbrixandersen henrikbrixandersen changed the title boards: Add support for XIAO ESP32S3 Sense boards: Extend XIAO ESPS3 board to support for XIAO ESP32S3 Sense Aug 31, 2023
@henrikbrixandersen henrikbrixandersen added Enhancement Changes/Updates/Additions to existing features and removed Hardware Support labels Aug 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement Changes/Updates/Additions to existing features Good first issue Good for a first time contributor to take platform: ESP32 Espressif ESP32
Projects
None yet
Development

No branches or pull requests

7 participants