Re: [M560x-driver-devel] PO1030 progress

[Laurent B. <du...@gm...>]

Another problem.

Since the recent changes, by default my image is very red.

When i read the red and blue balance values with v4lctrl,  i've got the 
default value 64, but if i set the same value with v4lctrl the image 
becomes good

Regards

Laurent

Erik Andrén a écrit :
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
>
>
> 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
>>>>>>>>>>> M56...@li...
>>>>>>>>>>> https://lists.sourceforge.net/lists/listinfo/m560x-driver-devel
>>>>>>>>>>>                               
>>>>>>>>>>>                       
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.9 (GNU/Linux)
>
> iEYEARECAAYFAkloqYoACgkQN7qBt+4UG0Hn+gCfR4JCR9JG3WJmOFfwqbd1fuWJ
> o6UAnRW/RBoeQ2yjiO1dc1J4V1e18ZbD
> =GHE1
> -----END PGP SIGNATURE-----
>