cron job: media_tree daily build: ERRORS

2018-05-05 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: Sun May 6 05:00:12 CEST 2018 media-tree git hash:f10379aad39e9da8bc7d1822e251b5f0673067ef media_build

Re: [PATCH 0/2] media: imx: add capture support for RGB565_2X8 on parallel bus

2018-05-05 Thread Steve Longerbeam
Hi Jan, Philipp, I reviewed this patch series, and while I don't have any objections to the code-level changes, but my question is more specifically about how the IPU/CSI deals with receiving RGB565 over a parallel bus. My understanding was that if the CSI receives RGB565 over a parallel 8-bit

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 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 (attached as .config) > > compiler:

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

2018-05-05 Thread Andy Shevchenko
On Sat, May 5, 2018 at 4:24 PM, Mauro Carvalho Chehab wrote: > 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 > ^~~ > Why not to go

Re: [PATCH] media: imx-csi: fix burst size for 16 bit

2018-05-05 Thread Steve Longerbeam
Acked-by: Steve Longerbeam On 05/03/2018 09:40 AM, Philipp Zabel wrote: On Thu, 2018-05-03 at 18:32 +0200, Jan Luebbe wrote: A burst_size of 4 does not work for the 16 bit passthrough formats, so we use 8 instead. Signed-off-by: Jan Luebbe

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

2018-05-05 Thread Steve Longerbeam
Acked-by: Steve Longerbeam On 05/03/2018 08:06 AM, Philipp Zabel wrote: Since commit 50b0f0aee839 ("gpu: ipu-csi: add 10/12-bit grayscale support to mbus_code_to_bus_cfg") the IPU CSI can be configured to capture 10-bit and 12-bit grayscale formats, expanded to

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

2018-05-05 Thread Randy Dunlap
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 (attached as .config) > compiler: or1k-linux-gcc (GCC) 6.0.0 20160327 (experimental) > > All errors (new ones prefixed by >>): > >

