cron job: media_tree daily build: ERRORS

2017-12-13 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 Dec 14 05:00:15 CET 2017 media-tree git hash:b32a2b42f76cdfd06b4b58a1ddf987ba329ae34e media_build

[PATCH] pvrusb2: correctly return V4L2_PIX_FMT_MPEG in enum_fmt

2017-12-13 Thread Hans Verkuil
The pvrusb2 code appears to have a some old workaround code for xawtv that causes a WARN() due to an unrecognized pixelformat 0 in v4l2_ioctl.c. Since all other MPEG drivers fill this in correctly, it is a safe assumption that this particular problem no longer exists. While I'm at it, clean up

Re: [PATCH v4 3/5] media: i2c: Add TDA1997x HDMI receiver driver

2017-12-13 Thread Tim Harvey
On Mon, Dec 4, 2017 at 9:30 AM, Tim Harvey wrote: > On Mon, Dec 4, 2017 at 4:50 AM, Hans Verkuil wrote: >> Hi Tim, >> >> Found a few more small issues. After that's fixed and you have the Ack for >> the >> bindings this can be merged I think. > > Hans,

Re: [PATCH 3/5] media: i2c: Add TDA1997x HDMI receiver driver

2017-12-13 Thread Tim Harvey
On Tue, Dec 12, 2017 at 4:18 AM, Hans Verkuil wrote: > Hi Tim, > > Sorry for the delay, I needed to find some time to think about this. > > On 11/16/17 05:30, Rob Herring wrote: >> On Wed, Nov 15, 2017 at 10:31:14AM -0800, Tim Harvey wrote: >>> On Wed, Nov 15, 2017 at 7:52 AM,

Re: Kernel Oopses from v4l_enum_fmt

2017-12-13 Thread Oleksandr Ostrenko
On 13.12.17 23:04, Nicolas Dufresne wrote: Le mercredi 13 décembre 2017 à 22:33 +0100, Oleksandr Ostrenko a écrit : Dear all, There is an issue in v4l_enum_fmt leading to kernel panic under certain circumstance. It happens while I try to capture video from my TV tuner. When I connect this USB

Re: Kernel Oopses from v4l_enum_fmt

2017-12-13 Thread Nicolas Dufresne
Le mercredi 13 décembre 2017 à 22:33 +0100, Oleksandr Ostrenko a écrit : > Dear all, > > There is an issue in v4l_enum_fmt leading to kernel panic under > certain > circumstance. It happens while I try to capture video from my TV > tuner. > > When I connect this USB TV tuner (WinTV HVR-1900) it

Kernel Oopses from v4l_enum_fmt

2017-12-13 Thread Oleksandr Ostrenko
Dear all, There is an issue in v4l_enum_fmt leading to kernel panic under certain circumstance. It happens while I try to capture video from my TV tuner. When I connect this USB TV tuner (WinTV HVR-1900) it gets recognized just fine. However, whenever I try to capture a video from the

Re: [PATCH v10 4/4] [media] platform: Add Synopsys DesignWare HDMI RX Controller Driver

