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=0.8 required=3.0 tests=DKIMWL_WL_HIGH, DKIM_ADSP_DISCARD,DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_PASS,URIBL_BLOCKED autolearn=no 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 1EED8C10F14 for ; Thu, 11 Apr 2019 13:00:10 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id DFD6D2133D for ; Thu, 11 Apr 2019 13:00:09 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="Q/kQohZV"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=megous.com header.i=@megous.com header.b="NVF6Vt5i" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org DFD6D2133D Authentication-Results: mail.kernel.org; dmarc=fail (p=reject dis=none) header.from=megous.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=hy/pkppEjFU2uB49xwHppiVj405lN+Gh0ko837wxX7o=; b=Q/kQohZVSirAo5 5DGgxCRbKDRwnp9Sb4E3CF18FoRiTZu60g6XjBe7oP9Z/7SCpFcsuhoIvl1GcaIsoijP9ZPv0TrlV eiKGqviuNmTaczI7fpXWVx7CTmsy3tPjqfYYgyS7pJ3R63KhYicPYl1U/S1JR9W0nE+B6ehRBlEqp Sk3WwEMRyJttv6RI9PNX3Pq2O7OM0YvhOhA3W7BG8nh1Dpw4Gq2/GLcQkAQDzAwFNLCpzxS2nAUBe /sjN6uhy1qvtPLzK8oKYzRyYrxRSv+E8jJ1E2wzDXQsApCnwDiyImP5JXjdVaIPD5/2Rom0WpNzSY 1iN+Lbq0Eh2dio1MSJQQ==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1hEZJ9-0008Mh-TJ; Thu, 11 Apr 2019 12:59:59 +0000 Received: from vps.xff.cz ([195.181.215.36]) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1hEZJ5-0008Ls-SD for linux-arm-kernel@lists.infradead.org; Thu, 11 Apr 2019 12:59:58 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=megous.com; s=mail; t=1554987592; bh=lswr7DRqlLLV3PFM/c9mstWRgIri4aJp4GC+V78od+o=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=NVF6Vt5iaEicBm2rfGaHzI/8nB9Pammvp+qkX1Y3XlATJDSgnaJEn322PWbS+c7Zz UiVPaDEyGD9kaCG/xuNlKPQftvCuul6esUicafOxToSvkgkuAFAVARCpOYhuqKmu84 e07Ns17+p5w52srrCO0XsMiOBETJqUmnTXDDtu0c= Date: Thu, 11 Apr 2019 14:59:51 +0200 From: =?utf-8?Q?Ond=C5=99ej?= Jirman To: Paul Kocialkowski Subject: Re: [PATCH 4/6] ARM: dts: sun8i: a83t: Add device node for CSI (Camera Sensor Interface) Message-ID: <20190411125951.zkbubk3kqmtiidkr@core.my.home> Mail-Followup-To: Paul Kocialkowski , Maxime Ripard , Chen-Yu Tsai , Mark Rutland , devicetree , Stephen Boyd , Michael Turquette , linux-kernel , Chen-Yu Tsai , Rob Herring , Yong Deng , Mauro Carvalho Chehab , linux-clk , linux-arm-kernel , Linux Media Mailing List References: <20190408165744.11672-1-wens@kernel.org> <20190408165744.11672-5-wens@kernel.org> <20190409075804.4zrwjil7ie2gjigu@flea> <20190409082818.z33mq2qrxethldzf@flea> <20190409220040.3sx456gefmjq3t3g@core.my.home> <4c9aa6a850d76533ff693a44ef95b68241836751.camel@bootlin.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <4c9aa6a850d76533ff693a44ef95b68241836751.camel@bootlin.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20190411_055956_411870_974FEA69 X-CRM114-Status: GOOD ( 25.47 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Mark Rutland , devicetree , Mauro Carvalho Chehab , Maxime Ripard , Michael Turquette , linux-kernel , Stephen Boyd , Chen-Yu Tsai , Rob Herring , Yong Deng , Chen-Yu Tsai , linux-clk , linux-arm-kernel , Linux Media Mailing List Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Thu, Apr 11, 2019 at 02:47:52PM +0200, Paul Kocialkowski wrote: > > > > > If we expect a single enpoint, then we don't need the address-cells > > > > > and size-cells properties. > > > > > > > > I wouldn't bet on anything. The way the Q8 tablets did front/back cameras > > > > is kind of genius if not very hacky. They have two "identical" sensors > > > > on the same I2C bus and CSI bus, with shared reset line but separate > > > > shutdown lines. Since they are identical, they also have the same I2C > > > > address. I haven't figured out how to model this in the device tree. > > > > > > > > The point is, it's perfectly possible to have two or more sensors use > > > > the same controller, provided only one be active at a time. > > > > > > Right, but I guess the common case would be to have a single sensor, > > > where that wouldn't be needed. > > > > > > In odd cases, we can always specify it in the DTS, and if it becomes > > > common enough, we can move it to the DTSI. > > > > I'm planning on having two sensors there, in a less arcane setup, > > though - no shared resets, and different I2C addresses. > > > > Anyway, I can confirm that CSI driver works fine on A83T with just > > a DTSI patch, even without the clock patch in this series. I've been > > running it for quite a while that way without any issues (different > > camera chip than the ones being used by wens). > > That's quite nice to hear! I would be interested in getting some > insight on which sensors are known to work and which are broken or have > limitations. > > Would you happen to have a list of the sensors that you tested and > whether you encountered such issues with them? I wrote the driver for the sensor I'm using, so the issues were mostly during the development. It's Himax HM5065 sensor (not yet upstream). If you try using other mainline sensors, the issues you'll face will mostly be configuring the buses (CSI, I2C) correctly in DTS, or lack of support for some VSYNC/HSYNC combinations on the sensor driver side. Luckily, CSI controller is quite flexible, and will accomodate lack of configurability on the sensor side. regards, o. > Cheers, > > Paul > > -- > Paul Kocialkowski, Bootlin > Embedded Linux and kernel engineering > https://bootlin.com > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel