[PATCH] media: platform: pxa_camera: convert to vb2

2016-03-09 Thread Robert Jarzmik
Convert pxa_camera from videobuf to videobuf2. As the soc_camera was already compatible with videobuf2, the port is quite straightforward. The only special port of this code is that if the vb2 to prepare is "too big" in terms of size for the new capture format, the pxa_camera will accept it

[PATCH 2/4] media: v4l2-mc add v4l_change_media_source() to invoke change_source

2016-03-09 Thread Shuah Khan
Add a common routine to invoke media device change_source handler. Signed-off-by: Shuah Khan --- drivers/media/v4l2-core/v4l2-mc.c | 14 ++ include/media/v4l2-mc.h | 20 +++- 2 files changed, 33 insertions(+), 1 deletion(-) diff

[PATCH 3/4] media: au0828 add media device change_source handler

2016-03-09 Thread Shuah Khan
Add media device change_source handler. Using the change_source handler, driver can disable current source and enable new one in one step when user selects a new input. Signed-off-by: Shuah Khan --- drivers/media/usb/au0828/au0828-core.c | 64

[PATCH 0/4] media add change_source handler support

2016-03-09 Thread Shuah Khan
This patch series: 1. Adds change_source function pointer to struct media_device. Using the change_source handler, driver can disable current source and enable new one in one step when user selects a new input. 2. Add a new common v4l interface to call change_source handler 3. Add

[PATCH 1/4] media: add change_source handler function pointer

2016-03-09 Thread Shuah Khan
Add change_source handler function pointer to struct media_device. Using the change_source handler, driver can disable current source and enable new one in one step when user selects a new input. Signed-off-by: Shuah Khan --- include/media/media-device.h | 18

Re: tw686x driver

2016-03-09 Thread Krzysztof Hałasa
Hans Verkuil writes: > Heck, if you prefer your driver can be added to staging first, then Ezequiel's > driver commit can directly refer to the staging driver as being derived from > it. Ok, I guess it's fair enough for me. Would you like me to send a patch with paths

cron job: media_tree daily build: OK

2016-03-09 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: Thu Mar 10 04:00:17 CET 2016 git branch: test git hash: de08b5a8be0df1eb7c796b0fe6b30cf1d03d14a6 gcc

Re: tw686x driver

2016-03-09 Thread Hans Verkuil
On 03/10/2016 08:16 AM, Krzysztof Hałasa wrote: > Hans Verkuil writes: > >> Heck, if you prefer your driver can be added to staging first, then >> Ezequiel's >> driver commit can directly refer to the staging driver as being derived from >> it. > > Ok, I guess it's fair

Re: [PATCH 4/4] media: au0828 change vidioc_s_input() to call v4l_change_media_source()

2016-03-09 Thread kbuild test robot
Hi Shuah, [auto build test ERROR on sailus-media/master] [cannot apply to v4.5-rc7 next-20160309] [if your patch is applied to the wrong git tree, please drop us a note to help improving the system] url: https://github.com/0day-ci/linux/commits/Shuah-Khan/media-add-change_source-handler

[PATCH 4/4] media: au0828 change vidioc_s_input() to call v4l_change_media_source()

2016-03-09 Thread Shuah Khan
Change vidioc_s_input() to call v4l_change_media_source() to disable current source and enable new source when user switches input. Signed-off-by: Shuah Khan --- drivers/media/usb/au0828/au0828-video.c | 8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) diff

Re: [PATCH v4 3/4] media: Add type field to struct media_entity

2016-03-09 Thread Sakari Ailus
Hi Mauro and Laurent, Mauro Carvalho Chehab wrote: > Em Fri, 4 Mar 2016 22:18:50 +0200 > Laurent Pinchart escreveu: > >> Code that processes media entities can require knowledge of the >> structure type that embeds a particular media entity instance in

Re: [PATCH v3 06/22] media: Media Controller enable/disable source handler API

2016-03-09 Thread Sakari Ailus
Hi Shuah, On Thu, Feb 11, 2016 at 04:41:22PM -0700, Shuah Khan wrote: > Add new fields to struct media_device to add enable_source, and > disable_source handlers, and source_priv to stash driver private > data that is used to run these handlers. The enable_source handler > finds source entity for

Re: i.mx6 camera interface (CSI) and mainline kernel

