[ragnatech:media-tree] BUILD INCOMPLETE 72c27a68a2a3f650f0dc7891ee98f02283fc11af

2017-12-12 Thread kbuild test robot
tree/branch: git://git.ragnatech.se/linux media-tree branch HEAD: 72c27a68a2a3f650f0dc7891ee98f02283fc11af media: pvrusb2: properly check endpoint types TIMEOUT after 1192m Sorry we cannot finish the testset for your branch within a reasonable time. It's our fault -- either some build server

cron job: media_tree daily build: ERRORS

2017-12-12 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 Dec 13 05:00:18 CET 2017 media-tree git hash:330dada5957e3ca0c8811b14c45e3ac42c694651 media_build

Re: [PATCH] au0828: fix use-after-free at USB probing

2017-12-12 Thread Gustavo A. R. Silva
Hey Andrey, Quoting Andrey Konovalov : On Thu, Nov 23, 2017 at 2:31 AM, Gustavo A. R. Silva wrote: Hi Andrey, I have successfully installed and tested syzkaller with QEMU. Can you please tell me how to reproduce this bug or share with me the

[PATCH 1/3] [media] staging/cxd2099: fix remaining checkpatch-strict issues

2017-12-12 Thread Daniel Scheller
From: Daniel Scheller Fix up all remaining cosmetic issues as reported by checkpatch.pl. Signed-off-by: Daniel Scheller Signed-off-by: Jasmin Jessich --- drivers/staging/media/cxd2099/cxd2099.c | 19 +--

[PATCH 2/3] [media] staging/cxd2099: fix debug message severity

2017-12-12 Thread Daniel Scheller
From: Daniel Scheller Debug messages should go to KERN_DEBUG, thus change the slot_shutdown() notice from dev_info() to dev_dbg(). Signed-off-by: Daniel Scheller Signed-off-by: Jasmin Jessich --- drivers/staging/media/cxd2099/cxd2099.c |

[PATCH 0/3] staging/cxd2099: cosmetics, checkpatch fixup

2017-12-12 Thread Daniel Scheller
From: Daniel Scheller These three small patches make the driver checkpatch-strict clean and improves a few strings. No functional changes. Essentially drivers/staging/media/cxd2099/ is now clean, esp.: $ scripts/checkpatch.pl --strict --file

[PATCH 3/3] [media] staging/cxd2099: cosmetics: improve strings

2017-12-12 Thread Daniel Scheller
From: Daniel Scheller Prefix dev_*() I2C address prints with 0x, change CXD2099 to CXD2099AR, change the MODULE_DESCRIPTION to a proper one and have a better (and shorter) description for the buffermode modparam. Signed-off-by: Daniel Scheller

[PATCH] [build] fixup v3.13_ddbridge_pcimsi.patch

2017-12-12 Thread Daniel Scheller
From: Jasmin Jessich Required after the ddbridge 0.9.32 bump in media_tree. Signed-off-by: Jasmin Jessich Signed-off-by: Daniel Scheller --- Fixes at least the patch issue with kernel <=3.13. Jasmin originally prepared the updated patch when

Re: [PATCH] tuners: tda8290: reduce stack usage with kasan

2017-12-12 Thread Joe Perches
On Tue, 2017-12-12 at 15:21 +0100, Arnd Bergmann wrote: > On Tue, Dec 12, 2017 at 1:45 PM, Mauro Carvalho Chehab > wrote: > > Em Tue, 12 Dec 2017 03:42:32 -0800 > > Joe Perches escreveu: > > > > > > I actually thought about marking them 'const' here before

Re: [PATCH 1/3] atomisp: Fix up the open v load race

2017-12-12 Thread Alan Cox
On Tue, 12 Dec 2017 09:03:50 -0200 Mauro Carvalho Chehab wrote: > Em Mon, 06 Nov 2017 23:36:36 + > Alan escreveu: > > > This isn't the ideal final solution but it stops the main problem for now > > where an open (often from udev) races the

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

2017-12-12 Thread Hans Verkuil
Hi Jose, Some more comments: On 11/12/17 18:41, Jose Abreu wrote: > This is an initial submission for the Synopsys DesignWare HDMI RX > Controller Driver. This driver interacts with a phy driver so that > a communication between them is created and a video pipeline is > configured. > > The

Re: [PATCH/RFC 1/2] v4l: v4l2-dev: Add infrastructure to protect device unplug race

2017-12-12 Thread Laurent Pinchart
Hi Mauro, On Tuesday, 12 December 2017 14:39:32 EET Mauro Carvalho Chehab wrote: > Em Thu, 23 Nov 2017 15:21:01 +0100 Greg Kroah-Hartman escreveu: > > On Thu, Nov 23, 2017 at 11:07:51AM -0200, Mauro Carvalho Chehab wrote: > >> Em Thu, 16 Nov 2017 02:33:48 +0200 Laurent Pinchart escreveu: > >>>

[GIT PULL FOR v4.16] staging/media: add NVIDIA Tegra video decoder driver

