cron job: media_tree daily build: ERRORS

2018-02-27 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 Feb 28 05:00:18 CET 2018 media-tree git hash:e3e389f931a14ddf43089c7db92fc5d74edf93a4 media_build git

Re: [PATCH for v3.2 00/12] v4l2-compat-ioctl32.c: remove set_fs(KERNEL_DS)

2018-02-27 Thread Ben Hutchings
On Wed, 2018-02-14 at 13:03 +0100, Hans Verkuil wrote: > From: Hans Verkuil > > This patch series fixes a number of bugs and culminates in the removal > of the set_fs(KERNEL_DS) call in v4l2-compat-ioctl32.c. > > This was tested with a VM running 3.2, the vivi driver (a poor substitute for > the

Re: [PATCH for v3.16 00/14] v4l2-compat-ioctl32.c: remove set_fs(KERNEL_DS)

2018-02-27 Thread Ben Hutchings
On Wed, 2018-02-14 at 12:59 +0100, Hans Verkuil wrote: > From: Hans Verkuil > > This patch series fixes a number of bugs and culminates in the removal > of the set_fs(KERNEL_DS) call in v4l2-compat-ioctl32.c. > > This was tested with a VM running 3.16, the vivi driver (a poor substitute for > th

[linuxtv-media:fixes 3/12] drivers/media/dvb-core/dvb_vb2.c:183: undefined reference to `vb2_vmalloc_memops'

2018-02-27 Thread kbuild test robot
tree: git://linuxtv.org/media_tree.git fixes head: 7dbdd16a79a9d27d7dca0a49029fc8966dcfecc5 commit: ec5b100462543aee1f3e139e168699fd3b05cdc6 [3/12] media: dvb: fix DVB_MMAP symbol name config: x86_64-randconfig-s2-02280514 (attached as .config) compiler: gcc-6 (Debian 6.4.0-9) 6.4.0 20171026 r

Re: [PATCH 1/2] usbtv: Use same decoder sequence as Windows driver

2018-02-27 Thread Hugo "Bonstra" Grostabussiat
Le 26/02/2018 à 15:12, Hans Verkuil a écrit : > Thanks for this patch, but I am a bit hesitant to apply it. Did you test > that PAL and NTSC still work after this change? I did test with both a PAL and a NTSC source before submitting the patch. It seemed to work fine. However, after reading your

Re: [v5 2/2] media: dt-bindings: Add bindings for Dongwoon DW9807 voice coil

2018-02-27 Thread Rob Herring
On Fri, Feb 23, 2018 at 10:13 AM, Andy Yeh wrote: > From: Alan Chiang > > Dongwoon DW9807 is a voice coil lens driver. > > Also add a vendor prefix for Dongwoon for one did not exist previously. Where's that? > > Signed-off-by: Andy Yeh > --- > Documentation/devicetree/bindings/media/i2c/dong

With All Due Respect!!

2018-02-27 Thread Mrs.Louisa Benicio
Greetings, I am Mrs. Louisa Benicio; I have decided to donate what I have to You/Churches/ Motherless babies/Less privileged/Widows’ because I am dying and diagnosed for cancer for about 2 years ago. I have been touched by God Almighty to donate from what I have inherited from my late husband to y

[PATCH RFC] media: em28xx: don't use coherent buffer for DMA transfers

2018-02-27 Thread Mauro Carvalho Chehab
While coherent memory is cheap on x86, it has problems on arm. So, stop using it. Signed-off-by: Mauro Carvalho Chehab --- I wrote this patch in order to check if this would make things better for ISOCH transfers on Raspberry Pi3. It didn't. Yet, keep using coherent memory at USB drivers seem an

[PATCH] media: platform: renesas-ceu: Fix CSTRST_CPON mask

2018-02-27 Thread Jacopo Mondi
The CSTRST_CPON mask was wrongly assigned to BIT(1) instead of BIT(0). Fix that by changing the mask opportunely. Reported-by: Dylan Laduranty Signed-off-by: Jacopo Mondi --- Mauro: could you please pick up this patch since you already applied the CEU series to your tree if I'm not wrong? Laur

Re: [PATCH v3 05/10] pwm: add PWM mode to pwm_config()

2018-02-27 Thread Claudiu Beznea
On 27.02.2018 17:38, Daniel Thompson wrote: > On Tue, Feb 27, 2018 at 01:40:58PM +0200, Claudiu Beznea wrote: >> On 27.02.2018 12:54, Daniel Thompson wrote: >>> On Mon, Feb 26, 2018 at 04:24:15PM +0200, Claudiu Beznea wrote: On 26.02.2018 11:57, Jani Nikula wrote: > On Thu, 22 Feb 2018,

[PATCH 12/13] media: ov772x: Re-order variables declaration

2018-02-27 Thread Jacopo Mondi
Re-order variables declaration to respect 'reverse christmas tree' ordering whenever possible. Signed-off-by: Jacopo Mondi --- drivers/media/i2c/ov772x.c | 17 + 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/drivers/media/i2c/ov772x.c b/drivers/media/i2c/ov772x.c

[PATCH 11/13] media: ov772x: Empty line before end-of-function return

2018-02-27 Thread Jacopo Mondi
Add an empty line before return at the end of functions. Signed-off-by: Jacopo Mondi --- drivers/media/i2c/ov772x.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/media/i2c/ov772x.c b/drivers/media/i2c/ov772x.c index 8849da1..4f464ac 100644 --- a/drivers/media/i2c/ov772x.c +++ b

[PATCH 05/13] media: tw9910: Re-organize in-code comments

2018-02-27 Thread Jacopo Mondi
A lot of comments that would fit a single line were spread on two or more lines. Also fix capitalization and punctuation where appropriate. Signed-off-by: Jacopo Mondi --- drivers/media/i2c/tw9910.c | 44 +--- 1 file changed, 13 insertions(+), 31 deletions

[PATCH 13/13] media: ov772x: Replace msleep(1) with usleep_range

2018-02-27 Thread Jacopo Mondi
msleep() can sleep up to 20ms. As suggested by Documentation/timers/timers_howto.txt replace it with usleep_range() with up to 5ms delay. Signed-off-by: Jacopo Mondi --- drivers/media/i2c/ov772x.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/i2c/ov772x.c b/d

[PATCH 06/13] media: tw9910: Mixed style fixes

2018-02-27 Thread Jacopo Mondi
Two minor style fixes, align function parameter and remove un-necessary spaces. Signed-off-by: Jacopo Mondi --- drivers/media/i2c/tw9910.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/i2c/tw9910.c b/drivers/media/i2c/tw9910.c index f082f6d..f9b75c1 100644

[PATCH 02/13] media: tw9910: Empty line before end-of-function return

2018-02-27 Thread Jacopo Mondi
Add an empty line before return at the end of functions. Signed-off-by: Jacopo Mondi --- drivers/media/i2c/tw9910.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/drivers/media/i2c/tw9910.c b/drivers/media/i2c/tw9910.c index 70e0ae2..3e4b530 100644 --- a/drivers/media/i2c/tw9910.c +++

[PATCH 04/13] media: tw9910: Re-order variables declaration

2018-02-27 Thread Jacopo Mondi
Re-order variables declaration to respect 'reverse christmas tree' ordering whenever possible. Signed-off-by: Jacopo Mondi --- drivers/media/i2c/tw9910.c | 23 +++ 1 file changed, 11 insertions(+), 12 deletions(-) diff --git a/drivers/media/i2c/tw9910.c b/drivers/media/i2c/t

[PATCH 01/13] media: tw9910: Fix parameter alignment issue

2018-02-27 Thread Jacopo Mondi
Align parameters to first open brace. Signed-off-by: Jacopo Mondi --- drivers/media/i2c/tw9910.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/media/i2c/tw9910.c b/drivers/media/i2c/tw9910.c index cc5d383..70e0ae2 100644 --- a/drivers/media/i2c/tw9910.c +++

[PATCH 03/13] media: tw9910: Align function parameters

2018-02-27 Thread Jacopo Mondi
Align all function parameters to first open brace when declaring functions. Signed-off-by: Jacopo Mondi --- drivers/media/i2c/tw9910.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/media/i2c/tw9910.c b/drivers/media/i2c/tw9910.c index 3e4b530..45afdb9 100644

[PATCH 08/13] media: tw9910: Replace msleep(1) with usleep_range

2018-02-27 Thread Jacopo Mondi
msleep() can sleep up to 20ms. As suggested by Documentation/timers/timers_howto.txt replace it with usleep_range() with up to 5ms delay. Signed-off-by: Jacopo Mondi --- drivers/media/i2c/tw9910.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/i2c/tw9910.c b/d

[PATCH 09/13] media: ov772x: Align function parameters

2018-02-27 Thread Jacopo Mondi
Align all function parameters to first open brace when declaring functions. Signed-off-by: Jacopo Mondi --- drivers/media/i2c/ov772x.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/media/i2c/ov772x.c b/drivers/media/i2c/ov772x.c index 16665af..a418455 100644

[PATCH 07/13] media: tw9910: Sort includes alphabetically

2018-02-27 Thread Jacopo Mondi
Sort include directives alphabetically to ease maintenance. Signed-off-by: Jacopo Mondi --- drivers/media/i2c/tw9910.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/media/i2c/tw9910.c b/drivers/media/i2c/tw9910.c index f9b75c1..1043f7d 100644 --- a/drivers/med

[PATCH 10/13] media: ov772x: Re-organize in-code comments

2018-02-27 Thread Jacopo Mondi
A lot of comments that would fit a single line were spread on two or more lines. Also fix capitalization and punctuation where appropriate. Signed-off-by: Jacopo Mondi --- drivers/media/i2c/ov772x.c | 32 ++-- 1 file changed, 10 insertions(+), 22 deletions(-) diff --

[PATCH 00/13] media: ov772x/tw9910 cleanup

2018-02-27 Thread Jacopo Mondi
Hi Mauro, as you have started cleaning up those two drivers as they've been now moved away from soc_camera I have added a few style fixes for both of them on top of your two patches: commit ae24b8a1d5f9 ("media: tw9910: solve coding style issues") commit 2d595d14fe8b ("media: ov772x: fix whitesp

Re: [PATCH v3 05/10] pwm: add PWM mode to pwm_config()

2018-02-27 Thread Daniel Thompson
On Tue, Feb 27, 2018 at 01:40:58PM +0200, Claudiu Beznea wrote: > On 27.02.2018 12:54, Daniel Thompson wrote: > > On Mon, Feb 26, 2018 at 04:24:15PM +0200, Claudiu Beznea wrote: > >> On 26.02.2018 11:57, Jani Nikula wrote: > >>> On Thu, 22 Feb 2018, Daniel Thompson wrote: > On Thu, Feb 22, 20

Re: [Patch v8 12/12] Documention: v4l: Documentation for HEVC CIDs

2018-02-27 Thread Nicolas Dufresne
Le vendredi 02 février 2018 à 17:55 +0530, Smitha T Murthy a écrit : > Added V4l2 controls for HEVC encoder > > Signed-off-by: Smitha T Murthy > --- > Documentation/media/uapi/v4l/extended-controls.rst | 410 > + > 1 file changed, 410 insertions(+) > > diff --git a/Documenta

Re: [Patch v8 12/12] Documention: v4l: Documentation for HEVC CIDs

2018-02-27 Thread Nicolas Dufresne
Le vendredi 02 février 2018 à 17:55 +0530, Smitha T Murthy a écrit : > Added V4l2 controls for HEVC encoder > > Signed-off-by: Smitha T Murthy > --- > Documentation/media/uapi/v4l/extended-controls.rst | 410 > + > 1 file changed, 410 insertions(+) > > diff --git a/Documenta

Re: [PATCH V2 1/2] [media] Add Rockchip RK1608 driver

2018-02-27 Thread Heiko Stuebner
Hi Leo, Am Dienstag, 27. Februar 2018, 04:15:46 CET schrieb 温暖: > Thank you for your advice! I will revise it according to your suggestion. please also keep an eye on my reply to Linus' mail pointing out some other issues where the driver should not tie into soc-specific areas like the clock cont

[PATCH v2 1/3] media: i2c: adv748x: Simplify regmap configuration

2018-02-27 Thread Kieran Bingham
From: Kieran Bingham The ADV748x has identical map configurations for each register map. The duplication of each map can be simplified using a helper macro such that each map is represented on a single line. Define ADV748X_REGMAP_CONF for this purpose use it to create the tables. Signed-off-by:

[PATCH v2 0/3] media: i2c: adv748x: Fix CBUS page issue

2018-02-27 Thread Kieran Bingham
The ADV748x has 12 pages mapped on to I2C addresses. The existing implementation only has 11 of these in the map enumeration, and this can cause an off-by-one error when programming the map addresses. This short series simplifies the regmap configuration tables, and adds the missing CBUS page to

[PATCH v2 2/3] media: i2c: adv748x: Add missing CBUS page

2018-02-27 Thread Kieran Bingham
From: Kieran Bingham The ADV748x has 12 pages mapped onto I2C addresses. In the existing implementation only 11 are mapped correctly in the page enumerations, which causes an off-by-one fault on pages above the infoframe definition due to a missing 'CBUS' page. This causes the address for the C

[PATCH v2 3/3] media: i2c: adv748x: Add support for i2c_new_secondary_device

2018-02-27 Thread Kieran Bingham
From: Kieran Bingham The ADV748x has twelve 256-byte maps that can be accessed via the main I2C ports. Each map has it own I2C address and acts as a standard slave device on the I2C bus. Allow a device tree node to override the default addresses so that address conflicts with other devices on th

Re: [PATCH 2/2] media: tw9910: solve coding style issues

2018-02-27 Thread jacopo mondi
Hi Mauro, thanks for doing this. I didn't dare to touch this driver style issues as it was mainline already, but since you addressed this I now would have more changes to apply... On Mon, Feb 26, 2018 at 09:28:08AM -0500, Mauro Carvalho Chehab wrote: > As we're adding this as a new driver, mak

[PATCH] media: rc: lirc does not use LIRC_CAN_SEND_SCANCODE feature

2018-02-27 Thread Sean Young
Since commit 02d742f4b209 ("media: lirc: lirc daemon fails to detect raw IR device"), the feature LIRC_CAN_SEND_SCANCODE is no longer used as it tripped up lircd. The ability to send scancodes for IR Tx is implied by LIRC_CAN_SEND_PULSE (i.e. any device that can send can use IR Tx encoders). So, r

Re: [PATCH v2] Staging: bcm2048: Fix function argument alignment in radio-bcm2048.c.

2018-02-27 Thread Greg KH
On Tue, Feb 27, 2018 at 08:32:30AM +0100, Hans Verkuil wrote: > On 02/27/2018 02:53 AM, Quytelda Kahja wrote: > > Hans, > > > > Thank you very much for your input on the patch; however this patch > > has already been applied to the staging tree. Additionally: > > I have no record of this being a

[GIT PULL] Remove metag architecture

2018-02-27 Thread James Hogan
Hi Arnd, On Fri, Feb 23, 2018 at 01:26:09PM +0100, Arnd Bergmann wrote: > On Fri, Feb 23, 2018 at 12:02 PM, James Hogan wrote: > > I'm happy to put v2 in linux-next now (only patch 4 has changed, I just > > sent an updated version), and send you a pull request early next week so > > you can take

Re: [PATCH v3 05/10] pwm: add PWM mode to pwm_config()

2018-02-27 Thread Claudiu Beznea
On 27.02.2018 12:54, Daniel Thompson wrote: > On Mon, Feb 26, 2018 at 04:24:15PM +0200, Claudiu Beznea wrote: >> On 26.02.2018 11:57, Jani Nikula wrote: >>> On Thu, 22 Feb 2018, Daniel Thompson wrote: On Thu, Feb 22, 2018 at 02:01:16PM +0200, Claudiu Beznea wrote: > Add PWM mode to pwm_

Re: [PATCH v3 05/10] pwm: add PWM mode to pwm_config()

2018-02-27 Thread Daniel Thompson
On Mon, Feb 26, 2018 at 04:24:15PM +0200, Claudiu Beznea wrote: > On 26.02.2018 11:57, Jani Nikula wrote: > > On Thu, 22 Feb 2018, Daniel Thompson wrote: > >> On Thu, Feb 22, 2018 at 02:01:16PM +0200, Claudiu Beznea wrote: > >>> Add PWM mode to pwm_config() function. The drivers which uses pwm_con

[GIT PULL FOR v4.17] Minor RC fixes

2018-02-27 Thread Sean Young
Hi Mauro, Just two minor changes for RC. Thanks, Sean The following changes since commit 15ea2df9143729a2b722d4ca2b52cfa14a819d8e: media: ov2685: mark PM functions as __maybe_unused (2018-02-26 10:38:56 -0500) are available in the Git repository at: git://linuxtv.org/syoung/media_tree.git

Re: [PATCH 01/13] media: v4l2-fwnode: Let parse_endpoint callback decide if no remote is error

2018-02-27 Thread Laurent Pinchart
Hi Philipp, On Tuesday, 27 February 2018 11:13:04 EET Philipp Zabel wrote: > On Fri, 2018-02-23 at 14:47 +0200, Sakari Ailus wrote: > > On Fri, Feb 23, 2018 at 12:16:17PM +0100, Philipp Zabel wrote: > >> On Fri, 2018-02-23 at 12:05 +0200, Laurent Pinchart wrote: > >>> On Friday, 23 February 2018 1

Re: [PATCH] media: platform: Drop OF dependency of VIDEO_RENESAS_VSP1

2018-02-27 Thread Laurent Pinchart
Hi Geert, Thank you for the patch. On Monday, 26 February 2018 20:09:10 EET Geert Uytterhoeven wrote: > VIDEO_RENESAS_VSP1 depends on ARCH_RENESAS && OF. > As ARCH_RENESAS implies OF, the latter can be dropped. > > Signed-off-by: Geert Uytterhoeven Acked-by: Laurent Pinchart and taken in my

Re: [PATCH 01/13] media: v4l2-fwnode: Let parse_endpoint callback decide if no remote is error

2018-02-27 Thread Philipp Zabel
Hi Sakari, On Fri, 2018-02-23 at 14:47 +0200, Sakari Ailus wrote: > Hi Philipp, > > On Fri, Feb 23, 2018 at 12:16:17PM +0100, Philipp Zabel wrote: > > Hi Laurent, > > > > On Fri, 2018-02-23 at 12:05 +0200, Laurent Pinchart wrote: > > > Hi Philipp, > > > > > > On Friday, 23 February 2018 11:56:5

Re: [PATCH 02/15] v4l: vsp1: Remove outdated comment

2018-02-27 Thread Laurent Pinchart
Hi Sergei, On Tuesday, 27 February 2018 10:22:25 EET Sergei Shtylyov wrote: > On 2/27/2018 12:45 AM, Laurent Pinchart wrote: > > The entities in the pipeline are all started when the LIF is setup. > > Remove the outdated comment that state otherwise. > > States? You're right, will fix in v2. >

Re: [PATCH 3/3] media: imx: Don't initialize vars that won't be used

2018-02-27 Thread Philipp Zabel
On Mon, 2018-02-26 at 08:40 -0500, Mauro Carvalho Chehab wrote: > As reported by gcc: > > + drivers/staging/media/imx/imx-media-csi.c: warning: variable 'input_fi' > set but not used [-Wunused-but-set-variable]: => 671:33 > + drivers/staging/media/imx/imx-media-csi.c: warning: variable 'pinc

Re: [PATCH] media: platform: Drop OF dependency of VIDEO_RENESAS_VSP1

2018-02-27 Thread Simon Horman
On Mon, Feb 26, 2018 at 07:09:10PM +0100, Geert Uytterhoeven wrote: > VIDEO_RENESAS_VSP1 depends on ARCH_RENESAS && OF. > As ARCH_RENESAS implies OF, the latter can be dropped. > > Signed-off-by: Geert Uytterhoeven My reasoning is that ARCH_RENESAS depends on ARCH_MULTI_V7, which in turn depends

Re: [PATCH] media: imx: add 8-bit grayscale support

2018-02-27 Thread Philipp Zabel
Hi Hans, On Thu, 2018-02-15 at 15:43 +0100, Hans Verkuil wrote: > Hi Philipp, > > Can you let me know if/when I can merge this? It looks good, so when the other > patch is merged, then this can be merged as well. Thank you for the reminder, the required patch is now merged into v4.16-rc3, commit

Re: [PATCH v8 1/2] dt-bindings: media: Add Allwinner V3s Camera Sensor Interface (CSI)

2018-02-27 Thread Yong
On Tue, 27 Feb 2018 09:20:39 +0100 Philipp Zabel wrote: > On Tue, 2018-02-27 at 10:07 +0800, Yong Deng wrote: > > Add binding documentation for Allwinner V3s CSI. > > > > Acked-by: Maxime Ripard > > Acked-by: Sakari Ailus > > Reviewed-by: Rob Herring > > Signed-off-by: Yong Deng > > --- > >

Re: [PATCH 02/15] v4l: vsp1: Remove outdated comment

2018-02-27 Thread Sergei Shtylyov
Hello! On 2/27/2018 12:45 AM, Laurent Pinchart wrote: The entities in the pipeline are all started when the LIF is setup. Remove the outdated comment that state otherwise. States? Signed-off-by: Laurent Pinchart [...] MBR, Sergei

Re: [PATCH v8 1/2] dt-bindings: media: Add Allwinner V3s Camera Sensor Interface (CSI)

2018-02-27 Thread Philipp Zabel
On Tue, 2018-02-27 at 10:07 +0800, Yong Deng wrote: > Add binding documentation for Allwinner V3s CSI. > > Acked-by: Maxime Ripard > Acked-by: Sakari Ailus > Reviewed-by: Rob Herring > Signed-off-by: Yong Deng > --- > .../devicetree/bindings/media/sun6i-csi.txt| 59 >