luc-github / ESP3D-TFT

ESP3D Firmware for ESP32 based TFT
GNU General Public License v3.0
38 stars 9 forks source link

Add camera function to BZM motherboard #57

Closed liqijian101 closed 8 months ago

liqijian101 commented 10 months ago

Camera using OV2640

luc-github commented 10 months ago

is that working ? I see pinMode, digitalWrite and other API that is Arduino specific and not in IDF

liqijian101 commented 10 months ago

I have solved it.

luc-github commented 10 months ago

you need to add handler counter value for snap handler like : https://github.com/luc-github/ESP3D-TFT/blob/main/main/modules/http/esp3d_http_service.cpp#L39-L43 and add it here : https://github.com/luc-github/ESP3D-TFT/blob/main/main/modules/http/esp3d_http_service.cpp#L237

Do you plan to add camera related commands also ? [ESP170]https://esp3d.io/esp3d/v3.x/documentation/commands/esp170/index.html [ESP171]https://esp3d.io/esp3d/v3.x/documentation/commands/esp171/index.html

liqijian101 commented 10 months ago

I have add handler counter value,but I dont Know Why to add it. I plan to add related commands,but I am not goot at the code.

luc-github commented 10 months ago

I have add handler counter value,but I dont Know Why to add it.

This is related to IDF API you need to set how many handler you have in top of adding handler or the latest handler won't be added and won't be used

I plan to add related commands,but I am not goot at the code.

It is ok you did well in PR and you can use same code as ESP3D as code base, as you did, they should be almost compatible

liqijian101 commented 9 months ago

Hi luc, I tried adding 170 commands,But it always failde to compile. So I will not add any related commands. I hope to PR now.

luc-github commented 9 months ago

ok I will have a look - thank you

luc-github commented 8 months ago

why did you closed the PR ?

liqijian101 commented 8 months ago

I mistakenly received the lvgl9 branch.I am changing the code and restoring the previous program. Afterwards, I will resubmit Pr

luc-github commented 8 months ago

Ok - the lvgl 90 is definitly not for short terms so ignore it pls