Re: linux-next: Tree for Aug 18 (drivers/media/i2c/tc358743.c)

2015-08-18 Thread Hans Verkuil
On 08/18/2015 05:26 PM, Randy Dunlap wrote: > On 08/18/15 04:40, Stephen Rothwell wrote: >> Hi all, >> >> Changes since 20150817: >> > > on i386: > > when CONFIG_MEDIA_CONTROLLER is not enabled: > > ../drivers/media/i2c/tc358743.c: In function 'tc358743_probe': > ../drivers/media/i2c/tc358743.c:

Re: [PATCH 2/2] [media] tc358743: make reset gpio optional

2015-08-18 Thread Linus Walleij
On Tue, Aug 18, 2015 at 10:31 AM, Uwe Kleine-König wrote: > Commit 256148246852 ("[media] tc358743: support probe from device tree") > specified in the device tree binding documentation that the reset gpio > is optional. Make the implementation match accordingly. > > Signed-off-by: Uwe Kleine-Kön

Re: [PATCH 1/2] [media] tc358743: set direction of reset gpio using devm_gpiod_get

2015-08-18 Thread Linus Walleij
On Tue, Aug 18, 2015 at 10:31 AM, Uwe Kleine-König wrote: > Commit 256148246852 ("[media] tc358743: support probe from device tree") > failed to explicitly set the direction of the reset gpio. Use the > optional flag of devm_gpiod_get to make up leeway. > > This is also necessary because the flag

Re: [PATCHv8 07/15] cec: add HDMI CEC framework

2015-08-18 Thread Dmitry Torokhov
On Tue, Aug 18, 2015 at 11:00:20AM +0100, Russell King - ARM Linux wrote: > On Tue, Aug 18, 2015 at 10:26:32AM +0200, Hans Verkuil wrote: > > + /* Part 2: Initialize and register the character device */ > > + cdev_init(&cecdev->cdev, &cec_devnode_fops); > > + cecdev->cdev.owner = owner; > > +

cron job: media_tree daily build: WARNINGS

2015-08-18 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 Aug 19 04:00:19 CEST 2015 git branch: test git hash: 3a6b0605c73d1d695f6d4e49289deaa3fa3e73ee gcc versi

Re: [GIT PULL for v4.2] media fixes

2015-08-18 Thread Linus Torvalds
On Tue, Aug 18, 2015 at 1:18 PM, Mauro Carvalho Chehab wrote: > > - Revert the IR encode patches, as the API is not mature enough. > So, better to postpone the changes to a latter Kernel; What the hell have you done with the commit messages? The first line is completely corrupt

[GIT PULL for v4.2] media fixes

2015-08-18 Thread Mauro Carvalho Chehab
Hi Linus, Please pull from: git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-media tags/media/v4.2-2 For some fixes: - a regression fix at the videobuf2 core driver; - Fix error handling at mantis probing code; - Revert the IR encode patches, as the API is no

[PATCH RFC v5 8/8] [media] media: rename the function that create pad links

2015-08-18 Thread Mauro Carvalho Chehab
Now that a link can be either between two different graph objects, we'll need to add more functions to create links. So, rename the existing one that create links only between two pads as media_create_pad_link(). No functional changes. This patch was created via this shell script: for i i

Re: [PATCH RFC v5 0/8] MC preparation patches

2015-08-18 Thread Mauro Carvalho Chehab
Em Tue, 18 Aug 2015 17:04:13 -0300 Mauro Carvalho Chehab escreveu: Hmm... I forgot to remove the "RFC" prefix. For me, this patch series is OK to be merged at the media-controller topic branch. > Those are the initial patches from my RFCv3 series of MC changes, > plus one additional renaming pa

[PATCH RFC v5 7/8] [media] media: add a debug message to warn about gobj creation/removal

2015-08-18 Thread Mauro Carvalho Chehab
It helps to check if the media controller is doing the right thing with the object creation and removal. No extra code/data will be produced if DEBUG or CONFIG_DYNAMIC_DEBUG is not enabled. Signed-off-by: Mauro Carvalho Chehab diff --git a/drivers/media/media-entity.c b/drivers/media/media-enti

[PATCH RFC v5 0/8] MC preparation patches

