Am Montag 22 Juni 2009 02:20:12 schrieb Brian Johnson: > > This means: > > 1. there is a stream (otherwise mplayer would show green) > > 2. It's to dark in my cave. > > 3. The manual gain is broken or insufficient for SOI968 (0c45:624e). > > 4. We need autogain reimplemented. Why are all these features missing > > anyway? > > Did you try using the camera in a suitabe light environment so that > even with low gain and exposure you ought to be able to see something? > Also when running the camera using mplayer can you check to see what > the exposure value is? start v4lucp turn on the update every second > and see if the exposure setting is in fact getting adjusted. > The gain control itself should work for all omnivision sensors i think > the formula should be: (bit[6] + 1) * (but[5] + 1) * (but[4] + 1) * > (bits[3:0]/16 + 1). This should give a maximum 8x gain. > Can you check the debugfs and find out what values are in the > following registers: 0x2d, 0x2e and 0x00 as well as what the v4l2 > exposure and gain settings are set to? > > > The advantage of "branching" this code in our repo is that all microdia > > contributers can contribute to the gspca_sn9c20x rewrite. Otherwise you > > need to reimplement all these faetures on your own. > > Instead of branching it would make more sense to create new > repository. The other thing is since this relies on header file found > in gspca my only patchsets are against either the kernel or v4l-dvb > should e just clone a copy of v4l-dvb and make that available? > > > On Sun, Jun 21, 2009 at 5:39 PM, Josua Grawitter > > <[email protected]> wrote: > > Am Sonntag 21 Juni 2009 21:17:09 schrieb Brian Johnson: > > > Strange I've compiled the latest version of v4l-dvb against both a > > > 2.6.28 and a 2.6.30 kernel with no problems so it seems odds that it > > > would not compile against a 2.6.29 one as well. > > > > > > Anyways i've attached an updated patch which I'm thinking will be the > > > one i likely submit. This one has two additional changes first i > > > increased the exposure setting to have a max of 6000 for abit of > > > increased granularity. Second i re-added code for the gain control. > > > the control control has a range of 0-28, which represents 1x-8x gain > > > with a step size of .25x > > > > > > On Sat, Jun 20, 2009 at 3:13 PM, Josua Grawitter > > > > > > <[email protected]> wrote: > > > > Am Samstag 20 Juni 2009 18:26:51 schrieb Brian Johnson: > > > > > I think its probably easier just to checkout the v4l-dvb repo apply > > > > > the patch and compile the whole v4l subsystem. > > > > > > > > > > However if you want to compile it on its own the attached makefiile > > > > > should work for compiling. > > > > > You will need to copy the jpeg.h and gspca.h headers files from > > > > > v4l-dvb for it to work however. > > > > > Also make sure to add the following line to the sn9c20x.h header > > > > > file > > > > > > > > > > #define V4L2_PIX_FMT_SN9C20X_I420 v4l2_fourcc('S', '9', '2', '0') > > > > > > > > > > On Sat, Jun 20, 2009 at 11:48 AM, Josua Grawitter > > > > > > > > > > <[email protected]> wrote: > > > > > > Am Samstag 20 Juni 2009 10:54:49 schrieb Vasily Khoruzhick: > > > > > > > On Saturday 20 June 2009 06:33:49 Brian Johnson wrote: > > > > > > > > Well making a new branch on our current repo would be a bit > > > > > > > > odd since so when reworking this as a gspca module so much > > > > > > > > changes in the general structure tha from gits point of view > > > > > > > > i basically deleted all the files and replaced them with 3 > > > > > > > > new ones. I am postin the current patch whcih i've verified > > > > > > > > works properly with a 624f camera for people look at before > > > > > > > > submitting to linux-media. This patch was written against the > > > > > > > > latest kernel checkout but should be able to be applied > > > > > > > > cleanly against the latest version of v4l found at > > > > > > > > http://linuxtv.org/hg/v4l-dvb/ just hit enter and choose to > > > > > > > > skip patching the MAINTAINERS file when it complains. Using > > > > > > > > the v4l checkout is a lot quicker as well as you don't need > > > > > > > > to bother upgrading your kernel just to try the driver. > > > > > > > > > > > > > > Works for me with my 624f cam, but it's missing gain control. > > > > > > > Anyway, thanks for your work ;) > > > > > > > > > > > > > > Regards > > > > > > > Vasily > > > > > > > > > > > > I find myself unable to compile the module by itself. > > > > > > > > > > > > Attached is the modified Makefile. Maybe someone can point out > > > > > > what I'm doing wrong here. > > > > > > > > > > > > GWater > > > > > > > > Actually thats what I did. (downloading the whole v4l-dvb tree). It > > > > seems that one macro from jpeg.h is missing (QUANT_whatever). > > > > Therefore not even the tree builds. Anyway I'll try to get an older > > > > revision as I expect this is because of my a bit older 2.6.29 kernel > > > > headers. > > > > > > > > GWater > > > > So, > > I got todays HEAD + Patch built and insmodded. Unfortunatly I get no only > > blackness. > > > > This means: > > 1. there is a stream (otherwise mplayer would show green) > > 2. It's to dark in my cave. > > 3. The manual gain is broken or insufficient for SOI968 (0c45:624e). > > 4. We need autogain reimplemented. Why are all these features missing > > anyway? > > > > The advantage of "branching" this code in our repo is that all microdia > > contributers can contribute to the gspca_sn9c20x rewrite. Otherwise you > > need to reimplement all these faetures on your own. > > > > GWater > > --~--~---------~--~----~------------~-------~--~----~ > Lets make microdia webcams plug'n play, (currently plug'n pray) > To post to this group, send email to [email protected] > Visit us online https://groups.google.com/group/microdia > -~----------~----~----~----~------~----~------~--~---
A complete v4l-dvb tree sounds good. (I hadn't thought about this before, actually) I'll take a look at the gain code in a few hours. Anyway - there was enough light to make it work with the standalone driver. Thank you, GWater
signature.asc
Description: This is a digitally signed message part.
