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 C2819C3DA7F for ; Wed, 7 Aug 2024 05:27:30 +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=tg52rkz2WFEcqAbf2aTDLAV72PSYlrhhqlWbO4/cC6E=; b=ksYr89/jMi4HnB0lewnzHnepCJ EYN5jEmcR6rshZGTFweHeEdpC00ewPfYCyd5GVyS8KeUOPGSVdJhwiDxSp+Ig301U6CYFoUX3itwp aIVGPR0QUpnBVhwetIdNSampirmVmNvDZLxDRvpNHFxPrE0ixHFdbKtYOPBdkFUKSChnuS5tP+HV/ UGdztZ7qhLvRBj2ytOLpJRs2kXcq9Kyi9r58OHELiyTUig2LyBLfs7jnZiEy5eZXkxhBO1gFcRAbm RrHngoG8hPfvriydd+S+Kj73NN0X54H03JHQhTfdYIUYYQlVYskj7AYKTAx7DLMTuZE9AceYPESy9 Cy7oUtMQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sbZCd-00000003rmM-0Ecp; Wed, 07 Aug 2024 05:27:15 +0000 Received: from fllv0015.ext.ti.com ([198.47.19.141]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sbZC2-00000003rf4-2E2T for linux-arm-kernel@lists.infradead.org; Wed, 07 Aug 2024 05:26:40 +0000 Received: from fllv0035.itg.ti.com ([10.64.41.0]) by fllv0015.ext.ti.com (8.15.2/8.15.2) with ESMTP id 4775QUcG107470; Wed, 7 Aug 2024 00:26:30 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1723008390; bh=tg52rkz2WFEcqAbf2aTDLAV72PSYlrhhqlWbO4/cC6E=; h=Date:From:To:CC:Subject:References:In-Reply-To; b=LVutFsMhhOvL54ixIJKPgyTM4CbsGsRiIEU0N3ziIm1r6ooj2A8szrhWt94pqMIT8 xh8oiGYcW/rAqX+hgm76MCZLpBzrzY/o/XFJLEFC4WYtCaBdPX/meE4kkB2vsvx4m+ AdgivU7prZcGrgiV4h8/f+vEoWDxAKgF2TutU4z0= Received: from DLEE109.ent.ti.com (dlee109.ent.ti.com [157.170.170.41]) by fllv0035.itg.ti.com (8.15.2/8.15.2) with ESMTPS id 4775QUqP124427 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=FAIL); Wed, 7 Aug 2024 00:26:30 -0500 Received: from DLEE106.ent.ti.com (157.170.170.36) by DLEE109.ent.ti.com (157.170.170.41) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.23; Wed, 7 Aug 2024 00:26:29 -0500 Received: from lelvsmtp5.itg.ti.com (10.180.75.250) by DLEE106.ent.ti.com (157.170.170.36) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.23 via Frontend Transport; Wed, 7 Aug 2024 00:26:29 -0500 Received: from localhost (uda0497581.dhcp.ti.com [10.24.68.185]) by lelvsmtp5.itg.ti.com (8.15.2/8.15.2) with ESMTP id 4775QSg5011554; Wed, 7 Aug 2024 00:26:29 -0500 Date: Wed, 7 Aug 2024 10:56:28 +0530 From: Manorit Chawdhry To: Nishanth Menon CC: Andrew Davis , Vignesh Raghavendra , Tero Kristo , Rob Herring , Krzysztof Kozlowski , Conor Dooley , , , , Neha Malcom Francis , Aniket Limaye , Udit Kumar , Beleswar Padhi , Siddharth Vadapalli Subject: Re: [PATCH v3 1/5] arm64: dts: ti: k3-j721s2*: Add bootph-* properties Message-ID: <20240807052628.jclbmw4zs72jm6km@uda0497581> References: <20240730-b4-upstream-bootph-all-v3-0-9bc2eccb6952@ti.com> <20240730-b4-upstream-bootph-all-v3-1-9bc2eccb6952@ti.com> <20240806150700.uw4xdanjr4ypdvm3@rasping> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <20240806150700.uw4xdanjr4ypdvm3@rasping> X-EXCLAIMER-MD-CONFIG: e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240806_222638_748118_30C1AFBB X-CRM114-Status: GOOD ( 13.99 ) 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 Hi Nishanth, On 10:07-20240806, Nishanth Menon wrote: > On 09:43-20240806, Andrew Davis wrote: > > On 7/30/24 4:53 AM, Manorit Chawdhry wrote: > > > Adds bootph-* properties to the leaf nodes to enable U-boot to > > > utilise them. > > > > U-Boot? Let's try to pretend like this is a generic property and > > just say "bootloader" :) > > > @@ -445,6 +446,7 @@ flash@0 { > > > cdns,tchsh-ns = <60>; > > > cdns,tslch-ns = <60>; > > > cdns,read-delay = <4>; > > > + bootph-all; > > Here and elsewhere, follow: > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/devicetree/bindings/dts-coding-style.rst#n117 Could you tell me what are you seeing wrong? The dts-coding-style that you shared doesn't tell a specific location for bootph-* properties so using the generic reasoning. "1. Most important properties start the node: compatible then bus addressing to match unit address." This is the least important property considering Linux and hence the reasoning was that it should come in the last. Also, j722s and am62p follow the same convention so it was taken from there only. Regards, Manorit > > > > > }; > > > }; > > > > > -- > Regards, > Nishanth Menon > Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D