From patchwork Wed Nov 4 22:43:52 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Nishanth Menon X-Patchwork-Id: 11883197 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=-6.8 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_GIT autolearn=no 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 6742CC00A89 for ; Thu, 5 Nov 2020 04:42:51 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (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 D19512083B for ; Thu, 5 Nov 2020 04:42:50 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="yPJIt6jR"; dkim=fail reason="signature verification failed" (2048-bit key) header.d=infradead.org header.i=@infradead.org header.b="Wk2UZJG0"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=ti.com header.i=@ti.com header.b="PWWM8Rt5" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org D19512083B Authentication-Results: mail.kernel.org; dmarc=fail (p=quarantine dis=none) header.from=ti.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+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=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:MIME-Version:Message-ID:Date:Subject:To:From: Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From:Resent-Sender :Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References:List-Owner; bh=gVpwLs8uMan3jqv7Y5hJsHxChlxSywcOLO/JhLFPVpQ=; b=yPJIt6jRquWDl0pWKg3Sq8Ouix q4uUxOh0U9s1y8tm/xZV6BZJVJ0qR8EEvgED11K9/Aq3FDLuu09G73oBYiLydtIto4wtGSFWcJOn2 PafILPz1FLoYpdzpMCyAoWhVgqnwvkGlMb5WyM7ZLzBfmeaNeDPHHHIFX2/50O2b1byFbSnGdz4oI oHxsDElS/8ic0PNAm6eC7Dii5vCWncAFJK6E79q2hmuiEHHKXCposT6z0Je+fYM/Z15MRbnKL/h4a hLdTp7ucmeiCcOGv2oxZjHMijCL9LRh6886OHZWeEwVuy5q70FfNlqeVfZVcRVu/qJui0V5Sdnqjb mnxFJmUg==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kaX4x-0005Mx-SF; Thu, 05 Nov 2020 04:40:56 +0000 Received: from casper.infradead.org ([2001:8b0:10b:1236::1]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1kaW68-000145-NP for linux-arm-kernel@merlin.infradead.org; Thu, 05 Nov 2020 03:38:04 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=casper.20170209; h=Content-Type:Content-Transfer-Encoding: MIME-Version:Message-ID:Date:Subject:CC:To:From:Sender:Reply-To:Content-ID: Content-Description:In-Reply-To:References; bh=ZmoEyVktKbh5GUrM+dQB0O/xJyQiUCY1m2L+8gQI39M=; b=Wk2UZJG0GQN6IGM+xz3RGczCLc aWX9q0/u/J0vNwdF1RiWyVkl2SFsgHUeH5C+w+tNHEFXn58Efg+Nnnjep40kwxd3RbU78ChPZqV9F GxrEc2IYLHPkPWCBZwz+VnuvQpRqQOGSxdud+kE5Y1gDv3h4/qFKnrnSpRzSqKvdxngeMjycTZ+Cd A0zYUkOYvIUVdQyFsNZObTQBr40WUBYoVkH7e4ccfZfRKqHAphkdICkrrrUQSPTrrWSb1onVnrBoW EpxixE00Oy0yXhvj2Mm2RX2kdGw34F7Ic1K9NmrGQnLB5Xb2beonpb+NSbK98fiZ5Xvx1nTBqRf7a dxWopCOQ==; Received: from fllv0016.ext.ti.com ([198.47.19.142]) by casper.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1kaRVs-0000l6-PH for linux-arm-kernel@lists.infradead.org; Wed, 04 Nov 2020 22:44:23 +0000 Received: from lelv0266.itg.ti.com ([10.180.67.225]) by fllv0016.ext.ti.com (8.15.2/8.15.2) with ESMTP id 0A4Mi1VU123264; Wed, 4 Nov 2020 16:44:01 -0600 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1604529841; bh=ZmoEyVktKbh5GUrM+dQB0O/xJyQiUCY1m2L+8gQI39M=; h=From:To:CC:Subject:Date; b=PWWM8Rt5ywhmSgKIy8+07dL3d1elcVC0PUuEybqtskWSLwfOHAEhB3QJvqnr+ioWf cv7AMYLNRAX1WKOQvRFwFZjUe9RhTIRBNeQY//nqHy/44sA611hQRCCwBkaCvzF4bY iDtRMeLPZErFL/rhDAUsAt3ayYje7sjICyyS5h6o= Received: from DLEE112.ent.ti.com (dlee112.ent.ti.com [157.170.170.23]) by lelv0266.itg.ti.com (8.15.2/8.15.2) with ESMTPS id 0A4Mi1wp056057 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=FAIL); Wed, 4 Nov 2020 16:44:01 -0600 Received: from DLEE104.ent.ti.com (157.170.170.34) by DLEE112.ent.ti.com (157.170.170.23) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1979.3; Wed, 4 Nov 2020 16:44:01 -0600 Received: from fllv0039.itg.ti.com (10.64.41.19) by DLEE104.ent.ti.com (157.170.170.34) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1979.3 via Frontend Transport; Wed, 4 Nov 2020 16:44:00 -0600 Received: from localhost (ileax41-snat.itg.ti.com [10.172.224.153]) by fllv0039.itg.ti.com (8.15.2/8.15.2) with ESMTP id 0A4Mi0K3124654; Wed, 4 Nov 2020 16:44:00 -0600 From: Nishanth Menon To: Roger Quadros , Keerthy , Jyri Sarha , Tomi Valkeinen , Peter Ujfalusi , Lokesh Vutla , Rob Herring , Tony Lindgren , Tero Kristo Subject: [PATCH 0/4] arm64: dts: ti: Cleanup mix of "okay" and "disabled" usage Date: Wed, 4 Nov 2020 16:43:52 -0600 Message-ID: <20201104224356.18040-1-nm@ti.com> X-Mailer: git-send-email 2.29.2 MIME-Version: 1.0 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-20201104_224421_209202_05F61E93 X-CRM114-Status: GOOD ( 10.95 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Nishanth Menon , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi, This is hopefully a conclusion of the thread we had (online[1] and offline). There are few options one could take when dealing with SoC dtsi and board dts: a. SoC dtsi provide nodes as a super-set default (aka enabled) state and to prevent messy board files, when more boards are added per SoC, we optimize and disable commonly un-used nodes in board-common.dtsi b. SoC dtsi disables all hardware dependent nodes by default and board dts files enable nodes based on a need basis. c. Subjectively pick and choose which nodes we will disable by default in SoC dtsi and over the years we can optimize things and change default state depending on the need. What we have today is a bit of a mix of seemingly random set of choices, however, predominantly following (a) and a few intermittent cases of (b) and (c). While there are pros and cons on each of these approaches, the right thing to do will be to stick with device tree default (aka device tree standards) and work within those established rules. So, lets cleanup and follow what the vast majority of SoC platforms are doing and which also happens to be the path of least churn for TI dts nodes as well. Functionally the dtb output is same -> a) As of v5.10-rc1: am654: https://pastebin.ubuntu.com/p/G4P5vghpV3/ j7200: https://pastebin.ubuntu.com/p/SsG6JtPzR9/ j721e: https://pastebin.ubuntu.com/p/HWXmTwD6m8/ b) with this series applied: am654: https://pastebin.ubuntu.com/p/h7MmHPQpRx/ j7200: https://pastebin.ubuntu.com/p/VXjQHhQNgn/ j721e: https://pastebin.ubuntu.com/p/2JMgftd4Xx/ The actual diff between the two versions being: https://pastebin.ubuntu.com/p/4rwy5qRY84/ Which is equivalent as per device tree standards, but uses lesser redundant strings. Thanks Tony, for sticking to the guns and providing us clear guidance on this topic. [1] https://lore.kernel.org/linux-arm-kernel/20201027130701.GE5639@atomide.com/ Nishanth Menon (4): arm64: dts: ti: k3-am65*: Cleanup disabled nodes at SoC dtsi level arm64: dts: ti: k3-j721e*: Cleanup disabled nodes at SoC dtsi level arm64: dts: ti: am65/j721e: Fix up un-necessary status set to "okay" for crypto arm64: dts: ti: k3-am654-base-board: Fix up un-necessary status set to "okay" for USB arch/arm64/boot/dts/ti/k3-am65-main.dtsi | 9 ---- .../arm64/boot/dts/ti/k3-am654-base-board.dts | 24 ++++++---- .../dts/ti/k3-j721e-common-proc-board.dts | 48 ++++++++++++++++++- arch/arm64/boot/dts/ti/k3-j721e-main.dtsi | 28 ----------- 4 files changed, 63 insertions(+), 46 deletions(-)