> 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 -~----------~----~----~----~------~----~------~--~---
