zephyrproject-rtos / zephyr

Primary Git Repository for the Zephyr Project. Zephyr is a new generation, scalable, optimized, secure RTOS for multiple hardware architectures.
https://docs.zephyrproject.org
Apache License 2.0
10.51k stars 6.44k forks source link

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

Open kartben opened 1 year ago

kartben commented 1 year ago

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

Maxwelltoo commented 1 year ago

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 commented 1 year ago

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 commented 1 year ago

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'.

pblxptr commented 3 days ago

Any update on this issue? From what I've seen the support for LCD_CAM driver has already been merged.