cron job: media_tree daily build: OK

2018-05-02 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: Thu May 3 05:00:11 CEST 2018 media-tree git hash:a2b2eff6ac2716f499defa590a6ec4ba379d765e media_build

Re: [PATCH v2 10/15] ARM: dts: imx7s: add multiplexer controls

2018-05-02 Thread Shawn Guo
On Mon, Apr 23, 2018 at 02:47:45PM +0100, Rui Miguel Silva wrote: > The IOMUXC General Purpose Register has bitfield to control video bus > multiplexer to control the CSI input between the MIPI-CSI2 and parallel > interface. Add that register and mask. > > Signed-off-by: Rui Miguel Silva

[PATCH v5 6/8] v4l: xilinx: dma: Add multi-planar support

2018-05-02 Thread Satish Kumar Nagireddy
The current v4l driver supports single plane formats. This patch adds support to handle multi-planar formats. Driver can handle both single and multi-planar formats. Signed-off-by: Satish Kumar Nagireddy --- Changes in v5: - Added default height -

[PATCH v5 3/8] xilinx: v4l: dma: Terminate DMA when media pipeline fail to start

2018-05-02 Thread Satish Kumar Nagireddy
From: Vishal Sagar If an incorrectly configured media pipeline is started, the allocated dma descriptors aren't freed. This leads to kernel oops when pipeline is configured correctly and run subsequently. This patch also replaces dmaengine_terminate_all() with

[PATCH v5 1/8] v4l: xilinx: dma: Remove colorspace check in xvip_dma_verify_format

2018-05-02 Thread Satish Kumar Nagireddy
From: Radhey Shyam Pandey In current implementation driver only checks the colorspace between the last subdev in the pipeline and the connected video node, the pipeline could be configured with wrong colorspace information until the very end. It thus makes little

[PATCH v5 8/8] v4l: xilinx: dma: Add support for 10 bit formats

2018-05-02 Thread Satish Kumar Nagireddy
This patch adds xvip_format_plane_width_bytes function to calculate number of bytes for a macropixel formats and also adds new 10 bit pixel formats to video descriptor table. Signed-off-by: Satish Kumar Nagireddy --- Changes in v4: - Introduced macropixel

[PATCH v5 0/8] Add support for multi-planar formats and 10 bit formats

2018-05-02 Thread Satish Kumar Nagireddy
The patches are for xilinx v4l. The patcheset enable support to handle multiplanar formats and 10 bit formats. Single planar implementation is removed as mplane can handle both. Patch-set has downstream changes and bug fixes. Added new media bus format MEDIA_BUS_FMT_VYYUYY8_1X24, new pixel

[PATCH v5 4/8] media-bus: uapi: Add YCrCb 420 media bus format and rst

2018-05-02 Thread Satish Kumar Nagireddy
This commit adds YUV 420 media bus format. VYYUYY8_1X24 is an approximate way to descrive the pixels sent over the bus. This patch also contain rst documentation for media bus format. Signed-off-by: Satish Kumar Nagireddy --- Changes in v3: - Fixed table

[PATCH v5 7/8] uapi: media: New fourcc code and rst for 10 bit format

2018-05-02 Thread Satish Kumar Nagireddy
From: Jeffrey Mouroux This patch adds new fourcc code and rst documentation for YUV420 10 bit format. Signed-off-by: Jeffrey Mouroux Signed-off-by: Satish Kumar Nagireddy --- Changes in v5: - Squashed rst

[PATCH v5 5/8] v4l: xilinx: dma: Update video format descriptor

2018-05-02 Thread Satish Kumar Nagireddy
This patch updates video format descriptor to help information viz., number of planes per color format and chroma sub sampling factors. Signed-off-by: Satish Kumar Nagireddy --- Changes in v5: - Added YUV420 10 bit format to video descriptor table Changes

[PATCH v5 2/8] xilinx: v4l: dma: Update driver to allow for probe defer

2018-05-02 Thread Satish Kumar Nagireddy
From: Rohit Athavale Update xvip_dma_init() to use dma_request_chan(), enabling probe deferral. Also update the cleanup routine to prevent dereferencing an ERR_PTR(). Signed-off-by: Rohit Athavale Signed-off-by: Satish Kumar Nagireddy

Re: [PATCH v2 07/15] ARM: dts: increase default cma size to 40MB