2017-12-13 Thread Hans Verkuil
On 13/12/17 15:00, Jose Abreu wrote: > Hi Hans, > > On 13-12-2017 10:00, Hans Verkuil wrote: >> On 12/12/17 17:02, Jose Abreu wrote: >>> > +static int dw_hdmi_s_routing(struct v4l2_subdev *sd, u32 input, u32 > output, > + u32 config) > +{ > + struct dw_hdmi_dev

Re: [PATCH 2/2] [media] ddbridge: don't break on single/last port attach failure

2017-12-13 Thread Daniel Scheller
On Wed, 13 Dec 2017 17:44:37 -0200 Mauro Carvalho Chehab wrote: > Em Wed, 13 Dec 2017 18:40:52 +0100 > Daniel Scheller escreveu: > > > On Wed, 13 Dec 2017 13:26:02 -0200 > > Mauro Carvalho Chehab wrote: > > > > > Em Wed, 6

Re: [PATCH 2/2] [media] ddbridge: don't break on single/last port attach failure

2017-12-13 Thread Mauro Carvalho Chehab
Em Wed, 13 Dec 2017 18:40:52 +0100 Daniel Scheller escreveu: > On Wed, 13 Dec 2017 13:26:02 -0200 > Mauro Carvalho Chehab wrote: > > > Em Wed, 6 Dec 2017 18:59:15 +0100 > > Daniel Scheller escreveu: > > > > > From:

Re: [PATCH v4 00/12] [dt-bindings] [media] Add document file and driver for Sony CXD2880 DVB-T2/T tuner + demodulator

2017-12-13 Thread Mauro Carvalho Chehab
Em Fri, 13 Oct 2017 14:46:35 +0900 escreveu: > From: Yasunari Takiguchi > > Hi, > > This is the patch series (version 4) of Sony CXD2880 DVB-T2/T tuner + > demodulator driver.The driver supports DVB-API and interfaces through > SPI.

Re: [PATCH v4 07/12] [media] cxd2880: Add top level of the driver

2017-12-13 Thread Mauro Carvalho Chehab
Em Fri, 13 Oct 2017 15:09:34 +0900 escreveu: > From: Yasunari Takiguchi > > This provides the main dvb frontend operation functions > for the Sony CXD2880 DVB-T2/T tuner + demodulator driver. > > Signed-off-by: Yasunari Takiguchi

Re: [PATCH v4 06/12] [media] cxd2880: Add integration layer for the driver

2017-12-13 Thread Mauro Carvalho Chehab
Em Fri, 13 Oct 2017 15:08:34 +0900 escreveu: > From: Yasunari Takiguchi > > These functions monitor the driver and watch for task completion. > This is part of the Sony CXD2880 DVB-T2/T tuner + demodulator driver. If I understand well,

Re: [PATCH v4 05/12] [media] cxd2880: Add tuner part of the driver

2017-12-13 Thread Mauro Carvalho Chehab
Em Fri, 13 Oct 2017 15:07:25 +0900 escreveu: > From: Yasunari Takiguchi > > This part of the driver has the main routines to handle > the tuner and demodulator functionality. The tnrdmd_mon.* files > have monitor functions for the

[PATCH 5/5] v4l2: async: Add debug output to v4l2-async module

2017-12-13 Thread Jacopo Mondi
The v4l2-async module operations are quite complex to follow, due to the asynchronous nature of subdevices and notifiers registration and matching procedures. In order to help with debugging of failed or erroneous matching between a subdevice and the notifier collected async_subdevice it gets

[PATCH 3/5] include: v4l2_async: Add 'owner' field to notifier

2017-12-13 Thread Jacopo Mondi
Notifiers can be registered as root notifiers (identified by a 'struct v4l2_device *') or subdevice notifiers (identified by a 'struct v4l2_subdev *'). In order to identify a notifier no matter if it is root or not, add a 'struct fwnode_handle *owner' field, whose name can be printed out for debug

[PATCH 4/5] v4l2: async: Postpone subdev_notifier registration

2017-12-13 Thread Jacopo Mondi
Currently, subdevice notifiers are tested against all available subdevices as soon as they get registered. It often happens anyway that the subdevice they are connected to is not yet initialized, as it usually gets registered later in drivers' code. This makes debug of v4l2_async particularly

[PATCH 0/5] Add debug output to v4l2-async

2017-12-13 Thread Jacopo Mondi
Hello Sakari, while testing rcar-vin setup on top of your RFC (included in the series) that moves the framework to perform endpoint matching, I realized how hard is to follow what happens with asynchronous notifiers, sub-notifiers and sub-devices. In order to better understand what happens and

[PATCH 2/5] device property: Add fwnode_get_name() operation

2017-12-13 Thread Jacopo Mondi
Add operation to retrieve the device name from a fwnode handle. Signed-off-by: Jacopo Mondi --- drivers/acpi/property.c | 6 ++ drivers/base/property.c | 12 drivers/of/property.c| 6 ++ include/linux/fwnode.h | 2 ++

[PATCH 1/5] v4l: async: Use endpoint node, not device node, for fwnode match

2017-12-13 Thread Jacopo Mondi
From: Sakari Ailus V4L2 async framework can use both device's fwnode and endpoints's fwnode for matching the async sub-device with the sub-device. In order to proceed moving towards endpoint matching assign the endpoint to the async sub-device. As most async

Re: [PATCH v4 03/12] [media] cxd2880: Add common files for the driver

2017-12-13 Thread Mauro Carvalho Chehab
Em Fri, 13 Oct 2017 15:02:59 +0900 escreveu: > From: Yasunari Takiguchi > > These are common files for the driver for the > Sony CXD2880 DVB-T2/T tuner + demodulator. > These contains helper functions for the driver. > > Signed-off-by:

Re: [PATCH v4 02/12] [media] cxd2880-spi: Add support for CXD2880 SPI interface

2017-12-13 Thread Mauro Carvalho Chehab
Em Fri, 13 Oct 2017 14:59:28 +0900 escreveu: > From: Yasunari Takiguchi > > This is the SPI adapter part of the driver for the > Sony CXD2880 DVB-T2/T tuner + demodulator. > > Signed-off-by: Yasunari Takiguchi

Re: [PATCH 2/2] [media] ddbridge: don't break on single/last port attach failure

2017-12-13 Thread Daniel Scheller
On Wed, 13 Dec 2017 13:26:02 -0200 Mauro Carvalho Chehab wrote: > Em Wed, 6 Dec 2017 18:59:15 +0100 > Daniel Scheller escreveu: > > > From: Daniel Scheller > > > > As all error handling improved quite a bit, don't stop

[PATCH] media: ov9650: support VIDIOC_DBG_G/S_REGISTER ioctls

2017-12-13 Thread Akinobu Mita
This adds support VIDIOC_DBG_G/S_REGISTER ioctls. There are many device control registers contained in the OV9650. So this helps debugging the lower level issues by getting and setting the registers. Cc: Sylwester Nawrocki Cc: Sakari Ailus

Re: [PATCH 1/2 v6] uvcvideo: send a control event when a Control Change interrupt arrives

2017-12-13 Thread Guennadi Liakhovetski
Sorry, forgot to mention a change from the previous version: now autoupdate Control Change events are also delivered. Thanks Guennadi On Wed, 13 Dec 2017, Guennadi Liakhovetski wrote: > UVC defines a method of handling asynchronous controls, which sends a > USB packet over the interrupt pipe.

[ragnatech:media-tree] BUILD SUCCESS 330dada5957e3ca0c8811b14c45e3ac42c694651

2017-12-13 Thread kbuild test robot
tree/branch: git://git.ragnatech.se/linux media-tree branch HEAD: 330dada5957e3ca0c8811b14c45e3ac42c694651 media: dvb_frontend: fix return error code elapsed time: 72m configs tested: 622 The following configs have been built successfully. More configs may be tested in the coming days.

Re: [PATCH v3 03/12] media: rkisp1: Add user space ABI definitions

2017-12-13 Thread Hans Verkuil
On 06/12/17 12:19, Jacob Chen wrote: > From: Jeffy Chen > > Add the header for userspace > > Signed-off-by: Jeffy Chen > Signed-off-by: Jacob Chen > --- > include/uapi/linux/rkisp1-config.h | 785 >

[PATCH 2/2 v6] uvcvideo: handle control pipe protocol STALLs

2017-12-13 Thread Guennadi Liakhovetski
When a command ends up in a STALL on the control pipe, use the Request Error Code control to provide a more precise error information to the user. Signed-off-by: Guennadi Liakhovetski --- drivers/media/usb/uvc/uvc_video.c | 59

[PATCH 1/2 v6] uvcvideo: send a control event when a Control Change interrupt arrives

2017-12-13 Thread Guennadi Liakhovetski
UVC defines a method of handling asynchronous controls, which sends a USB packet over the interrupt pipe. This patch implements support for such packets by sending a control event to the user. Since this can involve USB traffic and, therefore, scheduling, this has to be done in a work queue.

[PATCH 0/2 v6] uvcvideo: asynchronous controls

2017-12-13 Thread Guennadi Liakhovetski
This is an update of the two patches, adding asynchronous control support to the uvcvideo driver. If a control is sent, while the camera is still processing an earlier control, it will generate a protocol STALL condition on the control pipe. Thanks Guennadi Guennadi Liakhovetski (2): uvcvideo:

Re: [PATCH 2/2] [media] ddbridge: don't break on single/last port attach failure

2017-12-13 Thread Mauro Carvalho Chehab
Em Wed, 6 Dec 2017 18:59:15 +0100 Daniel Scheller escreveu: > From: Daniel Scheller > > As all error handling improved quite a bit, don't stop attaching frontends > if one of them failed, since - if other tuner modules are connected to > the PCIe

Re: [PATCH 1/2] media: dt-bindings: coda: Add compatible for CodaHx4 on i.MX51

2017-12-13 Thread Philipp Zabel
Hi Baruch, On Wed, 2017-12-13 at 16:21 +0200, Baruch Siach wrote: > Hi Philipp, > > On Wed, Dec 13, 2017 at 03:09:17PM +0100, Philipp Zabel wrote: > > Add a compatible for the CodaHx4 VPU used on i.MX51. > > > > Signed-off-by: Philipp Zabel > > --- > >

Re: [PATCH 0/3] Add support compat in dvb_frontend.c

2017-12-13 Thread Mauro Carvalho Chehab
Em Sat, 2 Dec 2017 05:50:47 -0200 Mauro Carvalho Chehab escreveu: > Hi Jaedon, > > Em Fri, 1 Dec 2017 21:31:27 +0900 > Jaedon Shin escreveu: > > > This patch series supports compat ioctl for 32-bit user space applications > > in 64-bit system. > >

Re: [PATCH 1/2] media: dt-bindings: coda: Add compatible for CodaHx4 on i.MX51

2017-12-13 Thread Baruch Siach
Hi Philipp, On Wed, Dec 13, 2017 at 03:09:17PM +0100, Philipp Zabel wrote: > Add a compatible for the CodaHx4 VPU used on i.MX51. > > Signed-off-by: Philipp Zabel > --- > Documentation/devicetree/bindings/media/coda.txt | 1 + > 1 file changed, 1 insertion(+) > > diff

[PATCH 1/2] media: dt-bindings: coda: Add compatible for CodaHx4 on i.MX51

2017-12-13 Thread Philipp Zabel
Add a compatible for the CodaHx4 VPU used on i.MX51. Signed-off-by: Philipp Zabel --- Documentation/devicetree/bindings/media/coda.txt | 1 + 1 file changed, 1 insertion(+) diff --git a/Documentation/devicetree/bindings/media/coda.txt

[PATCH 2/2] media: coda: Add i.MX51 (CodaHx4) support

2017-12-13 Thread Philipp Zabel
Add support for the CodaHx4 VPU used on i.MX51. Decoding h.264, MPEG-4, and MPEG-2 video works, as well as encoding h.264. MPEG-4 encoding is not enabled, it currently produces visual artifacts. Signed-off-by: Philipp Zabel --- drivers/media/platform/coda/coda-bit.c

[BUG] cx88: a possible sleep-in-atomic bug in snd_cx88_switch_put

2017-12-13 Thread Jia-Ju Bai
The driver may sleep under a spinlock. The function call path is: snd_cx88_switch_put (acquire the spinlock) v4l2_ctrl_find mutex_lock --> may sleep I do not find a good way to fix it, so I only report. This possible bug is found by my static analysis tool (DSAC) and checked by my code

Re: [PATCH v10 4/4] [media] platform: Add Synopsys DesignWare HDMI RX Controller Driver

2017-12-13 Thread Jose Abreu
Hi Hans, On 13-12-2017 10:00, Hans Verkuil wrote: > On 12/12/17 17:02, Jose Abreu wrote: >> +static int dw_hdmi_s_routing(struct v4l2_subdev *sd, u32 input, u32 output, + u32 config) +{ + struct dw_hdmi_dev *dw_dev = to_dw_dev(sd); + + if

Re: [bug report] media: lirc: improve locking

2017-12-13 Thread Sean Young
On Wed, Dec 13, 2017 at 01:07:31PM +0300, Dan Carpenter wrote: > Hello Sean Young, > > The patch 131fd7fc3c01: "media: lirc: improve locking" from Nov 4, > 2017, leads to the following static checker warning: > > drivers/media/rc/lirc_dev.c:373 ir_lirc_transmit_ir() > error: 'txbuf'

Re: [PATCH/RFC] not use a DiSEqC switch

2017-12-13 Thread Mauro Carvalho Chehab
Em Mon, 27 Nov 2017 17:26:07 -0200 Mauro Carvalho Chehab escreveu: > Em Fri, 24 Nov 2017 10:52:04 +0200 > Maksym Veremeyenko escreveu: > > > Hi, > > > > there is a code in function *dvbsat_diseqc_set_input*: > > > > [...] > > /* Negative

Re: [PATCH] dvb-sat: do the best to tune if DiSEqC is disabled

2017-12-13 Thread Rafaël Carré
On 13/12/2017 13:58, Mauro Carvalho Chehab wrote: > If sat_number is not filled (e.g. it is -1), the dvb-sat > disables DiSEqC. However, currently, it also breaks support > for non-bandstacking LNBf. > > Change the logic to fix it. There is a drawback on this > approach, though: usually, on

Re: [PATCH v1 07/10] v4l: i2c: Copy ov772x soc_camera sensor driver

2017-12-13 Thread Philippe Ombredanne
Jacopo, On Wed, Nov 15, 2017 at 11:56 AM, Jacopo Mondi wrote: > Copy the soc_camera based driver in v4l2 sensor driver directory. > This commit just copies the original file without modifying it. > > Signed-off-by: Jacopo Mondi > ---

[PATCH] dvb-sat: do the best to tune if DiSEqC is disabled

2017-12-13 Thread Mauro Carvalho Chehab
If sat_number is not filled (e.g. it is -1), the dvb-sat disables DiSEqC. However, currently, it also breaks support for non-bandstacking LNBf. Change the logic to fix it. There is a drawback on this approach, though: usually, on bandstacking arrangements, only one device needs to feed power to

Re: [GIT PULL for v4.15-rc3] media fixes

2017-12-13 Thread Mauro Carvalho Chehab
Em Wed, 13 Dec 2017 13:27:47 +0100 Geert Uytterhoeven escreveu: > Hi Mauro, > > On Wed, Dec 13, 2017 at 12:53 PM, Mauro Carvalho Chehab > wrote: > > Em Wed, 13 Dec 2017 10:03:56 +0100 > > Geert Uytterhoeven escreveu: > >>

Re: [GIT PULL for v4.15-rc3] media fixes

2017-12-13 Thread Geert Uytterhoeven
Hi Mauro, On Wed, Dec 13, 2017 at 12:53 PM, Mauro Carvalho Chehab wrote: > Em Wed, 13 Dec 2017 10:03:56 +0100 > Geert Uytterhoeven escreveu: >> On Mon, Dec 11, 2017 at 12:12 PM, Mauro Carvalho Chehab >> wrote: >> > Without

Re: [PATCH v2] media: ABS macro parameter parenthesization

2017-12-13 Thread Mauro Carvalho Chehab
Em Fri, 17 Nov 2017 09:55:44 -0500 dgopst...@nyu.edu escreveu: > From: Dan Gopstein > > Two definitions of the ABS (absolute value) macro fail to parenthesize > their parameter properly. This can lead to a bad expansion for > low-precedence expression arguments. Add parens to

Re: [PATCH v1 10/10] media: i2c: tw9910: Remove soc_camera dependencies

2017-12-13 Thread Hans Verkuil
On 15/11/17 11:56, Jacopo Mondi wrote: > Remove soc_camera framework dependencies from tw9910 sensor driver. > - Handle clock directly > - Register async subdevice > - Add platform specific enable/disable functions > - Adjust build system > > This commit does not remove the original soc_camera

Re: [PATCH v1 07/10] v4l: i2c: Copy ov772x soc_camera sensor driver

2017-12-13 Thread Hans Verkuil
On 15/11/17 11:56, Jacopo Mondi wrote: > Copy the soc_camera based driver in v4l2 sensor driver directory. > This commit just copies the original file without modifying it. > > Signed-off-by: Jacopo Mondi > --- > drivers/media/i2c/ov772x.c | 1124 >

Re: [PATCH v1 03/10] v4l: platform: Add Renesas CEU driver

2017-12-13 Thread Hans Verkuil
On 15/11/17 11:55, Jacopo Mondi wrote: > Add driver for Renesas Capture Engine Unit (CEU). > > The CEU interface supports capturing 'data' (YUV422) and 'images' > (NV[12|21|16|61]). > > This driver aims to replace the soc_camera based sh_mobile_ceu one. > > Tested with ov7670 camera sensor,

Re: [GIT PULL for v4.15-rc3] media fixes

2017-12-13 Thread Mauro Carvalho Chehab
Em Wed, 13 Dec 2017 10:03:56 +0100 Geert Uytterhoeven escreveu: > Hi Mauro, > > On Mon, Dec 11, 2017 at 12:12 PM, Mauro Carvalho Chehab > wrote: > > Without this series, I was getting 809 lines of bogus warnings (see below), > > with was

[PATCH] dvbv5-zap: accept -S 0 option

2017-12-13 Thread Rafaël Carré
Signed-off-by: Rafaël Carré --- utils/dvb/dvbv5-zap.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/utils/dvb/dvbv5-zap.c b/utils/dvb/dvbv5-zap.c index a88500d1..1b6dabd0 100644 --- a/utils/dvb/dvbv5-zap.c +++ b/utils/dvb/dvbv5-zap.c @@ -930,7 +930,7 @@

[PATCH] Staging: media: atomisp: made function static

2017-12-13 Thread Sergiy Redko
Fixed sparse warning by making 'dtrace_dot' function static. Signed-off-by: Sergiy Redko --- .../media/atomisp/pci/atomisp2/css2400/runtime/debug/src/ia_css_debug.c | 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH] pxa_camera: rename the soc_camera_ prefix to pxa_camera_

2017-12-13 Thread Hans Verkuil
Rename soc_camera to pxa_camera as this has no longer anything to do with the old soc_camera driver/framework. It's confusing when grepping on soc_camera. Signed-off-by: Hans Verkuil --- diff --git a/drivers/media/platform/pxa_camera.c

[bug report] media: lirc: improve locking

2017-12-13 Thread Dan Carpenter
Hello Sean Young, The patch 131fd7fc3c01: "media: lirc: improve locking" from Nov 4, 2017, leads to the following static checker warning: drivers/media/rc/lirc_dev.c:373 ir_lirc_transmit_ir() error: 'txbuf' dereferencing possible ERR_PTR() drivers/media/rc/lirc_dev.c 330

Re: [PATCH v10 4/4] [media] platform: Add Synopsys DesignWare HDMI RX Controller Driver

2017-12-13 Thread Philippe Ombredanne
Jose, On 12/12/17 17:02, Jose Abreu wrote: >>> Signed-off-by: Jose Abreu >>> Cc: Joao Pinto >>> Cc: Mauro Carvalho Chehab >>> Cc: Hans Verkuil >>> Cc: Sylwester Nawrocki >>> Cc:

Re: [PATCH v10 4/4] [media] platform: Add Synopsys DesignWare HDMI RX Controller Driver

2017-12-13 Thread Hans Verkuil
On 12/12/17 17:02, Jose Abreu wrote: > Hi Hans, > > On 12-12-2017 15:47, Hans Verkuil wrote: >> Hi Jose, >> >> Some more comments: > > Thanks for the review! > >> >> On 11/12/17 18:41, Jose Abreu wrote: >>> This is an initial submission for the Synopsys DesignWare HDMI RX >>> Controller Driver.

vidioc-g-dv-timings.rst: fix typo (frontporch -> backporch)

2017-12-13 Thread Hans Verkuil
The description of V4L2_DV_FL_HALF_LINE mixed up frontporch with backporch. It's the backporch that has different sizes for interlaced formats, the frontporch remains constant. Signed-off-by: Hans Verkuil --- diff --git a/Documentation/media/uapi/v4l/

Re: [PATCH v8] uvcvideo: Add a metadata device node

2017-12-13 Thread Guennadi Liakhovetski
Hi Laurent, On Wed, 13 Dec 2017, Laurent Pinchart wrote: > Hi Guennadi, > > On Tuesday, 12 December 2017 10:30:39 EET Guennadi Liakhovetski wrote: > > On Mon, 11 Dec 2017, Laurent Pinchart wrote: > > > On Monday, 11 December 2017 23:44:09 EET Guennadi Liakhovetski wrote: > > >> On Mon, 11 Dec

Re: [PATCH v8] uvcvideo: Add a metadata device node

2017-12-13 Thread Laurent Pinchart
Hi Guennadi, On Tuesday, 12 December 2017 10:30:39 EET Guennadi Liakhovetski wrote: > On Mon, 11 Dec 2017, Laurent Pinchart wrote: > > On Monday, 11 December 2017 23:44:09 EET Guennadi Liakhovetski wrote: > >> On Mon, 11 Dec 2017, Laurent Pinchart wrote: > >>> On Monday, 11 December 2017 22:16:23

Re: [PATCH 0/2] uvcvideo: Refactor code to ease metadata implementation

2017-12-13 Thread Laurent Pinchart
Hi Guennadi, On Tuesday, 12 December 2017 09:45:11 EET Guennadi Liakhovetski wrote: > Hi Laurent, > > Thanks for the patches. Please feel free to add either or both of > > Reviewed-by: Guennadi Liakhovetski > Tested-by: Guennadi Liakhovetski

[PATCH v2 0/2] uvcvideo: Refactor code to ease metadata implementation

2017-12-13 Thread Laurent Pinchart
Hello This small patch series refactors the uvc_video_register() function to extract the code that you need into a new uvc_video_register_device() function. Please let me know if it can help. Compared to v1, the first patch has been updated not to rely on the fact that VFL_DIR_RX equals to zero.

[PATCH v2 2/2] uvcvideo: Report V4L2 device caps through the video_device structure

2017-12-13 Thread Laurent Pinchart
The V4L2 core populates the struct v4l2_capability device_caps field from the same field in video_device. There's no need to handle that manually in the driver. Signed-off-by: Laurent Pinchart Reviewed-by: Guennadi Liakhovetski

[PATCH v2 1/2] uvcvideo: Factor out video device registration to a function

2017-12-13 Thread Laurent Pinchart
The function will then be used to register the video device for metadata capture. Signed-off-by: Laurent Pinchart Reviewed-by: Guennadi Liakhovetski Tested-by: Guennadi Liakhovetski ---

Re: [GIT PULL for v4.15-rc3] media fixes

2017-12-13 Thread Geert Uytterhoeven
Hi Mauro, On Mon, Dec 11, 2017 at 12:12 PM, Mauro Carvalho Chehab wrote: > Without this series, I was getting 809 lines of bogus warnings (see below), > with was preventing me to see new warnings on my incremental builds > while applying new patches at the media tree. $

Re: [Patch v6 10/12] [media] v4l2: Add v4l2 control IDs for HEVC encoder

2017-12-13 Thread Smitha T Murthy
On Tue, 2017-12-12 at 10:46 +0100, Sylwester Nawrocki wrote: > On 12/12/2017 03:34 AM, Smitha T Murthy wrote: > >> s/Lay/Layer here and below > >> > > Ok I will change it. > > While it's fine to make such change for controls up to > V4L2_CID_MPEG_VIDEO_HEVC_HIER_CODING_L6_QP... > > >>> + case