Re: [PATCH] media: mtk-vcodec: add missing MODULE_LICENSE/DESCRIPTION

2017-11-21 Thread kbuild test robot
Hi Jesse, Thank you for the patch! Yet something to improve: [auto build test ERROR on linuxtv-media/master] [also build test ERROR on v4.14 next-20171121] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci

Re: [PATCH 2/5] media: dt-bindings: Add bindings for TDA1997X

2017-11-21 Thread Sakari Ailus
Hi Tim, On Thu, Nov 09, 2017 at 10:45:33AM -0800, Tim Harvey wrote: > Cc: Rob Herring > Signed-off-by: Tim Harvey > --- > v3: > - fix typo > > v2: > - add vendor prefix and remove _ from vidout-portcfg > - remove _ from labels > - remove

Re: [PATCH] media: mtk-vcodec: add missing MODULE_LICENSE/DESCRIPTION

2017-11-21 Thread Randy Dunlap
On 11/21/17 23:41, kbuild test robot wrote: > Hi Jesse, > > Thank you for the patch! Yet something to improve: missing #include Jesse, did you build all of these driver changes? > [auto build test ERROR on linuxtv-media/master] > [also build test ERROR on v4.14 next-20171

Re: [PATCH] c8sectpfe: fix potential NULL pointer dereference in c8sectpfe_timer_interrupt

2017-11-21 Thread Patrice CHOTARD
Hi Gustavo On 11/20/2017 03:00 PM, Gustavo A. R. Silva wrote: > _channel_ is being dereferenced before it is null checked, hence there is a > potential null pointer dereference. Fix this by moving the pointer dereference > after _channel_ has been null checked. > > This issue was detected with

Re: [PATCH] [media] c8sectpfe: Use resource_size function on memory resource

2017-11-21 Thread Patrice CHOTARD
Hi Vasyl On 11/20/2017 11:46 PM, Vasyl Gomonovych wrote: > To adapt fei->sram_size calculation via resource_size for memory size > calculation before, in fei->sram = devm_ioremap_resource(dev, res). > And make memory initialization range in > memset_io for fei->sram appropriate > >

iMX6q/coda encoder failures with ffmpeg/gstreamer m2m encoders

2017-11-21 Thread Neil Armstrong
Hi, I'm trying to make the coda960 h.264 encoder work on an i.MX6q SoC with Linux 4.14 and the 3.1.1 firmware. # dmesg | grep coda [4.846574] coda 204.vpu: Direct firmware load for vpu_fw_imx6q.bin failed with error -2 [4.901351] coda 204.vpu: Using fallback firmware

Re: [PATCH] reservation: don't wait when timeout=0

2017-11-21 Thread Chris Wilson
Quoting Christian König (2017-11-21 15:49:55) > Am 21.11.2017 um 15:59 schrieb Rob Clark: > > On Tue, Nov 21, 2017 at 9:38 AM, Chris Wilson > > wrote: > >> Quoting Rob Clark (2017-11-21 14:08:46) > >>> If we are testing if a reservation object's fences have been > >>>

Re: [PATCH v4 04/12] intel-ipu3: Add user space ABI definitions

2017-11-21 Thread sakari.ai...@linux.intel.com
Hi Rajmohan, My apologies for the late reply. On Sat, Nov 11, 2017 at 04:07:22AM +, Mani, Rajmohan wrote: > Hi Sakari, > > > -Original Message- > > From: Sakari Ailus [mailto:sakari.ai...@iki.fi] > > Sent: Friday, October 20, 2017 2:30 AM > > To: Zhi, Yong > >

Re: [PATCH v2 1/3] media: V3s: Add support for Allwinner CSI.

2017-11-21 Thread Maxime Ripard
Hi, On Thu, Jul 27, 2017 at 01:01:35PM +0800, Yong Deng wrote: > Allwinner V3s SoC have two CSI module. CSI0 is used for MIPI interface > and CSI1 is used for parallel interface. This is not documented in > datasheet but by testing and guess. > > This patch implement a v4l2 framework driver for

Re: [PATCH] reservation: don't wait when timeout=0

2017-11-21 Thread Christian König
Am 21.11.2017 um 15:59 schrieb Rob Clark: On Tue, Nov 21, 2017 at 9:38 AM, Chris Wilson wrote: Quoting Rob Clark (2017-11-21 14:08:46) If we are testing if a reservation object's fences have been signaled with timeout=0 (non-blocking), we need to pass 0 for timeout

