Re: RFC: vb2: replace alloc_ctx by struct device * in vb2_queue

2014-09-21 Thread Marek Szyprowski
Hello, On 2014-09-21 15:05, Hans Verkuil wrote: Hi Marek, Pawel, Currently for dma_config (and the dma_sg code that I posted before) drivers have to allocate a alloc_ctx context, but in practice that just contains a device pointer. Is there any reason why we can't just change in struct vb2_qu

[PATCH] cx231xx: use i2c mux instead of own switching

2014-09-21 Thread Matthias Schwarzott
There no longer is port switching unrelated to any access. The 4 (3 external and 1 internal) i2c busses can now be specified directly via symbolic constants. The handling of the i2c_clients and the bus scanning could be improved, so that the new muxed busses also have i2c_clients for direct usage

Re: RFC: vb2: replace alloc_ctx by struct device * in vb2_queue

2014-09-21 Thread Pawel Osciak
Hi Hans, I'm not very much against it, but I'm not sure how option 2 is significantly simpler than option 1. It's just one cast away from being the same, unless you have some more rework in mind, e.g. making vb2 aware of devices somehow and skip driver's involvement for example? Thanks, Pawel On S

[PATCH 1/1] driver/drx39xyj: fix some compiling warnings

2014-09-21 Thread Li, Zhen-Hua
When compiling kernel, in module drx39xyj, there are some warnings showing some variables may be used uninitialized, though they have been initialized in fact. drivers/media/dvb-frontends/drx39xyj/drxj.c: In function ‘drxj_dap_atomic_read_reg32.isra.17’: drivers/media/dvb-frontends/drx39xyj/drxj

Re: [PATCH] [media] soc_camera: Support VIDIOC_EXPBUF ioctl

2014-09-21 Thread Kazunori Kobayashi
Hi Guennadi, Thank you for your review. (2014/09/20 16:50), Guennadi Liakhovetski wrote: > Hi Kobayashi-san, > > Thanks for the patch. In principle it looks good, just one question below: > > On Thu, 11 Sep 2014, Kazunori Kobayashi wrote: > >> This patch allows for exporting a dmabuf descripto

cron job: media_tree daily build: ERRORS

2014-09-21 Thread Hans Verkuil
This message is generated daily by a cron job that builds media_tree for the kernels and architectures in the list below. Results of the daily build of media_tree: date: Mon Sep 22 04:00:28 CEST 2014 git branch: test git hash: c0aaf696d45e2a72048a56441e81dad78659c698 gcc versi

HOW ARE YOU?

2014-09-21 Thread Benjamin Siaka
Hello my Dear, I will greatly appreciate my correspondence meets you in good health condition. My name is Mr. Benjamin Siaka. I am seeking for your co-operation for investment partnership in your Country. I shall provide the FUND for the investment. When you acknowledged the receipt of this cor

Re: [PATCH v2 3/8] anysee: convert tda18212 tuner to I2C client

2014-09-21 Thread Mauro Carvalho Chehab
Em Thu, 18 Sep 2014 16:01:59 +0300 Antti Palosaari escreveu: > > > On 09/18/2014 03:31 PM, Mauro Carvalho Chehab wrote: > > Em Sun, 7 Sep 2014 04:59:55 +0300 > > Antti Palosaari escreveu: > > > >> Used tda18212 tuner is implemented as I2C driver. Implement I2C > >> client to anysee and use it

Re: Running Technisat DVB-S2 on ARM-NAS

2014-09-21 Thread Jannis
Am 21.09.2014 um 19:28 schrieb JPT: > Tommorrow I'll swap the sat cable just to make sure this isn't the cause. Hi Jan, Are we talking about this device: http://www.linuxtv.org/wiki/index.php/Technisat_SkyStar_USB_HD (You never mentioned the actual model AFAIK)? If so, it has two LEDs. A red one

Re: [GIT PULL for v3.17-rc6] media fixes

2014-09-21 Thread Mauro Carvalho Chehab
Em Sun, 21 Sep 2014 11:05:06 -0700 Linus Torvalds escreveu: > On Sun, Sep 21, 2014 at 7:50 AM, Mauro Carvalho Chehab > wrote: > > > > PS.: FYI, I'm now starting to use mche...@osg.samsung.com e-mail address. > > The old one (m.che...@samsung.com) is still valid, but we're using the > > OSG subdo

