ComputerNerd / ov7670-no-ram-arduino-uno

Allows you to use a non fifo ov7670 on the arudino uno without external spi ram like other examples. Sends the data to UART.
235 stars 111 forks source link

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

Open jnshadow opened 9 years ago

jnshadow commented 9 years ago

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 commented 9 years ago

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 commented 9 years ago

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