[PATCH 3/3] media: v4l: fix broken video4linux docs locations

2018-05-09 Thread Mauro Carvalho Chehab
/video4linux/videobuf Documentation/video4linux/Zoran Make them point to the new location where available, removing otherwise. Signed-off-by: Mauro Carvalho Chehab <mchehab+sams...@kernel.org> --- .../translations/zh_CN/video4linux/v4l2-framework.txt | 6 +++--- drivers/media/pci/bt8xx/K

[PATCH 1/3] media: dvb: fix location of get_dvb_firmware script

2018-05-09 Thread Mauro Carvalho Chehab
This script was moved out of Documentation/dvb, but the links weren't updated. Signed-off-by: Mauro Carvalho Chehab <mchehab+sams...@kernel.org> --- drivers/media/dvb-frontends/Kconfig | 18 +- drivers/media/dvb-frontends/nxt200x.c| 4 ++-- drivers/med

Re: [PATCHv13 03/28] media-request: implement media requests

2018-05-08 Thread Mauro Carvalho Chehab
Em Tue, 8 May 2018 13:52:33 +0300 Sakari Ailus <sakari.ai...@iki.fi> escreveu: > Hi Mauro, Hans, > > On Tue, May 08, 2018 at 07:21:16AM -0300, Mauro Carvalho Chehab wrote: > > Em Fri, 4 May 2018 15:27:50 +0300 > > Sakari Ailus <sakari.ai...@iki.fi&g

Re: [PATCHv13 12/28] v4l2-ctrls: add core request support

2018-05-08 Thread Mauro Carvalho Chehab
Em Tue, 8 May 2018 10:07:22 +0200 Hans Verkuil escreveu: > >> diff --git a/include/media/v4l2-ctrls.h b/include/media/v4l2-ctrls.h > >> index 76352eb59f14..a0f7c38d1a90 100644 > >> --- a/include/media/v4l2-ctrls.h > >> +++ b/include/media/v4l2-ctrls.h > >> @@ -250,6 +250,10

Re: [PATCHv13 06/28] v4l2-dev: lock req_queue_mutex

2018-05-08 Thread Mauro Carvalho Chehab
Em Tue, 8 May 2018 09:45:27 +0200 Hans Verkuil <hverk...@xs4all.nl> escreveu: > On 05/07/2018 07:20 PM, Mauro Carvalho Chehab wrote: > > Em Thu, 3 May 2018 16:52:56 +0200 > > Hans Verkuil <hverk...@xs4all.nl> escreveu: > > > >> From: Hans Verkuil

Re: [PATCHv13 04/28] media-request: add media_request_get_by_fd

2018-05-08 Thread Mauro Carvalho Chehab
Em Tue, 8 May 2018 09:34:11 +0200 Hans Verkuil <hverk...@xs4all.nl> escreveu: > On 05/07/2018 07:01 PM, Mauro Carvalho Chehab wrote: > > Em Thu, 3 May 2018 16:52:54 +0200 > > Hans Verkuil <hverk...@xs4all.nl> escreveu: > > > >> From: Hans Verk

Re: [PATCHv13 00/28] Request API

2018-05-08 Thread Mauro Carvalho Chehab
Em Thu, 3 May 2018 16:52:50 +0200 Hans Verkuil escreveu: > From: Hans Verkuil > Regarding locking: > > There are two request locks used: req_queue_mutex (part of media_device) > ensures high-level serialization of queueing/reiniting and canceling >

Re: [PATCHv13 03/28] media-request: implement media requests

2018-05-08 Thread Mauro Carvalho Chehab
Em Fri, 4 May 2018 15:27:50 +0300 Sakari Ailus escreveu: > Hi Hans, > > I've read this patch a large number of times and I think also the details > begin to seem sound. A few comments below. I'm sending this after analyzing the other patches in this series, as this is the

Re: [PATCHv13 16/28] videobuf2-core: embed media_request_object

2018-05-08 Thread Mauro Carvalho Chehab
Em Thu, 3 May 2018 16:53:06 +0200 Hans Verkuil escreveu: > From: Hans Verkuil > > Make vb2_buffer a request object. > > Signed-off-by: Hans Verkuil > --- > include/media/videobuf2-core.h | 3 +++ > 1 file changed, 3

Re: [PATCHv13 12/28] v4l2-ctrls: add core request support

2018-05-07 Thread Mauro Carvalho Chehab
Em Thu, 3 May 2018 16:53:02 +0200 Hans Verkuil escreveu: > From: Hans Verkuil > > Integrate the request support. This adds the v4l2_ctrl_request_complete > and v4l2_ctrl_request_setup functions to complete a request and (as a > helper function) to

Re: [PATCHv13 09/28] v4l2-ctrls: prepare internal structs for request API

2018-05-07 Thread Mauro Carvalho Chehab
Em Thu, 3 May 2018 16:52:59 +0200 Hans Verkuil escreveu: > From: Hans Verkuil > > Embed and initialize a media_request_object in struct v4l2_ctrl_handler. > > Add a p_req field to struct v4l2_ctrl_ref that will store the > request value. > >

Re: [PATCHv13 06/28] v4l2-dev: lock req_queue_mutex

2018-05-07 Thread Mauro Carvalho Chehab
Em Thu, 3 May 2018 16:52:56 +0200 Hans Verkuil escreveu: > From: Hans Verkuil > > We need to serialize streamon/off with queueing new requests. > These ioctls may trigger the cancellation of a streaming > operation, and that should not be mixed with

Re: [PATCHv13 05/28] media-request: add media_request_object_find

2018-05-07 Thread Mauro Carvalho Chehab
Em Fri, 4 May 2018 15:43:07 +0300 Sakari Ailus escreveu: > On Thu, May 03, 2018 at 04:52:55PM +0200, Hans Verkuil wrote: > > From: Hans Verkuil > > > > Add media_request_object_find to find a request object inside a > > request based on ops and/or

Re: [PATCHv13 04/28] media-request: add media_request_get_by_fd