Re: [PATCH v2 1/6] V4L2: Add Renesas R-Car JPEG codec driver.

2014-09-21 Thread Sergei Shtylyov
Hello. On 08/25/2014 04:49 PM, Hans Verkuil wrote: This patch contains driver for Renesas R-Car JPEG codec. Cnanges since v1: - s/g_fmt function simplified - default format for queues added - dumb vidioc functions added to be in compliance with standard api: jpu_s_pri

Re: [PATCH v2 1/6] V4L2: Add Renesas R-Car JPEG codec driver.

2014-09-21 Thread Sergei Shtylyov
Hello. On 08/25/2014 04:49 PM, Hans Verkuil wrote: This patch contains driver for Renesas R-Car JPEG codec. Cnanges since v1: - s/g_fmt function simplified - default format for queues added - dumb vidioc functions added to be in compliance with standard api: jpu_s_pri

Re: [GIT PULL for v3.17-rc6] media fixes

2014-09-21 Thread Linus Torvalds
On Sun, Sep 21, 2014 at 7:50 AM, Mauro Carvalho Chehab wrote: > > PS.: FYI, I'm now starting to use mche...@osg.samsung.com e-mail address. > The old one (m.che...@samsung.com) is still valid, but we're using the > OSG subdomain for the Samsung's Open Source Group. Thanks for signing this. When I

Re: Running Technisat DVB-S2 on ARM-NAS

2014-09-21 Thread JPT
> If I didn't remember wrong, that means allocated buffers are 8 * 32 * > 2048 = 524288 bytes. It sounds rather big for my taste. Probably even > wrong. IIRC USB2.0 frames are 1024 and there could be 1-3 frames. You > could use lsusb with all verbosity levels to see if it is > 1024/2048/3072. And s

Re: Running Technisat DVB-S2 on ARM-NAS

2014-09-21 Thread Antti Palosaari
On 09/21/2014 06:10 PM, JPT wrote: How my I find out more about the error -12? http://www.virtsync.com/c-error-codes-include-errno #define ENOMEM 12 /* Out of memory */ Likely allocating USB stream buffers fails. You could try request smaller buffers. Drop count to 1 and test. Drop fr

Re: Running Technisat DVB-S2 on ARM-NAS

2014-09-21 Thread JPT
>> How my I find out more about the error -12? > > http://www.virtsync.com/c-error-codes-include-errno > > #define ENOMEM 12 /* Out of memory */ > > Likely allocating USB stream buffers fails. You could try request > smaller buffers. Drop count to 1 and test. Drop framesperurb to 1 and >

[GIT PULL for v3.17-rc6] media fixes

2014-09-21 Thread Mauro Carvalho Chehab
Hi Linus, Please pull from: git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-media tags/media-v3.17-rc6 For some media bug fixes: - a Kconfig dependency issue; - Some fixes for af9033/it913x demod to be more reliable and address a performance regression;

[RFC PATCH 10/11] v4l2-ctrls: add multi-selection controls

2014-09-21 Thread Hans Verkuil
From: Hans Verkuil Signed-off-by: Hans Verkuil --- drivers/media/v4l2-core/v4l2-ctrls.c | 24 include/uapi/linux/v4l2-controls.h | 7 ++- 2 files changed, 30 insertions(+), 1 deletion(-) diff --git a/drivers/media/v4l2-core/v4l2-ctrls.c b/drivers/media/v4l2-cor

[RFC PATCH 05/11] v4l2-ctrls: add function to apply a configuration store.

2014-09-21 Thread Hans Verkuil
From: Hans Verkuil Drivers need to be able to select a specific store. Add a new function that can be used to apply a given store. Signed-off-by: Hans Verkuil --- drivers/media/v4l2-core/v4l2-ctrls.c | 61 include/media/v4l2-ctrls.h | 2 ++ 2 fil

[RFC PATCH 09/11] videodev2.h: add v4l2_ctrl_selection compound control type.

2014-09-21 Thread Hans Verkuil
From: Hans Verkuil This will be used by a new selection control. Signed-off-by: Hans Verkuil --- include/media/v4l2-ctrls.h | 2 ++ include/uapi/linux/videodev2.h | 8 2 files changed, 10 insertions(+) diff --git a/include/media/v4l2-ctrls.h b/include/media/v4l2-ctrls.h index 300

