Re: [PATCH 0/3] Add support compat in dvb_frontend.c

2017-12-01 Thread Mauro Carvalho Chehab
Hi Jaedon, Em Fri, 1 Dec 2017 21:31:27 +0900 Jaedon Shin escreveu: > This patch series supports compat ioctl for 32-bit user space applications > in 64-bit system. > > Jaedon Shin (3): > media: dvb_frontend: Add unlocked_ioctl in dvb_frontend.c > media:

[PATCH] [media] soc_camera: add MODULE_LICENSE to soc_scale_crop.c

2017-12-01 Thread Daniel Axtens
This fixes the following warning on an allmodconfig build: WARNING: modpost: missing MODULE_LICENSE() in drivers/media/platform/soc_camera/soc_scale_crop.o Use license "GPL" to match the GPLv2+ license from the header at the top of the file. Signed-off-by: Daniel Axtens ---

[PATCH] media: tegra-cec: add MODULE_LICENSE()

2017-12-01 Thread Daniel Axtens
This fixes the following warning in an allmodconfig build: WARNING: modpost: missing MODULE_LICENSE() in drivers/media/platform/tegra-cec/tegra_cec.o The license matches the header. Signed-off-by: Daniel Axtens --- drivers/media/platform/tegra-cec/tegra_cec.c | 2 ++ 1 file

cron job: media_tree daily build: ERRORS

2017-12-01 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: Sat Dec 2 05:00:16 CET 2017 media-tree git hash:781b045baefdabf7e0bc9f33672ca830d3db9f27 media_build

[PATCH v5 11/12] intel-ipu3: Add v4l2 driver based on media framework

2017-12-01 Thread Yong Zhi
This implements video driver that utilizes v4l2, vb2 queue support and media controller APIs. The driver exposes single subdevice and seven nodes. Signed-off-by: Yong Zhi Signed-off-by: Ramya Vijaykumar ---

[PATCH v5 12/12] intel-ipu3: Add imgu top level pci device driver

2017-12-01 Thread Yong Zhi
This patch adds support for the Intel IPU v3 as found on Skylake and Kaby Lake SoCs. The driver glues v4l2, css(camera sub system) and other pieces together to perform its functions, it also loads the IPU3 firmware binary as part of its initialization. Signed-off-by: Yong Zhi

[PATCH v5 10/12] intel-ipu3: Add css pipeline programming

2017-12-01 Thread Yong Zhi
This provides helper library to be used by v4l2 level to program imaging pipelines and control the streaming. Signed-off-by: Yong Zhi --- drivers/media/pci/intel/ipu3/ipu3-css.c | 1780 +++ 1 file changed, 1780 insertions(+) diff --git

[PATCH v5 08/12] intel-ipu3: css: Compute and program ccs

2017-12-01 Thread Yong Zhi
A collection of routines that are mainly used to calculate the parameters for accelerator cluster. Signed-off-by: Yong Zhi --- drivers/media/pci/intel/ipu3/ipu3-css-params.c | 3169 drivers/media/pci/intel/ipu3/ipu3-css-params.h | 36 +

[PATCH v5 06/12] intel-ipu3: css: Add support for firmware management

2017-12-01 Thread Yong Zhi
Introduce functions to load and install ImgU FW blobs. Signed-off-by: Yong Zhi --- drivers/media/pci/intel/ipu3/ipu3-abi.h| 1579 drivers/media/pci/intel/ipu3/ipu3-css-fw.c | 271 + drivers/media/pci/intel/ipu3/ipu3-css-fw.h | 212

[PATCH v5 09/12] intel-ipu3: css: Initialize css hardware

2017-12-01 Thread Yong Zhi
This patch implements the functions to initialize and configure IPU3 h/w such as clock, irq and power. Signed-off-by: Yong Zhi Signed-off-by: Tomasz Figa --- drivers/media/pci/intel/ipu3/ipu3-css.c | 523 1 file changed,

[PATCH v5 01/12] v4l: Add Intel IPU3 meta buffer formats

