Re: si2157 driver

2016-06-28 Thread Oleh Kravchenko
Hi Olli, thanks for fast reply. It's possible to improve driver to support analog mode? I try to find datasheet for this chip, but looks like only short version is a public. On 29.06.16 07:42, Olli Salonen wrote: > Hi Oleg, > > Correct, only digital TV is supported currently by the driver. >

Re: si2157 driver

2016-06-28 Thread Olli Salonen
Hi Oleg, Correct, only digital TV is supported currently by the driver. Cheers, -olli On 28 June 2016 at 23:22, Oleh Kravchenko wrote: > Hello linux media developers! > > I try add support for usb hybrid tuner, it based on: > CX23102-112, Si2158, Si2168 > > I updated cx231xx-cards.c with valid

cron job: media_tree daily build: ERRORS

2016-06-28 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: Wed Jun 29 04:00:25 CEST 2016 git branch: test git hash: d81295d1bed850335f9f4ccb6b1aa4f6a123d4f0 gcc versi

[PATCH 2/2] tw686x: Support VIDIOC_{S,G}_PARM ioctls

2016-06-28 Thread Ezequiel Garcia
Now that the frame rate can be properly set, this commit adds support for S_PARM and G_PARM. Signed-off-by: Ezequiel Garcia --- drivers/media/pci/tw686x/tw686x-video.c | 46 ++--- 1 file changed, 43 insertions(+), 3 deletions(-) diff --git a/drivers/media/pci/tw686x/

[PATCH 1/2] tw686x: use a formula instead of two tables for div

2016-06-28 Thread Ezequiel Garcia
From: Mauro Carvalho Chehab Instead of using two tables to estimate the temporal decimation factor, use a formula. This allows to get the closest fps, which sounds better than the current tables. Also, the current code doesn't store the real framerate. This patch fixes the above issues. Signed

Re: [PATCH v3] tw686x: use a formula instead of two tables for div

2016-06-28 Thread Ezequiel Garcia
On 27 June 2016 at 14:38, Mauro Carvalho Chehab wrote: > Em Mon, 27 Jun 2016 12:45:38 -0300 > Ezequiel Garcia escreveu: > >> Hi Mauro, >> >> Thanks a lot for the patch. >> >> On 27 April 2016 at 12:27, Mauro Carvalho Chehab >> wrote: >> > Instead of using two tables to estimate the temporal deci

Re: [PATCH v2 1/2] Add OV5647 device tree documentation

2016-06-28 Thread Rob Herring
On Tue, Jun 28, 2016 at 12:59:58PM +0100, Ramiro Oliveira wrote: > Signed-off-by: Ramiro Oliveira > --- > .../devicetree/bindings/media/i2c/ov5647.txt | 19 > +++ > 1 file changed, 19 insertions(+) > create mode 100644 Documentation/devicetree/bindings/media/i2c/ov5647.

si2157 driver

2016-06-28 Thread Oleh Kravchenko
Hello linux media developers! I try add support for usb hybrid tuner, it based on: CX23102-112, Si2158, Si2168 I updated cx231xx-cards.c with valid ids, but I don't have idea how to use Si2158. It is not listed in tuner-types.c Why si2157.c is absent in tuner-types.c? Or at the current state si2

Re: [PATCH 00/38] i.MX5/6 Video Capture

2016-06-28 Thread Hans Verkuil
On 06/28/2016 08:54 PM, Tim Harvey wrote: > On Tue, Jun 14, 2016 at 3:48 PM, Steve Longerbeam > wrote: >> Tested on imx6q SabreAuto with ADV7180, and imx6q SabreSD with >> mipi-csi2 OV5640. There is device-tree support also for imx6qdl >> SabreLite, but that is not tested. Also, this driver shoul

Tuning problem with Pinnacle PCTV 2000e

2016-06-28 Thread Guillaume Membré
Hi, I asked this question on the mythtv mailling but without success :'( I know that my question may not be appropriate, forgive me if I'm at the wrong place. I had a working mythtv 0.27.5 installation with a Pinnacle PCTV 2000e (dual dvb-t usb card) and I recently replaced my motherboard. I had