[RFC PATCH 02/11] videodev2.h: add config_store to v4l2_ext_controls

2014-09-21 Thread Hans Verkuil
From: Hans Verkuil The ctrl_class is fairly pointless when used with drivers that use the control framework: you can just fill in 0 and it will just work fine. There are still some old unconverted drivers that do not support 0 and instead want the control class there. The idea being that all cont

[RFC PATCH 01/11] videodev2.h: add V4L2_CTRL_FLAG_CAN_STORE

2014-09-21 Thread Hans Verkuil
From: Hans Verkuil Controls that have a configuration store will set this flag. Signed-off-by: Hans Verkuil --- include/uapi/linux/videodev2.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/uapi/linux/videodev2.h b/include/uapi/linux/videodev2.h index 0b1ba5c..199834b 100644 --- a

[RFC PATCH 06/11] videodev2.h: add new v4l2_ext_control flags field

2014-09-21 Thread Hans Verkuil
From: Hans Verkuil Replace reserved2 by a flags field. This is used to tell whether setting a new store value is applied only once or every time that v4l2_ctrl_apply_store() is called for that store. Signed-off-by: Hans Verkuil --- include/uapi/linux/videodev2.h | 6 +- 1 file changed, 5 i

[RFC PATCH 11/11] vivid: add crop/compose selection control support

2014-09-21 Thread Hans Verkuil
From: Hans Verkuil Signed-off-by: Hans Verkuil --- drivers/media/platform/vivid/vivid-core.h| 5 drivers/media/platform/vivid/vivid-ctrls.c | 37 drivers/media/platform/vivid/vivid-vid-cap.c | 10 ++-- drivers/media/platform/vivid/vivid-vid-cap.h |

[RFC PATCH 04/11] v4l2-ctrls: add config store support

2014-09-21 Thread Hans Verkuil
From: Hans Verkuil Signed-off-by: Hans Verkuil --- drivers/media/v4l2-core/v4l2-ctrls.c | 150 +-- drivers/media/v4l2-core/v4l2-ioctl.c | 4 +- include/media/v4l2-ctrls.h | 14 3 files changed, 141 insertions(+), 27 deletions(-) diff --git a/dr

[RFC PATCH 03/11] videodev2.h: rename reserved2 to config_store in v4l2_buffer.

2014-09-21 Thread Hans Verkuil
From: Hans Verkuil When queuing buffers allow for passing the configuration store ID that should be associated with this buffer. Use the 'reserved2' field for this. Signed-off-by: Hans Verkuil --- drivers/media/usb/cpia2/cpia2_v4l.c | 2 +- drivers/media/v4l2-core/v4l2-compat-ioctl32

[RFC PATCH 00/11] Add configuration store support

2014-09-21 Thread Hans Verkuil
This patch series adds support for configuration stores to the control framework. This allows you to store control values for a particular configuration (up to VIDEO_MAX_FRAME configuration stores are currently supported). When you queue a new buffer you can supply the store ID and the driver will

[RFC PATCH 07/11] v4l2-ctrls: implement 'ignore after use' support.

2014-09-21 Thread Hans Verkuil
From: Hans Verkuil Sometimes you want to apply a value every time v4l2_ctrl_apply_store is called, and sometimes you want to apply that value only once. This adds support for that feature. Signed-off-by: Hans Verkuil --- drivers/media/v4l2-core/v4l2-ctrls.c | 75 --

[RFC PATCH 08/11] vivid: add test config store for the contrast control

2014-09-21 Thread Hans Verkuil
From: Hans Verkuil Signed-off-by: Hans Verkuil --- drivers/media/platform/vivid/vivid-ctrls.c | 4 drivers/media/platform/vivid/vivid-vid-cap.c | 1 + 2 files changed, 5 insertions(+) diff --git a/drivers/media/platform/vivid/vivid-ctrls.c b/drivers/media/platform/vivid/vivid-ctrls.c i

Re: Running Technisat DVB-S2 on ARM-NAS

2014-09-21 Thread Antti Palosaari
On 09/21/2014 05:26 PM, JPT wrote: Hi, I want to turn my Netgear ReadyNAS RN104 into a VDR. I already run a self made kernel 3.16.3) and plain debian on it. For hardware and software details see http://natisbad.org/NAS3/ I recently compiled those DVB modules into the kernel. And after a lot o