[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" Cc: linux...@kvack.org Signed-off-by: Mauro Carvalho Chehab

[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" Cc: linux...@kvack.org Signed-off-by: Mauro Carvalho Chehab ---

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 --- drivers/media/common/siano/smscoreapi.c | 20

[linuxtv-media:master] BUILD REGRESSION 8d718e5376c602dfd41b599dcc2a7b1be07c7b6b

2018-05-05 Thread kbuild test robot
tree/branch: git://linuxtv.org/media_tree.git master branch HEAD: 8d718e5376c602dfd41b599dcc2a7b1be07c7b6b media: frontends: fix ops get_algo()'s return type Regressions in current branch: drivers/video/fbdev/omap2/omapfb/displays/connector-analog-tv.c:183:21: error:

[PATCH] [bug] cx231xx: Fix recursive dependency

2018-05-05 Thread Brad Love
0day build bot reported an unnoticed recursive dependency, fix it by removing the select statement. fixes: c66d4d99a8fb ("cx231xx: Add I2C_MUX dependency") Signed-off-by: Brad Love --- drivers/media/usb/cx231xx/Kconfig | 1 - 1 file changed, 1 deletion(-) diff --git

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
Em Fri, 27 Apr 2018 01:13:32 +0900 Akinobu Mita escreveu: > (This patch is in prototype stage) > > This adds SCCB helper functions (sccb_read_byte and sccb_write_byte) that > are intended to be used by some of Omnivision sensor drivers. What do you mean by "SCCB"? >

Charity Gift !!!

2018-05-05 Thread Mrs Mavis L. Wanczyk
-- This is the second time i am sending you this mail. I, Mavis Wanczyk donates $ 5 Million Dollars from part of my Powerball Jackpot Lottery of $ 758 Million Dollars, respond with your details for claims. I await your earliest response and God Bless you Good luck. Mavis Wanczyk

[linuxtv-media:master 167/254] drivers/video/fbdev//omap2/omapfb/dss/overlay.c:41:5: error: redefinition of 'omap_dss_get_num_overlays'

2018-05-05 Thread kbuild test robot
tree: git://linuxtv.org/media_tree.git master head: 8d718e5376c602dfd41b599dcc2a7b1be07c7b6b commit: 771f7be87ff921e9a3d744febd606af39a150e14 [167/254] media: omapfb: omapfb_dss.h: add stubs to build with COMPILE_TEST && DRM_OMAP config: sh-allmodconfig (attached as .config) compiler:

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

2018-05-05 Thread Laurent Pinchart
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 ? > > Not aware of any crc core work going on in drm, so has my ack. Thank you. > Worst case we do a topic branch or

[GIT PULL FOR v4.18] VSP1 & DU CRC calculation support

2018-05-05 Thread Laurent Pinchart
Hi Mauro, The following changes since commit f2bea20ec56234a8ca8ec4a3481b744b3e0e8813: media: cx231xx: remove a now unused var (2018-05-05 08:55:29 -0400) are available in the Git repository at: git://linuxtv.org/pinchartl/media.git v4l2/vsp1/discom for you to fetch changes up to

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

2018-05-05 Thread Laurent Pinchart
Hi Mauro, On Saturday, 5 May 2018 13:35:38 EEST Mauro Carvalho Chehab wrote: > Em Wed, 25 Apr 2018 03:37:00 +0300 Laurent Pinchart escreveu: > > Hi Mauro, > > > > The following changes since commit 60cc43fc888428bb2f18f08997432d426a243338: > > Linux 4.17-rc1 (2018-04-15 18:24:20 -0700) > > >

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

2018-05-05 Thread Mauro Carvalho Chehab
Em Thu, 19 Apr 2018 22:00:07 +0200 Wolfram Sang escreveu: > This header only contains platform_data. Move it to the proper directory. > > Signed-off-by: Wolfram Sang > --- > MAINTAINERS | 2 +- >

[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 Signed-off-by: Mauro Carvalho Chehab

[linuxtv-media:master 235/238] drivers/media//pci/pt1/pt1.c:1433:5: warning: "CONFIG_PM_SLEEP" is not defined, evaluates to 0

2018-05-05 Thread kbuild test robot
tree: git://linuxtv.org/media_tree.git master head: fbfb3a75bd52249fe6e91b420cf2ae135ad5 commit: 41cb54e20543d436ee6331cd66dd0413a7452804 [235/238] media: dvb: earth-pt1: add support for suspend/resume config: xtensa-allmodconfig (attached as .config) compiler: xtensa-linux-gcc (GCC)

Re: compile error media-build on 4.15 because of 'device_get_match_data'

2018-05-05 Thread Jasmin J.
Hello! I just pushed a fix to media-build. But I had to disable the driver for Kernels older than 4.16. -> VIDEO_I2C requires device_get_match_data which requires the function pointer device_get_match_data in fwnode_operations. The latter has been added first in Kernel 4.16. If someone wants

compile error media-build on 4.15 because of 'device_get_match_data'

2018-05-05 Thread Martin Dauskardt
I tried to compile media-build on Ubuntu 18.04. (gcc 7.3.0) with Kernel 4.15 and get this error: /home/martin/media_build/v4l/video-i2c.c: In function 'video_i2c_probe': /home/martin/media_build/v4l/video-i2c.c:456:16: error: implicit declaration of function 'device_get_match_data'; did you

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

2018-05-05 Thread Andy Shevchenko
On Sat, May 5, 2018 at 2:33 PM, Mauro Carvalho Chehab wrote: > 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. > +

[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 --- drivers/media/pci/pt1/pt1.c | 3 ++-

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] media: i2c: adv748x: Fix pixel rate values

2018-05-05 Thread Niklas Söderlund
Hi Laurent, On 2018-05-05 12:48:58 +0300, Laurent Pinchart wrote: > Hi Niklas, > > On Saturday, 5 May 2018 01:58:10 EEST Niklas Söderlund wrote: > > On 2018-04-25 01:36:42 +0200, Niklas Söderlund wrote: > > > On 2018-04-21 15:44:44 +0300, Laurent Pinchart wrote: > > >> The pixel rate, as

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 2/2] dvbsky: Add support for MyGica T230C v2

2018-05-05 Thread Thomas Hollstegge
Support for newer revisions of the MyGica T230C, shipping with a different USB pid. Although sometimes referred to as T230C2, the device is sold under its original name T230C. Besides a slightly different PCB layout and some different minor components, it utilizes the same bridge, demodulator and

[PATCH 0/2] dvbsky: Add support for MyGica T230C v2

2018-05-05 Thread Thomas Hollstegge
Add support for newer revisions of the USB DVB-C/DVB-T/DVB-T2 stick MyGica T230C, sometimes referred to as MyGica T230C2. The device needs a fixed TS clock frequency of 10MHz to be able to demod some channels. This is done by adding two new optional configuration options for the Si2168 demod

[PATCH 1/2] si2168: Set TS clock mode and frequency

2018-05-05 Thread Thomas Hollstegge
Some devices require a higher TS clock frequency to demodulate some muxes. This adds two optional parameters to control the TS clock frequency mode as well as the frequency. Signed-off-by: Thomas Hollstegge --- drivers/media/dvb-frontends/si2168.c | 20

Re: [PATCH] media: i2c: adv748x: Fix pixel rate values

2018-05-05 Thread Laurent Pinchart
Hi Niklas, On Saturday, 5 May 2018 01:58:10 EEST Niklas Söderlund wrote: > On 2018-04-25 01:36:42 +0200, Niklas Söderlund wrote: > > On 2018-04-21 15:44:44 +0300, Laurent Pinchart wrote: > >> The pixel rate, as reported by the V4L2_CID_PIXEL_RATE control, must > >> include both horizontal and

[PATCH] [media] gspca: Stop using GFP_DMA for buffers for USB bulk transfers

2018-05-05 Thread Hans de Goede
The recent "x86 ZONE_DMA love" discussion at LSF/MM pointed out that some gspca sub-drivvers are using GFP_DMA to allocate buffers which are used for USB bulk transfers, there is absolutely no need for this, drop it. Cc: "Luis R. Rodriguez" Signed-off-by: Hans de Goede

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

2018-05-05 Thread SF Markus Elfring
> @@ -656,18 +656,18 @@ static int dvb_dmxdev_start_feed(struct dmxdev *dmxdev, > tsfeed->priv = filter; > > ret = tsfeed->set(tsfeed, feed->pid, ts_type, ts_pes, timeout); > - if (ret < 0) { > - dmxdev->demux->release_ts_feed(dmxdev->demux, tsfeed); > -

Re: 4.17-rc3 regression in UVC driver

2018-05-05 Thread Kieran Bingham
-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: >> Hi, >> >> I just got the following error message every ms with 4.17-rc3 after >> upgrading to for first ~192 seconds after

Re: 4.17-rc3 regression in UVC driver

2018-05-05 Thread Kieran Bingham
Hi Sebastian, On 04/05/18 19:45, Sebastian Reichel wrote: > Hi, > > I just got the following error message every ms with 4.17-rc3 after > upgrading to for first ~192 seconds after system start (Debian > 4.17~rc3-1~exp1 kernel) on my Thinkpad X250: > >> uvcvideo: Failed to query (GET_MIN) UVC