weekend patches

12 views
Skip to first unread message

Brian Johnson

unread,
Aug 16, 2008, 1:44:11 AM8/16/08
to micr...@googlegroups.com
Ok heres my list of patches I've been working for testing this weekend.

Changes:
- Seperating sensor init from 624f start_stream
- Trimming alot of unnecessary register writes of of the 624f start_stream
- Autoexposure patch posted by greywater. I moved the exposure function
into the ov965x.c file. Also the v4l2 control being used was AUTOGAIN,
I created a new v4l2 control to use for auto exposure.
- Added in support for following camera models 0c45:62bb, 0c45:6253, and
0c45:627f. These models all use already working bridge/sensor combos
- Fixed a few warnings when compiled for rt kernel. The two reported errors
may still exist when compiled on amd64 though.
- Added hardware based resolution switching using bridge scaling and
windowing registers.
Supported resolutions are 640x480, 320x240, and 160x120

0001-Seperate-bridge-and-sensor-code-for-624f.patch
0002-Minimizing-624f_start_stream.patch
0003-Auto-exposure-for-ov96xx-sensors.patch
0004-Readded-support-for-627f-cameras.patch
0005-Added-support-for-the-0c45-6253-camera.patch
0006-Added-support-for-0c45-62bb.patch
0007-Fix-warnings-when-compiled-under-rt-kernel.patch
0008-Support-for-hardware-resolultion-change.patch
0009-Removed-start_stream-writes-to-10fb-10ff-and-1189.patch

GWater

unread,
Aug 16, 2008, 4:42:17 AM8/16/08
to micr...@googlegroups.com
Brian Johnson schrieb:
No checkpatch, apply, build or dmesg errors.

New features (resolution switching, AEC) work fine.

No problems due to the removal of 0x1189 or 0x10fb-0x10ff settings. Even
I2C still works. (Don't know what went wrong when I tried that.)

GO!

GWater

JoJo jojo

unread,
Aug 17, 2008, 1:17:00 AM8/17/08
to micr...@googlegroups.com
Hi Brian

All patches look OK, except
0009-Removed-start_stream-writes-to-10fb-10ff-and-1189.patch
(causes 627b to just show green screen)

I'll take a look at the 627b changes in that patch. Meanwhile if you
want to push them,
the rest can be pushed w/o
0009-Removed-start_stream-writes-to-10fb-10ff-and-1189.patch

-JoJo

Brian Johnson

unread,
Aug 17, 2008, 8:23:13 AM8/17/08
to micr...@googlegroups.com
JoJo,
Heres an updated version of
0009-Removed-start_stream-writes-to-10fb-10ff-and-1189.patch. Let me
know if this fixes the green screen issues.
0009-Removed-start_stream-writes-to-10fb-10ff-and-1189.patch

JoJo jojo

unread,
Aug 17, 2008, 9:46:50 AM8/17/08
to micr...@googlegroups.com
Hey,

No still green!
(Don't worry just push the ones # 1- 8 patches,
or remove the 627b changes & push your #9 & I'll get to it soon)

-JoJo

Anbreizh

unread,
Aug 19, 2008, 9:12:02 AM8/19/08
to microdia
Hi !
I have try microdia driver with git sources and this nine patch for a
627f webcam. The webcam work, but picture was filp.
After this commands :

sudo modprobe -r microdia
sudo modprobe microdia vflip=1

The picture is good, but totally violet ......
Can you help me ?

On 17 août, 15:46, "JoJo jojo" <onetwoj...@gmail.com> wrote:
> Hey,
>
> No still green!
> (Don't worry just push the ones # 1- 8 patches,
> or remove the 627b changes & push your #9 & I'll get to it soon)
>
> -JoJo
>
> On Sun, Aug 17, 2008 at 5:53 PM, Brian Johnson <brij...@gmail.com> wrote:
> > JoJo,
> > Heres an updated version of
> > 0009-Removed-start_stream-writes-to-10fb-10ff-and-1189.patch. Let me
> > know if this fixes the green screen issues.
>
> > On Sun, Aug 17, 2008 at 1:17 AM, JoJo jojo <onetwoj...@gmail.com> wrote:
>
> >> Hi Brian
>
> >> All patches look OK, except
> >> 0009-Removed-start_stream-writes-to-10fb-10ff-and-1189.patch
> >> (causes 627b to just show green screen)
>
> >> I'll take a look at the 627b changes in that patch. Meanwhile if you
> >> want to push them,
> >> the rest can be pushed w/o
> >> 0009-Removed-start_stream-writes-to-10fb-10ff-and-1189.patch
>
> >> -JoJo
>

JoJo jojo

unread,
Aug 19, 2008, 9:46:14 AM8/19/08
to micr...@googlegroups.com
Hi Anbreizh

can you attach & email us a screenshot ?

-JoJo

Brian Johnson

unread,
Aug 19, 2008, 2:32:55 PM8/19/08
to micr...@googlegroups.com
Yes currently vflip will cause the colors to be off. The attached
patch has a fix to make vflip keep the correct colors. It should
apply on top of the nine previous patches.
0001-Fix-wrong-colors-on-vflip-for-ov965x.patch

Anbreizh

unread,
Aug 20, 2008, 11:09:48 AM8/20/08
to microdia
Oh thanks ! It's ok. Thanks you very much

On 19 août, 20:32, "Brian Johnson" <brij...@gmail.com> wrote:
> Yes currently vflip will cause the colors to be off.  The attached
> patch has a fix to make vflip keep the correct colors.  It should
> apply on top of the nine previous patches.
>
> On Tue, Aug 19, 2008 at 9:46 AM, JoJo jojo <onetwoj...@gmail.com> wrote:
>
> > Hi Anbreizh
>
> > can you attach & email us a screenshot ?
>
> > -JoJo
>
>  0001-Fix-wrong-colors-on-vflip-for-ov965x.patch
> 1KAfficherTélécharger

Brian Johnson

unread,
Aug 21, 2008, 12:30:19 PM8/21/08
to micr...@googlegroups.com
Alright sorry its taken awhile but i've finally got around to pushing
thee patches minus the 9th one. JoJO have you been able to figure out
whats wrong with the 627b when using the ninth patch yet?

JoJo jojo

unread,
Aug 21, 2008, 2:57:53 PM8/21/08
to micr...@googlegroups.com
On Thu, Aug 21, 2008 at 10:00 PM, Brian Johnson <bri...@gmail.com> wrote:
>
> Alright sorry its taken awhile but i've finally got around to pushing
> thee patches minus the 9th one. JoJO have you been able to figure out
> whats wrong with the 627b when using the ninth patch yet?
>


Hi Brian

been a bit busy past couple of days, maybe this weekend ;-)
Which writes are to be removed from the bridge initialization ?
1189 alone ?

-JoJo

Brian Johnson

unread,
Aug 21, 2008, 3:13:02 PM8/21/08
to micr...@googlegroups.com
registers 10fb-10ff and register 1189 should be removed they control
window dimensions and image scaling.
Reply all
Reply to author
Forward
0 new messages