[PATCH 1/1] media: mx3_camera: Improve data bus width check code for probe

2012-08-05 Thread Liu Ying
This patch contains code change only to use the present macro- MX3_CAMERA_DATAWIDTH_MASK to check valid camera platform data bus width instead of enumerating every possible data bus width. Signed-off-by: Liu Ying --- drivers/media/video/mx3_camera.c |4 +--- 1 files changed, 1 insertions(+),

Re: [RFC PATCH 1/3] dma-fence: dma-buf synchronization (v5)

2012-08-05 Thread Sumit Semwal
Hi Maarten, On 27 July 2012 19:09, Maarten Lankhorst wrote: > A dma-fence can be attached to a buffer which is being filled or consumed > by hw, to allow userspace to pass the buffer without waiting to another > device. For example, userspace can call page_flip ioctl to display the > next frame o

Re: [PATCH 2/2] dma-buf: add helpers for attacher dma-parms

2012-08-05 Thread Semwal, Sumit
On Fri, Jul 20, 2012 at 10:09 PM, Rob Clark wrote: > Fyi, Daniel Vetter had suggested on IRC that it would be cleaner to > have a single helper fxn that most-restrictive union of all attached > device's dma_parms. Really this should include dma_mask and > coherent_dma_mask, I think. But that tou

Re: [PATCH] staging: media: cxd2099: remove memcpy of similar structure variables

2012-08-05 Thread Devendra Naga
Hi Ezequiel, On Mon, Aug 6, 2012 at 3:36 AM, Ezequiel Garcia wrote: > Hi Devendra, > > Thanks for the patch, > > On Sun, Aug 5, 2012 at 5:40 PM, Devendra Naga > wrote: >> structure variables can be assigned, no memcpy needed, >> remove the memcpy and use assignment for the cfg and en variables.

Re: boot slow down

2012-08-05 Thread James
On 08/05/12 17:20, Sakari Ailus wrote: > Hi Andy and James, > > On Sat, Aug 04, 2012 at 06:28:19PM -0400, James wrote: >> On 08/04/12 13:42, Andy Walls wrote: >>> James wrote: >>> There's a big pause before the 'unable' [2.243856] usb 4-1: Manufacturer: Logitech [ 62.739

Re: firmware directory

2012-08-05 Thread James
On 08/05/12 13:49, Oliver Schinagl wrote: > On 05-08-12 17:27, James wrote: >> [ 62.739097] cx25840 6-0044: unable to open firmware >> v4l-cx23885-avcore-01.fw >> >> Did the firmware directory change recently? >> >> # ls -l /lib/firmware/v4l-cx23885-avcore-01.fw >> -rw-r--r-- 1 root root 16382 O

Re: boot slow down

2012-08-05 Thread James
On 08/05/12 17:20, Sakari Ailus wrote: > Hi Andy and James, > > On Sat, Aug 04, 2012 at 06:28:19PM -0400, James wrote: >> On 08/04/12 13:42, Andy Walls wrote: >>> James wrote: >>> There's a big pause before the 'unable' [2.243856] usb 4-1: Manufacturer: Logitech [ 62.739

[PATCH] [media] Add USB dependency for IguanaWorks USB IR Transceiver

