-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1
Laurent BARRAILLE wrote: > Hi, > > Thanks, > > My sensor is detected again. > > I've got always the problem of the red and blue blance, the image is > very red. > > When i set manually (with v4lctrl) this controls to default value it's ok. > > They don't set in the init sequence however it seems to be necessary to > set them. > I saw you removed it from init sequence ( > http://linuxtv.org/hg/~eandren/gspca-m5602/rev/d22ff6898892 ) > They are now set to the current value after the init sequence (see po1030_init() not the default. But I did find a bug where I set the red balance twice and not the blue balance at all. This is now fixed. Thanks for reporting, Erik > > Erik Andrén a écrit : > > > Laurent BARRAILLE wrote: > >>>> Ok, the sensor dump seems to work again but the sensor detection >>>> doesn't work yet. >>>> >>>> I attach result of dump_sensor >>>> >>>> > Thanks, > When correlating your sensor dump with the datasheet it seems like > all reads are offsets of one. When sensor register address 0x00 is > read we get the contents of address 0x01 and so on. I have some > saves register dumps from earlier and it seems like we've always had > this problem. > > The po1030 window driver snoops I have never reads the sensor only > writes to it so I'll have to think about how to solve this. > > >> Good, i think this will help you to improve the driver. > > >> Regards. > >> Laurent > > > I've pushed the previous patch plus a temporary solution to the > probe routine so that you don't need to force the sensor. > > Thanks, > Erik > > > >>>> Regards >>>> >>>> Laurent >>>> >>>> Erik Andrén a écrit : >>>> >>>> >>>> Laurent BARRAILLE wrote: >>>> >>>> >>>>>>> Hi, >>>>>>> >>>>>>> This revision is the offender of the bad sensor dump : >>>>>>> >>>>>>> http://linuxtv.org/hg/~eandren/gspca-m5602/rev/14a865ca7f2a >>>>>>> >>>>>>> Regards >>>>>>> >>>>>>> Laurent >>>>>>> >>>> Thanks for your quick work. >>>> Does the attached patch fix the issue? >>>> >>>> Regards, >>>> Erik >>>> >>>> >>>> >>>>>>> Erik Andrén a écrit : >>>>>>> >>>>>>> >>>>>>> Laurent BARRAILLE wrote: >>>>>>> >>>>>>> >>>>>>>>>> Hi, >>>>>>>>>> >>>>>>>>>> The new sensor dump. >>>>>>>>>> >>>>>>>>>> Regards >>>>>>>>>> >>>>>>>>>> Laurent >>>>>>>>>> >>>>>>> Thanks, >>>>>>> Obviously I broke the routine reading the po1030 registers as all >>>>>>> registers now respond with 0xff. Is it possible for you to either do >>>>>>> a bisect of the last commits or back them out one by one doing a >>>>>>> dump_sensor to see what commit that is the offender? >>>>>>> >>>>>>> See the hg manual for more instructions on how to perform these >>>>>>> routines. >>>>>>> >>>>>>> Regards, >>>>>>> Erik >>>>>>> >>>>>>> >>>>>>> >>>>>>>>>> Erik Andrén a écrit : >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> Laurent BARRAILLE wrote: >>>>>>>>>> >>>>>>>>>> >>>>>>>>>>>>> Erik Andrén a écrit : >>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>> Laurent BARRAILLE wrote: >>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>>>>> Hi, >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> My sensor is no more automaticly recognized. >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> >>>>>>>>>>>>> Hm, this is bad. If you insert the gspca_m5602 with >>>>>>>>>>>>> dump_sensor=1, >>>>>>>>>>>>> could you please send me the produced kernel log. >>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>>> i attached the result of this command : >>>>>>>>>>>>>> sudo modprobe gspca_m5602 dump_sensor=1 force_sensor=5 >>>>>>>>>>>>>> >>>>>>>>>>>>>>>> It works only when i force the sensor in loading module. >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> >>>>>>>>>> Ok, the attached dump showed that all register values were bogus. >>>>>>>>>> I've now moved the dump_sensor function to a more "safe" position. >>>>>>>>>> Could you please update to the latest driver and attach a new dump >>>>>>>>>> result? >>>>>>>>>> >>>>>>>>>> Regards, >>>>>>>>>> Erik >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> >>>>>>>>>>>>>>>> The auto balance white control has almost no effect on my >>>>>>>>>>>>>>>> image >>>>>>>>>>>>>>>> maybe >>>>>>>>>>>>>>>> it's normal. >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> >>>>>>>>>>>>> Did you activate it? It's disabled by default. >>>>>>>>>>>>> >>>>>>>>>>>>>> Yes, with vlc and v4lctrl. >>>>>>>>>>>>>> I've tried many times, there is a very very small change >>>>>>>>>>>>>> when i >>>>>>>>>>>>>> activate >>>>>>>>>>>>>> it but useless. >>>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>>> Best Regards >>>>>>>>>>>>>> Laurent >>>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>>>>> Unfortunately, i have noticed no really improvement in >>>>>>>>>>>>>>>> the use >>>>>>>>>>>>>>>> of my >>>>>>>>>>>>>>>> webcam. >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> With the old and the new version of the driver, the image is >>>>>>>>>>>>>>>> too red, >>>>>>>>>>>>>>>> too pink. >>>>>>>>>>>>>>>> When i set the blue and red balance to 54 ( 36 in hex) >>>>>>>>>>>>>>>> instead of >>>>>>>>>>>>>>>> 64 (40 >>>>>>>>>>>>>>>> in hex) the colors seems better. >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> >>>>>>>>>>>>> Ok, I'll set these as new default values. >>>>>>>>>>>>> >>>>>>>>>>>>> Thanks for testing! >>>>>>>>>>>>> >>>>>>>>>>>>> Regards, >>>>>>>>>>>>> Erik >>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>>>>> Best Regards >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> Laurent >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> Erik Andrén a écrit : >>>>>>>>>>>>>>>> Hi, >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> I've recently been able to aquire the datasheet for the >>>>>>>>>>>>>>>> PO1030 >>>>>>>>>>>>>>>> from >>>>>>>>>>>>>>>> pixelplus, unfortunately I'm not allowed to distribute or >>>>>>>>>>>>>>>> share it >>>>>>>>>>>>>>>> with anyone. I've pushed some changes to the linuxtv tree >>>>>>>>>>>>>>>> for the >>>>>>>>>>>>>>>> po1030. >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> Couly you please test the latest version and see if it still >>>>>>>>>>>>>>>> works >>>>>>>>>>>>>>>> for you? >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> Thanks, >>>>>>>>>>>>>>>> Erik >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>>>>> ------------------------------------------------------------------------------ >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> Check out the new SourceForge.net Marketplace. >>>>>>>>>>>>>>>> It is the best place to buy or sell services for >>>>>>>>>>>>>>>> just about anything Open Source. >>>>>>>>>>>>>>>> http://p.sf.net/sfu/Xq1LFB >>>>>>>>>>>>>>>> _______________________________________________ >>>>>>>>>>>>>>>> M560x-driver-devel mailing list >>>>>>>>>>>>>>>> [email protected] >>>>>>>>>>>>>>>> https://lists.sourceforge.net/lists/listinfo/m560x-driver-devel >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.9 (GNU/Linux) iEYEARECAAYFAklqOY4ACgkQN7qBt+4UG0GqnQCaArnwPEALunYZFhGUznjdbXs5 6AgAoJlcyJ2Wg6utawUmBLpipVzs7Eph =+rlJ -----END PGP SIGNATURE----- ------------------------------------------------------------------------------ Check out the new SourceForge.net Marketplace. It is the best place to buy or sell services for just about anything Open Source. http://p.sf.net/sfu/Xq1LFB _______________________________________________ M560x-driver-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/m560x-driver-devel
