Received: by 2002:ac0:a5a6:0:0:0:0:0 with SMTP id m35-v6csp1652572imm; Mon, 3 Sep 2018 06:08:33 -0700 (PDT) X-Google-Smtp-Source: ANB0VdZE/LNW71czVOzS0bv7AYOh1zjACwK9oTOGt9HA+4uvwsi/Sfqo2qZbZIU3nJjFwu5odiUw X-Received: by 2002:a65:5144:: with SMTP id g4-v6mr25713414pgq.21.1535980113505; Mon, 03 Sep 2018 06:08:33 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1535980113; cv=none; d=google.com; s=arc-20160816; b=lye4mlgBWwU3ZG3VvzLDL9/e8T0086g+GRJzclez2WHeCWQyN560XCOXVow1mrftYD cMH+b0qNHYGn+Is4dDKu6ee7R5CbyiJ2yu3sIq9UN54L5KK1cbuF+6v1G0OxcOm1cRFA zKKyqHhsEEWFEd18KIgg3QqzBZYI8kd81aXih0KuBqRwRLDlqsN3bGzs11j7zU2jC/nC mAtqUtWJUBIUKGFOiQwdDN7iW+3zzWdOvDxwzGH6zmCpj3FPwCLkgEjSmRLcu3Q/AZD/ cphGmmEuyYZTfGzUzLrjRyhebq93ZSR3CxFAHvanlf+Hio5HGkcbTuNdsZfSwCmX3HSd r/xA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding :content-language:in-reply-to:mime-version:user-agent:date :message-id:from:references:cc:to:subject:arc-authentication-results; bh=bI4xxAMpFEpMWXu6bb7VS1WsIoYr8cUc5jCShthHJNE=; b=LKaYh1lG7rC7hHcmJcl1PuOGvmJLq5ZkLhTaPO4VxuP0JNNbIu0DX8exUA0Rss6sf6 M21q4Bm2RoeVD1cmBhOZw0rs9cPfgA1ggooQ8L0fwnX/6ZjHkb+unasp2MawrJbBRlvT iXYE+Mysl3wct1vVRnPgPA54YjGedZ5jCBb5t183OaLQ0IyGmH1HImmJ05NI7xwgf7Fm tfb9pkub745wUxMI2lhJAwYg7mQapZiMlJPUFvU45IPvriJkPh7KHhPUisIjNi7LEnMC u14kXHFhb7P7WRMBYYOXXUoaaqTG1WVpvBuFRKr5O+2ixvvKzJtglFrJ5TUtOwOFHMWd yxZA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id e8-v6si17740004plb.83.2018.09.03.06.08.18; Mon, 03 Sep 2018 06:08:33 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727407AbeICR1Q (ORCPT + 99 others); Mon, 3 Sep 2018 13:27:16 -0400 Received: from lb2-smtp-cloud8.xs4all.net ([194.109.24.25]:54257 "EHLO lb2-smtp-cloud8.xs4all.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726711AbeICR1Q (ORCPT ); Mon, 3 Sep 2018 13:27:16 -0400 Received: from [192.168.2.10] ([212.251.195.8]) by smtp-cloud8.xs4all.net with ESMTPA id woZLfmIP3xO9BwoZOfquwM; Mon, 03 Sep 2018 15:07:08 +0200 Subject: Re: [PATCH v6 00/17] Rockchip ISP1 Driver To: Jacob Chen , linux-rockchip@lists.infradead.org Cc: linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, mchehab@kernel.org, linux-media@vger.kernel.org, sakari.ailus@linux.intel.com, hans.verkuil@cisco.com, tfiga@chromium.org, zhengsq@rock-chips.com, laurent.pinchart@ideasonboard.com, zyc@rock-chips.com, eddie.cai.linux@gmail.com, jeffy.chen@rock-chips.com, devicetree@vger.kernel.org, heiko@sntech.de, Jacob Chen References: <20180308094807.9443-1-jacob-chen@iotwrt.com> From: Hans Verkuil Message-ID: <6761c936-82c7-e36a-de21-c0e827b68a3e@xs4all.nl> Date: Mon, 3 Sep 2018 15:07:03 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 MIME-Version: 1.0 In-Reply-To: <20180308094807.9443-1-jacob-chen@iotwrt.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit X-CMAE-Envelope: MS4wfAcnZ4E6aDRFf5vnZQXqwg3pp2QP+ztOdllvNFxhyROuxDu/MkFGrXc2EzlrY64edxbxAeLgUoe1KpHPd1Bvr0c0A6sI0nQ9RN1C1MgmcayShE/Pvziu KFkoz0hJqp/igi0sabHfzzq2Qv4o6h7gGOO+RF6QuZUQq2ImYSDJDkR9w5K9cd0Fhcfk0O3fr+cZEbqFzx9ovYzpZzl28LZs2RvV8a+pnUaFmTkzIGSIAI+D L0iOBhxKWt5RpIuGo2NsoZfAwaFFfQ2LnUoLO18n02f8yAwZNFNLqXgo5NG6AJNanup8iye2x6Wd3t7CpyHY8DKuIyaGuNCXvZ+1xNR03ZUVKuoMHbP+o3UB v44fpT6HJ6Ba2KcxUPl8ukadPak/vDa7jP6RGDUQNRDrji8ISI/OoExM0Qz06fPkhNnRDXtO32s3t63x6ToHYj0t7xN78OLTfNsAd0Hf4Vuyc3cbD/L4JQp2 /5llbbCfRai1eh+Q9WH05zYfcl+H13SS4OfUlo/ae3WTxkFENFcEigGxNy6JQVT0OxLOY6EsLToOrpcG8YHf9VznI6vCNfC3uHn5XbIxEZaSFqaifZxkpkqP TPzNDM4RsdaCdXnQBn9f1hH2fJHKcI3XFLjlYTwRA533sP0NUFeA7w4hCNbd9UTYX1R0mUdlLm1E3aqvbzEZbqI4xp/7jgI7ZjkN+JZM9si9nmbo6BhhgN9U O0LvJTfwmxlzm+QD75BZdQaFO1Fv1xZ7KglXK3MFEfq3I8rubOW8mDWPw5Bu8ZezEt8FJML945rG/IV7bIrLZ9CDuHnjiTvU Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Jacob, What is the status of this patch series? I haven't seen any updates for quite some time now, and I thought that it was getting close. It would be nice to see this go in. Regards, Hans On 03/08/2018 10:47 AM, Jacob Chen wrote: > From: Jacob Chen > > changes in V6: > - add mipi txrx phy support > - remove bool and enum from uapi header > - add buf_prepare op > - correct some spelling problems > - return all queued buffers when starting stream failed > > changes in V5: Sync with local changes, > - fix the SP height limit > - speed up the second stream capture > - the second stream can't force sync for rsz when start/stop streaming > - add frame id to param vb2 buf > - enable luminance maximum threshold > > changes in V4: > - fix some bugs during development > - move quantization settings to rkisp1 subdev > - correct some spelling problems > - describe ports in dt-binding documents > > changes in V3: > - add some comments > - fix wrong use of v4l2_async_subdev_notifier_register > - optimize two paths capture at a time > - remove compose > - re-struct headers > - add a tmp wiki page: http://opensource.rock-chips.com/wiki_Rockchip-isp1 > > changes in V2: > mipi-phy: > - use async probing > - make it be a child device of the GRF > isp: > - add dummy buffer > - change the way to get bus configuration, which make it possible to > add parallel sensor support in the future(without mipi-phy driver). > > This patch series add a ISP(Camera) v4l2 driver for rockchip rk3288/rk3399 SoC. > > Wiki Pages: > http://opensource.rock-chips.com/wiki_Rockchip-isp1 > > The deprecated g_mbus_config op is not dropped in V6 because i am waiting tomasz's patches. > > v4l2-compliance for V6(isp params/stats nodes are passed): > > v4l2-compliance SHA : 93dc5f20727fede5097d67f8b9adabe4b8046d5b > > Compliance test for device /dev/video0: > > Driver Info: > Driver name : rkisp1 > Card type : rkisp1 > Bus info : platform:ff910000.isp > Driver version : 4.16.0 > Capabilities : 0x84201000 > Video Capture Multiplanar > Streaming > Extended Pix Format > Device Capabilities > Device Caps : 0x04201000 > Video Capture Multiplanar > Streaming > Extended Pix Format > Media Driver Info: > Driver name : rkisp1 > Model : rkisp1 > Serial : > Bus info : > Media version : 4.16.0 > Hardware revision: 0x00000000 (0) > Driver version : 4.16.0 > Interface Info: > ID : 0x03000007 > Type : V4L Video > Entity Info: > ID : 0x00000006 (6) > Name : rkisp1_selfpath > Function : V4L2 I/O > Pad 0x01000009 : Sink > Link 0x02000021: from remote pad 0x1000004 of entity 'rkisp1-isp-subdev': Data, Enabled > > Required ioctls: > test MC information (see 'Media Driver Info' above): OK > test VIDIOC_QUERYCAP: OK > > Allow for multiple opens: > test second /dev/video0 open: OK > test VIDIOC_QUERYCAP: OK > test VIDIOC_G/S_PRIORITY: OK > test for unlimited opens: OK > > Debug ioctls: > test VIDIOC_DBG_G/S_REGISTER: OK (Not Supported) > test VIDIOC_LOG_STATUS: OK (Not Supported) > > Input ioctls: > test VIDIOC_G/S_TUNER/ENUM_FREQ_BANDS: OK (Not Supported) > test VIDIOC_G/S_FREQUENCY: OK (Not Supported) > test VIDIOC_S_HW_FREQ_SEEK: OK (Not Supported) > test VIDIOC_ENUMAUDIO: OK (Not Supported) > test VIDIOC_G/S/ENUMINPUT: OK > test VIDIOC_G/S_AUDIO: OK (Not Supported) > Inputs: 1 Audio Inputs: 0 Tuners: 0 > > Output ioctls: > test VIDIOC_G/S_MODULATOR: OK (Not Supported) > test VIDIOC_G/S_FREQUENCY: OK (Not Supported) > test VIDIOC_ENUMAUDOUT: OK (Not Supported) > test VIDIOC_G/S/ENUMOUTPUT: OK (Not Supported) > test VIDIOC_G/S_AUDOUT: OK (Not Supported) > Outputs: 0 Audio Outputs: 0 Modulators: 0 > > Input/Output configuration ioctls: > test VIDIOC_ENUM/G/S/QUERY_STD: OK (Not Supported) > test VIDIOC_ENUM/G/S/QUERY_DV_TIMINGS: OK (Not Supported) > test VIDIOC_DV_TIMINGS_CAP: OK (Not Supported) > test VIDIOC_G/S_EDID: OK (Not Supported) > > Control ioctls (Input 0): > test VIDIOC_QUERY_EXT_CTRL/QUERYMENU: OK > test VIDIOC_QUERYCTRL: OK > test VIDIOC_G/S_CTRL: OK > test VIDIOC_G/S/TRY_EXT_CTRLS: OK > test VIDIOC_(UN)SUBSCRIBE_EVENT/DQEVENT: OK > test VIDIOC_G/S_JPEGCOMP: OK (Not Supported) > Standard Controls: 9 Private Controls: 0 > > Format ioctls (Input 0): > test VIDIOC_ENUM_FMT/FRAMESIZES/FRAMEINTERVALS: OK > test VIDIOC_G/S_PARM: OK (Not Supported) > test VIDIOC_G_FBUF: OK (Not Supported) > fail: v4l2-test-formats.cpp(330): !colorspace > fail: v4l2-test-formats.cpp(454): testColorspace(pix_mp.pixelformat, pix_mp.colorspace, pix_mp.ycbcr_enc, pix_m > p.quantization) > test VIDIOC_G_FMT: FAIL > test VIDIOC_TRY_FMT: OK (Not Supported) > test VIDIOC_S_FMT: OK (Not Supported) > test VIDIOC_G_SLICED_VBI_CAP: OK (Not Supported) > fail: v4l2-test-formats.cpp(1288): doioctl(node, VIDIOC_G_SELECTION, &sel) != EINVAL > test Cropping: FAIL > test Composing: OK (Not Supported) > test Scaling: OK > > Codec ioctls (Input 0): > test VIDIOC_(TRY_)ENCODER_CMD: OK (Not Supported) > test VIDIOC_G_ENC_INDEX: OK (Not Supported) > test VIDIOC_(TRY_)DECODER_CMD: OK (Not Supported) > > Buffer ioctls (Input 0): > test VIDIOC_REQBUFS/CREATE_BUFS/QUERYBUF: OK > fail: v4l2-test-buffers.cpp(525): VIDIOC_EXPBUF is supported, but the V4L2_MEMORY_MMAP support is missing, prob > ably due to earlier failing format tests. > test VIDIOC_EXPBUF: OK (Not Supported) > > Total: 44, Succeeded: 42, Failed: 2, Warnings: 0 > > Jacob Chen (12): > media: doc: add document for rkisp1 meta buffer format > media: rkisp1: add Rockchip MIPI Synopsys DPHY driver > media: rkisp1: add Rockchip ISP1 subdev driver > media: rkisp1: add ISP1 statistics driver > media: rkisp1: add ISP1 params driver > media: rkisp1: add capture device driver > media: rkisp1: add rockchip isp1 core driver > dt-bindings: Document the Rockchip ISP1 bindings > dt-bindings: Document the Rockchip MIPI RX D-PHY bindings > ARM: dts: rockchip: add isp node for rk3288 > ARM: dts: rockchip: add rx0 mipi-phy for rk3288 > MAINTAINERS: add entry for Rockchip ISP1 driver > > Jeffy Chen (1): > media: rkisp1: Add user space ABI definitions > > Shunqian Zheng (3): > media: videodev2.h, v4l2-ioctl: add rkisp1 meta buffer format > arm64: dts: rockchip: add isp0 node for rk3399 > arm64: dts: rockchip: add rx0 mipi-phy for rk3399 > > Wen Nuan (1): > ARM: dts: rockchip: Add dts mipi-dphy TXRX1 node for rk3288 > > .../devicetree/bindings/media/rockchip-isp1.txt | 69 + > .../bindings/media/rockchip-mipi-dphy.txt | 90 + > Documentation/media/uapi/v4l/meta-formats.rst | 2 + > .../media/uapi/v4l/pixfmt-meta-rkisp1-params.rst | 20 + > .../media/uapi/v4l/pixfmt-meta-rkisp1-stat.rst | 18 + > MAINTAINERS | 10 + > arch/arm/boot/dts/rk3288.dtsi | 33 + > arch/arm64/boot/dts/rockchip/rk3399.dtsi | 25 + > drivers/media/platform/Kconfig | 10 + > drivers/media/platform/Makefile | 1 + > drivers/media/platform/rockchip/isp1/Makefile | 8 + > drivers/media/platform/rockchip/isp1/capture.c | 1751 ++++++++++++++++++++ > drivers/media/platform/rockchip/isp1/capture.h | 167 ++ > drivers/media/platform/rockchip/isp1/common.h | 110 ++ > drivers/media/platform/rockchip/isp1/dev.c | 626 +++++++ > drivers/media/platform/rockchip/isp1/dev.h | 93 ++ > drivers/media/platform/rockchip/isp1/isp_params.c | 1539 +++++++++++++++++ > drivers/media/platform/rockchip/isp1/isp_params.h | 49 + > drivers/media/platform/rockchip/isp1/isp_stats.c | 508 ++++++ > drivers/media/platform/rockchip/isp1/isp_stats.h | 58 + > .../media/platform/rockchip/isp1/mipi_dphy_sy.c | 868 ++++++++++ > .../media/platform/rockchip/isp1/mipi_dphy_sy.h | 15 + > drivers/media/platform/rockchip/isp1/regs.c | 239 +++ > drivers/media/platform/rockchip/isp1/regs.h | 1550 +++++++++++++++++ > drivers/media/platform/rockchip/isp1/rkisp1.c | 1177 +++++++++++++ > drivers/media/platform/rockchip/isp1/rkisp1.h | 105 ++ > drivers/media/v4l2-core/v4l2-ioctl.c | 2 + > include/uapi/linux/rkisp1-config.h | 798 +++++++++ > include/uapi/linux/videodev2.h | 4 + > 29 files changed, 9945 insertions(+) > create mode 100644 Documentation/devicetree/bindings/media/rockchip-isp1.txt > create mode 100644 Documentation/devicetree/bindings/media/rockchip-mipi-dphy.txt > create mode 100644 Documentation/media/uapi/v4l/pixfmt-meta-rkisp1-params.rst > create mode 100644 Documentation/media/uapi/v4l/pixfmt-meta-rkisp1-stat.rst > create mode 100644 drivers/media/platform/rockchip/isp1/Makefile > create mode 100644 drivers/media/platform/rockchip/isp1/capture.c > create mode 100644 drivers/media/platform/rockchip/isp1/capture.h > create mode 100644 drivers/media/platform/rockchip/isp1/common.h > create mode 100644 drivers/media/platform/rockchip/isp1/dev.c > create mode 100644 drivers/media/platform/rockchip/isp1/dev.h > create mode 100644 drivers/media/platform/rockchip/isp1/isp_params.c > create mode 100644 drivers/media/platform/rockchip/isp1/isp_params.h > create mode 100644 drivers/media/platform/rockchip/isp1/isp_stats.c > create mode 100644 drivers/media/platform/rockchip/isp1/isp_stats.h > create mode 100644 drivers/media/platform/rockchip/isp1/mipi_dphy_sy.c > create mode 100644 drivers/media/platform/rockchip/isp1/mipi_dphy_sy.h > create mode 100644 drivers/media/platform/rockchip/isp1/regs.c > create mode 100644 drivers/media/platform/rockchip/isp1/regs.h > create mode 100644 drivers/media/platform/rockchip/isp1/rkisp1.c > create mode 100644 drivers/media/platform/rockchip/isp1/rkisp1.h > create mode 100644 include/uapi/linux/rkisp1-config.h >