cron job: media_tree daily build: ERRORS

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: Sat Dec 9 05:00:18 CET 2017 media-tree git hash:0393e735649dc41358adb7b603bd57dad1ed3260 media_build

★ lnfo --“2018世界复合材料展览及会议”将于“3月”在“法国巴黎”举行 （地右P1-L-Me）

尊敬的 l...@noirol.cn 企业领导/公司负责人/业界专家，您好： 　　 　　 新材料为21世纪三大共性关键技术之一，已成为全球经济迅速增长的源动力和提升核心竞争力的战略焦点。材料作为制造业的基础，特别是新材料研究和产业发展的水平与规模，已经成为衡量一个国家科技进步和综合实力的重要标志。在新材料发展与应用中，复合材料占有相当重要的地位，特别广泛的应用在汽车、交通、风能、航空、航天、兵器、船舶、国防、机械、电子、化工、建筑、农业、渔业、纺织、运动器材等领域，一直是世界各国优先发展和竞争激烈的重要行业。 　　 　　“JEC世界复合材料展览及会议”（JEC world

Re: [PATCH] build: Remove patch v3.13_ddbridge_pcimsi

Hi! The code of this patch is already upstream, so it does no longer apply when compiling for Kernel 3.13. BR, Jasmin

[PATCH] build: Remove patch v3.13_ddbridge_pcimsi

From: Jasmin Jessich Signed-off-by: Jasmin Jessich --- backports/backports.txt | 3 --- backports/v3.13_ddbridge_pcimsi.patch | 29 - 2 files changed, 32 deletions(-) delete mode 100644

[PATCH] build: Added missing READ_ONCE

From: Jasmin Jessich Signed-off-by: Jasmin Jessich --- v4l/compat.h | 5 + v4l/scripts/make_config_compat.pl | 1 + 2 files changed, 6 insertions(+) diff --git a/v4l/compat.h b/v4l/compat.h index 89deae1..fba7a99 100644 --- a/v4l/compat.h

What to do with input_enable_softrepeat in av7110_ir.c

Hello Hans! I try to fix the compilation for Kernel 3.13 (the kernel I use on my VDR). In commit 5aeaa3e668de0782d1502f3d5751e2266a251d7c the timer handling in the driver has been changed and now it uses "input_enable_softrepeat". This function has been added with Kernel 4.4. I tried to define

RE: [RESEND GIT PULL for 4.16] Intel IPU3 CIO2 CSI-2 receiver driver

Hi, Mauro, > -Original Message- > From: Mauro Carvalho Chehab [mailto:mche...@s-opensource.com] > Sent: Friday, December 8, 2017 7:00 AM > To: Sakari Ailus > Cc: linux-media@vger.kernel.org; Mani, Rajmohan > ; Zhi, Yong >

Re: [PATCH] build: Added missing timer_setup_on_stack

Hello Vincent! > Not quite there yet however. I always test with several Kernel headers ... > patch -s -f -N -p1 -i ../backports/api_version.patch > patch -s -f -N -p1 -i ../backports/pr_fmt.patch > 1 out of 1 hunk FAILED I have tested the patches on Linux Mint 18.1 (Kernel 4.4.0-62-generic):

Re: [PATCH] build: Added missing timer_setup_on_stack

On Fri, Dec 08, 2017 at 10:12:05PM +0100, Hans Verkuil wrote: ... > > I've applied all your patches. Thank you very much for working on this. > > Let's see what the result of the nightly build will be. > > In general reverting kernel patches to make a driver compile is something of a > last

Re: [PATCH v2 1/2] media: exynos-gsc: fix lockdep warning

On 11/07/2017 09:53 AM, Shuah Khan wrote: > On 10/16/2017 09:18 AM, Hans Verkuil wrote: >> On 10/16/2017 05:16 PM, Shuah Khan wrote: >>> The driver mmap functions shouldn't take lock when calling vb2_mmap(). >>> Fix it to not take the lock. >>> >>> Reference: commit log for

