cron job: media_tree daily build: ERRORS

2017-08-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 Aug 10 05:00:17 CEST 2017 media-tree git hash:ec0c3ec497cabbf3bfa03a9eb5edcc252190a4e0 media_build gi

Re: [PATCH 3/3] drm/i915: add DisplayPort CEC-Tunneling-over-AUX support

2017-08-09 Thread Carlos Santa
On Tue, 2017-07-11 at 15:30 +0200, Hans Verkuil wrote: > From: Hans Verkuil > > Implement support for this DisplayPort feature. > > The cec device is created whenever it detects an adapter that > has this feature. It is only removed when a new adapter is connected > that does not support this. I

Re: [PATCH 1/3] drm: add support for DisplayPort CEC-Tunneling-over-AUX

2017-08-09 Thread Carlos Santa
On Tue, 2017-07-11 at 15:30 +0200, Hans Verkuil wrote: > From: Hans Verkuil > > This adds support for the DisplayPort CEC-Tunneling-over-AUX > feature that is part of the DisplayPort 1.3 standard. > > Unfortunately, not all DisplayPort/USB-C to HDMI adapters with a > chip that has this capabilit

[PATCH v2] media: ov5670: Fix incorrect frame timing reported to user

2017-08-09 Thread Chiranjeevi Rapolu
Previously, pixel-rate/(pixels-per-line * lines-per-frame) was yielding incorrect frame timing for the user. OV sensor is using internal timing and this requires conversion (internal timing -> PPL) for correct HBLANK calculation. Now, change pixels-per-line domain from internal sensor clock to pi

Re: [PATCH 2/2] cec: fix remote control passthrough

2017-08-09 Thread Sean Young
On Tue, Aug 08, 2017 at 10:11:13AM +0200, Hans Verkuil wrote: > On 07/08/17 22:58, Sean Young wrote: > > On Mon, Aug 07, 2017 at 03:31:24PM +0200, Hans Verkuil wrote: > >> From: Hans Verkuil > >> > >> The 'Press and Hold' operation was not correctly implemented, in > >> particular the requirement

Re: [PATCH v1] media: ov5670: Fix incorrect fps reported by user

2017-08-09 Thread Sakari Ailus
Hi Chiranjeevi, On the subject --- this isn't fps as such, but frame timing information in general. How about "Fix incorrect frame timing reported to user"? On Wed, Aug 09, 2017 at 12:17:42PM -0700, Chiranjeevi Rapolu wrote: > Previously, pixel-rate/(pixels-per-line * lines-per-frame) was > yield

[PATCH v3 09/12] [media] ddbridge: fix impossible condition warning

2017-08-09 Thread Daniel Scheller
From: Daniel Scheller Smatch and gcc complained: drivers/media/pci/ddbridge/ddbridge-core.c:3491 bpsnr_show() warn: impossible condition '(snr[0] == 255) => ((-128)-127 == 255)' drivers/media/pci/ddbridge/ddbridge-core.c: In function ‘bpsnr_show’: drivers/media/pci/ddbridge/ddbridge-core

[PATCH v3 05/12] [media] ddbridge: check pointers before dereferencing

2017-08-09 Thread Daniel Scheller
From: Daniel Scheller Fixes two warnings reported by smatch: drivers/media/pci/ddbridge/ddbridge-core.c:240 ddb_redirect() warn: variable dereferenced before check 'idev' (see line 238) drivers/media/pci/ddbridge/ddbridge-core.c:240 ddb_redirect() warn: variable dereferenced before check '

[PATCH v3 06/12] [media] ddbridge: only register frontends in fe2 if fe is not NULL

2017-08-09 Thread Daniel Scheller
From: Daniel Scheller Smatch reported: drivers/media/pci/ddbridge/ddbridge-core.c:1602 dvb_input_attach() error: we previously assumed 'dvb->fe' could be null (see line 1595) dvb->fe2 will ever only be populated when dvb->fe is set. So only handle registration of dvb->fe2 when dvb->fe got se

[PATCH v3 10/12] [media] ddbridge: fix dereference before check

