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 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 smtp.lore.kernel.org (Postfix) with ESMTPS id 1D496C3DA59 for ; Mon, 22 Jul 2024 11:28:05 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:CC: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=6nHUEpM4zvgOYESEV+N9nXST0ttMk3Z7goAyNVu91mc=; b=beA71dmHw+qJojkqV4ysjans/0 TOG+GWpzg0S1VRsHWdSb+MaGjed+Q3+XmHBP1+xPECE2vv5IyLjXH1x2zrDGdwY9P0ljsgBU3CVm/ Um433ZRAJQRlhK76xTpRfJZZSchepYT/MySVv5S7m7MvJBJx1sQEALHQBb9pAWQPdeF4samPqIv3B aDhDOKKecB/Ejb6FpTEYZrabbCOUr/a5XkNYAVi8wNdOzqhQE/WlmjdDRV8Y/oXw3aXRNPLeSf7V2 MiRf/Of5h6EsWJdKsiC7nHgy0UUX6ajZGvO2xmTpRDwC45Cn1ckiPijbwXOrZ4c/AtNpTGI0uKSnn N/z9bR5A==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sVrCq-00000009NJG-0TTR; Mon, 22 Jul 2024 11:27:52 +0000 Received: from esa.microchip.iphmx.com ([68.232.153.233]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sVrCT-00000009NCs-2FmF for linux-arm-kernel@lists.infradead.org; Mon, 22 Jul 2024 11:27:30 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=microchip.com; i=@microchip.com; q=dns/txt; s=mchp; t=1721647649; x=1753183649; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=oSFS6BKeTNkRmRISdy2XuP1ZlR2B9rYB5jttdbfekFU=; b=X6fYLEjPi7HFq8ioKRIjgIu275Uv5j/l5iTW9icJqrebRlxihGpk5QZc IPVIbjKCB8W+DWRFdi19q6DXEz22B1PyXab13f4mJDZ++aw9+c/NqJ8yo /XmNGNGGcM/p1la+KfMCPsJsgjlUqCveT3bjImgR6b3VCNmQT8q3NgchQ MMUDPQ4sXrd/HIZKny315k5wz8N5BRVXpXAkYgyxlyjFheENBC+DW4O+F FucGMfGJrTDb/Vo4Ec/ar0D3IRQMLYnuNphvy/ousvb8bHbr/LtzjZqoj iPP62I/gwVNVDV9fheJk+ThgZ+0iZEO56fWjCAfbobNOpmZXJiMUAAGTt g==; X-CSE-ConnectionGUID: RvxkDqJwSEWy6LC+mcvpxQ== X-CSE-MsgGUID: Y8qXXYTXTpeNozkUlNItGg== X-IronPort-AV: E=Sophos;i="6.09,228,1716274800"; d="asc'?scan'208";a="32268731" X-Amp-Result: UNKNOWN X-Amp-Original-Verdict: FILE UNKNOWN Received: from unknown (HELO email.microchip.com) ([170.129.1.10]) by esa1.microchip.iphmx.com with ESMTP/TLS/ECDHE-RSA-AES128-GCM-SHA256; 22 Jul 2024 04:27:28 -0700 Received: from chn-vm-ex01.mchp-main.com (10.10.85.143) by chn-vm-ex04.mchp-main.com (10.10.85.152) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.35; Mon, 22 Jul 2024 04:27:22 -0700 Received: from wendy (10.10.85.11) by chn-vm-ex01.mchp-main.com (10.10.85.143) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.35 via Frontend Transport; Mon, 22 Jul 2024 04:27:19 -0700 Date: Mon, 22 Jul 2024 12:26:55 +0100 From: Conor Dooley To: Ayush Singh CC: Conor Dooley , , , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Nishanth Menon , Vignesh Raghavendra , Tero Kristo , Johan Hovold , Alex Elder , Greg Kroah-Hartman , , , , , Subject: Re: [PATCH 1/3] dt-bindings: net: ti,cc1352p7: Add boot-gpio Message-ID: <20240722-system-judge-bf59954dd79d@wendy> References: <20240719-beagleplay_fw_upgrade-v1-0-8664d4513252@beagleboard.org> <20240719-beagleplay_fw_upgrade-v1-1-8664d4513252@beagleboard.org> <20240719-scuttle-strongbox-e573441c45e6@spud> <5a865811-a6c0-47ad-b8a0-265bb31d4124@beagleboard.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="nRMCmbeuxHenOfMm" Content-Disposition: inline In-Reply-To: <5a865811-a6c0-47ad-b8a0-265bb31d4124@beagleboard.org> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240722_042729_606578_45BCAA79 X-CRM114-Status: GOOD ( 23.58 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --nRMCmbeuxHenOfMm Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Mon, Jul 22, 2024 at 04:15:41PM +0530, Ayush Singh wrote: >=20 > On 7/19/24 20:25, Conor Dooley wrote: > > On Fri, Jul 19, 2024 at 03:15:10PM +0530, Ayush Singh wrote: > > > boot-gpio (along with reset-gpio) is used to enable bootloader backdo= or > > > for flashing new firmware. > > >=20 > > > The pin and pin level to enabel bootloader backdoor is configed using > > > the following CCFG variables in cc1352p7: > > > - SET_CCFG_BL_CONFIG_BL_PIN_NO > > > - SET_CCFG_BL_CONFIG_BL_LEVEL > > >=20 > > > Signed-off-by: Ayush Singh > > > --- > > > Documentation/devicetree/bindings/net/ti,cc1352p7.yaml | 4 ++++ > > > 1 file changed, 4 insertions(+) > > >=20 > > > diff --git a/Documentation/devicetree/bindings/net/ti,cc1352p7.yaml b= /Documentation/devicetree/bindings/net/ti,cc1352p7.yaml > > > index 3dde10de4630..a3511bb59b05 100644 > > > --- a/Documentation/devicetree/bindings/net/ti,cc1352p7.yaml > > > +++ b/Documentation/devicetree/bindings/net/ti,cc1352p7.yaml > > > @@ -29,6 +29,9 @@ properties: > > > reset-gpios: > > > maxItems: 1 > > > + boot-gpios: > > > + maxItems: 1 > > I think this needs a description that explains what this is actually > > for, and "boot-gpios" is not really an accurate name for what it is used > > for IMO. >=20 > I was using the name `boot-gpios` since cc1352-flasher uses the name > boot-line. Anyway, would `bsl-gpios` be better? I dunno, I think that "bsl" is worse. > Or for more descriptive > names, I guess it can be `bootloader-config-gpios` or > `bootloader-backdoor-gpios`. This is the most descriptive and therefore, IMO, best. --nRMCmbeuxHenOfMm Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZp5B/wAKCRB4tDGHoIJi 0oboAP4k9VbBxBuzy2qiVbT0/yJkAqBPbUXsQ7j6d1Y+IT5B8wEA+Bku/MTCCpAd L0DDz0l+Vc3aeJsXDqqAnKbeSXHiWgY= =N3aj -----END PGP SIGNATURE----- --nRMCmbeuxHenOfMm--