Running Technisat DVB-S2 on ARM-NAS

2014-09-21 Thread JPT
Hi, I want to turn my Netgear ReadyNAS RN104 into a VDR. I already run a self made kernel 3.16.3) and plain debian on it. For hardware and software details see http://natisbad.org/NAS3/ I recently compiled those DVB modules into the kernel. And after a lot of struggle to get a clean build, I succ

Re: [PATCH 3/4 v4] support for DVBSky dvb-s2 usb: add dvb-usb-v2driver for DVBSky dvb-s2 box, no ci support.

2014-09-21 Thread Antti Palosaari
Moikka Max, I have asked Mauro many times to handle these waiting patches, lastly yesterday. Mauro's ETA is tomorrow, Monday. If it does not happen tomorrow, I will pick those and send PULL request as those PULL request generally go higher priority than patches in patchwork :) There is also s

Re: Re: [PATCH 3/4 v4] support for DVBSky dvb-s2 usb: add dvb-usb-v2driver for DVBSky dvb-s2 box, no ci support.

2014-09-21 Thread Nibble Max
Hello Antti, Could you collect the following patches? It will make to support DVBSky DVB-S2 usb device. https://patchwork.linuxtv.org/patch/25313/ https://patchwork.linuxtv.org/patch/25202/ https://patchwork.linuxtv.org/patch/25201/ BR, Max >Looks OK >Reviewed-by: Antti Palosaari > >regards >

RFC: vb2: replace alloc_ctx by struct device * in vb2_queue

2014-09-21 Thread Hans Verkuil
Hi Marek, Pawel, Currently for dma_config (and the dma_sg code that I posted before) drivers have to allocate a alloc_ctx context, but in practice that just contains a device pointer. Is there any reason why we can't just change in struct vb2_queue: void*allo

[PATCH] v4l2-ioctl.c: fix inverted condition

2014-09-21 Thread Hans Verkuil
v4l_print_ext_controls() would print the 'size' if it was 0 and 'value' if size was non-zero, but it should have been the other way around. Signed-off-by: Hans Verkuil --- drivers/media/v4l2-core/v4l2-ioctl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/v4l2-

[PATCH 2/3] af9035: Add possibility to define which I2C adapter to use

2014-09-21 Thread Olli Salonen
Some I2C tuner drivers require that the I2C device of the tuner is added to the I2C adapter of the demodulator (Si2168+Si2157 for example). Add possibility to tell af9035_add_i2c_dev which I2C adapter should be used. Cc: cr...@iki.fi Signed-off-by: Olli Salonen --- drivers/media/usb/dvb-usb-v2

[PATCH 3/3] af9035: Add support for IT930x USB bridge

2014-09-21 Thread Olli Salonen
Add support for IT930x USB bridge and IT9303 reference design. It is a DVB-T/T2/C tuner with the following components: - IT9303 USB bridge - Si2168-B40 demodulator - Si2147-A30 tuner The IT9303 requires firmware that can be downloaded here: http://trsqr.net/olli/linux/firmwares/it930x/ The Si216

[PATCH 1/3] si2157: Add support for Si2147-A30

2014-09-21 Thread Olli Salonen
This patch adds support for Si2147-A30 tuner. Fairly trivial, no firmware needed for this tuner. However, command 14 00 02 07 01 00 seems to be mandatory. On Si2157 and Si2158 the value 0x0100 is the default value, so this patch does not impact the existing tuners/devices. On Si2147 the default is

[PATCH 0/3] IT930x USB DVB-T2/C tuner

2014-09-21 Thread Olli Salonen
This patch set adds support for IT930x reference design. It contains IT9303 USB bridge, Si2168-B40 demodulator and Si2147-A30 tuner. The patches should be applied on top of Antti's af9035 branch. http://git.linuxtv.org/cgit.cgi/anttip/media_tree.git/log/?h=af9035 Cc: cr...@iki.fi Olli Salonen (

Re: vb2_reqbufs() is not allowing more than VIDEO_MAX_FRAME