2012-08-05 Thread Guenter Roeck
This patch fixes the error drivers/usb/core/hub.c:3753: undefined reference to `usb_speed_string' seen in various random configurations. Cc: Sean Young Cc: Mauro Carvalho Chehab Signed-off-by: Guenter Roeck --- drivers/media/rc/Kconfig |1 + 1 file changed, 1 insertion(+) diff --g

Re: [PATCH 0/2] get rid of fe_ioctl_override()

2012-08-05 Thread Devin Heitmueller
On Sun, Aug 5, 2012 at 5:31 PM, Mauro Carvalho Chehab wrote: > This is not how it works. Patches are posted at the ML and developers can > review and comment about them. Does those patches break something? If not, > please stop flaming. I'm confused why you think this email was a flame. It's ask

[PATCH] [media] move dvb-usb-ids.h to dvb-core

2012-08-05 Thread Mauro Carvalho Chehab
While this header were meant to be used just by dvb-usb driver, it is now being used also by dvb-usb-v2 and cx231xx. So, move it to a better place. Signed-off-by: Mauro Carvalho Chehab --- This fixes dvb-usb-v2 build at the out-of-tree media_tree.git. V.2: Send it with git -M, to better show th

[PATCH] [media] move dvb-usb-ids.h to dvb-core

2012-08-05 Thread Mauro Carvalho Chehab
While this header were meant to be used just by dvb-usb driver, it is now being used also by dvb-usb-v2 and cx231xx. So, move it to a better place. Signed-off-by: Mauro Carvalho Chehab --- This fixes dvb-usb-v2 build at the out-of-tree media_tree.git. --- drivers/media/dvb/dvb-core/dvb-usb-ids

Re: [PATCH] staging: media: cxd2099: remove memcpy of similar structure variables

2012-08-05 Thread Ezequiel Garcia
Hi Devendra, Thanks for the patch, On Sun, Aug 5, 2012 at 5:40 PM, Devendra Naga wrote: > structure variables can be assigned, no memcpy needed, > remove the memcpy and use assignment for the cfg and en variables. > > Tested by Compilation Only > > Suggested-by: Ezequiel Garcia I'm not sure th

Re: [PATCH 0/2] get rid of fe_ioctl_override()

2012-08-05 Thread Mauro Carvalho Chehab
Em 05-08-2012 15:44, Devin Heitmueller escreveu: > On Sun, Aug 5, 2012 at 2:16 PM, Mauro Carvalho Chehab > wrote: >> There's just one driver using fe_ioctl_override(), and it can be >> replaced at tuner_attach call. This callback is evil, as only DVBv3 >> calls are handled. >> >> Removing it is al

Re: boot slow down

2012-08-05 Thread Sakari Ailus
Hi Andy and James, On Sat, Aug 04, 2012 at 06:28:19PM -0400, James wrote: > On 08/04/12 13:42, Andy Walls wrote: > > James wrote: > > > >> There's a big pause before the 'unable' > >> > >> [2.243856] usb 4-1: Manufacturer: Logitech > >> [ 62.739097] cx25840 6-0044: unable to open firmware

[PATCH] staging: media: cxd2099: remove memcpy of similar structure variables

2012-08-05 Thread Devendra Naga
structure variables can be assigned, no memcpy needed, remove the memcpy and use assignment for the cfg and en variables. Tested by Compilation Only Suggested-by: Ezequiel Garcia Signed-off-by: Devendra Naga --- drivers/staging/media/cxd2099/cxd2099.c |4 ++-- 1 file changed, 2 insertions(

Re: [PATCH 2/2] staging: media: cxd2099: use kzalloc to allocate ci pointer of type struct cxd in cxd2099_attach

2012-08-05 Thread Devendra Naga
Hello Ezequiel, Thanks, you wrote a full description of what i need to do.. i will definitely follow this. On Sun, Aug 5, 2012 at 11:57 PM, Ezequiel Garcia wrote: > Hi Devendra, > > On Sun, Aug 5, 2012 at 1:04 AM, Devendra Naga > wrote: >> Hello Ezequiel, >> >> On Sun, Aug 5, 2012 at 12:24 AM,

Re: [PATCH 0/2] get rid of fe_ioctl_override()

2012-08-05 Thread Devin Heitmueller
On Sun, Aug 5, 2012 at 2:16 PM, Mauro Carvalho Chehab wrote: > There's just one driver using fe_ioctl_override(), and it can be > replaced at tuner_attach call. This callback is evil, as only DVBv3 > calls are handled. > > Removing it is also a nice cleanup, as about 90 lines of code are > removed

[PATCH 2/2] [media] dvb: get rid of fe_ioctl_override callback

2012-08-05 Thread Mauro Carvalho Chehab
This callback were meant to allow overriding a FE callback, before its call, but it is not really needed, as the callback can be intercepted after tuner attachment. Worse than that, only DVBv3 calls are intercepted this way, so a DVBv5 application will produce different effects than a DVBv3 one.

[PATCH 1/2] [media] dvb core: remove support for post FE legacy ioctl intercept

2012-08-05 Thread Mauro Carvalho Chehab
This DVB_FE_IOCTL_POST isn't used, so remove it. Also, intercepting ioctl's like that only works with legacy ioctl's, due to the way it was implemented. so this design is broken. Document it. Signed-off-by: Mauro Carvalho Chehab --- drivers/media/dvb/dvb-core/dvb_frontend.c | 7 --- drivers

[PATCH 0/2] get rid of fe_ioctl_override()

2012-08-05 Thread Mauro Carvalho Chehab
There's just one driver using fe_ioctl_override(), and it can be replaced at tuner_attach call. This callback is evil, as only DVBv3 calls are handled. Removing it is also a nice cleanup, as about 90 lines of code are removed. Get rid of it! Mauro Carvalho Chehab (2): [media] dvb core: remove

Re: [PATCH 2/2] staging: media: cxd2099: use kzalloc to allocate ci pointer of type struct cxd in cxd2099_attach

2012-08-05 Thread Ezequiel Garcia
Hi Devendra, On Sun, Aug 5, 2012 at 1:04 AM, Devendra Naga wrote: > Hello Ezequiel, > > On Sun, Aug 5, 2012 at 12:24 AM, Ezequiel Garcia > wrote: >> Hi Devendra, >> >> On Sat, Aug 4, 2012 at 3:12 PM, Devendra Naga >> wrote: >>> >>> mutex_init(&ci->lock); >>> memcpy(&ci->cfg, cf

Re: firmware directory

2012-08-05 Thread Oliver Schinagl
On 05-08-12 17:27, James wrote: [ 62.739097] cx25840 6-0044: unable to open firmware v4l-cx23885-avcore-01.fw Did the firmware directory change recently? # ls -l /lib/firmware/v4l-cx23885-avcore-01.fw -rw-r--r-- 1 root root 16382 Oct 15 2011 /lib/firmware/v4l-cx23885-avcore-01.fw -- To unsub

[PATCH] [media] az6007: Update copyright

2012-08-05 Thread Mauro Carvalho Chehab
Update copyright comments after dvb-usb-v2 conversion. Signed-off-by: Mauro Carvalho Chehab --- drivers/media/dvb/dvb-usb-v2/az6007.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/dvb/dvb-usb-v2/az6007.c b/drivers/media/dvb/dvb-usb-v2/az6007.c index 420cb

[PATCH 3/3] [media] az6007: handle CI during suspend/resume

2012-08-05 Thread Mauro Carvalho Chehab
The dvb-usb-v2 core doesn't know anything about CI. So, the driver needs to handle it by hand. This patch stops CI just before stopping URB's/RC, and restarts it before URB/RC start. It should be noticed that suspend/resume is not yet working properly, as the PM model requires the implementation o

[PATCH 2/3] [media] az6007: make all functions static

2012-08-05 Thread Mauro Carvalho Chehab
There's no reason why those functions shouldn't be static. Signed-off-by: Mauro Carvalho Chehab --- drivers/media/dvb/dvb-usb-v2/az6007.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/dvb/dvb-usb-v2/az6007.c b/drivers/media/dvb/dvb-usb-v2/az6007.c index b

[PATCH 1/3] [media] az6007: rename "st" to "state" at az6007_power_ctrl()

2012-08-05 Thread Mauro Carvalho Chehab
On all other parts, this var is called state. So, use the same name here, to be consistent. Signed-off-by: Mauro Carvalho Chehab --- drivers/media/dvb/dvb-usb-v2/az6007.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/media/dvb/dvb-usb-v2/az6007.c b/drivers/

[PATCH 0/3] Some additional az6007 cleanup patches

2012-08-05 Thread Mauro Carvalho Chehab
Those are mostly cleanup patches. With regards to suspend/resume, this is not working properly yet. I suspect that it is due to the lack of dvb-usb-v2 support for reset_resume. So, document it. Mauro Carvalho Chehab (3): [media] az6007: rename "st" to "state" at az6007_power_ctrl() [media] az6

Re: Update VIP to videobuf2 and control framework

2012-08-05 Thread Federico Vaga
Hi Hans, > Did you run the latest v4l2-compliance tool from the v4l-utils.git > repository over your driver? I'm sure you didn't since VIP is missing > support for control events and v4l2-compliance would certainly > complain about that. > > Always check with v4l2-compliance whenever you make ch

firmware directory

2012-08-05 Thread James
[ 62.739097] cx25840 6-0044: unable to open firmware v4l-cx23885-avcore-01.fw Did the firmware directory change recently? # ls -l /lib/firmware/v4l-cx23885-avcore-01.fw -rw-r--r-- 1 root root 16382 Oct 15 2011 /lib/firmware/v4l-cx23885-avcore-01.fw -- To unsubscribe from this list: send the l

Re: [PATCH] lmedm04 v2.05 conversion to dvb-usb-v2

2012-08-05 Thread Mauro Carvalho Chehab
Em 05-08-2012 11:10, Malcolm Priestley escreveu: > Conversion of lmedm04 to dvb-usb-v2 > > functional changes are that callbacks have been moved to fe_ioctl_override. Don't do that: fe_ioctl_override has a broken design and only handles DVBv3 ioctl's. So, if userspace is using DVBv5, this will ca

[PATCH] lmedm04 v2.05 conversion to dvb-usb-v2

2012-08-05 Thread Malcolm Priestley
Conversion of lmedm04 to dvb-usb-v2 functional changes are that callbacks have been moved to fe_ioctl_override. This patch is applied on top of [BUG] Re: dvb_usb_lmedm04 crash Kernel (rs2000) http://patchwork.linuxtv.org/patch/13584/ Signed-off-by: Malcolm Priestley --- drivers/media/dvb/dvb-

[PATCH] [media] gspca: dubious one-bit signed bitfield

2012-08-05 Thread Emil Goode
This patch changes some signed integers to unsigned because they are not intended for negative values and sparse is making noise about it. Sparse gives eight of these errors: drivers/media/video/gspca/ov519.c:144:29: error: dubious one-bit signed bitfield Signed-off-by: Emil Goode --- drivers/m