2016-03-09 Thread Hans Verkuil
On 03/10/2016 01:12 AM, Steve Longerbeam wrote: > On 03/09/2016 02:44 PM, Tim Harvey wrote: >> On Tue, Mar 8, 2016 at 6:06 PM, Steve Longerbeam >> wrote: >>> On 03/07/2016 08:19 AM, Tim Harvey wrote: >> >>> >>> Hi Tim, good to hear it works for you on the Ventana

[RFC PATCH 0/3] [media] tvp515p: Proposal for MC input connector support

2016-03-09 Thread Javier Martinez Canillas
Hello, I was waiting for the MC input connector support discussion to settle before attempting to propose another patch series for the tvp5150 video decoder but IIUC you are going to continue the discussion at ELC so I'm posting a series that I believe is aligned with the latest conversations.

Re: Kernel docs: muddying the waters a bit

2016-03-09 Thread Mauro Carvalho Chehab
Em Tue, 8 Mar 2016 12:39:21 -0300 Mauro Carvalho Chehab escreveu: > Pandoc failed to fully convert it, but at least it left all the texts, > with prevented rewriting it from scratch. This is the manual fix > I applied to it: > >

Re: i.mx6 camera interface (CSI) and mainline kernel

2016-03-09 Thread Tim Harvey
On Tue, Mar 8, 2016 at 6:06 PM, Steve Longerbeam wrote: > On 03/07/2016 08:19 AM, Tim Harvey wrote: > > > Hi Tim, good to hear it works for you on the Ventana boards. > > I've just pushed some more commits to the mx6-media-staging branch that > get the

[PATCH 1/2] smipcie: add support for TechnoTrend S2-4200 Twin

2016-03-09 Thread Olli Salonen
Add support for TechnoTrend TT-budget S2-4200 Twin DVB-S2 tuner. The device seems to be rather similar to DVBSky S952 V3. This is a PCIe card with 2 tuners. SMI PCIe bridge is used and the card has two Montage M88RS6000 demod/tuners. The M88RS6000 demod/tuner package needs firmware. You can

[PATCH 2/2] smipcie: MAC address printout formatting

2016-03-09 Thread Olli Salonen
Modify the printout for MAC address to be more vendor agnostic. Print also the port number. Signed-off-by: Olli Salonen --- drivers/media/pci/smipcie/smipcie-main.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

[RFC PATCH 2/3] [media] tvp5150: Add input connectors DT bindings

2016-03-09 Thread Javier Martinez Canillas
The tvp5150 and tvp5151 decoders support different video input source connections to their AIP1A and AIP1B pins. Either two Composite input signals or a S-Video (with separate Y and C signals) are supported. The possible configurations are as follows: - Analog Composite signal connected to AIP1A

[RFC PATCH 3/3] [media] tvp5150: Replace connector support according to DT binding

2016-03-09 Thread Javier Martinez Canillas
The tvp5150 and tvp5151 decoders support different video input source connections to their AIP1A and AIP1B pins. Either two Composite input signals or a S-Video signal are supported. The MC input connector support was added before, but the Device Tree binding was found to be inadecuate so it was

[RFC PATCH 1/3] [media] v4l2-mc.h: Add a S-Video C input PAD to demod enum

2016-03-09 Thread Javier Martinez Canillas
The enum demod_pad_index list the PADs that an analog TV demod has but in some decoders the S-Video Y (luminance) and C (chrominance) signals are carried by different connectors. So a single DEMOD_PAD_IF_INPUT is not enough and an additional PAD is needed in the case of S-Video for the additional

Re: [PATCH] dmabuf: allow exporter to define customs ioctls

2016-03-09 Thread Daniel Vetter
On Wed, Mar 9, 2016 at 10:40 AM, Benjamin Gaignard wrote: > dmabuf have just accept one ioctl for cache management but I think > that some exporter may need also need custom iotcl. > > For example I'm working since a while on a way to secure buffer (SMAF) > [1] which

DVBv5 Tools: VDR support seems to be broken

2016-03-09 Thread Markus Heiser
Hi, I tested DVBv5 tools, creating vdr channel lists. My first attemp was to convert a dvbv5 channel list: - # file: test_convert_in.conf # # converted with: dvb-format-convert -I DVBV5 -O VDR test_convert_in.conf test_convert_out.conf # [Das Erste HD]

Re: [RFC PATCH v0] Add tw5864 driver