2014-09-21 Thread Hans Verkuil
On 07/21/2014 08:03 AM, Divneil Wadhawan wrote: > > Hi Hans, > >> This patch is all messed up and doesn't apply. >> >> Check your mailer settings: it clearly replaced hard tabs by a space. >> >> Can you repost? > > I tried to find out if I can change the mailer settings. Seems, that is the > pr

[PATCH] saa7134: add saa7134-go7007

2014-09-21 Thread Hans Verkuil
This patch adds support to saa7134 for 'WIS Voyager or compatible' PCI boards such as the Sensoray model 614 with which this patch was tested. It is a saa7134-based PCI board with a go7007 MPEG encoder. This was a patch when the go7007 was still in staging and was not applied when go7007 was moved

Re: [PATCHv2 1/3] vb2: fix VBI/poll regression

2014-09-21 Thread Laurent Pinchart
Hi Hans, On Sunday 21 September 2014 11:34:26 Hans Verkuil wrote: > On 09/21/2014 11:30 AM, Laurent Pinchart wrote: > > On Sunday 21 September 2014 11:00:02 Hans Verkuil wrote: > >> On 09/20/2014 09:26 PM, Laurent Pinchart wrote: > >>> On Saturday 20 September 2014 21:16:35 Hans Verkuil wrote: > >

[PATCH] saa7134: also capture the WSS signal for 50 Hz VBI capture

2014-09-21 Thread Hans Verkuil
The saa7134 driver missed capturing line 23 of the VBI area for the 50 Hz formats. Include that line in the VBI capture. Signed-off-by: Hans Verkuil Tested-by: Hans Verkuil --- drivers/media/pci/saa7134/saa7134-vbi.c | 2 +- drivers/media/pci/saa7134/saa7134-video.c | 2 +- 2 files changed, 2

[GIT PULL FOR v3.18] Fixes, add teletext to vivid

2014-09-21 Thread Hans Verkuil
Various 3.18 fixes and the teletext support for vivid. Regards, Hans The following changes since commit f5281fc81e9a0a3e80b78720c5ae2ed06da3bfae: [media] vpif: Fix compilation with allmodconfig (2014-09-09 18:08:08 -0300) are available in the git repository at: git://linuxtv.org/h

Re: [PATCHv2 1/3] vb2: fix VBI/poll regression

2014-09-21 Thread Hans Verkuil
On 09/21/2014 11:30 AM, Laurent Pinchart wrote: > Hi Hans, > > On Sunday 21 September 2014 11:00:02 Hans Verkuil wrote: >> On 09/20/2014 09:26 PM, Laurent Pinchart wrote: >>> On Saturday 20 September 2014 21:16:35 Hans Verkuil wrote: From: Hans Verkuil The recent conversion of saa7

Re: [PATCHv2 1/3] vb2: fix VBI/poll regression

2014-09-21 Thread Laurent Pinchart
Hi Hans, On Sunday 21 September 2014 11:00:02 Hans Verkuil wrote: > On 09/20/2014 09:26 PM, Laurent Pinchart wrote: > > On Saturday 20 September 2014 21:16:35 Hans Verkuil wrote: > >> From: Hans Verkuil > >> > >> The recent conversion of saa7134 to vb2 unconvered a poll() bug that > >> broke the

[GIT PULL FOR v3.17] Various bug/regression fixes for 3.17

2014-09-21 Thread Hans Verkuil
Hi Mauro, This collects my set of 3.17 patches, including the vb2 poll regression fix. I have also included the poll documentation fixes here, but if you prefer to add those to 3.18 instead, then that's fine by me. Besides this pull request for 3.17 I have two other pull requests for 3.17 as wel

Re: [PATCHv2 1/3] vb2: fix VBI/poll regression

2014-09-21 Thread Hans Verkuil
On 09/20/2014 09:26 PM, Laurent Pinchart wrote: > Hi Hans, > > Thank you for the patch. > > On Saturday 20 September 2014 21:16:35 Hans Verkuil wrote: >> From: Hans Verkuil >> >> The recent conversion of saa7134 to vb2 unconvered a poll() bug that >> broke the teletext applications alevt and mtt

RE: buffer delivery stops with cx23885

2014-09-21 Thread James Harper
> > > > And important for me, because if it IS related to the vb2 conversion then I > > need to know asap. > > > > Might take a few days to be completely sure. I've wondered if it's something > to do with the signal too (maybe some bug when signal error occurs?). So > many variables :( > I can r