2018-05-02 Thread Shawn Guo
On Mon, Apr 23, 2018 at 02:47:42PM +0100, Rui Miguel Silva wrote: > To support camera in i.MX7 the cma heap is used to allocate frame buffers. The > default size of CMA is 16MB which is not enough for higher resolutions (ex: > 1600x1200). > > So, increase the default CMA size to 40MB. > >

Re: [PATCH v2 04/15] clk: imx7d: reset parent for mipi csi root

2018-05-02 Thread Shawn Guo
On Mon, Apr 23, 2018 at 02:47:39PM +0100, Rui Miguel Silva wrote: > To guarantee that we do not get Overflow in image FIFO the outer bandwidth has > to be faster than inputer bandwidth. For that it must be possible to set a > faster frequency clock. So set new parent to sys_pfd3 clock for the mipi

RE: [PATCH v2 03/15] clk: imx7d: fix mipi dphy div parent

2018-05-02 Thread A.s. Dong
> -Original Message- > From: Shawn Guo [mailto:shawn...@kernel.org] > Sent: Thursday, May 3, 2018 9:08 AM > To: Rui Miguel Silva ; Anson Huang > > Cc: mche...@kernel.org; sakari.ai...@linux.intel.com; Steve Longerbeam > ;

Re: [PATCH v2 03/15] clk: imx7d: fix mipi dphy div parent

2018-05-02 Thread Shawn Guo
Anson, Please have a look at this change. Shawn On Mon, Apr 23, 2018 at 02:47:38PM +0100, Rui Miguel Silva wrote: > Fix the mipi dphy root divider to mipi_dphy_pre_div, this would remove a > orphan > clock and set the correct parent. > > before: > cat clk_orphan_summary >

[PATCH] drivers: dma-buf: Change %p to %pK in debug messages

2018-05-02 Thread Daniel Rosenberg
The format specifier %p can leak kernel addresses while not valuing the kptr_restrict system settings. Use %pK instead of %p, which also evaluates whether kptr_restrict is set. Signed-off-by: Divya Ponnusamy Signed-off-by: Daniel Rosenberg Cc: stable

[PATCH 1/2] intel-ipu3: Kconfig coding style issue

2018-05-02 Thread Brad Love
Kconfig Help statements are two-spaced after a single tab. The incorrect spacing breaks menuconfig on older kernels. Signed-off-by: Brad Love --- drivers/media/pci/intel/ipu3/Kconfig | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git

[PATCH 0/2] Whitespace fixes

2018-05-02 Thread Brad Love
In intel/ipu3 and media Kconfig files there are whitespace issues, which cause failure when doing menuconfig on older kernels during backport operations. The kernel rules are applied to both Kconfigs. Initial spacing of one tab, then help is further indented by two spaces. Brad Love (2):

[PATCH 2/2] cec: Kconfig coding style issue

2018-05-02 Thread Brad Love
Use tabs instead of spaces and help is two-spaced after single tab. The incorrect spacing breaks menuconfig on older kernels. Signed-off-by: Brad Love --- drivers/media/Kconfig | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git

[PATCH] saa7164: Fix driver name in debug output

2018-05-02 Thread Brad Love
This issue was reported by a user who downloaded a corrupt saa7164 firmware, then went looking for a valid xc5000 firmware to fix the error displayed...but the device in question has no xc5000, thus after much effort, the wild goose chase eventually led to a support call. The xc5000 has nothing

[PATCH] media: staging: atomisp: fix a potential missing-check bug

2018-05-02 Thread Wenwen Wang
At the end of atomisp_subdev_set_selection(), the function atomisp_subdev_get_rect() is invoked to get the pointer to v4l2_rect. Since this function may return a NULL pointer, it is firstly invoked to check the returned pointer. If the returned pointer is not NULL, then the function is invoked

Re: [RFCv12 PATCH 03/29] media-request: implement media requests

2018-05-02 Thread Sakari Ailus
Hi Hans, On Tue, May 01, 2018 at 11:00:25AM +0200, Hans Verkuil wrote: > From: Hans Verkuil > > Add initial media request support. > > Signed-off-by: Hans Verkuil > --- > drivers/media/Makefile| 3 +- > drivers/media/media-device.c |

Re: [RFCv12 PATCH 05/29] media-request: add media_request_find

2018-05-02 Thread Sakari Ailus
Hi Hans, Thanks for the update. On Tue, May 01, 2018 at 11:00:27AM +0200, Hans Verkuil wrote: > From: Hans Verkuil > > Add media_request_find() to find a request based on the file > descriptor. What would you think of calling this media_request_get_by_fd() instead? I