2017-08-09 Thread Daniel Scheller
From: Daniel Scheller Both ts_release() and ts_open() can use "output" before check (smatch): drivers/media/pci/ddbridge/ddbridge-core.c:816 ts_release() warn: variable dereferenced before check 'output' (see line 809) drivers/media/pci/ddbridge/ddbridge-core.c:836 ts_open() warn: variable

[PATCH v3 11/12] [media] ddbridge: Kconfig option to control the MSI modparam default

2017-08-09 Thread Daniel Scheller
From: Daniel Scheller It is known that MSI interrupts - while working quite well so far - can still cause issues on some hardware platforms (causing I2C timeouts due to unhandled interrupts). The msi variable/option is set to 1 by default. So, add a Kconfig option prefixed with "EXPERIMENTAL" tha

[PATCH v3 12/12] [media] MAINTAINERS: add entry for ddbridge

2017-08-09 Thread Daniel Scheller
From: Daniel Scheller Signed-off-by: Daniel Scheller --- MAINTAINERS | 8 1 file changed, 8 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 931abca006b7..0453a1365c3a 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -8421,6 +8421,14 @@ T: git git://linuxtv.org/media_tree

[PATCH v3 08/12] [media] ddbridge: remove unreachable code

2017-08-09 Thread Daniel Scheller
From: Daniel Scheller >From smatch: drivers/media/pci/ddbridge/ddbridge-core.c:3490 snr_store() info: ignoring unreachable code. In fact, the function immediately returns zero, so remove it and update ddb_attrs_snr[] to not reference it anymore. Cc: Ralph Metzler Signed-off-by: Daniel Sche

[PATCH v3 07/12] [media] ddbridge: fix possible buffer overflow in ddb_ports_init()

2017-08-09 Thread Daniel Scheller
From: Daniel Scheller Report from smatch: drivers/media/pci/ddbridge/ddbridge-core.c:2659 ddb_ports_init() error: buffer overflow 'dev->port' 32 <= u32max Fix by making sure "p" is greater than zero before checking for "dev->port[].type == DDB_CI_EXTERNAL_XO2". Cc: Ralph Metzler Signed-off

[PATCH v3 00/12] ddbridge: bump to ddbridge-0.9.29