2016-03-09 Thread Andrey Utkin
Hi Hans! Some improvements took place on the driver, including cleaner v4l2-compliance tests passing. But there's a single test failure I don't understand. In the code of v4l2-compliance, it seems like an API call CREATE_BUFS is supposed to fail with EINVAL. But in case of my driver, which

Re: i.mx6 camera interface (CSI) and mainline kernel

2016-03-09 Thread Steve Longerbeam
On 03/09/2016 02:44 PM, Tim Harvey wrote: > On Tue, Mar 8, 2016 at 6:06 PM, Steve Longerbeam > wrote: >> On 03/07/2016 08:19 AM, Tim Harvey wrote: > >> >> Hi Tim, good to hear it works for you on the Ventana boards. >> >> I've just pushed some more commits to the

Re: [PATCH 2/2] smipcie: MAC address printout formatting

2016-03-09 Thread Nibble Max
Reviewed-by: Max Nibble On 2016-03-10 06:39:27, Olli Salonen wrote: >Modify the printout for MAC address to be more vendor agnostic. >Print also the port number. > >Signed-off-by: Olli Salonen >--- >

Re: [PATCH 1/2] smipcie: add support for TechnoTrend S2-4200 Twin

2016-03-09 Thread Nibble Max
Reviewed-by: Max Nibble On 2016-03-10 06:39:26, Olli Salonen wrote: >Add support for TechnoTrend TT-budget S2-4200 Twin DVB-S2 tuner. The >device seems to be rather similar to DVBSky S952 V3. This is a PCIe >card with 2 tuners. SMI PCIe bridge is used

[PATCH] media: au0828 disable tuner to demod link in au0828_media_device_register()

2016-03-09 Thread Shuah Khan
Disable tuner to demod link in au0828_media_device_register(). This step should be done after dvb graph is created. Signed-off-by: Shuah Khan --- drivers/media/usb/au0828/au0828-core.c | 26 ++ drivers/media/usb/au0828/au0828-video.c | 20

[PATCH 5/7] [media] gspca: fix a v4l2-compliance failure about buffer timestamp

2016-03-09 Thread Antonio Ospite
v4l2-compliance fails with this message: fail: v4l2-test-buffers.cpp(250): \ timestamp != V4L2_BUF_FLAG_TIMESTAMP_MONOTONIC && \ timestamp != V4L2_BUF_FLAG_TIMESTAMP_COPY ... test VIDIOC_REQBUFS/CREATE_BUFS/QUERYBUF: FAIL When setting the frame time, gspca uses

[PATCH 1/7] [media] gspca: ov534/topro: use a define for the default framerate

2016-03-09 Thread Antonio Ospite
When writing the change in commit dcc7fdbec53a ("[media] gspca: ov534/topro: prevent a division by 0") I used magic numbers for the default framerate to minimize the code footprint to make it easier to backport the patch to the stable trees. However it's better if the default framerate has its

[PATCH 7/7] [media] gspca: fix a v4l2-compliance failure during read()

2016-03-09 Thread Antonio Ospite
v4l2-compliance fails with this message: fail: v4l2-test-buffers.cpp(512): Expected EBUSY, got 22 test VIDIOC_REQBUFS/CREATE_BUFS/QUERYBUF: FAIL Looking at the v4l2-compliance code reveals that this failure is about the read() callback. In gspca, dev_read() is calling vidioc_dqbuf() which

[PATCH 0/7] gspca: pass all v4l2-compliance tests + minor fixes

2016-03-09 Thread Antonio Ospite
Hi, after applying this patchset gspca passes all v4l2-compliance tests, at least it does with a PS3 Eye. - Patch 1 removes some magic numbers in subdrivers. - Patch 2 is a correctness fix, but it does not bring any functional changes. - Patch 3 is a readability improvement by itself, but

[PATCH 2/7] [media] gspca: fix setting frame interval type in vidioc_enum_frameintervals()

2016-03-09 Thread Antonio Ospite
Set the frame _interval_ type to V4L2_FRMIVAL_TYPE_DISCRETE instead of using V4L2_FRMSIZE_TYPE_DISCRETE which is meant for frame _size_. The old and new values happen to be the same so there is no functional change. Signed-off-by: Antonio Ospite --- drivers/media/usb/gspca/gspca.c