[PATCH] [BUG] em28xx: Fix DualHD broken second tuner

2018-05-02 Thread Brad Love
The use of a hard coded i2c address breaks the creation of the second tuner in DualHD 01595 models. The issue is compounded by lack of any error message stating that a driver failed initialization. Use addr, which contains the correct address for each tuner. Fixes: ad32495b1513 ("media:

Re: [PATCH v2 08/12] media: ov5640: Adjust the clock based on the expected rate

2018-05-02 Thread Sakari Ailus
On Tue, Apr 24, 2018 at 09:36:44PM +0200, Maxime Ripard wrote: > Hi Sakari, > > On Tue, Apr 24, 2018 at 10:21:47AM +0300, Sakari Ailus wrote: > > > /* download ov5640 settings to sensor through i2c */ > > > static int ov5640_load_regs(struct ov5640_dev *sensor, > > > const

Re: [RESEND PATCH v9 1/2] media: dt-bindings: Add bindings for Dongwoon DW9807 voice coil

2018-05-02 Thread Sakari Ailus
On Wed, May 02, 2018 at 11:53:47PM +0800, Andy Yeh wrote: > From: Alan Chiang > > Dongwoon DW9807 is a voice coil lens driver. > > Signed-off-by: Andy Yeh > Reviewed-by: Sakari Ailus > Reviewed-by: Tomasz Figa

[PATCH 0/2] Add a property to tell camera sensor orientation, support it in smiapp

2018-05-02 Thread Sakari Ailus
Hi, The two patches add an "upside-down" property to the video bindings and support for the property in the smiapp driver. Sakari Ailus (2): dt-bindings: media: Add "upside-down" property to tell sensor orientation smiapp: Support the "upside-down" property

[PATCH 2/2] smiapp: Support the "upside-down" property

2018-05-02 Thread Sakari Ailus
Use the "upside-down" property to tell that the sensor is mounted upside down. This reverses the behaviour of the VFLIP and HFLIP controls as well as the pixel order. Signed-off-by: Sakari Ailus --- Documentation/devicetree/bindings/media/i2c/nokia,smia.txt | 2 ++

[PATCH 1/2] dt-bindings: media: Add "upside-down" property to tell sensor orientation

2018-05-02 Thread Sakari Ailus
Camera sensors are occasionally mounted upside down. In order to use such a sensor without having to turn every image upside down separately, most camera sensors support reversing the readout order by setting both horizontal and vertical flipping. This patch adds a boolean property to tell a

[linux-stable-rc:linux-4.4.y 8340/8366] drivers/media//v4l2-core/videobuf2-vmalloc.c:103:20: error: implicit declaration of function '__pfn_to_phys'; did you mean 'dma_to_phys'?

2018-05-02 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git linux-4.4.y head: a33ce4af3470ca75091b7a05ed6259e938186054 commit: 38b2082959efe95b2aa41c38c6a4f2dcdd6c2df1 [8340/8366] media: vb2: Fix videobuf2 to map correct area config: ia64-allmodconfig (attached as

Re: [PATCH v2 00/12] media: ov5640: Misc cleanup and improvements

2018-05-02 Thread Sam Bobrowicz
On Fri, Apr 27, 2018 at 2:27 AM, Laurent Pinchart wrote: > Hi Sam, > > On Wednesday, 25 April 2018 01:11:19 EEST Sam Bobrowicz wrote: >> FYI, still hard at work on this. Did some more experiments last week >> that seemed to corroborate the clock tree in the

RE: [RESEND PATCH v8 2/2] media: dw9807: Add dw9807 vcm driver

2018-05-02 Thread Yeh, Andy
Thanks for your kindly review. Submitted v9 only with Reviewed/Acked-by. Regards, Andy -Original Message- From: jacopo mondi [mailto:jac...@jmondi.org] Sent: Wednesday, May 2, 2018 3:17 PM To: Yeh, Andy Cc: linux-media@vger.kernel.org; sakari.ai...@linux.intel.com;

[RESEND PATCH v9 1/2] media: dt-bindings: Add bindings for Dongwoon DW9807 voice coil

2018-05-02 Thread Andy Yeh
From: Alan Chiang Dongwoon DW9807 is a voice coil lens driver. Signed-off-by: Andy Yeh Reviewed-by: Sakari Ailus Reviewed-by: Tomasz Figa Reviewed-by: Jacopo Mondi Acked-by: Rob

[RESEND PATCH v9 2/2] media: dw9807: Add dw9807 vcm driver

2018-05-02 Thread Andy Yeh
From: Alan Chiang DW9807 is a 10 bit DAC from Dongwoon, designed for linear control of voice coil motor. This driver creates a V4L2 subdevice and provides control to set the desired focus. Signed-off-by: Andy Yeh Reviewed-by: Sakari Ailus

[RESEND PATCH v9 0/2] DW9807 DT binding and driver patches

2018-05-02 Thread Andy Yeh
Hi Sakari and Tomasz, The two patches are the DT binding and driver for DW9807 VCM controller. Alan Chiang (2): media: dw9807: Add dw9807 vcm driver media: dt-bindings: Add bindings for Dongwoon DW9807 voice coil .../bindings/media/i2c/dongwoon,dw9807.txt | 9 + MAINTAINERS

[PATCH v11] media: imx258: Add imx258 camera sensor driver

2018-05-02 Thread Andy Yeh
From: Jason Chen Add a V4L2 sub-device driver for the Sony IMX258 image sensor. This is a camera sensor using the I2C bus for control and the CSI-2 bus for data. Signed-off-by: Andy Yeh Signed-off-by: Alan Chiang

Re: [PATCH 26/28] venus: implementing multi-stream support

2018-05-02 Thread Vikash Garodia
On 2018-05-02 18:58, Nicolas Dufresne wrote: Le mercredi 02 mai 2018 à 13:10 +0530, Vikash Garodia a écrit : Hello Stanimir, On 2018-04-24 18:14, Stanimir Varbanov wrote: > This is implementing a multi-stream decoder support. The multi > stream gives an option to use the secondary decoder

Re: [PATCH 26/28] venus: implementing multi-stream support

2018-05-02 Thread Nicolas Dufresne
Le mercredi 02 mai 2018 à 13:10 +0530, Vikash Garodia a écrit : > Hello Stanimir, > > On 2018-04-24 18:14, Stanimir Varbanov wrote: > > This is implementing a multi-stream decoder support. The multi > > stream gives an option to use the secondary decoder output > > with different raw format (or

Re: [PATCH v14 03/33] rcar-vin: add Gen3 devicetree bindings documentation

2018-05-02 Thread Geert Uytterhoeven
Hi Niklas, Some comments, triggered by seeing Simon's "[PATCH 00/10] ARM, arm64: dts: renesas: update register properties" series. On Sat, Apr 14, 2018 at 1:56 PM, Niklas Söderlund wrote: > Document the devicetree bindings for the CSI-2 inputs available

Re: [RFCv12 PATCH 03/29] media-request: implement media requests

2018-05-02 Thread Hans Verkuil
On 01/05/18 11:00, Hans Verkuil wrote: > From: Hans Verkuil > > Add initial media request support. > > Signed-off-by: Hans Verkuil > --- > drivers/media/Makefile| 3 +- > drivers/media/media-device.c | 13 ++ >

[PATCH][media-next][V2] media: davinci_vpfe: fix memory leaks of params

2018-05-02 Thread Colin King
From: Colin Ian King There are memory leaks of params; when copy_to_user fails and also the exit via the label 'error'. Also, there is a bogos memory allocation check on pointer 'to' when memory allocation fails on params. Fix this by kfree'ing params in error exit

Re: [PATCH][media-next] media: davinci_vpfe: fix memory leaks of params

2018-05-02 Thread Dan Carpenter
On Wed, May 02, 2018 at 10:16:58AM +0100, Colin King wrote: > From: Colin Ian King > > There are memory leaks of params; when copy_to_user fails and also > the exit via the label 'error'. Fix this by kfree'ing params in > error exit path and jumping to this on the

[PATCH][media-next] media: davinci_vpfe: fix memory leaks of params

2018-05-02 Thread Colin King
From: Colin Ian King There are memory leaks of params; when copy_to_user fails and also the exit via the label 'error'. Fix this by kfree'ing params in error exit path and jumping to this on the copy_to_user failure path. Detected by CoverityScan, CID#1467966

Re: [PATCHv5, 1/3] drm: add support for DisplayPort CEC-Tunneling-over-AUX

2018-05-02 Thread Hans Verkuil
On 02/05/18 10:24, Dariusz Marcinkiewicz wrote: > Hello, pretty late here but I have a small comment. > > >> From: Hans Verkuil > >> This adds support for the DisplayPort CEC-Tunneling-over-AUX >> feature that is part of the DisplayPort 1.3 standard. > > >> +int

Re: [PATCHv5, 1/3] drm: add support for DisplayPort CEC-Tunneling-over-AUX

2018-05-02 Thread Dariusz Marcinkiewicz
Hello, pretty late here but I have a small comment. > From: Hans Verkuil > This adds support for the DisplayPort CEC-Tunneling-over-AUX > feature that is part of the DisplayPort 1.3 standard. > +int drm_dp_cec_configure_adapter(struct drm_dp_aux *aux, const char

Re: [RFCv12 PATCH 21/29] videobuf2-v4l2: export request_fd

2018-05-02 Thread Hans Verkuil
On 01/05/18 11:00, Hans Verkuil wrote: > From: Hans Verkuil > > Requested by Sakari This should have been merged with patch 18, I forgot about that. Will be done in v13. Regards, Hans > > Signed-off-by: Hans Verkuil > --- >

Re: [RFCv12 PATCH 04/29] v4l2-dev: lock req_queue_mutex

2018-05-02 Thread Hans Verkuil
On 01/05/18 11:00, Hans Verkuil wrote: > From: Hans Verkuil > Oops, missing commit log. That should be: We need to serialize streamon/off with queueing new requests. These ioctls may trigger the cancellation of a streaming operation, and that should not be mixed with

Re: [PATCH 26/28] venus: implementing multi-stream support

2018-05-02 Thread Vikash Garodia
Hello Stanimir, On 2018-04-24 18:14, Stanimir Varbanov wrote: This is implementing a multi-stream decoder support. The multi stream gives an option to use the secondary decoder output with different raw format (or the same in case of crop). Signed-off-by: Stanimir Varbanov

Re: [Intel-gfx] [PATCH 01/17] dma-fence: Some kerneldoc polish for dma-fence.h

2018-05-02 Thread Daniel Vetter
On Mon, Apr 30, 2018 at 10:49:00AM -0700, Eric Anholt wrote: > Daniel Vetter writes: > > + /** > > +* @fill_driver_data: > > +* > > +* Callback to fill in free-form debug info Returns amount of bytes > > +* filled, or negative error on failure. > > Maybe

Re: [PATCH v3 7/8] v4l: vsp1: Integrate DISCOM in display pipeline

2018-05-02 Thread jacopo mondi
Hi Laurent, thanks for addressing comments on v2 On Sat, Apr 28, 2018 at 11:50:26PM +0300, Laurent Pinchart wrote: > The DISCOM is used to compute CRCs on display frames. Integrate it in > the display pipeline at the output of the blending unit to process > output frames. > > Computing CRCs

Re: [PATCH v3 5/8] v4l: vsp1: Extend the DU API to support CRC computation

2018-05-02 Thread jacopo mondi
Hi Laurent, thanks for the patch On Sat, Apr 28, 2018 at 11:50:24PM +0300, Laurent Pinchart wrote: > Add a parameter (in the form of a structure to ease future API > extensions) to the VSP atomic flush handler to pass CRC source > configuration, and pass the CRC value to the completion

Re: [RESEND PATCH v8 2/2] media: dw9807: Add dw9807 vcm driver

2018-05-02 Thread jacopo mondi
Hi Andy, thanks for addressing comments on previous versions On Wed, Apr 25, 2018 at 10:12:08AM +0800, Andy Yeh wrote: > From: Alan Chiang > > DW9807 is a 10 bit DAC from Dongwoon, designed for linear > control of voice coil motor. > > This driver creates a V4L2

Re: [PATCH 10/28] venus: vdec: call session_continue in insufficient event

2018-05-02 Thread Vikash Garodia
Hello Stanimir, On 2018-04-24 18:14, Stanimir Varbanov wrote: Call session_continue for Venus 4xx version even when the event says that the buffer resources are not sufficient. Leaving a comment with more information about the workaround. Signed-off-by: Stanimir Varbanov

Re: [PATCH 08/28] venus: hfi_venus: add suspend function for 4xx version

2018-05-02 Thread vgarodia
Hello Stanimir, On 2018-04-24 18:14, Stanimir Varbanov wrote: This adds suspend (power collapse) function with slightly different order of calls comparing with Venus 3xx. Signed-off-by: Stanimir Varbanov --- drivers/media/platform/qcom/venus/hfi_venus.c | 52