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

Add camera function to BZM motherboard #57

Closed
wants to merge 12 commits into from
Closed

Add camera function to BZM motherboard #57

wants to merge 12 commits into from

Conversation

liqijian101
Copy link
Contributor

Camera using OV2640

1.Screen and SD card changed to SPI driver
2.Reserve OV2640 camera interface
@luc-github
Copy link
Owner

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

@liqijian101
Copy link
Contributor Author

I have solved it.

@liqijian101
Copy link
Contributor Author

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
Copy link
Owner

luc-github commented Jan 24, 2024

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
Copy link
Contributor Author

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
Copy link
Owner

ok I will have a look - thank you

@liqijian101 liqijian101 closed this by deleting the head repository Mar 17, 2024
@luc-github
Copy link
Owner

why did you closed the PR ?

@liqijian101
Copy link
Contributor Author

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

@luc-github
Copy link
Owner

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

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

Successfully merging this pull request may close these issues.

2 participants