2017-08-09 Thread Daniel Scheller
From: Daniel Scheller Even without DDB IOCTLs, target: 4.14. Changes from v2 to v3: * (private) IOCTLs (temporarily) removed. Proper implementation via UAPI and some documentation will be (re)added with a separate patch. Even with this functionality removed (which most prominently is used

[PATCH v3 03/12] [media] ddbridge: split off IRQ handling

2017-08-09 Thread Daniel Scheller
From: Daniel Scheller This not only helps keep the ddbridge-core tidy, but also gets rid of defined-but-unused-function warnings which might be triggered depending of CONFIG_PCI_MSI, without having to clutter the code with #ifdef'ery. Signed-off-by: Daniel Scheller Tested-by: Richard Scobie Te

[PATCH v3 02/12] [media] ddbridge: split I/O related functions off from ddbridge.h

2017-08-09 Thread Daniel Scheller
From: Daniel Scheller While it seems valid that headers can carry simple oneline static inline annotated functions, move them into their own header file to have the overall code more readable. Also, keep them as header (and don't put in a separate object) and static inline to help the compiler av

[PATCH v3 04/12] [media] ddbridge: split off hardware definitions and mappings

2017-08-09 Thread Daniel Scheller
From: Daniel Scheller Further cleanup of ddbridge-core and ddbridge-main, and moves all such hw definitions into one single place, making things easier to maintain. Signed-off-by: Daniel Scheller Tested-by: Richard Scobie Tested-by: Jasmin Jessich Tested-by: Dietmar Spingler Tested-by: Manfr

[GIT PULL for 4.14] Omap3isp CCP2 support

2017-08-09 Thread Sakari Ailus
Hi Mauro, These patches add functional CCP2 support for the omap3isp, as needed for the Nokia N900. Please pull. The following changes since commit ec0c3ec497cabbf3bfa03a9eb5edcc252190a4e0: media: ddbridge: split code into multiple files (2017-08-09 12:17:01 -0400) are available in the git

[PATCH v1] media: ov5670: Fix incorrect fps reported by user

2017-08-09 Thread Chiranjeevi Rapolu
Previously, pixel-rate/(pixels-per-line * lines-per-frame) was yielding incorrect fps for the user. This results in much lower fps reported by user than the actual fps on the bus. OV sensor is using internal timing and this requires conversion (internal timing -> PPL) for correct HBLANK calculatio

[PATCH 1/3] uapi: media-bus-format: Add Xilinx specific YCbCr 4:2:0 media bus format

2017-08-09 Thread Rohit Athavale
This commit adds Xilinx Video IP specific 8-bit color depth YCbCr 4:2:0 to the media bus format uapi. Signed-off-by: Rohit Athavale --- include/uapi/linux/media-bus-format.h | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/include/uapi/linux/media-bus-format.h b/include/ua

[PATCH 3/3] Documentation: subdev-formats: Add Xilinx YCbCr to Vendor specific area

2017-08-09 Thread Rohit Athavale
This commit adds the custom Xilinx IP specific 8-bit YCbCr 4:2:0 to the custom formats area in the subdev-formats documentation. Signed-off-by: Rohit Athavale --- Documentation/media/uapi/v4l/subdev-formats.rst | 5 + 1 file changed, 5 insertions(+) diff --git a/Documentation/media/uapi/v4l

[PATCH 0/3] Xilinx 8-bit color depth YCbCr 4:2:0 media bus format

2017-08-09 Thread Rohit Athavale
This patch set adds a vendor specific custom YCbCr 4:2:0 format. The way Xilinx Video IP sends data over the bus makes it unrepresented by existing media bus formats. YCbCr 4:2:0 transmission over the bus does not slice Chroma and Luma components over the bus width. Instead it sends sub-sampled Ch

[PATCH 2/3] media: xilinx-vip: Add 8-bit YCbCr 4:2:0 to formats table

2017-08-09 Thread Rohit Athavale
Add Xilinx YCbCr 4:2:0 to xvip formats table. This commit will allow driver to setup media pad codes to YUV 420 via DT properties. Signed-off-by: Rohit Athavale --- drivers/media/platform/xilinx/xilinx-vip.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/media/platform/xilinx/xil

[PATCH] [media] rc: per-protocol repeat period

2017-08-09 Thread Sean Young
CEC needs a keypress timeout of 550ms, which is too high for the IR protocols. Also fill in known repeat times, with 50ms error margin. Also, combine all protocol data into one structure. Signed-off-by: Sean Young Suggested-by: Hans Verkuil --- drivers/media/rc/rc-main.c | 138

Re: [PATCH v2 03/14] [media] ddbridge: bump ddbridge code to version 0.9.29

2017-08-09 Thread Daniel Scheller
On Wed, 9 Aug 2017 13:47:31 -0300 Mauro Carvalho Chehab wrote: > Em Sat, 29 Jul 2017 13:28:37 +0200 > Daniel Scheller escreveu: > > > From: Daniel Scheller > > > > This huge patch bumps the ddbridge driver to version 0.9.29. > > Compared to the vendor driver package, DD OctoNET including GTL

Re: [PATCH v2 03/14] [media] ddbridge: bump ddbridge code to version 0.9.29

2017-08-09 Thread Mauro Carvalho Chehab
Em Sat, 29 Jul 2017 13:28:37 +0200 Daniel Scheller escreveu: > From: Daniel Scheller > > This huge patch bumps the ddbridge driver to version 0.9.29. Compared to > the vendor driver package, DD OctoNET including GTL link support, and all > DVB-C Modulator card support has been removed since thi

Re: [PATCH v2 2/3] v4l2-flash-led-class: Create separate sub-devices for indicators

2017-08-09 Thread Rui Miguel Silva
Hi, On Wed, Aug 09, 2017 at 02:15:54PM +0300, Sakari Ailus wrote: > The V4L2 flash interface allows controlling multiple LEDs through a single > sub-devices if, and only if, these LEDs are of different types. This > approach scales badly for flash controllers that drive multiple flash LEDs > or for

Re: [PATCH v2 1/3] staging: greybus: light: fix memory leak in v4l2 register

2017-08-09 Thread Rui Miguel Silva
Hi Sakari, On Wed, Aug 09, 2017 at 06:36:59PM +0300, Sakari Ailus wrote: > Hi Rui, > > On Wed, Aug 09, 2017 at 02:20:02PM +0100, Rui Miguel Silva wrote: > > Hi Sakari, > > On Wed, Aug 09, 2017 at 02:15:53PM +0300, Sakari Ailus wrote: > > > From: Rui Miguel Silva > > > > > > We are allocating mem

Re: [PATCH v1.1 1/1] omap3isp: Skip CSI-2 receiver initialisation in CCP2 configuration

2017-08-09 Thread Pavel Machek
> If the CSI-2 receiver isn't part of the pipeline (or isn't there to begin > with), skip its initialisation. > > Signed-off-by: Sakari Ailus Looks good to me. Acked-by: Pavel Machek I should be able to test it later, but as Laurent already tested it, I believe it is good to go. Thanks,

Re: [GIT PULL for 4.14] Stream control documentation

2017-08-09 Thread Mauro Carvalho Chehab
Em Wed, 9 Aug 2017 18:35:07 +0300 Sakari Ailus escreveu: > Hi Mauro, > > On Wed, Aug 09, 2017 at 12:29:17PM -0300, Mauro Carvalho Chehab wrote: > > Em Wed, 9 Aug 2017 11:03:40 +0300 > > Sakari Ailus escreveu: > > > > > Hi Mauro, > > > > > > Add stream control documentation. > > > > > > We

Re: [GIT PULL for 4.14] Stream control documentation

2017-08-09 Thread Mauro Carvalho Chehab
Em Wed, 9 Aug 2017 12:29:17 -0300 Mauro Carvalho Chehab escreveu: > Em Wed, 9 Aug 2017 11:03:40 +0300 > Sakari Ailus escreveu: > > > Hi Mauro, > > > > Add stream control documentation. > > > > We have recently added support for hardware that makes it possible to have > > pipelines that are co

Re: [PATCH v2 1/3] staging: greybus: light: fix memory leak in v4l2 register

2017-08-09 Thread Sakari Ailus
Hi Rui, On Wed, Aug 09, 2017 at 02:20:02PM +0100, Rui Miguel Silva wrote: > Hi Sakari, > On Wed, Aug 09, 2017 at 02:15:53PM +0300, Sakari Ailus wrote: > > From: Rui Miguel Silva > > > > We are allocating memory for the v4l2 flash configuration structure and > > leak it in the normal path. Just u

Re: [GIT PULL for 4.14] Stream control documentation

2017-08-09 Thread Sakari Ailus
Hi Mauro, On Wed, Aug 09, 2017 at 12:29:17PM -0300, Mauro Carvalho Chehab wrote: > Em Wed, 9 Aug 2017 11:03:40 +0300 > Sakari Ailus escreveu: > > > Hi Mauro, > > > > Add stream control documentation. > > > > We have recently added support for hardware that makes it possible to have > > pipelin

Re: [GIT PULL for 4.14] Stream control documentation

2017-08-09 Thread Mauro Carvalho Chehab
Em Wed, 9 Aug 2017 11:03:40 +0300 Sakari Ailus escreveu: > Hi Mauro, > > Add stream control documentation. > > We have recently added support for hardware that makes it possible to have > pipelines that are controlled by more than two drivers. This necessitates > documenting V4L2 stream control

Re: [GIT PULL for 4.14] Sensor driver patches for 4.14

2017-08-09 Thread Mauro Carvalho Chehab
Em Wed, 9 Aug 2017 10:59:38 +0300 Sakari Ailus escreveu: > Hi Mauro, > > Here are a bunch of sensor driver fixes and improvements for 4.14. > > Please pull. > > > The following changes since commit da48c948c263c9d87dfc64566b3373a858cc8aa2: > > media: fix warning on v4l2_subdev_call() resul

Re: [PATCH 3/3] [media] cx231xx: only unregister successfully registered i2c adapters

2017-08-09 Thread Mauro Carvalho Chehab
Em Wed, 9 Aug 2017 16:43:03 +0200 Peter Rosin escreveu: > On 2017-08-09 16:27, Mauro Carvalho Chehab wrote: > > Em Mon, 31 Jul 2017 15:38:52 +0200 > > Peter Rosin escreveu: > > > >> This prevents potentially scary debug messages from the i2c core. > >> > >> Signed-off-by: Peter Rosin > >> --

Re: [PATCH 3/3] [media] cx231xx: only unregister successfully registered i2c adapters

2017-08-09 Thread Peter Rosin
On 2017-08-09 16:27, Mauro Carvalho Chehab wrote: > Em Mon, 31 Jul 2017 15:38:52 +0200 > Peter Rosin escreveu: > >> This prevents potentially scary debug messages from the i2c core. >> >> Signed-off-by: Peter Rosin >> --- >> drivers/media/usb/cx231xx/cx231xx-core.c | 3 +++ >> drivers/media/usb

Re: [PATCH v2 3/3] rc: add zx-irdec remote control driver

2017-08-09 Thread Sean Young
Hi Shawn, On Wed, Aug 09, 2017 at 09:48:18PM +0800, Shawn Guo wrote: > On Wed, Aug 09, 2017 at 02:00:29PM +0100, Sean Young wrote: > > On Sun, Jul 30, 2017 at 09:23:13PM +0800, Shawn Guo wrote: > > > From: Shawn Guo > > > > > > It adds the remote control driver and corresponding keymap file for

Re: [GIT PULL FOR v4.14] Fixes, fixes, ever more fixes :-)