Re: [GIT PULL FOR v4.16] media: imx: Add better OF graph support

On 12/08/2017 08:38 AM, Mauro Carvalho Chehab wrote: Em Fri, 8 Dec 2017 11:56:58 +0100 Hans Verkuil escreveu: Note: the new v4l2-async work makes it possible to simplify this. That will be done in follow-up patches. It's easier to do that if this is in first. Regards,

Re: [PATCH v2] kernel-doc: parse DECLARE_KFIFO and DECLARE_KFIFO_PTR()

On 12/08/2017 06:05 AM, Mauro Carvalho Chehab wrote: > On media, we now have an struct declared with: > > struct lirc_fh { > struct list_head list; > struct rc_dev *rc; > int carrier_low; > bool

Re: [git:media_tree/master] media: davinci: vpif_capture: add NULL check on devm_kzalloc return value

Quoting Mauro Carvalho Chehab : This is an automatic generated email to let you know that the following patch were queued: Subject: media: davinci: vpif_capture: add NULL check on devm_kzalloc return value Author: Gustavo A. R. Silva

Re: [GIT PULL for v4.15-rc3] media fixes

On Fri, Dec 8, 2017 at 7:56 AM, Mauro Carvalho Chehab wrote: > > - The largest amount of fixes in this series is with regards to comments > that aren't kernel-doc, but start with "/**". A new check added for > 4.15 makes it to produce a *huge* amount of new warnings

Re: [PATCH] build: Added missing timer_setup_on_stack

On 12/08/2017 10:06 PM, Jasmin J. wrote: > Hello Hans! > > With this patch it compiles for Kernel 4.4, but not on 3.13. I will work on > that soon. > > I am not sure if this patch keeps pvrusb2 working, but it compiles. I tried > first a solution by reverting

Re: [PATCH] build: Added missing timer_setup_on_stack

Hello Hans! With this patch it compiles for Kernel 4.4, but not on 3.13. I will work on that soon. I am not sure if this patch keeps pvrusb2 working, but it compiles. I tried first a solution by reverting 8da0edf2f90b6c74b69ad420fdd230c9bd2bd1ed. If you prefer this, I have it on a branch and can

[PATCH] build: Added missing timer_setup_on_stack

From: Jasmin Jessich Signed-off-by: Jasmin Jessich --- v4l/compat.h | 5 + v4l/scripts/make_config_compat.pl | 1 + 2 files changed, 6 insertions(+) diff --git a/v4l/compat.h b/v4l/compat.h index 4af407e..89deae1 100644 --- a/v4l/compat.h

Re: [PATCH v9 25/28] rcar-vin: extend {start,stop}_streaming to work with media controller

Hi Niklas, Thank you for the patch. On Friday, 8 December 2017 03:08:39 EET Niklas Söderlund wrote: > The procedure to start or stop streaming using the non-MC single > subdevice and the MC graph and multiple subdevices are quite different. > Create a new function to abstract which method is

Re: [PATCH v9 22/28] rcar-vin: add chsel information to rvin_info

Hi Niklas, Thank you for the patch. On Friday, 8 December 2017 03:08:36 EET Niklas Söderlund wrote: > Each Gen3 SoC has a limited set of predefined routing possibilities for > which CSI-2 device and virtual channel can be routed to which VIN > instance. Prepare to store this information in the

Re: [PATCH v9 21/28] rcar-vin: add group allocator functions

Hi Niklas, Thank you for the patch. On Friday, 8 December 2017 03:08:35 EET Niklas Söderlund wrote: > In media controller mode all VIN instances needs to be part of the same > media graph. There is also a need to each VIN instance to know and in > some cases be able to communicate with other VIN

Re: [PATCH v9 19/28] rcar-vin: use different v4l2 operations in media controller mode

Hi Hans, On Friday, 8 December 2017 12:24:26 EET Hans Verkuil wrote: > Hi Laurent, > > >> +static const struct v4l2_ioctl_ops rvin_mc_ioctl_ops = { > >> + .vidioc_querycap= rvin_querycap, > >> + .vidioc_try_fmt_vid_cap = rvin_mc_try_fmt_vid_cap, > >> +

Re: [PATCH v9 13/28] rcar-vin: fix handling of single field frames (top, bottom and alternate fields)

Hi Niklas, On Friday, 8 December 2017 16:06:58 EET Niklas Söderlund wrote: > On 2017-12-08 11:35:18 +0200, Laurent Pinchart wrote: > > On Friday, 8 December 2017 03:08:27 EET Niklas Söderlund wrote: > >> There was never proper support in the VIN driver to deliver ALTERNATING > >> field format to

Re: [PATCH v9 11/28] rcar-vin: do not allow changing scaling and composing while streaming

Hi Niklas, On Friday, 8 December 2017 16:14:23 EET Niklas Söderlund wrote: > On 2017-12-08 11:04:26 +0200, Laurent Pinchart wrote: > > On Friday, 8 December 2017 03:08:25 EET Niklas Söderlund wrote: > >> It is possible on Gen2 to change the registers controlling composing and > >> scaling while

Re: [PATCH v9 03/28] rcar-vin: unregister video device on driver removal

Hi Niklas, On Friday, 8 December 2017 15:09:21 EET Niklas Söderlund wrote: > On 2017-12-08 09:54:31 +0200, Laurent Pinchart wrote: > > On Friday, 8 December 2017 03:08:17 EET Niklas Söderlund wrote: > >> If the video device was registered by the complete() callback it should > >> be unregistered

Re: [PATCH] build: Fixed include compiler-gcc.h directly error

Hello Hans! I am not sure why compiler-gcc.h is used from the kernel tree and not from the installed kernel headers. So a better fix for this problem might be to remove compiler-gcc.h from media_build/linux/Makefile. This is something you will know better. Please note that this doesn't fix all

[PATCH] build: Fixed include compiler-gcc.h directly error

From: Jasmin Jessich The type definitions have been splitting from compiler.h into compiler_types.h. This requires to check for the new and the old include guard in compiler-gcc.h. Signed-off-by: Jasmin Jessich --- backports/backports.txt | 3 +++

[PATCH] build: Fixed pr_fmt.patch

From: Jasmin Jessich Signed-off-by: Jasmin Jessich --- backports/pr_fmt.patch | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/backports/pr_fmt.patch b/backports/pr_fmt.patch index 53f0273..d36e071 100644 --- a/backports/pr_fmt.patch +++

webcams not recognized on a Dell Latitude 5285 detachable laptop

Dear all, I recently installed debian 9 and ubuntu 17.10 on a Dell Latitude 5285 detachable laptop. I problem I have is that the webcams (both the rear one and the front one) are not recognized. There is no /dev/video* file. I am not sure what is the model of the webcam. The Dell support

Re: [GIT PULL FOR v4.16] media: imx: Add better OF graph support

Em Fri, 8 Dec 2017 11:56:58 +0100 Hans Verkuil escreveu: > Note: the new v4l2-async work makes it possible to simplify this. That > will be done in follow-up patches. It's easier to do that if this is in > first. > > Regards, > > Hans > > The following changes since

Re: [PATCH v2] kernel-doc: parse DECLARE_KFIFO and DECLARE_KFIFO_PTR()

FYI: added similar patch to the linuxdoc sphinx-extension https://github.com/return42/linuxdoc/commit/726af7a Thanks! -- Markus -- > Am 08.12.2017 um 15:05 schrieb Mauro Carvalho Chehab > : > > On media, we now have an struct declared with: > > struct lirc_fh { >

Re: [Patch v6 05/12] [media] videodev2.h: Add v4l2 definition for HEVC

Em Fri, 08 Dec 2017 10:47:24 -0500 Nicolas Dufresne escreveu: > Le vendredi 08 décembre 2017 à 14:38 +0530, Smitha T Murthy a écrit : > > Add V4L2 definition for HEVC compressed format > > > > Signed-off-by: Smitha T Murthy > > Reviewed-by: Andrzej

[GIT PULL for v4.15-rc3] media fixes

Hi Linus, Please pull from: git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-media tags/media/v4.15-2 For a series of fixes for the media subsytem: - The largest amount of fixes in this series is with regards to comments that aren't kernel-doc, but start with "/**". A new check

Re: [Patch v6 05/12] [media] videodev2.h: Add v4l2 definition for HEVC

Le vendredi 08 décembre 2017 à 14:38 +0530, Smitha T Murthy a écrit : > Add V4L2 definition for HEVC compressed format > > Signed-off-by: Smitha T Murthy > Reviewed-by: Andrzej Hajda > Reviewed-by: Stanimir Varbanov >

Re: [RESEND GIT PULL for 4.16] Intel IPU3 CIO2 CSI-2 receiver driver

Em Fri, 1 Dec 2017 16:31:36 +0200 Sakari Ailus escreveu: > Hi Mauro, > > Here's the Intel IPU3 CIO2 CSI-2 receiver driver, with the accompanying > format definitions. This patch generates two warnings: drivers/media/pci/intel/ipu3/ipu3-cio2.c:1899:16: warning: Variable

Re: [PATCH 1/1] imx274: Fix error handling, add MAINTAINERS entry

Em Wed, 1 Nov 2017 11:43:27 +0200 Sakari Ailus escreveu: > Add the missing MAINTAINERS entry for imx274, fix error handling in driver > probe and unregister the correct control handler in driver remove. > > Signed-off-by: Sakari Ailus

Re: [PATCH v9 11/28] rcar-vin: do not allow changing scaling and composing while streaming

Hi Laurent, Thanks for your feedback. On 2017-12-08 11:04:26 +0200, Laurent Pinchart wrote: > Hi Niklas, > > Thank you for the patch. > > On Friday, 8 December 2017 03:08:25 EET Niklas Söderlund wrote: > > It is possible on Gen2 to change the registers controlling composing and > > scaling

Re: [PATCH v3 26/26] kfifo: DECLARE_KIFO_PTR(fifo, u64) does not work on arm 32 bit

Em Thu, 30 Nov 2017 13:34:10 +0100 Stefani Seibold escreveu: > On Thu, 2017-11-30 at 10:29 -0200, Mauro Carvalho Chehab wrote: > > Em Tue, 10 Oct 2017 09:59:42 +0200 > > Sean Young escreveu: > > > > > If you try to store u64 in a kfifo (or a struct with

Re: [PATCH v9 13/28] rcar-vin: fix handling of single field frames (top, bottom and alternate fields)

Hi Laurent, Thanks for your feedback. On 2017-12-08 11:35:18 +0200, Laurent Pinchart wrote: > Hi Niklas, > > Thank you for the patch. > > On Friday, 8 December 2017 03:08:27 EET Niklas Söderlund wrote: > > There was never proper support in the VIN driver to deliver ALTERNATING > > field format

[PATCH v2] kernel-doc: parse DECLARE_KFIFO and DECLARE_KFIFO_PTR()

On media, we now have an struct declared with: struct lirc_fh { struct list_head list; struct rc_dev *rc; int carrier_low; boolsend_timeout_reports; DECLARE_KFIFO_PTR(rawir, unsigned int);

[PATCH] [media] vb2: clear V4L2_BUF_FLAG_LAST when filling vb2_buffer

V4L2_BUF_FLAG_LAST is a signal from the driver to userspace for buffers on the capture queue. When userspace queues back a capture buffer with the flag set, we should clear it. Otherwise, if userspace restarts streaming after EOS, without reallocating the buffers, mem2mem devices will erroneously

Re: [Patch v6 02/12] [media] s5p-mfc: Adding initial support for MFC v10.10

Smitha, On Fri, Dec 8, 2017 at 10:08 AM, Smitha T Murthy wrote: > Adding the support for MFC v10.10, with new register file and > necessary hw control, decoder, encoder and structural changes. > > CC: Rob Herring > CC: devicet...@vger.kernel.org >

Re: [PATCH v8 2/2] media: i2c: Add the ov7740 image sensor driver

Wenyou, On Fri, Dec 8, 2017 at 2:55 AM, Wenyou Yang wrote: > The ov7740 (color) image sensor is a high performance VGA CMOS > image snesor, which supports for output formats: RAW RGB and YUV > and image sizes: VGA, and QVGA, CIF and any size smaller. > > Signed-off-by:

Re: [PATCH v9 03/28] rcar-vin: unregister video device on driver removal

Hi Laurent, Thanks for your comments. On 2017-12-08 09:54:31 +0200, Laurent Pinchart wrote: > Hi Niklas, > > Thank you for the patch. > > On Friday, 8 December 2017 03:08:17 EET Niklas Söderlund wrote: > > If the video device was registered by the complete() callback it should > > be

Re: [PATCH v9 02/28] rcar-vin: rename poorly named initialize and cleanup functions

Hi Laurent, Thanks for your comments. On 2017-12-08 09:49:55 +0200, Laurent Pinchart wrote: > Hi Niklas, > > Thank you for the patch. > > On Friday, 8 December 2017 03:08:16 EET Niklas Söderlund wrote: > > The functions to initialize and cleanup the hardware and video device > > where poorly

Re: [PATCH v9 01/28] rcar-vin: add Gen3 devicetree bindings documentation

Hi Laurent, Thanks for your comments! On 2017-12-08 09:46:24 +0200, Laurent Pinchart wrote: > Hi Niklas, > > Thank you for the patch. > > On Friday, 8 December 2017 03:08:15 EET Niklas Söderlund wrote: > > Document the devicetree bindings for the CSI-2 inputs available on Gen3. > > > > There

[PATCH] media: v4l: xilinx: Use SPDX-License-Identifier

SPDX-License-Identifier is used for the Xilinx Video IP and related drivers. Signed-off-by: Dhaval Shah --- drivers/media/platform/xilinx/xilinx-dma.c | 5 + drivers/media/platform/xilinx/xilinx-dma.h | 5 + drivers/media/platform/xilinx/xilinx-tpg.c | 5

[GIT PULL FOR v4.16] media: imx: Add better OF graph support

Note: the new v4l2-async work makes it possible to simplify this. That will be done in follow-up patches. It's easier to do that if this is in first. Regards, Hans The following changes since commit 781b045baefdabf7e0bc9f33672ca830d3db9f27: media: imx274: Fix error handling, add

Re: [PATCH v9 27/28] rcar-vin: enable support for r8a7796

Hi Niklas, Thank you for the patch. On Friday, 8 December 2017 03:08:41 EET Niklas Söderlund wrote: > Add the SoC specific information for Renesas r8a7796. > > Signed-off-by: Niklas Söderlund > Reviewed-by: Hans Verkuil >

Re: [PATCH v9 19/28] rcar-vin: use different v4l2 operations in media controller mode

Hi Laurent, >> +static const struct v4l2_ioctl_ops rvin_mc_ioctl_ops = { >> +.vidioc_querycap= rvin_querycap, >> +.vidioc_try_fmt_vid_cap = rvin_mc_try_fmt_vid_cap, >> +.vidioc_g_fmt_vid_cap = rvin_g_fmt_vid_cap, >> +.vidioc_s_fmt_vid_cap

Re: [PATCH v9 26/28] rcar-vin: enable support for r8a7795

Hi Niklas, Thank you for the patch. On Friday, 8 December 2017 03:08:40 EET Niklas Söderlund wrote: > Add the SoC specific information for Renesas r8a7795 ES1.x and ES2.0. > > Signed-off-by: Niklas Söderlund > Reviewed-by: Hans Verkuil

Re: [PATCH v9 20/28] rcar-vin: prepare for media controller mode initialization

Hi Niklas, Thank you for the patch. On Friday, 8 December 2017 03:08:34 EET Niklas Söderlund wrote: > When running in media controller mode a media pad is needed, register > one. Also set the media bus format to CSI-2. This sounds a bit unclear to me. We don't need a pad for the sake of it,

Re: [PATCH v9 19/28] rcar-vin: use different v4l2 operations in media controller mode

Hi Niklas, Thank you for the patch. On Friday, 8 December 2017 03:08:33 EET Niklas Söderlund wrote: > When the driver runs in media controller mode it should not directly > control the subdevice instead userspace will be responsible for > configuring the pipeline. To be able to run in this mode

[Patch v6 07/12] Documentation: v4l: Documentation for HEVC v4l2 definition

Add V4L2 definition for HEVC compressed format which is also known as H.265. Signed-off-by: Smitha T Murthy Reviewed-by: Andrzej Hajda Reviewed-by: Stanimir Varbanov --- Documentation/media/uapi/v4l/pixfmt-compressed.rst

[Patch v6 03/12] [media] s5p-mfc: Use min scratch buffer size as provided by F/W

After MFC v8.0, mfc f/w lets the driver know how much scratch buffer size is required for decoder. If mfc f/w has the functionality, E_MIN_SCRATCH_BUFFER_SIZE, driver can know how much scratch buffer size is required for encoder too. Signed-off-by: Smitha T Murthy

[Patch v6 04/12] [media] s5p-mfc: Support MFCv10.10 buffer requirements

Aligning the luma_dpb_size, chroma_dpb_size, mv_size and me_buffer_size for MFCv10.10. Signed-off-by: Smitha T Murthy Reviewed-by: Andrzej Hajda Acked-by: Kamil Debski --- drivers/media/platform/s5p-mfc/regs-mfc-v10.h | 19 +

[Patch v6 05/12] [media] videodev2.h: Add v4l2 definition for HEVC

Add V4L2 definition for HEVC compressed format Signed-off-by: Smitha T Murthy Reviewed-by: Andrzej Hajda Reviewed-by: Stanimir Varbanov Acked-by: Hans Verkuil --- include/uapi/linux/videodev2.h |

[Patch v6 06/12] [media] v4l2-ioctl: add HEVC format description

HEVC is a video coding format Signed-off-by: Smitha T Murthy Reviewed-by: Stanimir Varbanov Acked-by: Hans Verkuil --- drivers/media/v4l2-core/v4l2-ioctl.c | 1 + 1 file changed, 1 insertion(+) diff --git

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

Add v4l2 controls for HEVC encoder Signed-off-by: Smitha T Murthy Reviewed-by: Andrzej Hajda --- drivers/media/v4l2-core/v4l2-ctrls.c | 118 +++ include/uapi/linux/v4l2-controls.h | 92 ++- 2

[Patch v6 08/12] [media] s5p-mfc: Add support for HEVC decoder

Add support for codec definition and corresponding buffer requirements for HEVC decoder. Signed-off-by: Smitha T Murthy Reviewed-by: Andrzej Hajda --- drivers/media/platform/s5p-mfc/regs-mfc-v10.h | 1 +

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

Added V4l2 controls for HEVC encoder Signed-off-by: Smitha T Murthy --- Documentation/media/uapi/v4l/extended-controls.rst | 395 + 1 file changed, 395 insertions(+) diff --git a/Documentation/media/uapi/v4l/extended-controls.rst

[Patch v6 09/12] [media] s5p-mfc: Add VP9 decoder support

Add support for codec definition and corresponding buffer requirements for VP9 decoder. Signed-off-by: Smitha T Murthy Reviewed-by: Andrzej Hajda --- drivers/media/platform/s5p-mfc/regs-mfc-v10.h | 6 ++

[Patch v6 11/12] [media] s5p-mfc: Add support for HEVC encoder

Add HEVC encoder support and necessary registers, V4L2 CIDs, and hevc encoder parameters Signed-off-by: Smitha T Murthy --- drivers/media/platform/s5p-mfc/regs-mfc-v10.h | 28 +- drivers/media/platform/s5p-mfc/s5p_mfc.c| 1 +

[Patch v6 02/12] [media] s5p-mfc: Adding initial support for MFC v10.10

Adding the support for MFC v10.10, with new register file and necessary hw control, decoder, encoder and structural changes. CC: Rob Herring CC: devicet...@vger.kernel.org Signed-off-by: Smitha T Murthy Reviewed-by: Andrzej Hajda

[Patch v6 01/12] [media] s5p-mfc: Rename IS_MFCV8 macro

This patch renames macro IS_MFCV8 to IS_MFCV8_PLUS so that the MFCv8 code can be resued for MFCv10.10 support. Since the MFCv8 specific code holds good for MFC v10.10 also. Signed-off-by: Smitha T Murthy Acked-by: Andrzej Hajda ---

Re: [PATCH v9 18/28] rcar-vin: break out format alignment and checking

Hi Niklas, Thank you for the patch. On Friday, 8 December 2017 03:08:32 EET Niklas Söderlund wrote: > Part of the format alignment and checking can be shared with the Gen3 > format handling. Break that part out to its own function. While doing > this clean up the checking and add more checks.

Re: [PATCH v9 17/28] rcar-vin: add flag to switch to media controller mode

Hi Niklas, Thank you for the patch. On Friday, 8 December 2017 03:08:31 EET Niklas Söderlund wrote: > On Gen3 a media controller API needs to be used to allow userspace to > configure the subdevices in the pipeline instead of directly controlling > a single source subdevice, which is and will

Re: [PATCH v9 16/28] rcar-vin: add function to manipulate Gen3 chsel value

Hi Niklas, Thank you for the patch. On Friday, 8 December 2017 03:08:30 EET Niklas Söderlund wrote: > On Gen3 the CSI-2 routing is controlled by the VnCSI_IFMD register. One > feature of this register is that it's only present in the VIN0 and VIN4 > instances. The register in VIN0 controls the

Re: [PATCH v9 15/28] rcar-vin: enable Gen3 hardware configuration

Hi Niklas, Thank you for the patch. On Friday, 8 December 2017 03:08:29 EET Niklas Söderlund wrote: > Add the register needed to work with Gen3 hardware. This patch adds > the logic for how to work with the Gen3 hardware. More work is required > to enable the subdevice structure needed to

Re: [PATCH v9 14/28] rcar-vin: move media bus configuration to struct rvin_info

Hi Niklas, Thank you for the patch. On Friday, 8 December 2017 03:08:28 EET Niklas Söderlund wrote: > Bus configuration will once the driver is extended to support Gen3 > contain information not specific to only the directly connected parallel > subdevice. Move it to struct rvin_info to show

[Patch v6 00/12] Add MFC v10.10 support

This patch series adds MFC v10.10 support. MFC v10.10 is used in some of Exynos7 variants. This adds support for following: * Add support for HEVC encoder and decoder * Add support for VP9 decoder * Update Documentation for control id definitions * Update computation of min scratch buffer size

Re: [PATCH v9 13/28] rcar-vin: fix handling of single field frames (top, bottom and alternate fields)

Hi Niklas, Thank you for the patch. On Friday, 8 December 2017 03:08:27 EET Niklas Söderlund wrote: > There was never proper support in the VIN driver to deliver ALTERNATING > field format to user-space, remove this field option. For sources using > this field format instead use the VIN hardware

Re: [PATCH v9 12/28] rcar-vin: read subdevice format for crop only when needed

Hi Niklas, Thank you for the patch. On Friday, 8 December 2017 03:08:26 EET Niklas Söderlund wrote: > Instead of caching the subdevice format each time the video device > format is set read it directly when it's needed. As it turns out the > format is only needed when figuring out the max

Re: [PATCH v9 11/28] rcar-vin: do not allow changing scaling and composing while streaming

Hi Niklas, Thank you for the patch. On Friday, 8 December 2017 03:08:25 EET Niklas Söderlund wrote: > It is possible on Gen2 to change the registers controlling composing and > scaling while the stream is running. It is however not a good idea to do > so and could result in trouble. There are

Re: [PATCH v9 03/28] rcar-vin: unregister video device on driver removal

Hi Hans, On Friday, 8 December 2017 10:46:34 EET Hans Verkuil wrote: > On 12/08/2017 08:54 AM, Laurent Pinchart wrote: > > On Friday, 8 December 2017 03:08:17 EET Niklas Söderlund wrote: > >> If the video device was registered by the complete() callback it should > >> be unregistered when the

Re: [PATCH v9 03/28] rcar-vin: unregister video device on driver removal

On 12/08/2017 08:54 AM, Laurent Pinchart wrote: > Hi Niklas, > > Thank you for the patch. > > On Friday, 8 December 2017 03:08:17 EET Niklas Söderlund wrote: >> If the video device was registered by the complete() callback it should >> be unregistered when the driver is removed. > > The

Re: [PATCH v9 09/28] rcar-vin: all Gen2 boards can scale simplify logic

Hi Niklas, Thank you for the patch. On Friday, 8 December 2017 03:08:23 EET Niklas Söderlund wrote: > The logic to preserve the requested format width and height are too > complex and come from a premature optimization for Gen3. All Gen2 SoC > can scale and the Gen3 implementation will not use

Re: [PATCH v9 08/28] rcar-vin: move functions regarding scaling

Hi Niklas, Thank you for the patch. On Friday, 8 December 2017 03:08:22 EET Niklas Söderlund wrote: > In preparation of refactoring the scaling code move the code regarding > scaling to to the top of the file to avoid the need to add forward > declarations. No code is changed in this commit only

Re: [PATCH v9 07/28] rcar-vin: change name of video device

Hi Niklas, (CC'ing Sakari) Thank you for the patch. On Friday, 8 December 2017 03:08:21 EET Niklas Söderlund wrote: > The rcar-vin driver needs to be part of a media controller to support > Gen3. Give each VIN instance a unique name so it can be referenced from > userspace. > > Signed-off-by:

Re: [PATCH v9 06/28] rcar-vin: move max width and height information to chip information

Hi Niklas, Thank you for the patch. On Friday, 8 December 2017 03:08:20 EET Niklas Söderlund wrote: > On Gen3 the max supported width and height will be different from Gen2. > Move the limits to the struct rvin_info to prepare for Gen3 support. > > Signed-off-by: Niklas Söderlund

Re: [PATCH v9 05/28] rcar-vin: move chip information to own struct

Hi Niklas, Thank you for the patch. On Friday, 8 December 2017 03:08:19 EET Niklas Söderlund wrote: > When Gen3 support is added to the driver more than chip ID will be > different for the different SoCs. To avoid a lot of if statements in the > code create a struct chip_info to store this

Re: [PATCH v9 04/28] rcar-vin: move subdevice handling to async callbacks

Hi Niklas, Thank you for the patch. On Friday, 8 December 2017 03:08:18 EET Niklas Söderlund wrote: > In preparation for Gen3 support move the subdevice initialization and > clean up from rvin_v4l2_{register,unregister}() directly to the async > callbacks. This simplifies the addition of Gen3