All of lore.kernel.org
 help / color / mirror / Atom feed
From: Yan Seiner <yan@seiner.com>
To: linux-media@vger.kernel.org
Subject: Re: [cx231xx] Support for Arm / Omap working at all?
Date: Tue, 22 Jan 2013 06:15:14 -0800	[thread overview]
Message-ID: <50FE9EF2.6060206@seiner.com> (raw)
In-Reply-To: <CAGoCfiw1oSohcU=LBUxco6A2EmuNi649YiQFPgjnG1r0E4rZmw@mail.gmail.com>

Devin Heitmueller wrote:
> On Tue, Jan 22, 2013 at 4:38 AM, Jan Stumpf <Jan.Stumpf@asctec.de> wrote:
>   
>> Thanks!
>>
>> I will try it with your patches!
>>
>> Regards
>> Jan
>>     
>
> FYI:  the cx231xx driver has worked in the past on ARM platforms,
> although I haven't tried the USBLive2 on OMAP specifically.  In fact,
> I merged the original driver support upstream as part of a project I
> did while developing a product that has it running on ARM.
>
> You may wish to try whatever kernel you have on an x86 platform, as
> people have a history of introducing regressions for the USBLive 2 in
> the past (I've fixed it multiple times since I originally submitted
> the support upstream).  It's possible that it's broken on x86 as well,
> and has nothing to do with your being on ARM at all.
>
> Regards,
>
> Devin
>
>   
I submitted this patch some time ago.  It fixes the driver for 
endiannes.  Without it will try to allocate some random amount of 
memory.  It was in the kernel for a little then it disappeared....

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 */



      reply	other threads:[~2013-01-22 14:52 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-01-17  7:31 [cx231xx] Support for Arm / Omap working at all? Jan Stumpf
2013-01-21  9:53 ` Hans Verkuil
2013-01-22  9:38   ` AW: " Jan Stumpf
2013-01-22 13:36     ` Devin Heitmueller
2013-01-22 14:15       ` Yan Seiner [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=50FE9EF2.6060206@seiner.com \
    --to=yan@seiner.com \
    --cc=linux-media@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.