2017-08-09 Thread Mauro Carvalho Chehab
Em Fri, 4 Aug 2017 13:40:18 +0200 Hans Verkuil escreveu: > Lots of constify patches and some random other fixes. Except for the solo > patch > which is an actual feature enhancement. > > Regards, > > Hans > > > The following changes since commit da48c948c263c9d87dfc64566b3373a858cc8aa2

Re: [PATCH 3/3] [media] cx231xx: only unregister successfully registered i2c adapters

2017-08-09 Thread Mauro Carvalho Chehab
Em Mon, 31 Jul 2017 15:38:52 +0200 Peter Rosin escreveu: > This prevents potentially scary debug messages from the i2c core. > > Signed-off-by: Peter Rosin > --- > drivers/media/usb/cx231xx/cx231xx-core.c | 3 +++ > drivers/media/usb/cx231xx/cx231xx-i2c.c | 3 ++- > 2 files changed, 5 inserti

Re: [PATCH v2 3/3] rc: add zx-irdec remote control driver

2017-08-09 Thread Shawn Guo
Hi Sean, On Wed, Aug 09, 2017 at 02:00:29PM +0100, Sean Young wrote: > On Sun, Jul 30, 2017 at 09:23:13PM +0800, Shawn Guo wrote: > > From: Shawn Guo > > > > It adds the remote control driver and corresponding keymap file for > > IRDEC block found on ZTE ZX family SoCs. > > > > Signed-off-by: S

