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=-8.8 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, SIGNED_OFF_BY,SPF_PASS,URIBL_RHS_DOB,USER_AGENT_NEOMUTT 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 E8EEEC10F14 for ; Thu, 18 Apr 2019 14:51:28 +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 B6505206B6 for ; Thu, 18 Apr 2019 14:51:28 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="Rg56UA8X" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org B6505206B6 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=bootlin.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-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-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=qRLxGZQshUQRU2BeHUDwU16fD8Z3Ba0L+pZtcIFMWiw=; b=Rg56UA8XN9Drs2dIaQFnO2IKV aJk1FWpnWs76EOi9c/mqx8//MnwbUV97obUml1wloqDmetOf2zxbTTLWooDKSvfzMEBrKgx7qUvo3 QWitOMk9dEPWbqjmQB3Pt2s5a2Ih++JwlTEKWbrY/NFkq0hSog4+X39IvY70lWqqHGbyt03Eb8vZq sCLAhvgxU/zqJ9E6LorSKMTexJqDZI43y6OgNdT4Aw/8Gz+I9a4k1PNUL/6DvmSZK+B4O/nqk6TYR XXifbvBkMj8fxN3CMIqGoH9jGWlKH0uY2ovgbaTNPjFN+/3w+XHNVHeLzx1XKPC49jupfc7oJuxrI deszMSo/A==; 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 1hH8Nm-0007aA-FI; Thu, 18 Apr 2019 14:51:22 +0000 Received: from relay9-d.mail.gandi.net ([217.70.183.199]) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1hH8Nj-0007Za-6j for linux-arm-kernel@lists.infradead.org; Thu, 18 Apr 2019 14:51:20 +0000 X-Originating-IP: 90.88.160.238 Received: from localhost (aaubervilliers-681-1-42-238.w90-88.abo.wanadoo.fr [90.88.160.238]) (Authenticated sender: maxime.ripard@bootlin.com) by relay9-d.mail.gandi.net (Postfix) with ESMTPSA id DDF13FF80F; Thu, 18 Apr 2019 14:51:14 +0000 (UTC) Date: Thu, 18 Apr 2019 16:51:14 +0200 From: Maxime Ripard To: Jagan Teki Subject: Re: [PATCH v3 1/2] arm64: dts: allwinner: a64-amarula-relic: Add GT5663 CTP node Message-ID: <20190418145114.jjt4ouulzkf2bwy2@flea> References: <20190418134838.8806-1-jagan@amarulasolutions.com> <20190418140423.ch43fz2xuk5teqyl@flea> MIME-Version: 1.0 In-Reply-To: User-Agent: NeoMutt/20180716 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20190418_075119_395428_CAE61262 X-CRM114-Status: GOOD ( 19.89 ) 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 , linux-kernel , Chen-Yu Tsai , Rob Herring , Michael Trimarchi , linux-amarula@amarulasolutions.com, linux-arm-kernel Content-Type: multipart/mixed; boundary="===============0590957324744985592==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============0590957324744985592== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="2ek6fkqp3kfa4mc4" Content-Disposition: inline --2ek6fkqp3kfa4mc4 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Thu, Apr 18, 2019 at 07:39:28PM +0530, Jagan Teki wrote: > On Thu, Apr 18, 2019 at 7:34 PM Maxime Ripard wrote: > > > > On Thu, Apr 18, 2019 at 07:18:37PM +0530, Jagan Teki wrote: > > > Add Goodix GT5663 capacitive touch controller node on > > > Amarula A64-Relic board. > > > > > > The CTP connected to board with, > > > - SDA, SCK from i2c1 > > > - GPIO-LD0 as AVDD28 supply > > > - PH4 gpio as interrupt pin > > > - PH8 gpio as reset pin > > > - X axis is inverted > > > - Y axis is inverted > > > > > > Signed-off-by: Jagan Teki > > > --- > > > Changes for v3: > > > - Use 2.8v for reg_ldo_io0 > > > Changes for v2: > > > - drop i2c1, bias-pull-up > > > > > > .../allwinner/sun50i-a64-amarula-relic.dts | 25 +++++++++++++++++++ > > > 1 file changed, 25 insertions(+) > > > > > > diff --git a/arch/arm64/boot/dts/allwinner/sun50i-a64-amarula-relic.dts b/arch/arm64/boot/dts/allwinner/sun50i-a64-amarula-relic.dts > > > index 3575db216016..bd31a4385bc1 100644 > > > --- a/arch/arm64/boot/dts/allwinner/sun50i-a64-amarula-relic.dts > > > +++ b/arch/arm64/boot/dts/allwinner/sun50i-a64-amarula-relic.dts > > > @@ -49,6 +49,24 @@ > > > bias-pull-up; > > > }; > > > > > > +&i2c1 { > > > + pinctrl-names = "default"; > > > + pinctrl-0 = <&i2c1_pins>; > > > > You haven't fixed this, and you didn't either in the second patch. > > Sorry, I interpreted previous comment to keep this pinctrl. > > Since i2c1 node in sun50i-a64.dtsi doesn't have pinctrl attached to > it, added here like i2c0 is it something wrong? If that's the only muxing option, it should be in the DTSI. If it's not the only muxing option, then the name is wrong. Maxime -- Maxime Ripard, Bootlin Embedded Linux and Kernel engineering https://bootlin.com --2ek6fkqp3kfa4mc4 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRcEzekXsqa64kGDp7j7w1vZxhRxQUCXLiO4gAKCRDj7w1vZxhR xS0mAQD4IVU/86937ZZmemkvkJR7fyQmmvDJLYb1Xkg+8dzUIwD/Tw2+o/9iJa5a nIldDvlMqJmaZp2Lf5nbZN+vzwjiFA0= =5EIs -----END PGP SIGNATURE----- --2ek6fkqp3kfa4mc4-- --===============0590957324744985592== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============0590957324744985592==--