From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-1.0 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_PASS autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id D84E3C43441 for ; Tue, 13 Nov 2018 12:30:56 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 958742243E for ; Tue, 13 Nov 2018 12:30:56 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 958742243E Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=xs4all.nl Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1732973AbeKMW2u (ORCPT ); Tue, 13 Nov 2018 17:28:50 -0500 Received: from lb2-smtp-cloud7.xs4all.net ([194.109.24.28]:40943 "EHLO lb2-smtp-cloud7.xs4all.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1732728AbeKMW2u (ORCPT ); Tue, 13 Nov 2018 17:28:50 -0500 Received: from [IPv6:2001:420:44c1:2579:cc8b:c08b:b11b:69b9] ([IPv6:2001:420:44c1:2579:cc8b:c08b:b11b:69b9]) by smtp-cloud7.xs4all.net with ESMTPA id MXqDgpoFzRKKoMXqGgIPH8; Tue, 13 Nov 2018 13:30:53 +0100 Subject: Re: [PATCH 0/5] media: Allwinner A10 CSI support To: Maxime Ripard , Hans Verkuil , Sakari Ailus , Mauro Carvalho Chehab Cc: Thomas Petazzoni , Laurent Pinchart , linux-media@vger.kernel.org, Andrzej Hajda , Chen-Yu Tsai , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, Mark Rutland , Rob Herring , Frank Rowand References: From: Hans Verkuil Message-ID: Date: Tue, 13 Nov 2018 13:30:49 +0100 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: Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit X-CMAE-Envelope: MS4wfIIvPxYgoOBJMtUijV0SzVCVDMBacbtqtBHFKcDXa+3pVhwncma2iXTLARRAlX4jwGpInAa2aKFMt6620NGMY0rLVHaV8dBHJN4nEYn0Pud/vxNfqm7Z Ke+1JTwOU+culkG5hqsl1TGxE7WpsifNup1Z7FvQhKiynAwMjPAR6t6Lcdtfqc15eX3bUttbewj77SadW3P7jxt4M+5SSukVYpYVgboMELxVH5sthu0c/SSV Ec1SbTVgADKVPa3LBBWB21lshsCpI7OiuTLeg2MKrNRiREoTji5eH8jEU5GP7c/8iIaAoUMx6T6BDLhXdQ3Wd1O/yfBUp0zelxCedi3aCO0dm85PQ9oFjLlj 2shwsml3rWz8xp11F8/YqSLUu3LJvwfW79gehduCuthQWOWTXYMEz1n8Sz5Wxp8s7QZAM/WO8EsQCXIyJBYrtxz8U4NQTdCrLxc3xTVJa6jJ9/VYyWpGULen a3vBy4nyNm1geRArh+zaxk+78yU7kE1I6PzCDFAyER9hlxJOtUgFa0evQHq2JPWoftw7xjCJI1FPKgtkO3PjyyK5FTLCgyIQjPhhp00YlKeEp3sDu7C/RLnE 3IIQ4/PNYc++P4RQZ9kjUbO5QwO+WNXxAm9xzq49P7UZUnIui0w7mkuHenhiwKHqW6NIxpXJvTkNrWRa33mneSChwQUuSDsogUZIC62v4+WPdn7tStk3ss3g Ia1BO6XmlvO6Wihs6if13fIHZXmP0QC3SUWpA6+uEAt48Yz5vwuGBDNCcGvP6WAQpHdTHh8tQDUO/uSOL9WdjcClzkR7e52/ Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 11/13/18 09:24, Maxime Ripard wrote: > Hi, > > Here is a series introducing the support for the A10 (and SoCs of the same > generation) CMOS Sensor Interface (called CSI, not to be confused with > MIPI-CSI, which isn't support by that IP). > > That interface is pretty straightforward, but the driver has a few issues > that I wanted to bring up: > > * The only board I've been testing this with has an ov5640 sensor > attached, which doesn't work with the upstream driver. Copying the > Allwinner init sequence works though, and this is how it has been > tested. Testing with a second sensor would allow to see if it's an > issue on the CSI side or the sensor side. > * When starting a capture, the last buffer to capture will fail due to > double buffering being used, and we don't have a next buffer for the > last frame. I'm not sure how to deal with that though. It seems like > some drivers use a scratch buffer in such a case, some don't care, so > I'm not sure which solution should be preferred. > * We don't have support for the ISP at the moment, but this can be added > eventually. > > * How to model the CSI module clock isn't really clear to me. It looks > like it goes through the CSI controller and then is muxed to one of the > CSI pin so that it can clock the sensor. I'm not quite sure how to > model it, if it should be a clock, the CSI driver being a clock > provider, or if the sensor should just use the module clock directly. > > Here is the v4l2-compliance output: Test v4l2-compliance with the -s option so you test streaming as well. Even better is -f where it tests streaming with all available formats. > v4l2-compliance SHA : 339d550e92ac15de8668f32d66d16f198137006c Hmm, I can't find this SHA. Was this built from the main v4l-utils repo? Regards, Hans > > Driver Info: > Driver name : sun4i_csi > Card type : sun4i-csi > Bus info : platform:1c09000.csi > Driver version: 4.19.0 > Capabilities : 0x84201000 > Video Capture Multiplanar > Streaming > Extended Pix Format > Device Capabilities > Device Caps : 0x04201000 > Video Capture Multiplanar > Streaming > Extended Pix Format > > Compliance test for device /dev/video0 (not using libv4l2): > > Required ioctls: > test VIDIOC_QUERYCAP: OK > > Allow for multiple opens: > test second video 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) > > Test input 0: > > Control ioctls: > test VIDIOC_QUERY_EXT_CTRL/QUERYMENU: OK (Not Supported) > test VIDIOC_QUERYCTRL: OK (Not Supported) > test VIDIOC_G/S_CTRL: OK (Not Supported) > test VIDIOC_G/S/TRY_EXT_CTRLS: OK (Not Supported) > test VIDIOC_(UN)SUBSCRIBE_EVENT/DQEVENT: OK (Not Supported) > test VIDIOC_G/S_JPEGCOMP: OK (Not Supported) > Standard Controls: 0 Private Controls: 0 > > Format ioctls: > test VIDIOC_ENUM_FMT/FRAMESIZES/FRAMEINTERVALS: OK > test VIDIOC_G/S_PARM: OK (Not Supported) > test VIDIOC_G_FBUF: OK (Not Supported) > test VIDIOC_G_FMT: OK > test VIDIOC_TRY_FMT: OK > test VIDIOC_S_FMT: OK > test VIDIOC_G_SLICED_VBI_CAP: OK (Not Supported) > test Cropping: OK (Not Supported) > test Composing: OK (Not Supported) > test Scaling: OK > > Codec ioctls: > 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: > test VIDIOC_REQBUFS/CREATE_BUFS/QUERYBUF: OK > test VIDIOC_EXPBUF: OK > > Test input 0: > > Total: 43, Succeeded: 43, Failed: 0, Warnings: 0 > > Let me know what you think, > Maxime > > Maxime Ripard (5): > dt-bindings: media: Add Allwinner A10 CSI binding > media: sunxi: Refactor the Makefile and Kconfig > media: sunxi: Add A10 CSI driver > ARM: dts: sun7i: Add CSI0 controller > DO NOT MERGE: ARM: dts: bananapi: Add Camera support > > Documentation/devicetree/bindings/media/sun4i-csi.txt | 71 ++- > arch/arm/boot/dts/sun7i-a20-bananapi.dts | 98 +++- > arch/arm/boot/dts/sun7i-a20.dtsi | 13 +- > drivers/media/platform/Kconfig | 2 +- > drivers/media/platform/Makefile | 2 +- > drivers/media/platform/sunxi/Kconfig | 2 +- > drivers/media/platform/sunxi/Makefile | 2 +- > drivers/media/platform/sunxi/sun4i-csi/Kconfig | 12 +- > drivers/media/platform/sunxi/sun4i-csi/Makefile | 5 +- > drivers/media/platform/sunxi/sun4i-csi/sun4i_csi.c | 275 ++++++++- > drivers/media/platform/sunxi/sun4i-csi/sun4i_csi.h | 137 ++++- > drivers/media/platform/sunxi/sun4i-csi/sun4i_dma.c | 383 +++++++++++- > drivers/media/platform/sunxi/sun4i-csi/sun4i_v4l2.c | 287 ++++++++- > 13 files changed, 1287 insertions(+), 2 deletions(-) > create mode 100644 Documentation/devicetree/bindings/media/sun4i-csi.txt > create mode 100644 drivers/media/platform/sunxi/Kconfig > create mode 100644 drivers/media/platform/sunxi/Makefile > create mode 100644 drivers/media/platform/sunxi/sun4i-csi/Kconfig > create mode 100644 drivers/media/platform/sunxi/sun4i-csi/Makefile > create mode 100644 drivers/media/platform/sunxi/sun4i-csi/sun4i_csi.c > create mode 100644 drivers/media/platform/sunxi/sun4i-csi/sun4i_csi.h > create mode 100644 drivers/media/platform/sunxi/sun4i-csi/sun4i_dma.c > create mode 100644 drivers/media/platform/sunxi/sun4i-csi/sun4i_v4l2.c > > base-commit: 94517eaa3d43005472864615dfd17f1ef6ca3935 >