2017-12-01 Thread Yong Zhi
Add IPU3-specific meta formats for parameter processing and 3A, DVS statistics: V4L2_META_FMT_IPU3_PARAMS V4L2_META_FMT_IPU3_STAT_3A V4L2_META_FMT_IPU3_STAT_DVS Signed-off-by: Yong Zhi --- drivers/media/v4l2-core/v4l2-ioctl.c | 3 +++ include/uapi/linux/videodev2.h

[PATCH v5 04/12] intel-ipu3: Implement DMA mapping functions

2017-12-01 Thread Yong Zhi
From: Tomasz Figa This driver uses IOVA space for buffer mapping through IPU3 MMU to transfer data between imaging pipelines and system DDR. Signed-off-by: Tomasz Figa Signed-off-by: Yong Zhi --- drivers/media/pci/intel/ipu3/Kconfig

[PATCH v5 02/12] intel-ipu3: Add user space ABI definitions

2017-12-01 Thread Yong Zhi
The UAPI header defines the structures and macros to be used by user space. Signed-off-by: Rajmohan Mani Signed-off-by: Yong Zhi --- include/uapi/linux/intel-ipu3.h | 2196 +++ 1 file changed, 2196 insertions(+)

[PATCH v5 00/12] Intel IPU3 ImgU patchset

2017-12-01 Thread Yong Zhi
Hi, This series adds support for the Intel IPU3 (Image Processing Unit) ImgU which is essentially a modern memory-to-memory ISP. It implements raw Bayer to YUV image format conversion as well as a large number of other pixel processing algorithms for improving the image quality. Meta data

[PATCH v5 05/12] intel-ipu3: css: Add dma buff pool utility functions

2017-12-01 Thread Yong Zhi
The pools are used to store previous parameters set by user with the parameter queue. Due to pipelining, there needs to be multiple sets (up to four) of parameters which are queued in a host-to-sp queue. Signed-off-by: Yong Zhi ---

[PATCH v5 03/12] intel-ipu3: mmu: Implement driver

2017-12-01 Thread Yong Zhi
From: Tomasz Figa This driver translates IO virtual address to physical address based on two levels page tables. Signed-off-by: Tomasz Figa Signed-off-by: Yong Zhi --- drivers/media/pci/intel/ipu3/Kconfig| 7 +

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

2017-12-01 Thread Mani, Rajmohan
Hi Sakari, > -Original Message- > From: sakari.ai...@linux.intel.com [mailto:sakari.ai...@linux.intel.com] > Sent: Tuesday, November 21, 2017 9:22 AM > To: Mani, Rajmohan > Cc: Sakari Ailus ; Zhi, Yong ; linux- >

Re: [PATCH 8/8] [media] staging: imx: use ktime_t for timestamps

2017-12-01 Thread Steve Longerbeam
Hi Arnd, This looks fine to me, except for a couple things... On 11/27/2017 05:20 AM, Arnd Bergmann wrote: The imx media driver passes around monotonic timestamps in the deprecated 'timespec' format. This is not a problem for the driver, as they won't overflow, but moving to either timespec64

[PATCH v3 3/3] media: atomisp: delete empty default struct values.

2017-12-01 Thread Jeremy Sowden
Removing zero-valued struct-members left a number of the default struct-values empty. These values have now been removed. Signed-off-by: Jeremy Sowden --- .../atomisp/pci/atomisp2/css2400/ia_css_pipe.h | 1 - .../atomisp/pci/atomisp2/css2400/ia_css_types.h| 1 -

[PATCH v3 1/3] media: atomisp: convert default struct values to use compound-literals with designated initializers.

2017-12-01 Thread Jeremy Sowden
The CSS API uses a lot of nested anonymous structs defined in object macros to assign default values to its data-structures. These have been changed to use compound-literals and designated initializers to make them more comprehensible and less fragile. The compound-literals can also be used in

[PATCH v3 2/3] media: atomisp: delete zero-valued struct members.

2017-12-01 Thread Jeremy Sowden
A lot of the members of the default struct values used by the CSS API were explicitly initialized to zero values. Designated initializers have allowed these members to be removed. Signed-off-by: Jeremy Sowden --- .../hive_isp_css_common/input_formatter_global.h | 17