Re: [PATCH v2 1/3] staging: greybus: light: fix memory leak in v4l2 register

2017-08-09 Thread Rui Miguel Silva
Hi Sakari, On Wed, Aug 09, 2017 at 02:15:53PM +0300, Sakari Ailus wrote: > From: Rui Miguel Silva > > We are allocating memory for the v4l2 flash configuration structure and > leak it in the normal path. Just use the stack for this as we do not > use it outside of this function. > > Fixes: 2870b

Re: [PATCH v2 3/3] rc: add zx-irdec remote control driver

2017-08-09 Thread Sean Young
On Sun, Jul 30, 2017 at 09:23:13PM +0800, Shawn Guo wrote: > From: Shawn Guo > > It adds the remote control driver and corresponding keymap file for > IRDEC block found on ZTE ZX family SoCs. > > Signed-off-by: Shawn Guo > --- > drivers/media/rc/Kconfig | 11 ++ > drivers/media/

Re: [PATCH 0/4] drm/bridge/adv7511: add CEC support

2017-08-09 Thread Archit Taneja
On 07/30/2017 06:37 PM, Hans Verkuil wrote: From: Hans Verkuil This patch series adds CEC support to the drm adv7511/adv7533 drivers. I have tested this with the Qualcomm Dragonboard C410 (adv7533 based) and the Renesas R-Car Koelsch board (adv7511 based). Note: the Dragonboard needs this p

Re: [PATCH] media: i2c: adv748x: Export I2C device table entries as module aliases