2018-05-07 Thread Mauro Carvalho Chehab
Em Thu, 3 May 2018 16:52:54 +0200 Hans Verkuil escreveu: > From: Hans Verkuil > > Add media_request_get_by_fd() to find a request based on the file > descriptor. > > The caller has to call media_request_put() for the returned > request since this

Re: [PATCHv13 01/28] v4l2-device.h: always expose mdev

2018-05-07 Thread Mauro Carvalho Chehab
Em Fri, 4 May 2018 13:51:28 +0300 Sakari Ailus escreveu: > On Thu, May 03, 2018 at 04:52:51PM +0200, Hans Verkuil wrote: > > From: Hans Verkuil > > > > The mdev field is only present if CONFIG_MEDIA_CONTROLLER is set. > > But since we will need to

Re: [PATCH] media: vsp1: cleanup a false positive warning

2018-05-07 Thread Mauro Carvalho Chehab
Em Mon, 07 May 2018 17:05:24 +0300 Laurent Pinchart <laurent.pinch...@ideasonboard.com> escreveu: > Hi Mauro, > > Thank you for the patch. > > On Friday, 4 May 2018 15:13:58 EEST Mauro Carvalho Chehab wrote: > > With the new vsp1 code changes introduced by changes

Re: Are media drivers abusing of GFP_DMA? - was: Re: [LSF/MM TOPIC NOTES] x86 ZONE_DMA love

2018-05-07 Thread Mauro Carvalho Chehab
Em Mon, 07 May 2018 16:26:08 +0300 Laurent Pinchart <laurent.pinch...@ideasonboard.com> escreveu: > Hi Mauro, > > On Saturday, 5 May 2018 19:08:15 EEST Mauro Carvalho Chehab wrote: > > There was a recent discussion about the use/abuse of GFP_DMA flag when > > allocat

[PATCH] media: v4l2-dev.h: document VFL_TYPE_MAX

2018-05-07 Thread Mauro Carvalho Chehab
Cleanup this Sphinx warning: read./include/media/v4l2-dev.h:42: warning: Enum value 'VFL_TYPE_MAX' not described in enum 'vfl_devnode_type' Signed-off-by: Mauro Carvalho Chehab <mchehab+sams...@kernel.org> --- include/media/v4l2-dev.h | 2 ++ 1 file changed, 2 insertions(+) diff

Re: [PATCH 1/2] media: siano: don't use GFP_DMA

2018-05-06 Thread Mauro Carvalho Chehab
Em Sun, 6 May 2018 08:05:05 +0900 Tomoki Sekiyama <tomoki.sekiy...@gmail.com> escreveu: > 2018/5/6 1:09 Mauro Carvalho Chehab <mchehab+sams...@kernel.org>: > > > I can't think on a single reason why this driver would be using > > GFP_DMA. The typical usag

Re: [PATCH] media: include/video/omapfb_dss.h: use IS_ENABLED()