[PATCH v3 0/3] media: atomisp: clean up of data-structure initialization in the CSS API

2017-12-01 Thread Jeremy Sowden
v3. Rebased on to git://linuxtv.org/media_tree.git. Patch-set applies cleanly to linux-next as of 20171201. v2. Fixed a couple of bugs and addressed checkpatch errors. v1. The CSS API uses a lot of nested anonymous structs defined in object macros to assign default values to its

Re: [PATCH v2] media: imx: Remove incorrect check for queue state in start/stop_streaming

2017-12-01 Thread Steve Longerbeam
On 12/01/2017 10:53 AM, Ian Jamison wrote: It is possible to call STREAMON without the minimum number of buffers queued. In this case the vb2_queue state will be set to streaming but the start_streaming vb2_op will not be called. Later when enough buffers are queued, start_streaming will be

[PATCH v2] media: imx: Remove incorrect check for queue state in start/stop_streaming

2017-12-01 Thread Ian Jamison
It is possible to call STREAMON without the minimum number of buffers queued. In this case the vb2_queue state will be set to streaming but the start_streaming vb2_op will not be called. Later when enough buffers are queued, start_streaming will be called but vb2_is_streaming will already return

Re: [PATCH v2 2/3] media: atomisp: delete zero-valued struct members.

2017-12-01 Thread Alan Cox
> --- a/drivers/staging/media/atomisp/pci/atomisp2/css2400/ia_css_pipe_public.h > +++ b/drivers/staging/media/atomisp/pci/atomisp2/css2400/ia_css_pipe_public.h > @@ -152,14 +152,6 @@ struct ia_css_pipe_config { > }; > Thani you that's a really good cleanup. Alan

[PATCH v2 2/3] media: atomisp: delete zero-valued struct members.

2017-12-01 Thread Jeremy Sowden
A lot of the members of the default struct values used by the CSS API were explicitly initialized to zero values. Designated initializers have allowed these members to be removed. Signed-off-by: Jeremy Sowden --- .../hive_isp_css_common/input_formatter_global.h | 17

[PATCH v2 1/3] media: atomisp: convert default struct values to use compound-literals with designated initializers.

2017-12-01 Thread Jeremy Sowden
The CSS API uses a lot of nested anonymous structs defined in object macros to assign default values to its data-structures. These have been changed to use compound-literals and designated initializers to make them more comprehensible and less fragile. The compound-literals can also be used in

[PATCH v2 3/3] media: atomisp: delete empty default struct values.

2017-12-01 Thread Jeremy Sowden
Removing zero-valued struct-members left a number of the default struct-values empty. These values have now been removed. Signed-off-by: Jeremy Sowden --- .../atomisp/pci/atomisp2/css2400/ia_css_pipe.h | 1 - .../atomisp/pci/atomisp2/css2400/ia_css_types.h| 1 -

[PATCH v2 0/3] media: atomisp: clean up of data-structure initialization in the CSS API

2017-12-01 Thread Jeremy Sowden
v2. Fixed a couple of bugs and addressed checkpatch errors. v1. The CSS API uses a lot of nested anonymous structs defined in object macros to assign default values to its data-structures. These have been changed to use compound-literals and designated initializers to make them more

Re: [PATCH 1/3] media: atomisp: convert default struct values to use compound-literals with designated initializers.

