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

Request corresponding frame grabber for desktop side to be added into GitHub repository #2

Open
jnshadow opened this issue Dec 31, 2014 · 2 comments

Comments

@jnshadow
Copy link

Hi Mr Arduino,

Excellent efficient code for the Arduino-side -- nice use of register shifts, ports, etc. in order to reduce processing time!

To duplicate what you have done, I have an Arduino, a non-FIFO OV7670, and an understanding of the required electrical hookups, i.e., the required pull-ups to 3.3V for SIO_C and SIO_D, the voltage divider for pin 11 PWM.

I would like to use the exact frame grabber you have used on the desktop side. It appears you have kindly provided 2 different versions of frame grabber code for the desktop-side: (1) a version in the link off the Arduino forum (not to GitHub) but this code uses a 2MB baud rate but fails to compile on my MAC (only change I made is to line 279 for my UART port name and baud rate to 1MB); and (2) what appears to be an older version of the frame grabber on GitHub under sst39sf040-computerside that uses RS232.c/RS232.h but also is not aligned with the ov7670-no-ram-arduino-uno. It also does not compile.

I request that you post to this repository the version of the frame grabber you successfully used with the current ov7670-no-ram-ardunio-uno that is already in this repository -- I will then work with it and provide feedback on issues I can't solve myself. Thanks!

@ComputerNerd
Copy link
Owner

The code is no where near as good for the frame grabber as this code is. I wrote the frame grabber when I was a newer coder and I forgot to do a rewrite which is needed by the way. Things such as a fixed width and height controlled by
#define Width_G 640
#define Height_G 480
This does not make sense for a frame grabber. Instead there should be GUI control or command line arguments to specify width and height. Also I would prefer to make the protocol not fixed in code. Maybe Lua support or even a basic configuration file.
The code I have currently is modified to use a slightly different protocol. You can for now use the one I posted on the Arduino form until I find time to write a better one. Or maybe someone from the community contributes a better one. That would be very good if such were to happen.

@ComputerNerd
Copy link
Owner

You can try this:
https://github.com/ComputerNerd/simpleFrameGrabber

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

No branches or pull requests

2 participants