Re: cx231xx kernel oops

[Yan Seiner <yan@seiner.com>]

Yan Seiner wrote:
> Yan Seiner wrote:
>> Andy Walls wrote:
>>> 800 MB for 320x420 frames? It sounds like your app has gooned its 
>>> requested buffer size.
>>>   
>>
>> That's an understatement.  :-)
>>
>>> <wild speculation>
>>> This might be due to endianess differences between MIPS abd x86 and 
>>> your app only being written and tested on x86.
>>> </wild speculation>
>>>   
>>
>> My speculation too.  I don't know where that number comes from; the 
>> same app works fine with the saa7115 driver if I switch frame 
>> grabbers.  I'll have to do some fiddling with the code to figure out 
>> where the problem lies.  It's some interaction between the app and 
>> the cx231xx driver.
>
> HAH!
>
> This simple patch fixes it on my MIPS platform - not tested on other 
> architectures as I don't have them readily available running a newer 
> kernel:
>
OK, too groggy last night with my own success... Here's the better 
solution.  Should work on all architectures.

diff -U3 
/data10/home/yan/openwrt/backfire/trunk/build_dir/linux-brcm47xx/linux-3.0.3/drivers/media/video/cx231xx/cx231xx-pcb-cfg.c 
cx231xx-pcb-cfg.c
--- 
/data10/home/yan/openwrt/backfire/trunk/build_dir/linux-brcm47xx/linux-3.0.3/drivers/media/video/cx231xx/cx231xx-pcb-cfg.c    
2011-08-17 10:57:16.000000000 -0700
+++ cx231xx-pcb-cfg.c    2011-12-13 05:50:36.000000000 -0800
@@ -672,7 +672,9 @@
     pcb config it is related to */
     cx231xx_read_ctrl_reg(dev, VRT_GET_REGISTER, BOARD_CFG_STAT, data, 4);
 
-    config_info = *((u32 *) data);
+    //config_info = *((u32 *) data);
+    config_info = ((u32)(*(data))) + (((u32)(*(data+1))) << 8) + 
(((u32)(*(data+2))) << 16) + (((u32)(*(data+3))) << 24);
+    cx231xx_info("config_info %x\n",config_info);
     usb_speed = (u8) (config_info & 0x1);
 
     /* Verify this device belongs to Bus power or Self power device */



-- 
Few people are capable of expressing with equanimity opinions which differ from the prejudices of their social environment. Most people are even incapable of forming such opinions.
    Albert Einstein