Re: [PATCH v2 0/2] Media Device Allocator API

2016-06-28 Thread Shuah Khan
On 05/24/2016 05:39 PM, Shuah Khan wrote: > Media Device Allocator API to allows multiple drivers share a media device. > Using this API, drivers can allocate a media device with the shared struct > device as the key. Once the media device is allocated by a driver, other > drivers can get a referen

Re: [PATCH] media: s5p-mfc fix memory leak in s5p_mfc_remove()

2016-06-28 Thread Shuah Khan
On 06/28/2016 09:55 AM, Shuah Khan wrote: > On 06/28/2016 07:39 AM, Shuah Khan wrote: >> On 06/28/2016 03:57 AM, Kamil Debski wrote: >>> HI Shuah, >>> >>> Which branch do you base your patches on? >>> >>> I have trouble applying this path >>> (https://patchwork.linuxtv.org/patch/34577/) and >>> "s

[PATCH REBASE 2/3] media: s5p-mfc fix memory leak in s5p_mfc_remove()

2016-06-28 Thread Shuah Khan
s5p_mfc_remove() fails to release encoder and decoder video devices. Signed-off-by: Shuah Khan Reviewed-by: Javier Martinez Canillas --- drivers/media/platform/s5p-mfc/s5p_mfc.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/media/platform/s5p-mfc/s5p_mfc.c b/drivers/media/platf

[PATCH v3 4/4] ARM64: dts: meson-gxbb: Enable the the IR decoder on supported boards

2016-06-28 Thread Martin Blumenstingl
Enable the Infrared Remote Controller on boards which have an Infrared receiver. Signed-off-by: Martin Blumenstingl --- arch/arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dts | 4 arch/arm64/boot/dts/amlogic/meson-gxbb-p20x.dtsi | 4 arch/arm64/boot/dts/amlogic/meson-gxbb-vega-s95.dt

[PATCH v3 0/4] Add Meson 8b / GXBB support to the IR driver

2016-06-28 Thread Martin Blumenstingl
Newer Amlogic platforms (Meson 8b and GXBB) use a slightly different register layout for their Infrared Remoete Controller. The decoder mode is now configured in another register. Without the changes to the meson-ir driver we are simply getting incorrect "durations" reported from the hardware (beca

[PATCH v3 3/4] ARM64: meson-gxbb: Add Infrared Remote Controller decoder

2016-06-28 Thread Martin Blumenstingl
From: Neil Armstrong This adds the Infrared Remote Controller node so boards with an IR remote can simply enable it. Signed-off-by: Neil Armstrong Signed-off-by: Martin Blumenstingl --- arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi | 7 +++ 1 file changed, 7 insertions(+) diff --git a/arch

[PATCH v3 1/4] dt-bindings: media: meson-ir: Add Meson8b and GXBB compatible strings

2016-06-28 Thread Martin Blumenstingl
From: Neil Armstrong New bindings are needed as the register layout on the newer platforms is slightly different compared to Meson6b. Signed-off-by: Neil Armstrong Signed-off-by: Martin Blumenstingl --- Documentation/devicetree/bindings/media/meson-ir.txt | 5 - 1 file changed, 4 insertio

[PATCH v3 2/4] media: rc: meson-ir: Add support for newer versions of the IR decoder

2016-06-28 Thread Martin Blumenstingl
From: Neil Armstrong Newer SoCs (Meson 8b and GXBB) are using REG2 (offset 0x20) instead of REG1 to configure the decoder mode. This makes it necessary to introduce new bindings so the driver knows which register has to be used. Signed-off-by: Neil Armstrong Signed-off-by: Martin Blumenstingl

[PATCH REBASE 1/3] media: s5p-mfc fix video device release double release in probe error path

2016-06-28 Thread Shuah Khan
Fix Decoder and encoder video device double release in probe error path. video_device_release(dev->vfd_dec) get called twice if decoder register fails. Also, video_device_release(dev->vfd_enc) get called twice if encoder register fails. Signed-off-by: Shuah Khan --- drivers/media/platform/s5p-mf

[PATCH REBASE 0/3] Rebased on linuxtv/master

2016-06-28 Thread Shuah Khan
Rebased on linuxtv/master latest with the top commit: commit 80aa26593e3eb48f16c4222aa27ff40806f57c45 Author: Ulrich Hecht Date: Wed May 11 11:02:53 2016 -0300 Shuah Khan (3): media: s5p-mfc fix video device release double release in probe error path media: s5p-mfc fix memory leak in s5p_

[PATCH REBASE 3/3] media: s5p-mfc fix null pointer deference in clk_core_enable()

2016-06-28 Thread Shuah Khan
Fix null pointer deference in clk_core_enable() when driver unbind is run when there is an application has an active pipeline playing. s5p_mfc_release() gets called after s5p_mfc_final_pm() disables and does clk_put() and s5p_mfc_release() attempts to enable clock and runs into null pointer defere

Re: [PATCH 00/38] i.MX5/6 Video Capture

2016-06-28 Thread Tim Harvey
On Tue, Jun 14, 2016 at 3:48 PM, Steve Longerbeam wrote: > Tested on imx6q SabreAuto with ADV7180, and imx6q SabreSD with > mipi-csi2 OV5640. There is device-tree support also for imx6qdl > SabreLite, but that is not tested. Also, this driver should > theoretically work on i.MX5 targets, but that

Re: [PATCH] media: s5p-mfc fix memory leak in s5p_mfc_remove()

2016-06-28 Thread Shuah Khan
On 06/28/2016 07:39 AM, Shuah Khan wrote: > On 06/28/2016 03:57 AM, Kamil Debski wrote: >> HI Shuah, >> >> Which branch do you base your patches on? >> >> I have trouble applying this path >> (https://patchwork.linuxtv.org/patch/34577/) and >> "s5p-mfc fix null pointer deference in clk_core_enable

[GIT PULL FOR v4.8] Two more CEC patches

2016-06-28 Thread Hans Verkuil
This adds the missing rc-cec keymap module and a bug fix patch. I originally thought the rc-cec module was already merged in the cec topic branch, but I later discovered that it wasn't. Regards, Hans The following changes since commit c7169ad5616229b87cabf886bc5f9cbd1fc35a5f: [media

[GIT PULL FOR v4.8] Two more CEC patches

2016-06-28 Thread Hans Verkuil
(apologies if this is a duplicate, the first attempt to mail this seemed to fail). This adds the missing rc-cec keymap module and a bug fix patch. I originally thought the rc-cec module was already merged in the cec topic branch, but I later discovered that it wasn't. Regards, Hans T

[PATCH] cec-adap: on reply, restore the tx_status value from the,transmit

2016-06-28 Thread Hans Verkuil
When a reply to an earlier transmit is received, the tx_status of that transmit needs to be restored. Otherwise the status that the transmit was successful would be lost. Signed-off-by: Hans Verkuil --- Note: this patch sits on top of the cec topic branch. --- drivers/staging/media/cec/cec-adap.

[PATCH 2/2] [media] s5p_cec: get rid of an unused var

2016-06-28 Thread Mauro Carvalho Chehab
drivers/staging/media/s5p-cec/s5p_cec.c: In function 's5p_cec_adap_enable': drivers/staging/media/s5p-cec/s5p_cec.c:42:6: warning: variable 'ret' set but not used [-Wunused-but-set-variable] int ret; ^~~ Signed-off-by: Mauro Carvalho Chehab --- drivers/staging/media/s5p-cec/s5p_cec.c |

[PATCH 1/2] [media] move s5p-cec to staging

2016-06-28 Thread Mauro Carvalho Chehab
As the CEC core is currently at staging, it doesn't make any sense to put a dependent driver outside staging. So, move it also to staging. Signed-off-by: Mauro Carvalho Chehab --- MAINTAINERS| 2 +- drivers/media/platform/Kconfig

Re: [GIT PULL FOR v4.8 v6] Add HDMI CEC framework

2016-06-28 Thread Mauro Carvalho Chehab
Em Sat, 25 Jun 2016 15:47:41 +0200 Hans Verkuil escreveu: > Hi Mauro, > > Here is the pull request for the HDMI CEC framework. The code of this pull > request is identical to the v19 patch series and is against the media_tree cec > topic branch. > ... > create mode 100644 drivers/media/platf

[PATCH] v4l2-tpg: ignore V4L2_DV_RGB_RANGE setting for YUV formats

2016-06-28 Thread Hans Verkuil
The V4L2_DV_RGB_RANGE_* settings are, as the name says, for RGB formats only. So they should be ignored for non-RGB formats. Signed-off-by: Hans Verkuil --- diff --git a/drivers/media/common/v4l2-tpg/v4l2-tpg-core.c b/drivers/media/common/v4l2-tpg/v4l2-tpg-core.c index cf1dadd..3ec3ceb 100644 --

Re: [PATCH] media: s5p-mfc fix memory leak in s5p_mfc_remove()

2016-06-28 Thread Shuah Khan
On 06/28/2016 03:57 AM, Kamil Debski wrote: > HI Shuah, > > Which branch do you base your patches on? > > I have trouble applying this path > (https://patchwork.linuxtv.org/patch/34577/) and > "s5p-mfc fix null pointer deference in clk_core_enable()" > (https://patchwork.linuxtv.org/patch/34751/

[PATCH] v4l: ioctl: Clear the v4l2_pix_format_mplane reserved field

2016-06-28 Thread Laurent Pinchart
The S_FMT and TRY_FMT handlers in multiplane mode attempt at clearing the reserved fields of the v4l2_format structure after the pix_mp member. However, the reserved fields are inside pix_mp, not after it. Signed-off-by: Laurent Pinchart --- drivers/media/v4l2-core/v4l2-ioctl.c | 8 1 f

Re: [PATCH] [media] rcar-csi2: add Renesas R-Car MIPI CSI-2 driver

2016-06-28 Thread Geert Uytterhoeven
Hi Niklas, On Mon, Jun 27, 2016 at 7:45 PM, Niklas Söderlund wrote: > --- /dev/null > +++ b/Documentation/devicetree/bindings/media/rcar-csi2.txt > @@ -0,0 +1,79 @@ > +Renesas R-Car MIPI CSI-2 driver (rcar-csi2) > +--- > + > +The rcar-csi2 device provides M

Submit media entity without media device

2016-06-28 Thread Ramiro Oliveira
Hi all, We at Synopsys have a media device driver and in that media device we have a media entity for our CSI-2 Host. At the moment we aren't ready to submit the entire media device, so I was wondering if it was possible to submit a media entity driver separately, without the rest of the architec

[PATCH v2 2/2] Add support for Omnivision OV5647

2016-06-28 Thread Ramiro Oliveira
Signed-off-by: Ramiro Oliveira --- MAINTAINERS| 7 + drivers/media/i2c/Kconfig | 12 + drivers/media/i2c/Makefile | 1 + drivers/media/i2c/ov5647.c | 891 + 4 files changed, 911 insertions(+) create mode 100644 drivers/media/i2c/o

[PATCH v2 0/2] OV5647 sensor driver

2016-06-28 Thread Ramiro Oliveira
Hello, This patch adds support for the Omnivision OV5647 sensor. At the moment it only supports 640x480 in Raw 8. Ramiro Oliveira (2): Add OV5647 device tree documentation Add support for Omnivision OV5647 .../devicetree/bindings/media/i2c/ov5647.txt | 19 + MAINTAINERS

Re: [GIT PULL FOR v4.8] rcar-vin patches

2016-06-28 Thread Hans Verkuil
Try again, this time Cc-ed to Ulrich's correct email address. Sorry Ulrich, Hans On 06/28/16 13:52, Hans Verkuil wrote: > Updated these patches. > > Ulrich, sorry, the compile error was my fault: I added these patches in the > wrong order. > > Regards, > > Hans > > The followin

Re: [PATCH v2 1/2] solo6x10: Set FRAME_BUF_SIZE to 200KB

2016-06-28 Thread Hans Verkuil
On 06/28/16 13:48, Andrey Utkin wrote: > On Mon, Jun 27, 2016 at 11:12:42AM +0200, Hans Verkuil wrote: >> Andrey, >> >> Since you are the original author, can you give me your Signed-off-by line? > > No, as increasing buffer size by few kilobytes doesn't change anything. I've > increased it from 2

Re: [PATCH v4 3/8] media: rcar_vin: Use correct pad number in try_fmt

2016-06-28 Thread Geert Uytterhoeven
Hi Mauro, On Tue, Jun 28, 2016 at 1:32 PM, Mauro Carvalho Chehab wrote: > Em Wed, 11 May 2016 16:02:51 +0200 > Ulrich Hecht escreveu: > >> Fix rcar_vin_try_fmt's use of an inappropriate pad number when calling >> the subdev set_fmt function - for the ADV7612, IDs should be non-zero. >> >> Signed

[PATCH v2 1/2] Add OV5647 device tree documentation

2016-06-28 Thread Ramiro Oliveira
Signed-off-by: Ramiro Oliveira --- .../devicetree/bindings/media/i2c/ov5647.txt | 19 +++ 1 file changed, 19 insertions(+) create mode 100644 Documentation/devicetree/bindings/media/i2c/ov5647.txt diff --git a/Documentation/devicetree/bindings/media/i2c/ov5647.txt b/Do

[GIT PULL FOR v4.8] rcar-vin patches

2016-06-28 Thread Hans Verkuil
Updated these patches. Ulrich, sorry, the compile error was my fault: I added these patches in the wrong order. Regards, Hans The following changes since commit 904aef0f9f6deff94223c0ce93eb598c47dd3aad: [media] v4l2-ctrl.h: fix comments (2016-06-28 08:07:04 -0300) are available in t

Re: [PATCH v2 1/2] solo6x10: Set FRAME_BUF_SIZE to 200KB

2016-06-28 Thread Andrey Utkin
On Mon, Jun 27, 2016 at 11:12:42AM +0200, Hans Verkuil wrote: > Andrey, > > Since you are the original author, can you give me your Signed-off-by line? No, as increasing buffer size by few kilobytes doesn't change anything. I've increased it from 200 to 204, then found new occurances of the issue

Re: [GIT PULL FOR v4.8] Various fixes

2016-06-28 Thread Mauro Carvalho Chehab
Em Mon, 27 Jun 2016 14:31:02 +0200 Hans Verkuil escreveu: > Note for Ulrich's patches: these are prerequisites for two other patch > series (one from Ulrich for HDMI support and one from Niklas for Gen3 > support). It doesn't hurt to add these now, and it will simplify future > development. > >

Re: [PATCH v4 3/8] media: rcar_vin: Use correct pad number in try_fmt

2016-06-28 Thread Mauro Carvalho Chehab
Em Wed, 11 May 2016 16:02:51 +0200 Ulrich Hecht escreveu: > Fix rcar_vin_try_fmt's use of an inappropriate pad number when calling > the subdev set_fmt function - for the ADV7612, IDs should be non-zero. > > Signed-off-by: William Towle > Reviewed-by: Rob Taylor > Acked-by: Hans Verkuil > [ul

RE: [PATCH] media: s5p-mfc fix memory leak in s5p_mfc_remove()

2016-06-28 Thread Kamil Debski
HI Shuah, Which branch do you base your patches on? I have trouble applying this path (https://patchwork.linuxtv.org/patch/34577/) and "s5p-mfc fix null pointer deference in clk_core_enable()" (https://patchwork.linuxtv.org/patch/34751/) onto current linuxtv/master. The top commit of linuxtv/m