Skip to content
This repository has been archived by the owner on Jun 15, 2020. It is now read-only.

Add RGB565 mode for OV7725 #33

Merged
merged 2 commits into from
Nov 7, 2017
Merged

Add RGB565 mode for OV7725 #33

merged 2 commits into from
Nov 7, 2017

Conversation

igrr
Copy link
Owner

@igrr igrr commented Jun 8, 2017

Implements #30

Adds support for RGB565 bitmaps. These can be downloaded as single files or streamed like JPEGs.


static EventGroupHandle_t wifi_event_group;
const int CONNECTED_BIT = BIT0;
static ip4_addr_t s_ip_addr;
static camera_pixelformat_t s_pixel_format;

#define CAMERA_PIXEL_FORMAT CAMERA_PF_RGB565
#define CAMERA_FRAME_SIZE CAMERA_FS_QQVGA
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should add this configurations to menuconfig, depending on the ov7725 option

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Recently I've been thinking that rebuilding the whole project and the whole IDF each time I want to change camera configuration is not nice. OTOH, it's nice to have stuff configurable from a GUI. Maybe it's time we have a decent web server and configure everything through it? I've been looking at ureq, which is really small and easy to integrate, unfortunately it doesn't really have a way of sending (part of) the data in NETCONN_NOCOPY mode, which is essential when streaming bit chunks of framebuffer.
civetweb (MIT-licensed for of Mongoose) is another nice solution, although might take more time to integrate.
I guess i need to open a separate issue to discuss which embedded web server to use...

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

how about integrate Jeroen's httpd to this demo, It is quite small and easy to use
https://github.com/tuanpmt/esp32-cam-demo/blob/feature/httpd/main/app_main.c#L117

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Opened #35 :) libesphttpd is certainly an option, but as of now httpSend always needs to copy data into an internal buffer, there's no way to utilize NETCONN_NOCOPY. This way it's either much higher memory use, or lower bandwidth.

@igrr igrr merged commit e4f922f into master Nov 7, 2017
@igrr igrr deleted the feature/rgb_bitmap branch November 7, 2017 15:15
@igrr
Copy link
Owner Author

igrr commented Nov 7, 2017

Merging this now; will add configuration via esp_console in a separate commit.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants