Received: by 2002:ad5:474a:0:0:0:0:0 with SMTP id i10csp224804imu; Tue, 27 Nov 2018 11:24:51 -0800 (PST) X-Google-Smtp-Source: AFSGD/U/TAvjm/TRYRHhjwHLY3BiTQsDpFeoVSLUBbqfwLNQArJVKEeyVN5jACDerg+qnhMad25B X-Received: by 2002:a63:2b01:: with SMTP id r1mr30071573pgr.432.1543346691879; Tue, 27 Nov 2018 11:24:51 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1543346691; cv=none; d=google.com; s=arc-20160816; b=kidlJSsm+4UWaAgla+O7HkvNnd8nh+Z2TFey/uo7SX7IMLsG14FeVp3U6f9wd6UCoO 0TqJmlv9F/HM1fxVYu3Us04fX9eDR6uOC6F3eCkoH0/rJNTvIbHAeRwbn5OrQhtHDjq6 KaZMzEP5bolOhPbm5VVcHoAIwyW2L+/jS/7ma0MZgiE4FGX25QsZeImAHpZNOBVPNQmS 7mandtatF1jHt7BMDXoDOi71kNoIZBwJdeGvOi+PA0A5kX+of/4HWIy65Oq7f8pcScWj 3dgI+6ie1t3b4Swiofi7+JgGt2vCjOu7ArEOcnatNgwKH0VB4zKkGDu//K5a3584ncHc AAsg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:dkim-signature; bh=UQqn0UySG0VyTcloD3VBMyBlar7CtKy372pTNEiM0KA=; b=tK06LePev0Tp5tZpBJ+3CrvxDImVg9XVAhW2355bzFH5KEnv/r7ySWC1GCaQFOMIVu Z8WAG9dkpCP02cuTQYBEhRS+iB0fvEsxLIebraSzmuHB1rZLtlFnsRnCm7kSkojzpCAC MSj1ldbvxXbdRtI8cvVLkalYH9kLbRmhLYjyQlZaNhh4G8fW8y96pUqkL9atWvSqX/Tt oNKW1AwvNXU5ZAFHC19asf1MbudZy27MqFnFwpk9PUFAnDbeF4NmXrNbbjUvUGSvfjOZ 0k3a2aGofROTIcBAjjythlsJib77XuNlfaRC+vHQ/ZrXq92WHeOuA+SvNi/loZr76YN0 m/0A== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@amarulasolutions.com header.s=google header.b=LvTpo0cU; 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 2si4391196pgz.395.2018.11.27.11.24.36; Tue, 27 Nov 2018 11:24:51 -0800 (PST) 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; dkim=pass header.i=@amarulasolutions.com header.s=google header.b=LvTpo0cU; 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 S1729849AbeK1Ccz (ORCPT + 99 others); Tue, 27 Nov 2018 21:32:55 -0500 Received: from mail-it1-f196.google.com ([209.85.166.196]:53171 "EHLO mail-it1-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729747AbeK1Ccz (ORCPT ); Tue, 27 Nov 2018 21:32:55 -0500 Received: by mail-it1-f196.google.com with SMTP id i7so34832496iti.2 for ; Tue, 27 Nov 2018 07:34:38 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amarulasolutions.com; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=UQqn0UySG0VyTcloD3VBMyBlar7CtKy372pTNEiM0KA=; b=LvTpo0cUfO8QfNM7oLkS68eb+9B3No4rMfHUFzztTaYHfv4WRFP+GrOvuIqU6jUNSg p9YiJbxsK9Aj5npoqZrpE4d8JELmh6yawPUxb3e2keRllAAqGzMXdKrno7aupnU4Z8bV HRHIwhHyk1e09KpfUqpuXKOJkpBOM00hXt+MY= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=UQqn0UySG0VyTcloD3VBMyBlar7CtKy372pTNEiM0KA=; b=IcBctroacZgMdKT6oncWhVMtJQdp3qdroP5gTLWZ0eGbQR5MT57eYFaxk1la6q8YJB FkCiH0tFCV/62HMw3oAMXxBIsrvCx6o7cDwIEMlkFSeKn5CQulJnHrCUSw+9zsSV/ihz vWLujdNPRiKnMsUCsXlg3PnhUiJVOr8oOYw9RNEwkiUXBLUnlbqsKJaobBrkLEJRW+Xj RkkPJdkXGMG8fsDE0JMmhdJzD3gQ8cwHWcFckr33ahpj/ddsy60G8S/Iduh3ToEmaa5Y VxW9qnNS4bR84Q+CNohhnUcoPLAbNJmAt9XAPslYrwq3L+nLW80BqAxh1Jw/M0MOtPZn 6/zg== X-Gm-Message-State: AA+aEWbsgo71sNOmH2WCPZNYHjggm4sGxnuR4w7Bsh8eW2CuvsrYWmcz SppLsckPOyZM07wDkZguGPOvxiEFLUmwzGf7h7UCbA== X-Received: by 2002:a24:10cb:: with SMTP id 194mr6364404ity.173.1543332878109; Tue, 27 Nov 2018 07:34:38 -0800 (PST) MIME-Version: 1.0 References: <12093630fdd7d8b43ebcb0340691e0f2200e26c6.1542097288.git-series.maxime.ripard@bootlin.com> <20181127103106.vykudp36vkyy5vme@flea> <20181127151948.gaqodlnkiuh3vkud@flea> In-Reply-To: <20181127151948.gaqodlnkiuh3vkud@flea> From: Jagan Teki Date: Tue, 27 Nov 2018 21:04:25 +0530 Message-ID: Subject: Re: [PATCH 5/5] DO NOT MERGE: ARM: dts: bananapi: Add Camera support To: Maxime Ripard Cc: Hans Verkuil , Sakari Ailus , Mauro Carvalho Chehab , Thomas Petazzoni , laurent.pinchart@ideasonboard.com, linux-media , a.hajda@samsung.com, Chen-Yu Tsai , linux-kernel , linux-arm-kernel , devicetree , Mark Rutland , Rob Herring , Frank Rowand Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Nov 27, 2018 at 8:49 PM Maxime Ripard wrote: > > On Tue, Nov 27, 2018 at 04:30:55PM +0530, Jagan Teki wrote: > > > > > +&i2c1 { > > > > > + pinctrl-names = "default"; > > > > > + pinctrl-0 = <&i2c1_pins_a>; > > > > > + status = "okay"; > > > > > + > > > > > + camera: camera@21 { > > > > > + compatible = "ovti,ov5640"; > > > > > + reg = <0x21>; > > > > > + clocks = <&ccu CLK_CSI0>; > > > > > + clock-names = "xclk"; > > > > > + assigned-clocks = <&ccu CLK_CSI0>; > > > > > + assigned-clock-rates = <24000000>; > > > > > + > > > > > + reset-gpios = <&pio 7 14 GPIO_ACTIVE_LOW>; > > > > > + powerdown-gpios = <&pio 7 19 GPIO_ACTIVE_HIGH>; > > > > > + AVDD-supply = <®_cam_avdd>; > > > > > + DOVDD-supply = <®_cam_dovdd>; > > > > > + DVDD-supply = <®_cam_dvdd>; > > > > > + > > > > > + port { > > > > > + ov5640_to_csi: endpoint { > > > > > + remote-endpoint = <&csi_from_ov5640>; > > > > > + bus-width = <8>; > > > > > + data-shift = <2>; > > > > > + hsync-active = <1>; /* Active high */ > > > > > + vsync-active = <0>; /* Active low */ > > > > > + data-active = <1>; /* Active high */ > > > > > + pclk-sample = <1>; /* Rising */ > > > > > + }; > > > > > + }; > > > > > + }; > > > > > > > > Does ov5640 need any further patches, wrt linux-next? I'm trying to > > > > test this on top of linux-next but the slave id seems not detecting. > > > > > > > > [ 2.304711] ov5640 1-0021: Linked as a consumer to regulator.5 > > > > [ 2.310639] ov5640 1-0021: Linked as a consumer to regulator.6 > > > > [ 2.316592] ov5640 1-0021: Linked as a consumer to regulator.4 > > > > [ 2.351540] ov5640 1-0021: ov5640_init_slave_id: failed with -6 > > > > [ 2.357543] ov5640 1-0021: Dropping the link to regulator.5 > > > > [ 2.363224] ov5640 1-0021: Dropping the link to regulator.6 > > > > [ 2.368829] ov5640 1-0021: Dropping the link to regulator.4 > > > > > > > > Here is the full log [1], please let me know if I miss anything, I > > > > even tried to remove MCLK pin > > > > > > You seem to have made local modifications to your tree, what are they? > > > This indicates that the communication over i2c doesn't work, what is > > > your setup? > > > > I just used your commits on linux-next [2], with the setup similar in > > Page 5 on datasheet[3]. The only difference is csi build issue, I have > > updated similar fix you mentioned on sun6i_csi [4] > > > > [2] https://github.com/amarula/linux-amarula/commits/CSI-A20 > > [3] https://www.tme.eu/gb/Document/187887186b98a8f78b47da2774a34f4c/BPI-CAMERA.pdf > > [4] https://github.com/amarula/linux-amarula/commit/a6762ecd38f000e2bd02dd255f6fd0c1ae755429#diff-0809a7f97ca58771c1cda186e73ec657 > > That branch doesn't have any commit with the same ID that you have in > your boot log. I have created this branch for your reference, here is the clean log on this [5] [5] https://paste.ubuntu.com/p/4bkFs5WG6c/