add OV2640 support to sdkconfig for esp32s3_eye build

[jerryneedell]

fixes #8883

[dhalbert]

Thanks!

[tannewt]

Do we want to update this instead? https://github.com/adafruit/circuitpython/blob/main/ports/espressif/esp-idf-config/sdkconfig.defaults#L113-L125 That's the global setting for camera support.

[jerryneedell]

That probably makes sense. It is confusing. Where does OV5640 get enabled? It seems like it is globally enabled but I don't see it in there.

[jerryneedell]

for example, I just built the esp32s3_metro and in the generated sdkconfig it has

# CONFIG_OV7670_SUPPORT is not set
CONFIG_OV7725_SUPPORT=y
# CONFIG_NT99141_SUPPORT is not set
# CONFIG_OV2640_SUPPORT is not set
CONFIG_OV3660_SUPPORT=y
CONFIG_OV5640_SUPPORT=y

but the sdkconfig in boards/adafruit_metro_esp32s3 does not have any camera specified. It would probably be good to get bot the OV5640 and OV2640 here as well

[tannewt]

That probably makes sense. It is confusing. Where does OV5640 get enabled? It seems like it is globally enabled but I don't see it in there.

It is on by default in the kconfig: https://github.com/espressif/esp32-camera/blob/master/Kconfig#L40

With 9.x we only include non-default settings in our kconfigs.

[jerryneedell]

That probably makes sense. It is confusing. Where does OV5640 get enabled? It seems like it is globally enabled but I don't see it in there.

It is on by default in the kconfig: https://github.com/espressif/esp32-camera/blob/master/Kconfig#L40

With 9.x we only include non-default settings in our kconfigs.

but in line 26 https://github.com/espressif/esp32-camera/blob/master/Kconfig#L26 OV2640 is also enabled, but we don't get it without specifying it...

[dhalbert]

Yes, I am confused by why the Kconfig has them all on by default but we are not seeing that in the generated sdkconfig in the build. Maybe we are not using the esp32-camera Kconfig properly.

[tannewt]

The is not set are not actually just comments. They actually set the value to off. See zephyrproject-rtos/zephyr#5443 for background.