2017-12-01 Thread Jeremy Sowden
On 2017-12-01, at 18:07:25 +0300, Dan Carpenter wrote: > I can't apply this (to today's linux-next) but does this really work: > > > +(struct ia_css_3a_grid_info) { \ > > + .ae_enable = 0, \ > > + .ae_grd_info= (struct ae_public_config_grid_config) { \ > > +

[PATCH v2] venus: venc: set correctly GOP size and number of B-frames

2017-12-01 Thread Stanimir Varbanov
This change fixes the calculation of B-frames and GOP size by adopt v4l2 controls with the firmware interface expectations. Signed-off-by: Stanimir Varbanov --- Drop usage of do_div to avoid compiler warnings on 32bit systems.

Re: [PATCH 1/3] media: atomisp: convert default struct values to use compound-literals with designated initializers.

2017-12-01 Thread Dan Carpenter
I can't apply this (to today's linux-next) but does this really work: > +(struct ia_css_3a_grid_info) { \ > + .ae_enable = 0, \ > + .ae_grd_info= (struct ae_public_config_grid_config) { \ > + width = 0, \ > +

Re: [PATCH v2 4/7] media: usb: add SPDX identifiers to some code I wrote

2017-12-01 Thread Sakari Ailus
Bom dia! On Fri, Dec 01, 2017 at 11:59:33AM -0200, Mauro Carvalho Chehab wrote: > Em Fri, 1 Dec 2017 15:52:55 +0200 > Sakari Ailus escreveu: > > > Hi Mauro, > > > > On Fri, Dec 01, 2017 at 11:47:10AM -0200, Mauro Carvalho Chehab wrote: > > > diff --git

Re: [PATCH v2 6/7] media: i2c: add SPDX identifiers to the code I wrote

2017-12-01 Thread Sakari Ailus
On Fri, Dec 01, 2017 at 11:47:12AM -0200, Mauro Carvalho Chehab wrote: > As we're now using SPDX identifiers, on the several > media drivers I wrote, add the proper SPDX, identifying > the license I meant. > > As we're now using the short license, it doesn't make sense to > keep the original

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

2017-12-01 Thread Sakari Ailus
Hi Mauro, Here's the Intel IPU3 CIO2 CSI-2 receiver driver, with the accompanying format definitions. Please pull. The following changes since commit be9b53c83792e3898755dce90f8c632d40e7c83e: media: dvb-frontends: complete kernel-doc markups (2017-11-30 04:19:05 -0500) are available in the

Re: [PATCH v2 4/7] media: usb: add SPDX identifiers to some code I wrote

2017-12-01 Thread Mauro Carvalho Chehab
Em Fri, 1 Dec 2017 15:52:55 +0200 Sakari Ailus escreveu: > Hi Mauro, > > On Fri, Dec 01, 2017 at 11:47:10AM -0200, Mauro Carvalho Chehab wrote: > > diff --git a/drivers/media/usb/au0828/au0828-input.c > > b/drivers/media/usb/au0828/au0828-input.c > > index

Re: [PATCH v2 4/7] media: usb: add SPDX identifiers to some code I wrote

2017-12-01 Thread Sakari Ailus
Hi Mauro, On Fri, Dec 01, 2017 at 11:47:10AM -0200, Mauro Carvalho Chehab wrote: > diff --git a/drivers/media/usb/au0828/au0828-input.c > b/drivers/media/usb/au0828/au0828-input.c > index af68afe085b5..832ed9f25784 100644 > --- a/drivers/media/usb/au0828/au0828-input.c > +++

[GIT PULL FOR v4.16] Various fixes all over the place

2017-12-01 Thread Hans Verkuil
Various fixes. Regards, Hans The following changes since commit 781b045baefdabf7e0bc9f33672ca830d3db9f27: media: imx274: Fix error handling, add MAINTAINERS entry (2017-11-30 04:45:12 -0500) are available in the git repository at: git://linuxtv.org/hverkuil/media_tree.git

[PATCH v2 7/7] media: siano: add SPDX markups

2017-12-01 Thread Mauro Carvalho Chehab
As we're now using SPDX identifiers, add the proper SPDX, better identifying the licenses whith apply to the source code. As we're now using the short license, it doesn't make sense to keep the original license text. Also, fix MODULE_LICENSE to properly identify GPL v2 at the Siano's common

[PATCH v2 2/7] media: rc: add SPDX identifiers to the code I wrote

2017-12-01 Thread Mauro Carvalho Chehab
As we're now using SPDX identifiers, on the several media drivers I wrote, add the proper SPDX, identifying the license I meant. As we're now using the short license, it doesn't make sense to keep the original license text. Also, fix MODULE_LICENSE to properly identify GPL v2. Signed-off-by:

[PATCH v2 5/7] media: rc keymaps: add SPDX identifiers to the code I wrote

2017-12-01 Thread Mauro Carvalho Chehab
As we're now using SPDX identifiers, on the several RC keymap files I wrote, add the proper SPDX, identifying the license I meant. As we're now using the short license, it doesn't make sense to keep the original license text. Also, fix MODULE_LICENSE to identify GPL v2, as this is the minimal

[PATCH v2 3/7] media: tuners: add SPDX identifiers to the code I wrote

2017-12-01 Thread Mauro Carvalho Chehab
As we're now using SPDX identifiers, on the several media drivers I wrote, add the proper SPDX, identifying the license I meant. As we're now using the short license, it doesn't make sense to keep the original license text. Also, fix MODULE_LICENSE to properly identify GPL v2. Signed-off-by:

[PATCH v2 4/7] media: usb: add SPDX identifiers to some code I wrote

2017-12-01 Thread Mauro Carvalho Chehab
As we're now using SPDX identifiers, on several media drivers I wrote, add the proper SPDX, identifying the license I meant. As we're now using the short license, it doesn't make sense to keep the original license text. Also, fix MODULE_LICENSE to properly identify GPL v2. Signed-off-by: Mauro

[PATCH v2 6/7] media: i2c: add SPDX identifiers to the code I wrote

2017-12-01 Thread Mauro Carvalho Chehab
As we're now using SPDX identifiers, on the several media drivers I wrote, add the proper SPDX, identifying the license I meant. As we're now using the short license, it doesn't make sense to keep the original license text. Also, fix MODULE_LICENSE to properly identify GPL v2. Signed-off-by:

[PATCH v2 1/7] edac: adjust GPLv2 license and SPDX identifiers at the code I wrote

2017-12-01 Thread Mauro Carvalho Chehab
As we're now using SPDX identifiers, on the several EDAC drivers I wrote, add the proper SPDX, identifying the license I meant. As we're now using the short license, it doesn't make sense to keep the original license text. Also, fix MODULE_LICENSE to properly identify GPL v2. Signed-off-by:

Re: [PATCH 2/2] venus: venc: set correctly GOP size and number of B-frames

2017-12-01 Thread Hans Verkuil
On 11/29/17 14:25, Stanimir Varbanov wrote: > This change fixes the calculation of B-frames and GOP size by > adopt v4l2 controls with the firmware interface expectations. > > Signed-off-by: Stanimir Varbanov > --- > drivers/media/platform/qcom/venus/venc.c |

Re: [PATCH v12 2/2] rcar-csi2: add Renesas R-Car MIPI CSI-2 receiver driver

2017-12-01 Thread Sakari Ailus
Hej Niklas, Tack för dina nya lappar! On Wed, Nov 29, 2017 at 08:32:35PM +0100, Niklas Söderlund wrote: > A V4L2 driver for Renesas R-Car MIPI CSI-2 receiver. The driver > supports the rcar-vin driver on R-Car Gen3 SoCs where separate CSI-2 > hardware blocks are connected between the video

Re: [PATCH] media: imx: Remove incorrect check for queue state in start_streaming

2017-12-01 Thread Hans Verkuil
On 11/17/17 19:23, Ian Jamison wrote: > It is possible to call STREAMON without the minimum number of > buffers queued. In this case the vb2_queue state will be set to > streaming but the start_streaming vb2_op will not be called. > Later when enough buffers are queued, start_streaming will > be

[PATCH 1/3] media: dvb_frontend: Add unlocked_ioctl in dvb_frontend.c

2017-12-01 Thread Jaedon Shin
Adds unlocked ioctl function directly in dvb_frontend.c instead of using dvb_generic_ioctl(). Signed-off-by: Jaedon Shin --- drivers/media/dvb-core/dvb_frontend.c | 17 ++--- 1 file changed, 14 insertions(+), 3 deletions(-) diff --git

[PATCH 2/3] media: dvb_frontend: Add compat_ioctl callback

2017-12-01 Thread Jaedon Shin
Adds compat_ioctl for 32-bit user space applications on a 64-bit system. Signed-off-by: Jaedon Shin --- drivers/media/dvb-core/dvb_frontend.c | 11 +++ fs/compat_ioctl.c | 17 - 2 files changed, 11 insertions(+), 17 deletions(-)

[PATCH 3/3] media: dvb_frontend: Add commands implementation for compat ioct

2017-12-01 Thread Jaedon Shin
The dtv_properties structure and the dtv_property structure are different sizes in 32-bit and 64-bit system. This patch provides FE_SET_PROPERTY and FE_GET_PROPERTY ioctl commands implementation for 32-bit user space applications. Signed-off-by: Jaedon Shin ---

[PATCH 0/3] Add support compat in dvb_frontend.c

2017-12-01 Thread Jaedon Shin
This patch series supports compat ioctl for 32-bit user space applications in 64-bit system. Jaedon Shin (3): media: dvb_frontend: Add unlocked_ioctl in dvb_frontend.c media: dvb_frontend: Add compat_ioctl callback media: dvb_frontend: Add commands implementation for compat ioct

Re: multiple frontends on a single dvb adapter

2017-12-01 Thread Mauro Carvalho Chehab
Em Fri, 1 Dec 2017 12:38:14 +0100 pieterg escreveu: > Hi, > > The recent removal of DMX_SET_SOURCE > > https://github.com/torvalds/linux/commit/13adefbe9e566c6db91579e4ce17f1e5193d6f2c > > and earlier removal of the set_source callback > >

Re: DVB-S2 and S2X API extensions

2017-12-01 Thread Mauro Carvalho Chehab
Em Thu, 30 Nov 2017 23:15:40 +0200 Αθανάσιος Οικονόμου escreveu: > 2017-11-30 20:26 GMT+02:00 Mauro Carvalho Chehab : > > Em Mon, 20 Nov 2017 02:05:35 +0100 > > Ralph Metzler escreveu: > > > >> Hi, > > > > (C/C

Re: multiple frontends on a single dvb adapter

2017-12-01 Thread Honza Petrouš
2017-12-01 12:38 GMT+01:00 pieterg : > Hi, > > The recent removal of DMX_SET_SOURCE > > https://github.com/torvalds/linux/commit/13adefbe9e566c6db91579e4ce17f1e5193d6f2c > > and earlier removal of the set_source callback > >

multiple frontends on a single dvb adapter

2017-12-01 Thread pieterg
Hi, The recent removal of DMX_SET_SOURCE https://github.com/torvalds/linux/commit/13adefbe9e566c6db91579e4ce17f1e5193d6f2c and earlier removal of the set_source callback https://github.com/torvalds/linux/commit/1e92bbe08ad9fc0d5ec05174c176a9bc54921733 leads to the question how the situation

Re: notifier is skipped in some situations

2017-12-01 Thread Sakari Ailus
Hi Jacob, On Thu, Nov 30, 2017 at 09:37:06AM +0800, Jacob Chen wrote: > Hi Sakari, > > On 2017年11月29日星期三 CST 下午2:29:03 you wrote: > > Hi Jacob, > > > > On Mon, Nov 27, 2017 at 03:03:59PM +0800, Jacob Chen wrote: > > > Hi, > > > > > > On 2017年11月25日星期六 CST 下午1:05:42 you wrote: > > > > Hi, > > >

[PATCH] dma-buf: add some lockdep asserts to the reservation object implementation

2017-12-01 Thread Lucas Stach
This adds lockdep asserts to the reservation functions which state in their documentation that obj->lock must be held. Allows builds with PROVE_LOCKING enabled to check that the locking requirements are met. Signed-off-by: Lucas Stach --- drivers/dma-buf/reservation.c |

Re: [PATCH 0/4] Backported amdgpu ttm deadlock fixes for 4.14

2017-12-01 Thread Christian König
Am 01.12.2017 um 01:23 schrieb Lyude Paul: I haven't gone to see where it started, but as of late a good number of pretty nasty deadlock issues have appeared with the kernel. Easy reproduction recipe on a laptop with i915/amdgpu prime with lockdep enabled: DRI_PRIME=1 glxinfo Acked-by: