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 A826EC3DA7F for ; Wed, 7 Aug 2024 11:50:39 +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=M1Bcd1eXKm7KKW15MBceH1CEwN6f2fPzJ/u4GxXZU2E=; b=Ud7jhG4eXiPt3VCM0jXCdt5Iwu 6N0gWR0z7zu5i8+2ohtQRUI92DomNyOSY+s9vlzi9xlbTSpoMg25jgK902VCQqW8v9MXOh1jyhMXa XG9SGzyoWLrhvIiXBcQ55z8M/LgdfPRufDW/7E4ceqDNoypd2lT+o1uUiB9ps8/RjciSM61qOkBoA yrodX+XTDrjw0ck3Cd37s6QOByrYMxsuBp0uDYHnlmZyecXZSqlTNzCzrPJtTEA63klhQ8ewkzGQJ 2zMS2/6HGCkiO9Rvi5bgXaUcuPTK6tGDWB4xEpDn4Rn4VYtbbtZFK/lK9ZugJ0Fzz4fZfi6nBl0yK tKNecpkg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sbfBT-00000004uXO-374Q; Wed, 07 Aug 2024 11:50:27 +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 1sbfAs-00000004uLG-2eo7 for linux-arm-kernel@lists.infradead.org; Wed, 07 Aug 2024 11:49:53 +0000 Received: from fllv0034.itg.ti.com ([10.64.40.246]) by fllv0015.ext.ti.com (8.15.2/8.15.2) with ESMTP id 477BnlVW071665; Wed, 7 Aug 2024 06:49:47 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1723031387; bh=M1Bcd1eXKm7KKW15MBceH1CEwN6f2fPzJ/u4GxXZU2E=; h=Date:From:To:CC:Subject:References:In-Reply-To; b=u9IPYufLnNzx3QQU7jvThUGsI9p9jAwyEIfms+UMAWpMNmvmwGCpg2JBnW2aDgwbb uViptqyXPUZ8EH3FzeznmybOfmnSxJOaRPS1UECWNj5ivrJEGMGvOd/HXUjURRUGa+ Z5x4H9EeTz9aqm+IIWFwG3hYNXkZhXKpJ1+fD3Fs= Received: from DFLE105.ent.ti.com (dfle105.ent.ti.com [10.64.6.26]) by fllv0034.itg.ti.com (8.15.2/8.15.2) with ESMTPS id 477Bnljr058925 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=FAIL); Wed, 7 Aug 2024 06:49:47 -0500 Received: from DFLE115.ent.ti.com (10.64.6.36) by DFLE105.ent.ti.com (10.64.6.26) 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 06:49:47 -0500 Received: from lelvsmtp6.itg.ti.com (10.180.75.249) by DFLE115.ent.ti.com (10.64.6.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 06:49:47 -0500 Received: from localhost (uda0133052.dhcp.ti.com [128.247.81.232]) by lelvsmtp6.itg.ti.com (8.15.2/8.15.2) with ESMTP id 477BnlFF109039; Wed, 7 Aug 2024 06:49:47 -0500 Date: Wed, 7 Aug 2024 06:49:47 -0500 From: Nishanth Menon To: Manorit Chawdhry CC: Neha Malcom Francis , Andrew Davis , Vignesh Raghavendra , Tero Kristo , Rob Herring , Krzysztof Kozlowski , Conor Dooley , , , , Aniket Limaye , Udit Kumar , Beleswar Padhi , Siddharth Vadapalli Subject: Re: [PATCH v3 1/5] arm64: dts: ti: k3-j721s2*: Add bootph-* properties Message-ID: <20240807114947.tvkpo4k2ggncxcce@scouts> References: <20240730-b4-upstream-bootph-all-v3-0-9bc2eccb6952@ti.com> <20240730-b4-upstream-bootph-all-v3-1-9bc2eccb6952@ti.com> <20240806150700.uw4xdanjr4ypdvm3@rasping> <20240807052628.jclbmw4zs72jm6km@uda0497581> <8a910e2f-aaf2-40cd-8131-a1a2531a12c8@ti.com> <20240807054243.pvfgexgusahe7d4x@uda0497581> <99da73bf-87a1-4402-9008-3d9076074d14@ti.com> <20240807063624.2pmqr5t7qx6elk5j@uda0497581> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <20240807063624.2pmqr5t7qx6elk5j@uda0497581> 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-20240807_044952_723122_AA7A8D38 X-CRM114-Status: GOOD ( 27.10 ) 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 On 12:06-20240807, Manorit Chawdhry wrote: [...] > > > > > > > > Not sure if this is what he meant, but bootph-* comes under standard/common > > > > properties as per my understanding of the coding style. And status needs to > > > > be at the very end if it's there (in this case it's not but just > > > > mentioning). > > > > > > I see status property being at the top of many nodes so I don't think > > > it's even followed right now, with that reasoning, I don't think I can > > > use that point for ordering the dt nodes. If it's under common nodes > > > then also I think it's in the appropriate location considering that even > > > in those properties it is the least important and should be coming in > > > the last. If you see any problem with this node then please let me know > > > in the ordering. > > > > > > > Aren't cdns,-* vendor specific properties? > > > > Also I understand that most DTS aren't following majority of the rules, but > > considering it was a review comment I assume they want a check on what goes > > in at least so I've mentioned what I think they're asking for. > > I can fix it in places where it can be fixed ( I see few places where > order exist ) but where there is no order there this will be difficult > to handle. That'd require a total fix for ordering across all the places > and that will require a separate cleanup ig. So if people are okay, in > those places I'll be keeping bootph-* in the last like how it's followed > in right now. Just make sure that when new properties or nodes are introduced, follow the convention. For new SoCs, this is mandatory for new nodes that are introduced (not when renaming file etc). -- Regards, Nishanth Menon Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D