2017-08-09 Thread Javier Martinez Canillas
On 08/09/2017 01:05 PM, Kieran Bingham wrote: > On 09/08/17 11:58, Javier Martinez Canillas wrote: >> Hi Kieran, >> >> On 08/09/2017 12:29 PM, Kieran Bingham wrote: >>> Hi Javier, >>> >>> Thankyou for the patch >> >> You are welcome. >> >>> On 09/08/17 10:37, Javier Martinez Canillas wrote:

[PATCH v2 3/3] v4l2-flash-led-class: Document v4l2_flash_init() references

2017-08-09 Thread Sakari Ailus
The v4l2_flash_init() keeps a reference to the ops struct but not to the config struct (nor anything it contains). Document this. Signed-off-by: Sakari Ailus Acked-by: Pavel Machek --- include/media/v4l2-flash-led-class.h | 6 ++ 1 file changed, 6 insertions(+) diff --git a/include/media/v

[PATCH v2 2/3] v4l2-flash-led-class: Create separate sub-devices for indicators

2017-08-09 Thread Sakari Ailus
The V4L2 flash interface allows controlling multiple LEDs through a single sub-devices if, and only if, these LEDs are of different types. This approach scales badly for flash controllers that drive multiple flash LEDs or for LED specific associations. Essentially, the original assumption of a LED

[PATCH v2 0/3] Create sub-device per LED

2017-08-09 Thread Sakari Ailus
Hi folks, The original design decision in the V4L2 flash API allows controlling a two LEDs (an indicator and a flash) through a single sub-device. This covered virtually all flash driver chips back then but this no longer holds as there are many LED driver chips with multiple flash LED outputs. Th

[PATCH v2 1/3] staging: greybus: light: fix memory leak in v4l2 register

2017-08-09 Thread Sakari Ailus
From: Rui Miguel Silva We are allocating memory for the v4l2 flash configuration structure and leak it in the normal path. Just use the stack for this as we do not use it outside of this function. Fixes: 2870b52bae4c ("greybus: lights: add lights implementation") Reported-by: Sakari Ailus Signe

Re: [PATCH] media: i2c: adv748x: Export I2C device table entries as module aliases

2017-08-09 Thread Kieran Bingham
On 09/08/17 11:58, Javier Martinez Canillas wrote: > Hi Kieran, > > On 08/09/2017 12:29 PM, Kieran Bingham wrote: >> Hi Javier, >> >> Thankyou for the patch > > You are welcome. > >> On 09/08/17 10:37, Javier Martinez Canillas wrote: >>> The I2C core always reports a MODALIAS of the form i2c: e

Re: [PATCH] media: i2c: adv748x: Export I2C device table entries as module aliases

2017-08-09 Thread Javier Martinez Canillas
Hi Kieran, On 08/09/2017 12:29 PM, Kieran Bingham wrote: > Hi Javier, > > Thankyou for the patch > You are welcome. > On 09/08/17 10:37, Javier Martinez Canillas wrote: >> The I2C core always reports a MODALIAS of the form i2c: even if the >> device was registered via OF, and the driver is onl

Re: [PATCH] media: i2c: adv748x: Export I2C device table entries as module aliases

2017-08-09 Thread Kieran Bingham
Hi Javier, Thankyou for the patch On 09/08/17 10:37, Javier Martinez Canillas wrote: > The I2C core always reports a MODALIAS of the form i2c: even if the > device was registered via OF, and the driver is only exporting the OF ID > table entries as module aliases. > > So if the driver is built a

Re: [PATCH 0/3] drm/i915: add DisplayPort CEC-Tunneling-over-AUX support

2017-08-09 Thread Hans Verkuil
On 11/07/17 15:30, Hans Verkuil wrote: > From: Hans Verkuil > > This patch series adds support for the DisplayPort CEC-Tunneling-over-AUX > feature. This patch series is based on the latest mainline kernel (as of > today) > which has all the needed cec and drm 4.13 patches merged. > > This patc

[PATCH] media: i2c: adv748x: Export I2C device table entries as module aliases

2017-08-09 Thread Javier Martinez Canillas
The I2C core always reports a MODALIAS of the form i2c: even if the device was registered via OF, and the driver is only exporting the OF ID table entries as module aliases. So if the driver is built as module, autoload won't work since udev/kmod won't be able to match the registered OF device wit

