-----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
>>>>>>>>>> [email protected]
>>>>>>>>>> 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-----
diff -r 1c1090824398 linux/drivers/media/video/gspca/m5602/m5602_po1030.h
--- a/linux/drivers/media/video/gspca/m5602/m5602_po1030.h Sat Jan 10 14:55:07 2009 +0100
+++ b/linux/drivers/media/video/gspca/m5602/m5602_po1030.h Sat Jan 10 14:57:12 2009 +0100
@@ -250,6 +250,14 @@
{BRIDGE, M5602_XB_SEN_CLK_DIV, 0x04},
{BRIDGE, M5602_XB_SEN_CLK_CTRL, 0xb0},
/*end of sequence 2 stop */
+
+ {BRIDGE, M5602_XB_GPIO_DIR, 0x05},
+ {BRIDGE, M5602_XB_GPIO_DAT, 0x00},
+ {BRIDGE, M5602_XB_GPIO_EN_H, 0x06},
+ {BRIDGE, M5602_XB_GPIO_EN_L, 0x00},
+
+ {SENSOR, PO1030_AUTOCTRL2, 0x04},
+
};
static const unsigned char start_po1030[][3] =
------------------------------------------------------------------------------
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