[PATCH 6/7] [media] gspca: fix a v4l2-compliance failure during VIDIOC_REQBUFS

2016-03-09 Thread Antonio Ospite
When calling VIDIOC_REQBUFS v4l2-compliance fails with this message: fail: v4l2-test-buffers.cpp(476): q.reqbufs(node, 1) test VIDIOC_REQBUFS/CREATE_BUFS/QUERYBUF: FAIL By looking at the v4l2-compliance code the failure happens when trying to request V4L2_MEMORY_USERPTR buffers without

[PATCH 3/7] [media] gspca: rename wxh_to_mode() to wxh_to_nearest_mode()

2016-03-09 Thread Antonio Ospite
The name wxh_to_nearest_mode() reflects better what the function does. Signed-off-by: Antonio Ospite --- drivers/media/usb/gspca/gspca.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/media/usb/gspca/gspca.c b/drivers/media/usb/gspca/gspca.c

[PATCH 4/7] [media] gspca: fix a v4l2-compliance failure about VIDIOC_ENUM_FRAMEINTERVALS

2016-03-09 Thread Antonio Ospite
According to v4l2-compliance VIDIOC_ENUM_FRAMEINTERVALS should fail for unsupported frame sizes, but gspca is too tolerant and tries to find the frame intervals for the frame size nearest to the requested one. This makes v4l2-compliance fail with this message: fail: v4l2-test-formats.cpp(123):

Re: [PATCH 6/7] [media] gspca: fix a v4l2-compliance failure during VIDIOC_REQBUFS

2016-03-09 Thread Hans Verkuil
On 03/09/16 17:03, Antonio Ospite wrote: > When calling VIDIOC_REQBUFS v4l2-compliance fails with this message: > > fail: v4l2-test-buffers.cpp(476): q.reqbufs(node, 1) > test VIDIOC_REQBUFS/CREATE_BUFS/QUERYBUF: FAIL > > By looking at the v4l2-compliance code the failure happens when trying

Re: Kernel docs: muddying the waters a bit

2016-03-09 Thread Jani Nikula
On Wed, 09 Mar 2016, Dan Allen wrote: > On Tue, Mar 8, 2016 at 6:58 AM, Jani Nikula wrote: > >> I need to look into this again. Is there a specific option or directive >> to produce split output for includes? When I tried this, the result was >> just

Re: [PATCH] dmabuf: allow exporter to define customs ioctls

2016-03-09 Thread Christian König
Am 09.03.2016 um 10:03 schrieb Daniel Vetter: On Wed, Mar 9, 2016 at 9:54 AM, Benjamin Gaignard wrote: In addition of the already existing operations allow exporter to use it own custom ioctls. Signed-off-by: Benjamin Gaignard

[PATCH] dmabuf: allow exporter to define customs ioctls

2016-03-09 Thread Benjamin Gaignard
In addition of the already existing operations allow exporter to use it own custom ioctls. Signed-off-by: Benjamin Gaignard --- drivers/dma-buf/dma-buf.c | 3 +++ include/linux/dma-buf.h | 5 + 2 files changed, 8 insertions(+) diff --git

Re: [PATCH] dmabuf: allow exporter to define customs ioctls

2016-03-09 Thread Benjamin Gaignard
dmabuf have just accept one ioctl for cache management but I think that some exporter may need also need custom iotcl. For example I'm working since a while on a way to secure buffer (SMAF) [1] which is a central allocator + 2 ioctl for set/get secure status of the buffer. This not depend on one

Re: [PATCH] media: i2c/adp1653: fix check of devm_gpiod_get() error code

2016-03-09 Thread Sakari Ailus
Hi Vladimir, On Mon, Mar 07, 2016 at 08:39:32PM +0200, Vladimir Zapolskiy wrote: > The devm_gpiod_get() function returns either a valid pointer to > struct gpio_desc or ERR_PTR() error value, check for NULL is bogus. > > Signed-off-by: Vladimir Zapolskiy Thanks! Applied to my

Re: [PATCH] dmabuf: allow exporter to define customs ioctls

2016-03-09 Thread Daniel Vetter
On Wed, Mar 9, 2016 at 9:54 AM, Benjamin Gaignard wrote: > In addition of the already existing operations allow exporter > to use it own custom ioctls. > > Signed-off-by: Benjamin Gaignard First reaction: No way ever! More seriously,