2015-08-18 Thread Mauro Carvalho Chehab
Those are the initial patches from my RFCv3 series of MC changes, plus one additional renaming patch and two debug ones. They address the comments received on PATCH v4 series. The first 5 patches on this series ensures that all existing object types (entities, pads and links) will have an unique I

[PATCH RFC v5 6/8] [media] media: add messages when media device gets (un)registered

2015-08-18 Thread Mauro Carvalho Chehab
We can only free the media device after being sure that no graph object is used. In order to help tracking it, let's add debug messages that will print when the media controller gets registered or unregistered. Signed-off-by: Mauro Carvalho Chehab diff --git a/drivers/media/media-device.c b/dri

[PATCH RFC v5 3/8] [media] media: use media_gobj inside entities

2015-08-18 Thread Mauro Carvalho Chehab
As entities are graph elements, let's embed media_gobj on it. That ensures an unique ID for entities that can be global along the entire media controller. For now, we'll keep the already existing entity ID. Such field need to be dropped at some point, but for now, let's not do this, to avoid needi

[PATCH RFC v5 1/8] [media] media: create a macro to get entity ID

2015-08-18 Thread Mauro Carvalho Chehab
Instead of accessing directly entity.id, let's create a macro, as this field will be moved into a common struct later on. Signed-off-by: Mauro Carvalho Chehab Acked-by: Hans Verkuil Signed-off-by: Mauro Carvalho Chehab diff --git a/drivers/media/media-device.c b/drivers/media/media-device.c in

[PATCH RFC v5 4/8] [media] media: use media_gobj inside pads

2015-08-18 Thread Mauro Carvalho Chehab
PADs also need unique object IDs that won't conflict with the entity object IDs. The pad objects are currently created via media_entity_init() and, once created, never change. While this will likely change in the future in order to support dynamic changes, for now we'll keep PADs as arrays and in

[PATCH RFC v5 5/8] [media] media: use media_gobj inside links

2015-08-18 Thread Mauro Carvalho Chehab
Just like entities and pads, links also need to have unique Object IDs along a given media controller. So, let's add a media_gobj inside it and initialize the object then a new link is created. Signed-off-by: Mauro Carvalho Chehab diff --git a/drivers/media/media-device.c b/drivers/media/media-

[PATCH RFC v5 2/8] [media] media: add a common struct to be embed on media graph objects

2015-08-18 Thread Mauro Carvalho Chehab
Due to the MC API proposed changes, we'll need to have an unique object ID for all graph objects, and have some shared fields that will be common on all media graph objects. Right now, the only common object is the object ID, but other fields will be added later on. Signed-off-by: Mauro Carvalho

Re: linux-next: Tree for Aug 18 (drivers/media/i2c/tc358743.c)

2015-08-18 Thread Randy Dunlap
On 08/18/15 04:40, Stephen Rothwell wrote: > Hi all, > > Changes since 20150817: > on i386: when CONFIG_MEDIA_CONTROLLER is not enabled: ../drivers/media/i2c/tc358743.c: In function 'tc358743_probe': ../drivers/media/i2c/tc358743.c:1890:29: error: 'struct v4l2_subdev' has no member named 'ent

[linux-next:master 7256/9424] drivers/media/pci/netup_unidvb/netup_unidvb_core.c:191:41: sparse: incorrect type in argument 2 (different address spaces)

2015-08-18 Thread kbuild test robot
tree: git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master head: ebe96538f4817c765690e28fcd33dc1da1504d26 commit: 52b1eaf4c59a3bbd07afbb4ab4f43418a807d02e [7256/9424] [media] netup_unidvb: NetUP Universal DVB-S/S2/T/T2/C PCI-E card driver reproduce: # apt-get install spar

[linux-next:master 7177/9424] drivers/media/i2c/s5c73m3/s5c73m3-core.c:170:39: sparse: incorrect type in argument 1 (different base types)

2015-08-18 Thread kbuild test robot
tree: git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master head: ebe96538f4817c765690e28fcd33dc1da1504d26 commit: 5cc596c66fe7c725ec049ae2093e7242034c97d6 [7177/9424] [media] i2c: Make all i2c devices visible if COMPILE_TEST=y reproduce: # apt-get install sparse git che

Re: [PATCH] s5p-mfc: fix state check from encoder queue_setup

2015-08-18 Thread Andrzej Hajda
On 05/13/2015 09:25 AM, Seung-Woo Kim wrote: > MFCINST_GOT_INST state is set to encoder context with set_format > only for catpure buffer. In queue_setup of encoder called during > reqbufs, it is checked MFCINST_GOT_INST state for both capture > and output buffer. So this patch fixes to encoder to

[PATCH] [media] c8sectpfe: Use %pad to print 'dma_addr_t'

2015-08-18 Thread Fabio Estevam
Use %pad to print 'dma_addr_t' in order to fix the following build warning: drivers/media/platform/sti/c8sectpfe/c8sectpfe-core.c:588:2: warning: format '%x' expects argument of type 'unsigned int', but argument 5 has type 'dma_addr_t' [-Wformat=] Reported-by: Olof's autobuilder Signed-off-by:

[renesas-drivers:master 4533/5645] drivers/media/platform/rcar_jpu.c:1147:51: sparse: incorrect type in assignment (different base types)

2015-08-18 Thread kbuild test robot
tree: git://git.kernel.org/pub/scm/linux/kernel/git/geert/renesas-drivers.git master head: d7ba9d1e0b037c24ee7f278fdcf9c44f80f3dbe7 commit: 2c42cdbaec56a9565a2717b450506150c9c55103 [4533/5645] [media] V4L2: platform: Add Renesas R-Car JPEG codec driver reproduce: # apt-get install sparse

Re: VIMC: API proposal, configuring the topology through user space

2015-08-18 Thread Mauro Carvalho Chehab
Em Tue, 18 Aug 2015 09:35:14 +0300 Laurent Pinchart escreveu: > Hi Hans, > > On Friday 14 August 2015 12:54:44 Hans Verkuil wrote: > > On 08/13/2015 07:29 PM, Laurent Pinchart wrote: > > > On Tuesday 11 August 2015 17:07:04 Helen Fornazier wrote: > > >> On Tue, Aug 11, 2015 at 6:36 AM, Mauro Car

Re: [PATCHv8 07/15] cec: add HDMI CEC framework