2017-12-12 Thread Hans Verkuil
This adds a new NVIDIA Tegra video decoder driver. It is depending on the request API work since it is a stateless codec, so for now park this in staging. The dts patches should go through nvidia's tree. Regards, Hans The following changes since commit

Re: [PATCH/RFC 1/2] v4l: v4l2-dev: Add infrastructure to protect device unplug race

2017-12-12 Thread Laurent Pinchart
Hi Greg and Mauro, On Thursday, 23 November 2017 16:21:01 EET Greg Kroah-Hartman wrote: > On Thu, Nov 23, 2017 at 11:07:51AM -0200, Mauro Carvalho Chehab wrote: > > Em Thu, 16 Nov 2017 02:33:48 +0200 Laurent Pinchart escreveu: > >> Device unplug being asynchronous, it naturally races with

Re: [PATCH/RFC 1/2] v4l: v4l2-dev: Add infrastructure to protect device unplug race

2017-12-12 Thread Laurent Pinchart
Hi Mauro, On Thursday, 23 November 2017 15:07:51 EET Mauro Carvalho Chehab wrote: > Em Thu, 16 Nov 2017 02:33:48 +0200 Laurent Pinchart escreveu: > > Device unplug being asynchronous, it naturally races with operations > > performed by userspace through ioctls or other file operations on video >

Re: [PATCH/RFC 1/2] v4l: v4l2-dev: Add infrastructure to protect device unplug race

2017-12-12 Thread Laurent Pinchart
Hi Hans, On Friday, 17 November 2017 13:09:20 EET Hans Verkuil wrote: > On 16/11/17 01:33, Laurent Pinchart wrote: > > Device unplug being asynchronous, it naturally races with operations > > performed by userspace through ioctls or other file operations on video > > device nodes. > > > > This

Re: [PATCH/RFC 1/2] v4l: v4l2-dev: Add infrastructure to protect device unplug race

2017-12-12 Thread Laurent Pinchart
Hi Kieran, On Thursday, 16 November 2017 16:47:11 EET Kieran Bingham wrote: > On 16/11/17 12:32, Sakari Ailus wrote: > > Hi Laurent, > > > > Thank you for the initiative to bring up and address the matter! > > I concur - this looks like a good start towards managing the issue. > > One

Re: [PATCH/RFC 1/2] v4l: v4l2-dev: Add infrastructure to protect device unplug race

2017-12-12 Thread Laurent Pinchart
Hi Sakari, On Thursday, 16 November 2017 14:32:37 EET Sakari Ailus wrote: > On Thu, Nov 16, 2017 at 02:33:48AM +0200, Laurent Pinchart wrote: > > Device unplug being asynchronous, it naturally races with operations > > performed by userspace through ioctls or other file operations on video > >

Re: [PATCH] tuners: tda8290: reduce stack usage with kasan

2017-12-12 Thread Arnd Bergmann
On Tue, Dec 12, 2017 at 1:45 PM, Mauro Carvalho Chehab wrote: > Em Tue, 12 Dec 2017 03:42:32 -0800 > Joe Perches escreveu: > >> > I actually thought about marking them 'const' here before sending >> > (without noticing the changelog text) and then ran into

[PATCH 2/2] bdisp: Fix a possible sleep-in-atomic bug in bdisp_hw_save_request

2017-12-12 Thread Jia-Ju Bai
The driver may sleep under a spinlock. The function call path is: bdisp_device_run (acquire the spinlock) bdisp_hw_update bdisp_hw_save_request devm_kzalloc(GFP_KERNEL) --> may sleep To fix it, GFP_KERNEL is replaced with GFP_ATOMIC. This bug is found by my static analysis tool(DSAC)

[PATCH 1/2] bdisp: Fix a possible sleep-in-atomic bug in bdisp_hw_reset

2017-12-12 Thread Jia-Ju Bai
The driver may sleep under a spinlock. The function call path is: bdisp_device_run (acquire the spinlock) bdisp_hw_reset msleep --> may sleep To fix it, msleep is replaced with mdelay. This bug is found by my static analysis tool(DSAC) and checked by my code review. Signed-off-by: Jia-Ju

[PATCH] media: dvb_frontend: fix return error code

2017-12-12 Thread Mauro Carvalho Chehab
The correct error code when a function is not defined is -ENOTSUPP. It was typoed wrong as -EOPNOTSUPP, with, unfortunately, exists, but it is not used by the DVB core. Thanks-to: Geert Uytterhoeven Thanks-to: Arnd Bergmann To make me revisit this code.

Re: [PATCH v4 3/5] staging: Introduce NVIDIA Tegra video decoder driver

2017-12-12 Thread Hans Verkuil
On 10/12/17 19:56, Dmitry Osipenko wrote: > On 05.12.2017 16:03, Hans Verkuil wrote: >> On 12/05/17 13:17, Dmitry Osipenko wrote: >>> Hi Hans, >>> >>> On 04.12.2017 17:04, Hans Verkuil wrote: Hi Dmitry, As you already mention in the TODO, this should become a v4l2 codec driver.

Re: [PATCH] tuners: tda8290: reduce stack usage with kasan