[bug report] v4l: vsp1: Add support for the BRS entity

2017-08-09 Thread Dan Carpenter
Hello Laurent Pinchart, This is a semi-automatic email about new static checker warnings. The patch 6134148f6098: "v4l: vsp1: Add support for the BRS entity" from May 25, 2017, leads to the following Smatch complaint: drivers/media/platform/vsp1/vsp1_wpf.c:456 wpf_configure() error: we p

Re: [PATCH] media: i2c: add KConfig dependencies

2017-08-09 Thread Arnd Bergmann
On Wed, Aug 9, 2017 at 9:48 AM, Sakari Ailus wrote: > Hi Arnd, > > Thanks for the patch. > > On Tue, Jul 25, 2017 at 05:36:45PM +0200, Arnd Bergmann wrote: >> @@ -618,8 +618,9 @@ config VIDEO_OV6650 >> >> config VIDEO_OV5670 >> tristate "OmniVision OV5670 sensor support" >> - depends on

Re: [PATCH -next] media: ov5670: add depends to fix build errors

2017-08-09 Thread Sakari Ailus
On Thu, Jul 20, 2017 at 04:47:38PM -0700, Randy Dunlap wrote: > From: Randy Dunlap > > Fix build errors by adding dependency on VIDEO_V4L2_SUBDEV_API: Thanks for the patch, Randy, but I've already applied Arnd's patch with very similar content. -- Sakari Ailus e-mail: sakari.ai...@iki.fi X

[GIT PULL for 4.14] Stream control documentation

2017-08-09 Thread Sakari Ailus
Hi Mauro, Add stream control documentation. We have recently added support for hardware that makes it possible to have pipelines that are controlled by more than two drivers. This necessitates documenting V4L2 stream control behaviour for such pipelines. Please pull. The following changes sinc

[GIT PULL for 4.14] Sensor driver patches for 4.14

2017-08-09 Thread Sakari Ailus
Hi Mauro, Here are a bunch of sensor driver fixes and improvements for 4.14. Please pull. The following changes since commit da48c948c263c9d87dfc64566b3373a858cc8aa2: media: fix warning on v4l2_subdev_call() result interpreted as bool (2017-07-26 13:43:17 -0400) are available in the git re

[GIT PULL for 4.14] Atomisp cleanups

2017-08-09 Thread Sakari Ailus
Hi Mauro, Here's a usual bunch of atomisp cleanups. Please pull. The following changes since commit da48c948c263c9d87dfc64566b3373a858cc8aa2: media: fix warning on v4l2_subdev_call() result interpreted as bool (2017-07-26 13:43:17 -0400) are available in the git repository at: https://l

[GIT PULL for 4.13] atomisp bugfix

2017-08-09 Thread Sakari Ailus
Hi Mauro, The atomisp was broken by da22013f7df4 ("atomisp: remove indirection from sh_css_malloc") which caused that sh_css_calloc() wrapper function always returned NULL. Please pull. The following changes since commit da48c948c263c9d87dfc64566b3373a858cc8aa2: media: fix warning on v4l2_su

Re: [PATCH v1 1/2] uapi: media: New fourcc codes needed by Xilinx Video IP

2017-08-09 Thread Laurent Pinchart
Hi Jeffrey, Thank you for the patch. On Tuesday 08 Aug 2017 18:31:17 Jeffrey Mouroux wrote: > The Xilinx Video Mixer andn Xilinx Video Framebuffer DMA IP > support video memory formats that are not represented in the > current V4L2 fourcc library. This patch adds those missing > fourcc codes. C

Re: [PATCH] media: i2c: add KConfig dependencies

2017-08-09 Thread Sakari Ailus
Hi Arnd, Thanks for the patch. On Tue, Jul 25, 2017 at 05:36:45PM +0200, Arnd Bergmann wrote: > @@ -618,8 +618,9 @@ config VIDEO_OV6650 > > config VIDEO_OV5670 > tristate "OmniVision OV5670 sensor support" > - depends on I2C && VIDEO_V4L2 > + depends on I2C && VIDEO_V4L2 && VIDEO