2015-08-18 Thread Russell King - ARM Linux
On Tue, Aug 18, 2015 at 10:26:32AM +0200, Hans Verkuil wrote: > + /* Part 2: Initialize and register the character device */ > + cdev_init(&cecdev->cdev, &cec_devnode_fops); > + cecdev->cdev.owner = owner; > + > + ret = cdev_add(&cecdev->cdev, MKDEV(MAJOR(cec_dev_t), cecdev->minor),

Re: [PATCHv2 0/4] cec-ctl/compliance: new CEC utilities

2015-08-18 Thread Hans Verkuil
Oops, this cover letter is incomplete, here is the full cover letter: This patch series adds two new utilities to the v4l-utils git repository (http://git.linuxtv.org/cgit.cgi/v4l-utils.git/). It assumes that the new CEC framework (v8) is available in the kernel: http://www.mail-archive.com/linux

[PATCHv2 0/4] cec-ctl/compliance: new CEC utilities

2015-08-18 Thread Hans Verkuil
This patch series adds two new utilities to the v4l-utils git repository (http://git.linuxtv.org/cgit.cgi/v4l-utils.git/). It assumes that the new CEC framework available in the kernel: http://www.mail-archive.com/linux-media@vger.kernel.org/msg90085.html The first patch adds the new cec headers

[PATCHv8 06/15] rc: Add HDMI CEC protocol handling

2015-08-18 Thread Hans Verkuil
From: Kamil Debski Add handling of remote control events coming from the HDMI CEC bus. This patch includes a new keymap that maps values found in the CEC messages to the keys pressed and released. Also, a new protocol has been added to the core. Signed-off-by: Kamil Debski Signed-off-by: Hans V

[PATCHv2 4/4] cec-ctl: CEC control utility

2015-08-18 Thread Hans Verkuil
Generic CEC utility that can be used to send/receive/monitor CEC messages. Signed-off-by: Hans Verkuil --- configure.ac |1 + utils/Makefile.am |1 + utils/cec-ctl/Makefile.am |8 + utils/cec-ctl/cec-ctl.cpp | 1296 + u

[PATCHv8 04/15] input.h: add BUS_CEC type

2015-08-18 Thread Hans Verkuil
Inputs can come in over the HDMI CEC bus, so add a new type for this. Signed-off-by: Hans Verkuil Acked-by: Dmitry Torokhov --- include/uapi/linux/input.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/uapi/linux/input.h b/include/uapi/linux/input.h index 731417c..a32bff1 100644 --

[PATCHv8 02/15] dts: exynos4: add node for the HDMI CEC device

2015-08-18 Thread Hans Verkuil
From: Kamil Debski This patch adds HDMI CEC node specific to the Exynos4210/4x12 SoC series. Signed-off-by: Kamil Debski Signed-off-by: Hans Verkuil Acked-by: Krzysztof Kozlowski --- arch/arm/boot/dts/exynos4.dtsi | 12 1 file changed, 12 insertions(+) diff --git a/arch/arm/boo

[PATCHv8 14/15] cec: s5p-cec: Add s5p-cec driver

2015-08-18 Thread Hans Verkuil
From: Kamil Debski Add CEC interface driver present in the Samsung Exynos range of SoCs. The following files were based on work by SangPil Moon: - exynos_hdmi_cec.h - exynos_hdmi_cecctl.c Signed-off-by: Kamil Debski Signed-off-by: Hans Verkuil --- .../devicetree/bindings/media/s5p-cec.txt

[PATCHv8 01/15] dts: exynos4*: add HDMI CEC pin definition to pinctrl

2015-08-18 Thread Hans Verkuil
From: Kamil Debski Add pinctrl nodes for the HDMI CEC device to the Exynos4210 and Exynos4x12 SoCs. These are required by the HDMI CEC device. Signed-off-by: Kamil Debski Signed-off-by: Hans Verkuil Acked-by: Krzysztof Kozlowski --- arch/arm/boot/dts/exynos4210-pinctrl.dtsi | 7 +++ arch

[PATCHv2 3/4] cec-compliance: add new CEC compliance utility

2015-08-18 Thread Hans Verkuil
This utility will attempt to test whether the CEC protocol was implemented correctly. Signed-off-by: Hans Verkuil --- configure.ac| 1 + utils/Makefile.am | 1 + utils/cec-compliance/Makefile.am| 3 + utils/cec-compliance/cec-compli

[PATCHv8 00/15] HDMI CEC framework

2015-08-18 Thread Hans Verkuil
Hi all, The eighth version of this patchset addresses comments on the mailing list and many changes due to the work I did on the cec-compliance and cec-ctl utilities and due to stress-testing that uncovered a serious core design problem that required a partial rewrite. As far as I am concerned th

[PATCHv8 09/15] DocBook/media: add CEC documentation

2015-08-18 Thread Hans Verkuil
From: Hans Verkuil Add DocBook documentation for the CEC API. Signed-off-by: Hans Verkuil [k.deb...@samsung.com: add documentation for passthrough mode] [k.deb...@samsung.com: minor fixes and change of reserved field sizes] Signed-off-by: Kamil Debski Signed-off-by: Hans Verkuil --- Document

[PATCHv8 15/15] cobalt: add cec support

2015-08-18 Thread Hans Verkuil
Add CEC support to the cobalt driver. Signed-off-by: Hans Verkuil --- drivers/media/pci/cobalt/cobalt-driver.c | 50 +-- drivers/media/pci/cobalt/cobalt-driver.h | 2 + drivers/media/pci/cobalt/cobalt-irq.c| 3 + drivers/media/pci/cobalt/cobalt-v4l2.c | 107 +++

[PATCHv2 1/4] Makefile.am: copy cec headers with make sync-with-kernel

2015-08-18 Thread Hans Verkuil
Copy the new cec headers. Signed-off-by: Hans Verkuil --- Makefile.am | 4 1 file changed, 4 insertions(+) diff --git a/Makefile.am b/Makefile.am index 1a61592..b8c450d 100644 --- a/Makefile.am +++ b/Makefile.am @@ -21,6 +21,8 @@ sync-with-kernel: ! -f $(KERNEL_DIR)/usr/inclu

[PATCHv8 05/15] HID: add HDMI CEC specific keycodes

2015-08-18 Thread Hans Verkuil
From: Kamil Debski Add HDMI CEC specific keycodes to the keycodes definition. Signed-off-by: Kamil Debski Signed-off-by: Hans Verkuil --- include/uapi/linux/input.h | 28 1 file changed, 28 insertions(+) diff --git a/include/uapi/linux/input.h b/include/uapi/linu

[PATCHv8 10/15] v4l2-subdev: add HDMI CEC ops

2015-08-18 Thread Hans Verkuil
Add CEC callbacks to the v4l2_subdev_video_ops. These are the low-level CEC ops that subdevs that support CEC have to implement. Signed-off-by: Hans Verkuil [k.deb...@samsung.com: Merged changes from CEC Updates commit by Hans Verkuil] Signed-off-by: Kamil Debski Signed-off-by: Hans Verkuil ---

[PATCHv8 12/15] cec: adv7842: add cec support

2015-08-18 Thread Hans Verkuil
Add CEC support to the adv7842 driver. Signed-off-by: Hans Verkuil --- drivers/media/i2c/adv7842.c | 250 +--- 1 file changed, 233 insertions(+), 17 deletions(-) diff --git a/drivers/media/i2c/adv7842.c b/drivers/media/i2c/adv7842.c index b7269b8..73c67c4

[PATCHv8 13/15] cec: adv7511: add cec support.

2015-08-18 Thread Hans Verkuil
Add CEC support to the adv7511 driver. Signed-off-by: Hans Verkuil [k.deb...@samsung.com: Merged changes from CEC Updates commit by Hans Verkuil] Signed-off-by: Kamil Debski Signed-off-by: Hans Verkuil --- drivers/media/i2c/adv7511.c | 359 +++- include/

[PATCHv2 2/4] sync-with-kernel

2015-08-18 Thread Hans Verkuil
Signed-off-by: Hans Verkuil --- contrib/freebsd/include/linux/input.h | 29 + contrib/freebsd/include/linux/v4l2-controls.h |4 + include/linux/cec-funcs.h | 1771 + include/linux/cec.h | 781 +++ include

[PATCHv8 07/15] cec: add HDMI CEC framework

2015-08-18 Thread Hans Verkuil
The added HDMI CEC framework provides a generic kernel interface for HDMI CEC devices. Signed-off-by: Hans Verkuil [k.deb...@samsung.com: Merged CEC Updates commit by Hans Verkuil] [k.deb...@samsung.com: Merged Update author commit by Hans Verkuil] [k.deb...@samsung.com: change kthread handling w

[PATCHv8 03/15] dts: exynos4412-odroid*: enable the HDMI CEC device

2015-08-18 Thread Hans Verkuil
From: Kamil Debski Add a dts node entry and enable the HDMI CEC device present in the Exynos4 family of SoCs. Signed-off-by: Kamil Debski Signed-off-by: Hans Verkuil Acked-by: Krzysztof Kozlowski --- arch/arm/boot/dts/exynos4210-universal_c210.dts | 4 1 file changed, 4 insertions(+) d

[PATCHv8 11/15] cec: adv7604: add cec support.

2015-08-18 Thread Hans Verkuil
From: Hans Verkuil Add CEC support to the adv7604 driver. Signed-off-by: Hans Verkuil [k.deb...@samsung.com: Merged changes from CEC Updates commit by Hans Verkuil] [k.deb...@samsung.com: add missing methods cec/io_write_and_or] [k.deb...@samsung.com: change adv7604 to adv76xx in added function

[PATCHv8 08/15] cec.txt: add CEC framework documentation

2015-08-18 Thread Hans Verkuil
From: Hans Verkuil Document the new HDMI CEC framework. Signed-off-by: Hans Verkuil [k.deb...@samsung.com: add DocBook documentation by Hans Verkuil, with Signed-off-by: Kamil Debski Signed-off-by: Hans Verkuil --- Documentation/cec.txt | 166 +

[PATCH 1/2] [media] tc358743: set direction of reset gpio using devm_gpiod_get

2015-08-18 Thread Uwe Kleine-König
Commit 256148246852 ("[media] tc358743: support probe from device tree") failed to explicitly set the direction of the reset gpio. Use the optional flag of devm_gpiod_get to make up leeway. This is also necessary because the flag parameter will become mandatory soon. Signed-off-by: Uwe Kleine-Kön

[PATCH 2/2] [media] tc358743: make reset gpio optional

2015-08-18 Thread Uwe Kleine-König
Commit 256148246852 ("[media] tc358743: support probe from device tree") specified in the device tree binding documentation that the reset gpio is optional. Make the implementation match accordingly. Signed-off-by: Uwe Kleine-König --- drivers/media/i2c/tc358743.c | 6 -- 1 file changed, 4 i