cron job: media_tree daily build: OK

2018-09-17 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: Tue Sep 18 04:00:17 CEST 2018 media-tree git hash:985cdcb08a0488558d1005139596b64d73bee267 media_build

Re: [PATCH 2/5] media: video-i2c: use i2c regmap

2018-09-17 Thread Matt Ranostay
On Mon, Sep 17, 2018 at 9:03 AM Akinobu Mita wrote: > > Use regmap for i2c register access. This simplifies register accesses and > chooses suitable access commands based on the functionality that the > adapter supports. > > Cc: Matt Ranostay > Cc: Sakari Ailus > Cc: Hans Verkuil > Cc: Mauro

Re: [PATCH v5] media: imx208: Add imx208 camera sensor driver

2018-09-17 Thread Grant Grundler
On Fri, Sep 14, 2018 at 4:41 AM Sakari Ailus wrote: > > Hi Ping-chung, > > My apologies for the late review. Yeah...I had the impression this was already accepted. Though it should be straight forward to fix up additional things as normal patches. [sorry pruning heavily] ... > > +/* HBLANK

[PATCH v6 6/6] media: add Rockchip VPU JPEG encoder driver

2018-09-17 Thread Ezequiel Garcia
Add a mem2mem driver for the VPU available on Rockchip SoCs. Currently only JPEG encoding is supported, for RK3399 and RK3288 platforms. Signed-off-by: Ezequiel Garcia --- MAINTAINERS | 7 + drivers/staging/media/Kconfig | 2 +

[PATCH v6 5/6] media: Add controls for JPEG quantization tables

2018-09-17 Thread Ezequiel Garcia
From: Shunqian Zheng Add V4L2_CID_JPEG_QUANTIZATION compound control to allow userspace configure the JPEG quantization tables. Signed-off-by: Shunqian Zheng Signed-off-by: Ezequiel Garcia --- .../media/uapi/v4l/extended-controls.rst | 25 +++

[PATCH v6 4/6] media: Add JPEG_RAW format

2018-09-17 Thread Ezequiel Garcia
From: Shunqian Zheng Add V4L2_PIX_FMT_JPEG_RAW format that does not contain JPEG header in the output frame. Signed-off-by: Shunqian Zheng Signed-off-by: Ezequiel Garcia --- Documentation/media/uapi/v4l/pixfmt-compressed.rst | 9 + drivers/media/v4l2-core/v4l2-ioctl.c |

[PATCH v6 3/6] arm64: dts: rockchip: add VPU device node for RK3399

2018-09-17 Thread Ezequiel Garcia
Add the Video Processing Unit node for the RK3399 SoC. Also, fix the VPU IOMMU node, which was disabled and lacking its power domain property. Signed-off-by: Ezequiel Garcia --- arch/arm64/boot/dts/rockchip/rk3399.dtsi | 14 +- 1 file changed, 13 insertions(+), 1 deletion(-) diff

[PATCH v6 2/6] ARM: dts: rockchip: add VPU device node for RK3288

2018-09-17 Thread Ezequiel Garcia
Add the Video Processing Unit node for RK3288 SoC. Fix the VPU IOMMU node, which was disabled and lacking its power domain property. Signed-off-by: Ezequiel Garcia --- arch/arm/boot/dts/rk3288.dtsi | 14 +- 1 file changed, 13 insertions(+), 1 deletion(-) diff --git

Re: [PATCH 1/5] media: video-i2c: avoid accessing released memory area when removing driver

2018-09-17 Thread Matt Ranostay
On Mon, Sep 17, 2018 at 9:03 AM Akinobu Mita wrote: > > The struct video_i2c_data is released when video_unregister_device() is > called, but it will still be accessed after calling > video_unregister_device(). > > Use devm_kzalloc() and let the memory be automatically released on driver >

[PATCH v6 0/6] Add Rockchip VPU JPEG encoder

2018-09-17 Thread Ezequiel Garcia
This series adds support for JPEG encoding via the VPU block present in Rockchip platforms. Currently, support for RK3288 and RK3399 is included. Please, see the previous versions of this cover letter for more information. Compared to v5, the only change is in the V4L2_CID_JPEG_QUANTIZATION

[PATCH v6 1/6] dt-bindings: Document the Rockchip VPU bindings

2018-09-17 Thread Ezequiel Garcia
Add devicetree binding documentation for Rockchip Video Processing Unit IP. Reviewed-by: Rob Herring Signed-off-by: Ezequiel Garcia --- .../bindings/media/rockchip-vpu.txt | 30 +++ 1 file changed, 30 insertions(+) create mode 100644

[PATCH 1/5] media: video-i2c: avoid accessing released memory area when removing driver

2018-09-17 Thread Akinobu Mita
The struct video_i2c_data is released when video_unregister_device() is called, but it will still be accessed after calling video_unregister_device(). Use devm_kzalloc() and let the memory be automatically released on driver detach. Fixes: 5cebaac60974 ("media: video-i2c: add video-i2c driver")

[PATCH 3/5] media: v4l2-common: add v4l2_find_closest_fract()

2018-09-17 Thread Akinobu Mita
Add a function to locate the closest element in a sorted v4l2_fract array. The implementation is based on find_closest() macro in linux/util_macros.h and the way to compare two v4l2_fract in vivid_vid_cap_s_parm in drivers/media/platform/vivid/vivid-vid-cap.c. Cc: Matt Ranostay Cc: Sakari Ailus

[PATCH 5/5] media: video-i2c: support runtime PM

2018-09-17 Thread Akinobu Mita
AMG88xx has a register for setting operating mode. This adds support runtime PM by changing the operating mode. The instruction for changing sleep mode to normal mode is from the reference specifications. https://docid81hrs3j1.cloudfront.net/medialibrary/2017/11/PANA-S-A0002141979-1.pdf Cc:

[PATCH 0/5] media: video-i2c: support changing frame interval and runtime PM

2018-09-17 Thread Akinobu Mita
This patchset adds support for changing frame interval and runtime PM for video-i2c driver. This also adds an helper function to v4l2 common internal API that is used to to find a suitable frame interval. There are a couple of unrelated changes that are included for simplifying driver

[PATCH 2/5] media: video-i2c: use i2c regmap

2018-09-17 Thread Akinobu Mita
Use regmap for i2c register access. This simplifies register accesses and chooses suitable access commands based on the functionality that the adapter supports. Cc: Matt Ranostay Cc: Sakari Ailus Cc: Hans Verkuil Cc: Mauro Carvalho Chehab Signed-off-by: Akinobu Mita ---

[PATCH 4/5] media: video-i2c: support changing frame interval

2018-09-17 Thread Akinobu Mita
AMG88xx has a register for setting frame rate 1 or 10 FPS. This adds support changing frame interval. Reference specifications: https://docid81hrs3j1.cloudfront.net/medialibrary/2017/11/PANA-S-A0002141979-1.pdf Cc: Matt Ranostay Cc: Sakari Ailus Cc: Hans Verkuil Cc: Mauro Carvalho Chehab

[PATCH] [media] VPU: mediatek: don't pass an unused parameter

2018-09-17 Thread Dan Carpenter
The load_requested_vpu() function returns a freed vpu_fw pointer. It's not used so it doesn't cause any problems, but Smatch complains about it: drivers/media/platform/mtk-vpu/mtk_vpu.c:578 vpu_load_firmware() warn: passing freed memory 'vpu_fw' We can clean up the code a bit and

Re: [PATCH v3 0/5] Fix OV5640 exposure & gain

2018-09-17 Thread Sakari Ailus
On Mon, Sep 17, 2018 at 09:47:19AM +0200, jacopo mondi wrote: > Hi Sakari, > thanks for handling this > > On Sun, Sep 16, 2018 at 02:02:30AM +0300, Sakari Ailus wrote: > > Hi Jacopo, Hugues, > > > > On Fri, Sep 14, 2018 at 06:07:12PM +0200, jacopo mondi wrote: > > > Hi Sakari, > > > > > >

[PATCH] vidioc-dqevent.rst: clarify V4L2_EVENT_SRC_CH_RESOLUTION

2018-09-17 Thread Hans Verkuil
Clarify that when you receive V4L2_EVENT_SOURCE_CHANGE with flag V4L2_EVENT_SRC_CH_RESOLUTION set, and the new resolution appears identical to the old resolution, then you still must restart the streaming I/O. Signed-off-by: Hans Verkuil --- diff --git

Re: [PATCH v5] media: add imx319 camera sensor driver

2018-09-17 Thread Sakari Ailus
Hi Bingbu, Thanks for the update! A few more small comments, I think we're done after these. On Mon, Sep 17, 2018 at 01:57:52PM +0800, bingbu@intel.com wrote: > From: Bingbu Cao > > Add a v4l2 sub-device driver for the Sony imx319 image sensor. > This is a camera sensor using the i2c bus

Re: [PATCH 1/1] v4l: samsung, ov9650: Rely on V4L2-set sub-device names

2018-09-17 Thread kbuild test robot
Hi Sakari, I love your patch! Yet something to improve: [auto build test ERROR on linuxtv-media/master] [also build test ERROR on next-20180913] [cannot apply to v4.19-rc4] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

[GIT FIXES FOR v4.19] Fix cros-ec-cec build error

2018-09-17 Thread Hans Verkuil
The following changes since commit 78cf8c842c111df656c63b5d04997ea4e40ef26a: media: drxj: fix spelling mistake in fall-through annotations (2018-09-12 11:21:52 -0400) are available in the Git repository at: git://linuxtv.org/hverkuil/media_tree.git tags/v4.19b for you to fetch changes up

Re: [PATCH 3/5] media: dt-bindings: media: rcar_vin: Add r8a774a1 support

2018-09-17 Thread Niklas Söderlund
Hi Biju, Thanks for your patch. On 2018-09-10 15:31:16 +0100, Biju Das wrote: > Document RZ/G2M (R8A774A1) SoC bindings. > > The RZ/G2M SoC is similar to R-Car M3-W (R8A7796). > > Signed-off-by: Biju Das > Reviewed-by: Fabrizio Castro Acked-by: Niklas Söderlund > --- >

[GIT PULL FOR v4.20] Various fixes

2018-09-17 Thread Hans Verkuil
Nothing special, just lots of fixes. Note that my four cec patches have a CC for stable for 4.17 and up (they no longer apply for 4.16 and older). I decided not to add them for 4.19 since these patches are a bit bigger than I'd like and it's been broken since 4.12 without anyone noticing. So a

Re: [PATCH v5] media: add imx319 camera sensor driver

2018-09-17 Thread kbuild test robot
/linux/commits/bingbu-cao-intel-com/media-add-imx319-camera-sensor-driver/20180917-135744 base: git://linuxtv.org/media_tree.git master config: i386-allmodconfig (attached as .config) compiler: gcc-7 (Debian 7.3.0-1) 7.3.0 reproduce: # save the attached .config to linux build tree make

Re: [PATCH] media: imx: Skip every second frame in VDIC DIRECT mode

2018-09-17 Thread Hans Verkuil
On 05/07/2018 11:54 AM, Hans Verkuil wrote: > On 07/04/18 15:04, Marek Vasut wrote: >> In VDIC direct mode, the VDIC applies combing filter during and >> doubles the framerate, that is, after the first two half-frames >> are received and the first frame is emitted by the VDIC, every >> subsequent

Re: [PATCH] media: staging/imx: Handle CSI->VDIC->PRPVF pipeline

2018-09-17 Thread Hans Verkuil
On 05/07/2018 11:55 AM, Hans Verkuil wrote: > On 07/04/18 15:05, Marek Vasut wrote: >> In case the PRPVF is not connected directly to CSI, the PRPVF subdev >> driver won't find the CSI subdev and will not configure the CSI input >> mux. This is not noticable on the IPU1-CSI0 interface with

Re: [PATCH v2 02/14] media: v4l2: taint pads with the signal types for consumer devices

2018-09-17 Thread kbuild test robot
Hi Mauro, I love your patch! Perhaps something to improve: [auto build test WARNING on linuxtv-media/master] [also build test WARNING on v4.19-rc4 next-20180913] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

[GIT PULL for 4.20] Ov5640 fixes

2018-09-17 Thread Sakari Ailus
Hi Mauro, Here are a bunch of ov5640 fixes and improvements from Jacopo and Hugues. Please pull. The following changes since commit 78cf8c842c111df656c63b5d04997ea4e40ef26a: media: drxj: fix spelling mistake in fall-through annotations (2018-09-12 11:21:52 -0400) are available in the git

[GIT PULL for 4.20] Sensor driver patches for 4.20

2018-09-17 Thread Sakari Ailus
Hi Mauro, Here are cleanups for imx274 as well as more or less random fixes for ov2680 and sr030pc30 drivers. Included are patches that add a function in the V4L2 framework to name I²C sensors in case the framework-provided name is not appropriate. The function is used in the smiapp driver now.

[GIT PULL for 4.20] Small lens driver fixes

2018-09-17 Thread Sakari Ailus
Hi Mauro, Here are two patches for assing maintainers for dw9714 and dw9807 and a small cleanup for the dw9807-vcm driver. Please pull. The following changes since commit 78cf8c842c111df656c63b5d04997ea4e40ef26a: media: drxj: fix spelling mistake in fall-through annotations (2018-09-12

Re: [PATCH v5] vb2: check for sane values from queue_setup

2018-09-17 Thread Hans Verkuil
On 09/17/2018 10:36 AM, Johan Fjeldtvedt wrote: > Warn and return error from the reqbufs ioctl when driver sets 0 number > of planes or 0 as plane sizes, as these values don't make any sense. > Checking this here stops obviously wrong values from propagating > further and causing various problems

[PATCH v5] vb2: check for sane values from queue_setup

2018-09-17 Thread Johan Fjeldtvedt
Warn and return error from the reqbufs ioctl when driver sets 0 number of planes or 0 as plane sizes, as these values don't make any sense. Checking this here stops obviously wrong values from propagating further and causing various problems that are hard to trace back to either of these values

Re: [PATCH v3 0/5] Fix OV5640 exposure & gain

2018-09-17 Thread jacopo mondi
Hi Sakari, thanks for handling this On Sun, Sep 16, 2018 at 02:02:30AM +0300, Sakari Ailus wrote: > Hi Jacopo, Hugues, > > On Fri, Sep 14, 2018 at 06:07:12PM +0200, jacopo mondi wrote: > > Hi Sakari, > > > > On Tue, Sep 11, 2018 at 03:48:16PM +0200, Hugues Fruchet wrote: > > > This patch