2018-05-05 Thread Mauro Carvalho Chehab
Em Sat, 5 May 2018 10:59:23 -0700 Randy Dunlap <rdun...@infradead.org> escreveu: > On 05/04/2018 01:49 PM, Mauro Carvalho Chehab wrote: > > Just checking for ifdefs cause build issues as reported by > > kernel test: > > > > config: openrisc-allmodconfig (attach

[PATCH 1/2] media: siano: don't use GFP_DMA

2018-05-05 Thread Mauro Carvalho Chehab
I can't think on a single reason why this driver would be using GFP_DMA. The typical usage is as an USB driver. Any DMA restrictions should be handled inside the HCI driver, if any. Cc: "Luis R. Rodriguez" <mcg...@kernel.org> Cc: linux...@kvack.org Signed-off-by: Mauro Carvalho

[PATCH 2/2] media: gp8psk: don't abuse of GFP_DMA

2018-05-05 Thread Mauro Carvalho Chehab
There's s no reason why it should be using GFP_DMA there. This is an USB driver. Any restriction should be, instead, at HCI core, if any. Cc: "Luis R. Rodriguez" <mcg...@kernel.org> Cc: linux...@kvack.org Signed-off-by: Mauro Carvalho Chehab <mchehab+sams...@kernel.org> ---

Are media drivers abusing of GFP_DMA? - was: Re: [LSF/MM TOPIC NOTES] x86 ZONE_DMA love

2018-05-05 Thread Mauro Carvalho Chehab
There was a recent discussion about the use/abuse of GFP_DMA flag when allocating memories at LSF/MM 2018 (see Luis notes enclosed). The idea seems to be to remove it, using CMA instead. Before doing that, better to check if what we have on media is are valid use cases for it, or if it is there

[PATCH] media: siano: don't use GFP_DMA

2018-05-05 Thread Mauro Carvalho Chehab
I can't think on a single reason why this driver would be using GFP_DMA. The typical usage is as an USB driver. Any DMA restrictions should be handled inside the HCI driver, if any. Signed-off-by: Mauro Carvalho Chehab <mchehab+sams...@kernel.org> --- drivers/media/common/siano/smscoreapi.

Re: [PATCH v3 0/8] R-Car DU: Support CRC calculation

2018-05-05 Thread Mauro Carvalho Chehab
Em Sat, 05 May 2018 17:06:50 +0300 Laurent Pinchart escreveu: > Hi Daniel, > > (CC'ing Mauro) > > On Thursday, 3 May 2018 16:45:36 EEST Daniel Vetter wrote: > > On Thu, May 3, 2018 at 2:06 PM, Laurent Pinchart wrote: > > > Hi Dave, > > > > > > Ping ? > >

Re: [RFC PATCH] media: i2c: add SCCB helpers

2018-05-05 Thread Mauro Carvalho Chehab
i.org> > Cc: Laurent Pinchart <laurent.pinch...@ideasonboard.com> > Cc: Hans Verkuil <hans.verk...@cisco.com> > Cc: Sakari Ailus <sakari.ai...@linux.intel.com> > Cc: Mauro Carvalho Chehab <mche...@s-opensource.com> > Signed-off-by: Akinobu Mita <akinob

Re: [PATCH 1/7] i2c: i2c-gpio: move header to platform_data

2018-05-05 Thread Mauro Carvalho Chehab
| 2 +- > arch/mips/alchemy/board-gpr.c| 2 +- > drivers/i2c/busses/i2c-gpio.c| 2 +- > drivers/media/platform/marvell-ccic/mmp-driver.c | 2 +- Acked-by: Mauro Carvalho Chehab <mchehab+sams...@kernel.org> > drivers/mfd/

[PATCH] media: pt1: use #ifdef CONFIG_PM_SLEEP instead of #if

2018-05-05 Thread Mauro Carvalho Chehab
As pointed by ktest: >> drivers/media//pci/pt1/pt1.c:1433:5: warning: "CONFIG_PM_SLEEP" is not >> defined, evaluates to 0 [-Wundef] #if CONFIG_PM_SLEEP ^~~ Reported-by: kbuild test robot <l...@intel.com> Signed-off-by: Mauro Carvalho Chehab

[PATCH] media: pt1: fix strncmp() size warning

2018-05-05 Thread Mauro Carvalho Chehab
As warned by smatch: drivers/media/pci/pt1/pt1.c:213 config_demod() error: strncmp() '"tc90522sat"' too small (11 vs 20) Use the same strncmp() syntax as pt1_init_frontends() does. Signed-off-by: Mauro Carvalho Chehab <mchehab+sams...@kernel.org> --- drivers/media/p

Re: [PATCH v3 2/5] tuners: add new i2c driver for Sharp qm1d1b0004 ISDB-S tuner

2018-05-05 Thread Mauro Carvalho Chehab
Em Mon, 9 Apr 2018 02:39:50 +0900 tsk...@gmail.com escreveu: > From: Akihiro Tsukada > > The tuner is used in Earthsoft PT1/PT2 DVB boards, > and the driver was extraced from (the former) va1j5jf8007s.c of PT1. > it might contain PT1 specific configs. > > Signed-off-by:

Re: [PATCH v5 5/5] dvb-usb-v2/gl861: ensure USB message buffers DMA'able

2018-05-05 Thread Mauro Carvalho Chehab
Em Mon, 9 Apr 2018 02:21:38 +0900 tsk...@gmail.com escreveu: > From: Akihiro Tsukada > > i2c message buf might be on stack. That patch also applied without changes. So, just patch 3/5 was not applied. > > Signed-off-by: Akihiro Tsukada > --- > Changes

Re: [PATCH v5 4/5] dvb-usb-v2/gl861: use usleep_range() for short delay

2018-05-05 Thread Mauro Carvalho Chehab
Em Mon, 9 Apr 2018 02:21:37 +0900 tsk...@gmail.com escreveu: > From: Akihiro Tsukada > > As the kernel doc "timers-howto.txt" reads, > short delay with msleep() can take much longer. > In a case of raspbery-pi platform where CONFIG_HZ_100 was set, > it actually affected the

Re: [PATCH v6 3/5] dvb-usb/friio, dvb-usb-v2/gl861: decompose friio and merge with gl861

2018-05-05 Thread Mauro Carvalho Chehab
Em Mon, 9 Apr 2018 02:21:36 +0900 tsk...@gmail.com escreveu: > From: Akihiro Tsukada > > Friio device contains "gl861" bridge and "tc90522" demod, > for which the separate drivers are already in the kernel. > But friio driver was monolithic and did not use them, > practically

Re: [GIT FIXES FOR v4.17] UVC fixes

2018-05-05 Thread Mauro Carvalho Chehab
Em Wed, 25 Apr 2018 03:37:00 +0300 Laurent Pinchart escreveu: > Hi Mauro, > > The following changes since commit 60cc43fc888428bb2f18f08997432d426a243338: > >

Re: 4.17-rc3 regression in UVC driver

2018-05-05 Thread Mauro Carvalho Chehab
Hi Kieran, Em Sat, 5 May 2018 08:46:50 +0100 Kieran Bingham escreveu: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA512 > > Hello again, > > On 05/05/18 08:34, Kieran Bingham wrote: > > Hi Sebastian, > > > > On 04/05/18 19:45, Sebastian Reichel wrote: >

[PATCH] media: include/video/omapfb_dss.h: use IS_ENABLED()

2018-05-04 Thread Mauro Carvalho Chehab
olnier...@samsung.com> Cc: Randy Dunlap <rdun...@infradead.org> Cc: tomi.valkei...@ti.com Cc: linux-o...@vger.kernel.org Cc: linux-fb...@vger.kernel.org Fixes: 771f7be87ff9 ("media: omapfb: omapfb_dss.h: add stubs to build with COMPILE_TEST && DRM_OMAP") Signed-off-b

Re: [PATCH 1/1] v4l: Add macros for printing V4L 4cc values

2018-05-04 Thread Mauro Carvalho Chehab
Em Wed, 4 Apr 2018 16:02:10 +0300 Sakari Ailus escreveu: > Add two macros that facilitate printing V4L fourcc values with printf > family of functions. v4l2_fourcc_conv provides the printf conversion > specifier for printing formats and v4l2_fourcc_to_conv provides

Re: [PATCH 04/15] media: pxa_camera: remove the dmaengine compat need

2018-05-04 Thread Mauro Carvalho Chehab
f-by: Robert Jarzmik <robert.jarz...@free.fr> > > Acked-by: Hans Verkuil <hans.verk...@cisco.com> I'm assuming that you'll be applying it together with other PXA patches. So: Acked-by: Mauro Carvalho Chehab <mchehab+sams...@kernel.org> Regards, Mauro > > Regards, &g

Re: [PATCH v3][RESEND] media: i2c: tda1997: replace codec to component

2018-05-04 Thread Mauro Carvalho Chehab
Tim Harvey <thar...@gateworks.com> > Acked-by: Tim Harvey <thar...@gateworks.com> In order to keep it together with the patches doing the removal of the old API, it is probably better to apply this via ALSA tree: Acked-by: Mauro Carvalho Chehab <mchehab+sams...@kernel.org> > > Regards, > > Tim Thanks, Mauro

[PATCH] media: pt3: no need to check if null for dvb_module_release()

2018-05-04 Thread Mauro Carvalho Chehab
Such check is already there at the routine. So, no need to repeat it outside. Cc: Akihiro Tsukada <tsk...@gmail.com> Signed-off-by: Mauro Carvalho Chehab <mchehab+sams...@kernel.org> --- drivers/media/pci/pt3/pt3.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --g

Re: [v3] [media] Use common error handling code in 19 functions

2018-05-04 Thread Mauro Carvalho Chehab
Em Fri, 4 May 2018 18:08:59 +0200 SF Markus Elfring escreveu: > > Adjust jump targets so that a bit of exception handling can be better > > reused at the end of these functions. > > Why was this update suggestion rejected once more a moment ago? > >

Re: [PATCH v3 1/3] [media] dvb_frontend: add S2X and misc. other enums

2018-05-04 Thread Mauro Carvalho Chehab
Em Tue, 13 Mar 2018 23:18:03 +0100 Daniel Scheller escreveu: > From: Daniel Scheller > > Additional enums: > * FEC ratios 1/4 and 1/3 > * 64/128/256-APSK modulations (DVB-S2X) > * 15%, 10% and 5% rolloff factors (DVB-S2X) > * 64K transmission

Re: [PATCH] media: vsp1: cleanup a false positive warning

2018-05-04 Thread Mauro Carvalho Chehab
Em Fri, 4 May 2018 16:37:23 +0200 Geert Uytterhoeven <ge...@linux-m68k.org> escreveu: > Hi Mauro, > > On Fri, May 4, 2018 at 2:13 PM, Mauro Carvalho Chehab > <mchehab+sams...@kernel.org> wrote: > > With the new vsp1 code changes introduced by changeset > > f81

[PATCH] media: lgdt330x: don't use an uninitialized state

2018-05-04 Thread Mauro Carvalho Chehab
' could be null (see line 878) drivers/media/dvb-frontends/lgdt330x.c:920 lgdt330x_probe() error: dereferencing freed memory 'state' Fixes: 23ba635d45f5 ("media: lgdt330x: convert it to the new I2C binding way") Signed-off-by: Mauro Carvalho Chehab <mchehab+sams...@kernel.org&

Re: [PATCH v2 15/19] [media] ddbridge: initial support for MCI-based MaxSX8 cards

2018-05-04 Thread Mauro Carvalho Chehab
Em Mon, 9 Apr 2018 18:47:48 +0200 Daniel Scheller escreveu: > From: Daniel Scheller > > This adds initial support for the new MCI-based (micro-code interface) > DD cards, with the first one being the MaxSX8 eight-tuner DVB-S/S2/S2X > PCIe card.

[PATCH] media: video-i2c: get rid of two gcc warnings

2018-05-04 Thread Mauro Carvalho Chehab
of declaration [-Wold-style-declaration] const static struct v4l2_frmsize_discrete amg88xx_size = { ^ Signed-off-by: Mauro Carvalho Chehab <mchehab+sams...@kernel.org> --- drivers/media/i2c/video-i2c.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/med

Re: atomisp: drop from staging ?

2018-05-04 Thread Mauro Carvalho Chehab
Em Thu, 3 May 2018 11:30:50 +0300 Sakari Ailus escreveu: > On Mon, Apr 30, 2018 at 12:41:00PM +0300, Sakari Ailus wrote: > > Hi Alan, > > > > On Sun, Apr 29, 2018 at 01:18:37AM +0100, Alan Cox wrote: > > > > > > I think this is going to be the best option. When I started

[PATCH 2/7 v2] media: meye: allow building it with COMPILE_TEST on non-x86

2018-05-04 Thread Mauro Carvalho Chehab
From: Mauro Carvalho Chehab <mche...@s-opensource.com> This driver depends on sony-laptop driver, but this is available only for x86. So, add a stub function, in order to allow building it on non-x86 too. Signed-off-by: Mauro Carvalho Chehab <mche...@s-opensource.com> Signed-o

Re: [PATCH v2 7/7] media: via-camera: allow build on non-x86 archs with COMPILE_TEST

2018-05-04 Thread Mauro Carvalho Chehab
ply on yours: Reviewed-by: Mauro Carvalho Chehab <mchehab+sams...@kernel.org> Thanks! Mauro > > > From: Bartlomiej Zolnierkiewicz <b.zolnier...@samsung.com> > Subject: [PATCH] video: fbdev: via: allow COMPILE_TEST build > > This patch allows viafb driver to b

Re: [PATCH 5/7] omapfb: omapfb_dss.h: add stubs to build with COMPILE_TEST && DRM_OMAP

2018-05-04 Thread Mauro Carvalho Chehab
Em Wed, 25 Apr 2018 12:47:34 +0200 Bartlomiej Zolnierkiewicz <b.zolnier...@samsung.com> escreveu: > On Monday, April 23, 2018 10:55:57 AM Mauro Carvalho Chehab wrote: > > Em Mon, 23 Apr 2018 14:47:28 +0200 > > Bartlomiej Zolnierkiewicz <b.zolnier...@samsung.com> escre

Re: [PATCH v2 7/7] media: via-camera: allow build on non-x86 archs with COMPILE_TEST

2018-05-04 Thread Mauro Carvalho Chehab
Em Mon, 23 Apr 2018 14:19:31 +0200 Bartlomiej Zolnierkiewicz <b.zolnier...@samsung.com> escreveu: > Hi Mauro, > > On Friday, April 20, 2018 04:03:21 PM Mauro Carvalho Chehab wrote: > > This driver depends on FB_VIA for lots of things. Provide stubs > > for the f

[PATCH] media: vsp1: cleanup a false positive warning

2018-05-04 Thread Mauro Carvalho Chehab
be null (see line 180) This is a false positive, as, if pipe->bru is NULL, the brx var will be different, with ends by calling a code that will set pipe->bru to another value. Yet, cleaning this false positive is as easy as adding an explicit check if pipe->bru is NULL. Signed-off-by:

Re: [PATCH] MAINTAINERS & files: Canonize the e-mails I use at files

2018-05-04 Thread Mauro Carvalho Chehab
Em Fri, 04 May 2018 13:58:39 +0300 Jani Nikula <jani.nik...@linux.intel.com> escreveu: > On Fri, 04 May 2018, Mauro Carvalho Chehab <mchehab+sams...@kernel.org> wrote: > > From now on, I'll start using my @kernel.org as my development e-mail. > > > > As such, le

[GIT PULL for v4.17-rc4] media fixes and a MAINTAINERS file update with my email

2018-05-04 Thread Mauro Carvalho Chehab
-rc4 From: Gustavo A. R. Silva (1): media: imx-media-csi: Fix inconsistent IS_ERR and PTR_ERR Mauro Carvalho Chehab (1): MAINTAINERS & files: Canonize the e-mails I use at files Documentation/doc-guide/parse-headers.rst | 4 ++-- Documentation/media

[PATCH] MAINTAINERS & files: Canonize the e-mails I use at files

2018-05-04 Thread Mauro Carvalho Chehab
off-by: Mauro Carvalho Chehab <mche...@s-opensource.com> Signed-off-by: Brian Warner <brian.war...@samsung.com> Signed-off-by: Mauro Carvalho Chehab <mchehab+sams...@kernel.org> --- Documentation/doc-guide/parse-headers.rst | 4 ++-- Documentation/media/uapi/rc/key

Re: [PATCH] media: Revert cleanup ktime_set() usage

2018-04-27 Thread Mauro Carvalho Chehab
Em Fri, 27 Apr 2018 20:15:00 +0200 "Jasmin J." escreveu: > Hello Mauro! > > > This patch looks fine, but not for the above-mentioned. > So I shall reword the commit message? Yes. > > > The thing is that it is not consistent to have some places with > > things like: > >

Re: Regression: DVBSky S960 USB tuner doesn't work in 4.10 or newer

2018-04-27 Thread Mauro Carvalho Chehab
thing like the patch below (untested). Regards, Mauro media: dvbsky: use just one mutex for serializing device R/W ops Right now, there are two mutexes serializing r/w ops: one "generic" and another one specifically for stream on/off. Clean it a little bit, getting rid of one of the mutexes. Si

Re: [PATCH 01/11] media: tm6000: fix potential Spectre variant 1

2018-04-26 Thread Mauro Carvalho Chehab
Em Thu, 26 Apr 2018 16:41:56 -0500 "Gustavo A. R. Silva" <gust...@embeddedor.com> escreveu: > Hi Mauro, > > On 04/23/2018 02:17 PM, Mauro Carvalho Chehab wrote: > > Em Mon, 23 Apr 2018 14:11:02 -0500 > > > > Thanks, I 'll mark this series as rejecte

Re: [PATCH] media: zoran: move to dma-mapping interface

2018-04-25 Thread Mauro Carvalho Chehab
Em Wed, 25 Apr 2018 17:58:25 +0200 Arnd Bergmann escreveu: > On Wed, Apr 25, 2018 at 5:26 PM, Christoph Hellwig wrote: > > On Wed, Apr 25, 2018 at 01:15:18PM +0200, Arnd Bergmann wrote: > >> That thought had occurred to me as well. I removed the oldest ISDN

Re: [PATCH] Add new dvb-t board ":Zolid Hybrid Tv Stick"

2018-04-25 Thread Mauro Carvalho Chehab
Em Wed, 25 Apr 2018 14:24:09 +0200 mjs <mjst...@gmail.com> escreveu: > Op Wed, 25 Apr 2018 08:18:55 -0300 > Mauro Carvalho Chehab <mchehab+sams...@kernel.org> schreef: > > > Em Wed, 25 Apr 2018 12:11:10 +0200 > > mjs <mjst...@gmail.com> escreveu: > >

Re: [PATCH] Add new dvb-t board ":Zolid Hybrid Tv Stick"

2018-04-25 Thread Mauro Carvalho Chehab
Em Wed, 25 Apr 2018 12:11:10 +0200 mjs <mjst...@gmail.com> escreveu: > Op Wed, 25 Apr 2018 06:16:20 -0300 > Mauro Carvalho Chehab <mchehab+sams...@kernel.org> schreef: > > > Em Wed, 25 Apr 2018 11:09:50 +0200 > > mjs <mjst...@gm

Re: [PATCH] Add new dvb-t board ":Zolid Hybrid Tv Stick"

2018-04-25 Thread Mauro Carvalho Chehab
Em Wed, 25 Apr 2018 11:09:50 +0200 mjs escreveu: > From 0a3355b47dc465c6372d30fa4a36d1c5db6c0fe2 Mon Sep 17 00:00:00 2001 > From: Marcel Stork > Date: Wed, 25 Apr 2018 10:53:34 +0200 > Subject: [PATCH] Add new dvb-t board ":Zolid Hybrid Tv Stick". > >

Re: [PATCH 01/11] media: tm6000: fix potential Spectre variant 1

2018-04-24 Thread Mauro Carvalho Chehab
Em Tue, 24 Apr 2018 12:36:09 +0200 Peter Zijlstra <pet...@infradead.org> escreveu: > On Tue, Apr 24, 2018 at 12:35:00PM +0300, Dan Carpenter wrote: > > On Mon, Apr 23, 2018 at 03:24:55PM -0300, Mauro Carvalho Chehab wrote: > > > Em Mon, 23 Apr 2018 12:38:03 -0500 &g

Re: [PATCH 1/7] asm-generic, media: allow COMPILE_TEST with virt_to_bus

2018-04-24 Thread Mauro Carvalho Chehab
Em Tue, 24 Apr 2018 09:55:55 +0200 Arnd Bergmann <a...@arndb.de> escreveu: > On Fri, Apr 20, 2018 at 7:42 PM, Mauro Carvalho Chehab > <mche...@s-opensource.com> wrote: > > The virt_to_bus/bus_to_virt macros are arch-specific. Some > > archs don't support it. Yet,

Re: [PATCH 01/11] media: tm6000: fix potential Spectre variant 1

2018-04-24 Thread Mauro Carvalho Chehab
arrays. Peter do you know the answer here? > > regards, > dan carpenter > > On Mon, Apr 23, 2018 at 03:24:55PM -0300, Mauro Carvalho Chehab wrote: > > Em Mon, 23 Apr 2018 12:38:03 -0500 > > "Gustavo A. R. Silva" <gust...@embeddedor.com> escreveu

Re: How to proceed ? => [PATCH ?] EM28xx driver ?

2018-04-23 Thread Mauro Carvalho Chehab
Em Mon, 23 Apr 2018 22:21:29 +0200 mjs <mjst...@gmail.com> escreveu: > Op Mon, 23 Apr 2018 14:15:59 -0300 > Mauro Carvalho Chehab <mche...@s-opensource.com> schreef: > > > Em Mon, 23 Apr 2018 12:35:06 -0400 > > Alexandre-Xavier Labonté-Lamoureux

Re: [PATCH 5/7] omapfb: omapfb_dss.h: add stubs to build with COMPILE_TEST && DRM_OMAP

2018-04-23 Thread Mauro Carvalho Chehab
Em Mon, 23 Apr 2018 22:48:06 +0300 Laurent Pinchart <laurent.pinch...@ideasonboard.com> escreveu: > Hi Mauro, > > On Monday, 23 April 2018 17:22:27 EEST Mauro Carvalho Chehab wrote: > > Em Mon, 23 Apr 2018 15:56:53 +0200 Bartlomiej Zolnierkiewicz escreveu: > > > O

Re: [PATCH 01/11] media: tm6000: fix potential Spectre variant 1

2018-04-23 Thread Mauro Carvalho Chehab
Em Mon, 23 Apr 2018 14:11:02 -0500 "Gustavo A. R. Silva" <gust...@embeddedor.com> escreveu: > On 04/23/2018 01:24 PM, Mauro Carvalho Chehab wrote: > > Em Mon, 23 Apr 2018 12:38:03 -0500 > > "Gustavo A. R. Silva" <gust...@embeddedor.com> escreveu:

Re: [PATCH 01/11] media: tm6000: fix potential Spectre variant 1

2018-04-23 Thread Mauro Carvalho Chehab
Em Mon, 23 Apr 2018 12:38:03 -0500 "Gustavo A. R. Silva" escreveu: > f->index can be controlled by user-space, hence leading to a > potential exploitation of the Spectre variant 1 vulnerability. > > Smatch warning: > drivers/media/usb/tm6000/tm6000-video.c:879

Re: How to proceed ? => [PATCH ?] EM28xx driver ?

2018-04-23 Thread Mauro Carvalho Chehab
Em Mon, 23 Apr 2018 12:35:06 -0400 Alexandre-Xavier Labonté-Lamoureux escreveu: > This should give you an idea of what you need to do: > https://linuxtv.org/wiki/index.php/Development:_How_to_submit_patches Yes. In particular, you should send it against the upstream

Re: [PATCH 0/5] Remaining COMPILE_TEST and smatch cleanups

2018-04-23 Thread Mauro Carvalho Chehab
Em Mon, 23 Apr 2018 17:09:19 +0300 Sakari Ailus <sakari.ai...@iki.fi> escreveu: > Hi Mauro, > > On Thu, Apr 19, 2018 at 07:42:28AM -0300, Mauro Carvalho Chehab wrote: > > Em Wed, 18 Apr 2018 12:04:14 +0300 > > Sakari Ailus <sakari.ai...@iki.fi> escreveu: > &

Re: [PATCH 5/7] omapfb: omapfb_dss.h: add stubs to build with COMPILE_TEST && DRM_OMAP

2018-04-23 Thread Mauro Carvalho Chehab
Em Mon, 23 Apr 2018 17:11:14 +0300 Tomi Valkeinen escreveu: > On 23/04/18 16:56, Bartlomiej Zolnierkiewicz wrote: > > > Ideally we should be able to build both drivers in the same kernel > > (especially as modules). > > > > I was hoping that it could be fixed easily but

Re: [PATCH 5/7] omapfb: omapfb_dss.h: add stubs to build with COMPILE_TEST && DRM_OMAP

2018-04-23 Thread Mauro Carvalho Chehab
Em Mon, 23 Apr 2018 15:56:53 +0200 Bartlomiej Zolnierkiewicz <b.zolnier...@samsung.com> escreveu: > On Monday, April 23, 2018 02:47:28 PM Bartlomiej Zolnierkiewicz wrote: > > On Friday, April 20, 2018 01:42:51 PM Mauro Carvalho Chehab wrote: > > > Add stubs for o

Re: [RFCv11 PATCH 04/29] media-request: core request support

2018-04-23 Thread Mauro Carvalho Chehab
Em Mon, 23 Apr 2018 14:23:28 +0200 Hans Verkuil escreveu: > >> + spin_lock_irqsave(>lock, flags); > >> + state = req->state; > >> + spin_unlock_irqrestore(>lock, flags); > > > > IMO, it would be better to use an atomic var for state, having a > > lockless access to it.

Re: [PATCH 5/7] omapfb: omapfb_dss.h: add stubs to build with COMPILE_TEST && DRM_OMAP

2018-04-23 Thread Mauro Carvalho Chehab
Em Mon, 23 Apr 2018 14:47:28 +0200 Bartlomiej Zolnierkiewicz <b.zolnier...@samsung.com> escreveu: > On Friday, April 20, 2018 01:42:51 PM Mauro Carvalho Chehab wrote: > > Add stubs for omapfb_dss.h, in the case it is included by > > some driver when CONFIG_FB_OMAP2 is

Re: [RFCv11 PATCH 03/29] media-request: allocate media requests

2018-04-23 Thread Mauro Carvalho Chehab
Em Mon, 23 Apr 2018 13:49:11 +0200 Hans Verkuil <hverk...@xs4all.nl> escreveu: > On 04/10/2018 11:52 AM, Mauro Carvalho Chehab wrote: > > Em Mon, 9 Apr 2018 16:20:00 +0200 > > Hans Verkuil <hverk...@xs4all.nl> escreveu: > > > >> From: Hans Verk

Re: [RFCv11 PATCH 02/29] uapi/linux/media.h: add request API

2018-04-23 Thread Mauro Carvalho Chehab
Em Mon, 23 Apr 2018 13:41:31 +0200 Hans Verkuil <hverk...@xs4all.nl> escreveu: > On 04/10/2018 01:00 PM, Sakari Ailus wrote: > > On Tue, Apr 10, 2018 at 06:38:56AM -0300, Mauro Carvalho Chehab wrote: > >> Em Mon, 9 Apr 2018 16:19:59 +0200 > >> Hans Verk

[PATCH] media: siano: be sure to not override devpath size

2018-04-23 Thread Mauro Carvalho Chehab
ent is a parameter of the current function. So, explicitly limit strcmp() and strcpy() to ensure that the devpath size (32) will be respected. Signed-off-by: Mauro Carvalho Chehab <mche...@s-opensource.com> --- drivers/media/common/siano/smscoreapi.c | 10 +- 1 file changed, 5 inserti

[PATCH] media: flexcop-i2c: get rid of KERN_CONT

2018-04-23 Thread Mauro Carvalho Chehab
per call with different data, with sounds messy. Simplify it by getting rid of DUMP_I2C_MESSAGES and adding a new flag to debug (0x40), and making the debug logic there more consistent. Signed-off-by: Mauro Carvalho Chehab <mche...@s-opensource.com> --- drivers/media/common/b2c2/flex

[PATCH] media: davinci: don't override the error code

2018-04-23 Thread Mauro Carvalho Chehab
-sero value. It toesn't make sense to override such error code with a videobuf_streamoff() error. Signed-off-by: Mauro Carvalho Chehab <mche...@s-opensource.com> --- drivers/media/platform/davinci/vpfe_capture.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers

[PATCH] media: em28xx: Don't use ops->resume if NULL

2018-04-23 Thread Mauro Carvalho Chehab
ixes: be7fd3c3a8c5 ("media: em28xx: Hauppauge DualHD second tuner functionality") Signed-off-by: Mauro Carvalho Chehab <mche...@s-opensource.com> --- drivers/media/usb/em28xx/em28xx-core.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/media/usb/em28xx/em28x

Re: [PATCH stable v4.15 1/3] media: staging: lirc_zilog: broken reference counting

2018-04-23 Thread Mauro Carvalho Chehab
Em Mon, 23 Apr 2018 10:04:20 +0100 Sean Young escreveu: > On Sun, Apr 22, 2018 at 11:47:51AM +0200, Greg KH wrote: > > On Mon, Apr 16, 2018 at 10:15:28AM +0100, Sean Young wrote: > > > On Mon, Apr 16, 2018 at 10:50:15AM +0200, Greg KH wrote: > > > > On Mon, Apr 16, 2018 at

[PATCH v2 7/7] media: via-camera: allow build on non-x86 archs with COMPILE_TEST

2018-04-20 Thread Mauro Carvalho Chehab
This driver depends on FB_VIA for lots of things. Provide stubs for the functions it needs, in order to allow building it with COMPILE_TEST outside x86 architecture. Signed-off-by: Mauro Carvalho Chehab <mche...@s-opensource.com> diff --git a/drivers/media/platform/Kconfig b/drivers

[PATCH] media: s5p-jpeg: don't return a value on a void function

2018-04-20 Thread Mauro Carvalho Chehab
Building this driver on arm64 gives this warning: drivers/media/platform/s5p-jpeg/jpeg-hw-exynos3250.c:430:16: error: return expression in void function Signed-off-by: Mauro Carvalho Chehab <mche...@s-opensource.com> --- drivers/media/platform/s5p-jpeg/jpeg-hw-exynos3250.c | 4 +

[PATCH 4/7] media: ipu3: allow building it with COMPILE_TEST on non-x86 archs

2018-04-20 Thread Mauro Carvalho Chehab
Despite depending on ACPI, this driver builds fine on non-x86 archtecture with COMPILE_TEST, as it doesn't depend on ACPI-specific functions/structs. Signed-off-by: Mauro Carvalho Chehab <mche...@s-opensource.com> --- drivers/media/pci/intel/ipu3/Kconfig | 3 +-- 1 file changed, 1 ins

[PATCH 5/7] omapfb: omapfb_dss.h: add stubs to build with COMPILE_TEST && DRM_OMAP

2018-04-20 Thread Mauro Carvalho Chehab
Add stubs for omapfb_dss.h, in the case it is included by some driver when CONFIG_FB_OMAP2 is not defined, with can happen on ARM when DRM_OMAP is not 'n'. That allows building such driver(s) with COMPILE_TEST. Signed-off-by: Mauro Carvalho Chehab <mche...@s-opensource.com> --- include

[PATCH 0/7] Enable most media drivers to build on ARM

2018-04-20 Thread Mauro Carvalho Chehab
://git.linuxtv.org/mchehab/experimental.git/log/?h=compile_test_v7 Mauro Carvalho Chehab (7): asm-generic, media: allow COMPILE_TEST with virt_to_bus media: meye: allow building it with COMPILE_TEST on non-x86 media: rc: allow build pnp-dependent drivers with COMPILE_TEST media: ipu3: allow building

[PATCH 7/7] media: via-camera: allow build on non-x86 archs with COMPILE_TEST

2018-04-20 Thread Mauro Carvalho Chehab
This driver depends on FB_VIA for lots of things. Provide stubs for the functions it needs, in order to allow building it with COMPILE_TEST outside x86 architecture. Signed-off-by: Mauro Carvalho Chehab <mche...@s-opensource.com> --- drivers/media/platform/Kconfig | 2 +- drivers

[PATCH 1/7] asm-generic, media: allow COMPILE_TEST with virt_to_bus

2018-04-20 Thread Mauro Carvalho Chehab
Carvalho Chehab <mche...@s-opensource.com> --- drivers/media/pci/sta2x11/Kconfig | 4 ++-- drivers/media/pci/zoran/Kconfig | 3 ++- include/asm-generic/io.h | 2 +- 3 files changed, 5 insertions(+), 4 deletions(-) diff --git a/drivers/media/pci/sta2x11/Kconfig b/drivers/med

[PATCH 3/7] media: rc: allow build pnp-dependent drivers with COMPILE_TEST

2018-04-20 Thread Mauro Carvalho Chehab
The pnp header already provide enough stub to build those drivers with COMPILE_TEST on non-x86 archs. Signed-off-by: Mauro Carvalho Chehab <mche...@s-opensource.com> --- drivers/media/rc/Kconfig | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/me

[PATCH 6/7] media: omap2: allow building it with COMPILE_TEST && DRM_OMAP

2018-04-20 Thread Mauro Carvalho Chehab
Now that FB_OMAP has stubs, the omap2 media drivers can be built on ARM with COMPILE_TEST && DRM_OMAP. Signed-off-by: Mauro Carvalho Chehab <mche...@s-opensource.com> --- drivers/media/platform/omap/Kconfig | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a

[PATCH 2/7] media: meye: allow building it with COMPILE_TEST on non-x86

2018-04-20 Thread Mauro Carvalho Chehab
This driver depends on sony-laptop driver, but this is available only for x86. So, add a stub function, in order to allow building it on non-x86 too. Signed-off-by: Mauro Carvalho Chehab <mche...@s-opensource.com> --- drivers/media/pci/meye/Kconfig | 3 ++- include/linux/sony-laptop.h

Re: [PATCH 3/4] sound, media: allow building ISA drivers it with COMPILE_TEST

2018-04-20 Thread Mauro Carvalho Chehab
Em Fri, 20 Apr 2018 14:58:15 +0200 Takashi Iwai <ti...@suse.de> escreveu: > On Fri, 20 Apr 2018 14:51:29 +0200, > Mauro Carvalho Chehab wrote: > > > > Em Fri, 20 Apr 2018 14:37:46 +0200 > > Takashi Iwai <ti...@suse.de> escreveu: > > > > &

Re: [PATCH 3/4] sound, media: allow building ISA drivers it with COMPILE_TEST

2018-04-20 Thread Mauro Carvalho Chehab
Em Fri, 20 Apr 2018 09:51:29 -0300 Mauro Carvalho Chehab <mche...@s-opensource.com> escreveu: > Em Fri, 20 Apr 2018 14:37:46 +0200 > Takashi Iwai <ti...@suse.de> escreveu: > > > On Fri, 20 Apr 2018 14:32:15 +0200, > > Mauro Carvalho Chehab wrote: > >

[PATCH] sound, isapnp: allow building more drivers with COMPILE_TEST

2018-04-20 Thread Mauro Carvalho Chehab
Drivers that depend on ISAPNP currently can't be built with COMPILE_TEST. However, looking at isapnp.h, there are already stubs there to allow drivers to include it even when isa PNP is not supported. So, remove such dependencies when COMPILE_TEST. Signed-off-by: Mauro Carvalho Chehab <mche..

Re: [PATCH 3/4] sound, media: allow building ISA drivers it with COMPILE_TEST

2018-04-20 Thread Mauro Carvalho Chehab
Em Fri, 20 Apr 2018 14:37:46 +0200 Takashi Iwai <ti...@suse.de> escreveu: > On Fri, 20 Apr 2018 14:32:15 +0200, > Mauro Carvalho Chehab wrote: > > > > All sound drivers that don't depend on PNP can be safelly > > build with COMPILE_TEST, as ISA provides

[PATCH 3/4] sound, media: allow building ISA drivers it with COMPILE_TEST

2018-04-20 Thread Mauro Carvalho Chehab
All sound drivers that don't depend on PNP can be safelly build with COMPILE_TEST, as ISA provides function stubs to be used for such purposes. As a side effect, with this change, the radio-miropcm20 can now be built outside i386 with COMPILE_TEST. Signed-off-by: Mauro Carvalho Chehab <mche..

[PATCH 0/4] More COMPILE_TEST patches to build all media drivers on x86_64

2018-04-20 Thread Mauro Carvalho Chehab
Carvalho Chehab (4): media: radio: allow building ISA drivers with COMPILE_TEST media: sta2x11_vip: allow build with COMPILE_TEST sound, media: allow building ISA drivers it with COMPILE_TEST media: siano: get rid of __le32/__le16 cast warnings drivers/media/common/siano/smsendian.c | 14

[PATCH 4/4] media: siano: get rid of __le32/__le16 cast warnings

2018-04-20 Thread Mauro Carvalho Chehab
-off-by: Mauro Carvalho Chehab <mche...@s-opensource.com> --- drivers/media/common/siano/smsendian.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/media/common/siano/smsendian.c b/drivers/media/common/siano/smsendian.c index bfe831c10b1c..b95a63

[PATCH 2/4] media: sta2x11_vip: allow build with COMPILE_TEST

2018-04-20 Thread Mauro Carvalho Chehab
This driver doesn't use any weird API. So, allow building it with COMPILE_TEST. Signed-off-by: Mauro Carvalho Chehab <mche...@s-opensource.com> --- drivers/media/pci/sta2x11/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/pci/sta2x11/Kconfig b/d

<    1   2   3   4   5   6   7   8   9   10   >