Re: [PATCH] reservation: don't wait when timeout=0

2017-11-21 Thread Christian König
Am 21.11.2017 um 16:58 schrieb Chris Wilson: Quoting Christian König (2017-11-21 15:49:55) Am 21.11.2017 um 15:59 schrieb Rob Clark: On Tue, Nov 21, 2017 at 9:38 AM, Chris Wilson wrote: Quoting Rob Clark (2017-11-21 14:08:46) If we are testing if a reservation

[PATCH] [media] stb0899: remove redundant self assignment of k_indirect

2017-11-21 Thread Colin King
From: Colin Ian King The self assignment of k_indirect is redundant and can be removed. Detected using coccinelle. Signed-off-by: Colin Ian King --- drivers/media/dvb-frontends/stb0899_algo.c | 3 +-- 1 file changed, 1 insertion(+), 2

usb/media/em28xx: use-after-free in dvb_unregister_frontend

2017-11-21 Thread Andrey Konovalov
Hi! I've got the following report while fuzzing the kernel with syzkaller. On commit e1d1ea549b57790a3d8cf6300e6ef86118d692a3 (4.15-rc1). em28xx 1-1:9.0: Disconnecting tc90522 1-0015: Toshiba TC90522 attached. qm1d1c0042 2-0061: Sharp QM1D1C0042 attached. dvbdev: DVB: registering new adapter

Re: [PATCH 1/2] drivers/video/hdmi: allow for larger-than-needed vendor IF

2017-11-21 Thread Ville Syrjälä
On Mon, Nov 20, 2017 at 04:02:14PM +0100, Hans Verkuil wrote: > On 11/20/2017 03:51 PM, Ville Syrjälä wrote: > > On Mon, Nov 20, 2017 at 02:41:28PM +0100, Hans Verkuil wrote: > >> From: Hans Verkuil > >> > >> Some devices (Windows Intel driver!) send a Vendor InfoFrame that >

Re: [PATCH] reservation: don't wait when timeout=0

2017-11-21 Thread Christian König
Am 21.11.2017 um 15:08 schrieb Rob Clark: If we are testing if a reservation object's fences have been signaled with timeout=0 (non-blocking), we need to pass 0 for timeout to dma_fence_wait_timeout(). Plus bonus spelling correction. Signed-off-by: Rob Clark

[PATCH] [media] dvb-frontends/stv0367: remove redundant self assignment of temporary

2017-11-21 Thread Colin King
From: Colin Ian King The self assignment of temporary is redundant and can be removed. Detected using coccinelle. Signed-off-by: Colin Ian King --- drivers/media/dvb-frontends/stv0367.c | 2 -- 1 file changed, 2 deletions(-) diff --git

[PATCH] reservation: don't wait when timeout=0

2017-11-21 Thread Rob Clark
If we are testing if a reservation object's fences have been signaled with timeout=0 (non-blocking), we need to pass 0 for timeout to dma_fence_wait_timeout(). Plus bonus spelling correction. Signed-off-by: Rob Clark --- drivers/dma-buf/reservation.c | 11 +-- 1

Re: [PATCH] reservation: don't wait when timeout=0

2017-11-21 Thread Chris Wilson
Quoting Rob Clark (2017-11-21 14:08:46) > If we are testing if a reservation object's fences have been > signaled with timeout=0 (non-blocking), we need to pass 0 for > timeout to dma_fence_wait_timeout(). > > Plus bonus spelling correction. > > Signed-off-by: Rob Clark >

Re: [PATCH] media: rc: double keypresses due to timeout expiring to early

2017-11-21 Thread Matthias Reichl
Hi Sean! On Sun, Nov 19, 2017 at 09:57:27PM +, Sean Young wrote: > I think for now the best solution is to revert to 250ms for all protocols > (except for cec which needs 550ms), and reconsider for another kernel. Thanks, this sounds like a good idea! > >>From

Re: [PATCH v3 2/4] [media] dt-bindings: Document BCM283x CSI2/CCP2 receiver

2017-11-21 Thread Eric Anholt
Dave Stevenson writes: > Hi Rob > > On 27 September 2017 at 22:51, Rob Herring wrote: >> On Fri, Sep 22, 2017 at 05:07:22PM +0100, Dave Stevenson wrote: >>> Hi Stefan >>> >>> On 22 September 2017 at 07:45, Stefan Wahren

