-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1
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. 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) iEYEARECAAYFAklqFwgACgkQN7qBt+4UG0G6BgCgo48jMi/S3jxfhWae88DIAGK+ d1EAniy3FrG8DHxvfbEqCnOy2TrFJilb =3n/R -----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