2017-12-12 Thread Mauro Carvalho Chehab
Em Tue, 12 Dec 2017 03:42:32 -0800 Joe Perches escreveu: > > I actually thought about marking them 'const' here before sending > > (without noticing the changelog text) and then ran into what must > > have led me to drop the 'const' originally: tuner_i2c_xfer_send() > > takes a

Re: [PATCH/RFC 1/2] v4l: v4l2-dev: Add infrastructure to protect device unplug race

2017-12-12 Thread Mauro Carvalho Chehab
Em Thu, 23 Nov 2017 15:21:01 +0100 Greg Kroah-Hartman escreveu: > On Thu, Nov 23, 2017 at 11:07:51AM -0200, Mauro Carvalho Chehab wrote: > > Hi Laurent, > > > > Em Thu, 16 Nov 2017 02:33:48 +0200 > > Laurent Pinchart

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

2017-12-12 Thread Hans Verkuil
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, Rob Herring wrote: >>> On Thu, Nov 09, 2017 at 10:45:34AM -0800,

Re: [PATCH] tuners: tda8290: reduce stack usage with kasan

2017-12-12 Thread Joe Perches
On Tue, 2017-12-12 at 11:24 +0100, Arnd Bergmann wrote: > On Mon, Dec 11, 2017 at 10:17 PM, Michael Ira Krufky > wrote: > > On Mon, Dec 11, 2017 at 2:34 PM, Joe Perches wrote: > > > On Mon, 2017-12-11 at 13:06 +0100, Arnd Bergmann wrote: > > > > With

Re: [PATCH v4 0/9] vsp1: TLB optimisation and DL caching

2017-12-12 Thread Geert Uytterhoeven
Hi Kieran, On Fri, Nov 17, 2017 at 4:47 PM, Kieran Bingham wrote: > Each display list currently allocates an area of DMA memory to store register > settings for the VSP1 to process. Each of these allocations adds pressure to > the IPMMU TLB entries. > >

Re: [PATCH 1/3] atomisp: Fix up the open v load race

2017-12-12 Thread Mauro Carvalho Chehab
Em Mon, 06 Nov 2017 23:36:36 + Alan escreveu: > This isn't the ideal final solution but it stops the main problem for now > where an open (often from udev) races the device initialization and we try > and load the firmware twice at the same time. This needless to say

[PATCH] s5p-mfc: Fix encoder menu controls initialization

2017-12-12 Thread Sylwester Nawrocki
This patch fixes the menu_skip_mask field initialization and addresses a following issue found by the SVACE static analysis: * NO_EFFECT.SELF: assignment to self in expression 'cfg.menu_skip_mask = cfg.menu_skip_mask' No effect at drivers/media/platform/s5p-mfc/s5p_mfc_enc.c:2083

Re: [PATCH 2/2] dt-bindings: Document the Rockchip RK1608 bindings

2017-12-12 Thread Heiko Stuebner
Hi Leo, Am Dienstag, 12. Dezember 2017, 14:28:15 CET schrieb Leo Wen: > +Optional properties: > + > +- spi-max-frequency : Maximum SPI clocking speed of the device; > + (for RK1608) > +- spi-min-frequency : Minimum SPI clocking speed of the device; > +

Re: [PATCH] tuners: tda8290: reduce stack usage with kasan

2017-12-12 Thread Arnd Bergmann
On Mon, Dec 11, 2017 at 10:17 PM, Michael Ira Krufky wrote: > On Mon, Dec 11, 2017 at 2:34 PM, Joe Perches wrote: >> On Mon, 2017-12-11 at 13:06 +0100, Arnd Bergmann wrote: >>> With CONFIG_KASAN enabled, we get a relatively large stack frame in one >>>

Re: [PATCH v1 05/10] arch: sh: migor: Use new renesas-ceu camera driver

2017-12-12 Thread Laurent Pinchart
Hi Jacopo, Thank you for the patch. On Wednesday, 15 November 2017 12:55:58 EET Jacopo Mondi wrote: > Migo-R platform uses sh_mobile_ceu camera driver, which is now being > replaced by a proper V4L2 camera driver named 'renesas-ceu'. > > Move Migo-R platform to use the v4l2 renesas-ceu camera

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

2017-12-12 Thread Sylwester Nawrocki
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 V4L2_CID_MPEG_VIDEO_HEVC_HIER_CODING_L1_QP:return "HEVC >>>

Re: [PATCH 0/2] Add timers to en50221 protocol driver

2017-12-12 Thread Ralph Metzler
Yes, acked-by: Ralph Metzler Jasmin J. writes: > Hi! > > > I don't know if you are also waiting for a comment from me. > All positive ones are welcome ;) > > > I am fine with those changes. They are even desperately needed > > to get some "problem CAMs" working. >

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

2017-12-12 Thread Guennadi Liakhovetski
Hi Laurent, On Mon, 11 Dec 2017, Laurent Pinchart wrote: > Hi Guennadi, > > 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 EET Laurent Pinchart wrote: > > >> On Wednesday, 6