Re: 'LITE-ON USB2.0 DVB-T Tune' driver crash with kernel 4.13 / ubuntu 17.10

2017-11-21 Thread Sean Young
Hi Laurent, On Sun, Nov 12, 2017 at 09:38:47AM +0100, Laurent Caumont wrote: > Thank you for the changes, It's better like this, I will test it. Just wondering if you have had the time to test this patch. It would be great if it could be included after being tested. Thanks, Sean

Re: [PATCH v3 2/4] [media] dt-bindings: Document BCM283x CSI2/CCP2 receiver

2017-11-21 Thread Rob Herring
On Tue, Nov 21, 2017 at 1:26 PM, Eric Anholt wrote: > Dave Stevenson writes: > >> Hi Rob >> >> On 27 September 2017 at 22:51, Rob Herring wrote: >>> On Fri, Sep 22, 2017 at 05:07:22PM +0100, Dave Stevenson wrote: Hi Stefan

[PATCH] media: imon: auto-config ffdc 30 device

2017-11-21 Thread Sean Young
Another device with the 0xffdc device id, this one with 0x30 in the config byte. Its an iMON VFD + iMON IR (it does not understand rc6). Signed-off-by: Sean Young --- drivers/media/rc/imon.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/media/rc/imon.c

Re: [PATCH v3 2/4] [media] dt-bindings: Document BCM283x CSI2/CCP2 receiver

2017-11-21 Thread Eric Anholt
Rob Herring writes: > On Tue, Nov 21, 2017 at 1:26 PM, Eric Anholt wrote: >> Dave Stevenson writes: >> >>> Hi Rob >>> >>> On 27 September 2017 at 22:51, Rob Herring wrote: On Fri, Sep 22, 2017 at

Hi Dear

2017-11-21 Thread Ronald Bern
hello baby. Can I will like to know about you. I am a single man searching for a nice female friend.

Re: 'LITE-ON USB2.0 DVB-T Tune' driver crash with kernel 4.13 / ubuntu 17.10

2017-11-21 Thread Laurent Caumont
Hi Sean, Yes, it is working well. Laurent 2017-11-21 22:03 UTC+01:00, Sean Young : > Hi Laurent, > > On Sun, Nov 12, 2017 at 09:38:47AM +0100, Laurent Caumont wrote: >> Thank you for the changes, It's better like this, I will test it. > > Just wondering if you have had the time to

Re: [PATCH] reservation: don't wait when timeout=0

2017-11-21 Thread Rob Clark
On Tue, Nov 21, 2017 at 9:38 AM, Chris Wilson wrote: > Quoting Rob Clark (2017-11-21 14:08:46) >> If we are testing if a reservation object's fences have been >> signaled with timeout=0 (non-blocking), we need to pass 0 for >> timeout to dma_fence_wait_timeout(). >> >>

Re: [PATCH v2 1/3] media: V3s: Add support for Allwinner CSI.

2017-11-21 Thread Yong
On Tue, 21 Nov 2017 16:48:27 +0100 Maxime Ripard wrote: > Hi, > > On Thu, Jul 27, 2017 at 01:01:35PM +0800, Yong Deng wrote: > > Allwinner V3s SoC have two CSI module. CSI0 is used for MIPI interface > > and CSI1 is used for parallel interface. This is not

Re: [PATCH v4 00/12] [dt-bindings] [media] Add document file and driver for Sony CXD2880 DVB-T2/T tuner + demodulator

2017-11-21 Thread Takiguchi, Yasunari
Hi, all I sent the patch series of Sony CXD2880 DVB-T2/T tuner + demodulator driver version 4 on 13th/Oct. I'd like to get better understanding of current review status for our codes. Are there any comments, advices and review results for them? Regards, Takiguchi

cron job: media_tree daily build: WARNINGS

2017-11-21 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 Nov 22 05:00:16 CET 2017 media-tree git hash:30b4e122d71cbec2944a5f8b558b88936ee42f10 media_build

[PATCH 23/30] [media] atomisp: deprecate pci_get_bus_and_slot()

2017-11-21 Thread Sinan Kaya
pci_get_bus_and_slot() is restrictive such that it assumes domain=0 as where a PCI device is present. This restricts the device drivers to be reused for other domain numbers. Use pci_get_domain_bus_and_slot() with a domain number of 0 where we can't extract the domain number. Other places, use