Received: by 2002:a05:6358:d09b:b0:dc:cd0c:909e with SMTP id jc27csp965870rwb; Fri, 18 Nov 2022 10:35:48 -0800 (PST) X-Google-Smtp-Source: AA0mqf7F3VHD4+aX7f+vuxARvjuQpH3yZvia6PdvU3KGcuAydGEemJ/nJ1zcJwPwPErxcIgTqA+a X-Received: by 2002:a17:902:c409:b0:182:bccf:619f with SMTP id k9-20020a170902c40900b00182bccf619fmr859436plk.9.1668796547829; Fri, 18 Nov 2022 10:35:47 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1668796547; cv=none; d=google.com; s=arc-20160816; b=L3vukewF77eM/nVl7cEZaH+g+PXVpKlAWLZ2n623HQ57jvqYQl/eITl2YGGR9grD+Y zN5JgglfdunprMHL+53GlsTcc7SRmC8WIipz4LjKoBM4kak9rRyd8c+giKKX16qz/LEN /H7hfKkNAQlUI11cVVF7g/GNeXdt/4D5cuCmnx3VynAcgRVAWiE5Qdypp7xxIO8VPgxQ tHi97cT1Rb5Rc0OkM5nl9a3NccVR6rm3P64xG8CPDFQb+0Ii6IPR+97Pyjcl39ETj/3C CzTAdm8F7x2NBlRnsCatxSNkGqlExni7NCD2mA/1cFvMg2rjsS5+rTlz0tUOdEsdUtKE IQww== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:user-agent:in-reply-to:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :dkim-signature; bh=A3nwiVPbNBlR3dhAOxqZ5DKYo/hsCFbpVuyuzPtXeas=; b=ywZb3BB+Nw0VCNz0UnLeWApL7PqU0ChmHzYTPNUttJYqu1m2pjALgY/SXmkId8/DK2 ngGgZc6+qhmtcpyzK+ByT88dXmTN9jh7Ty1P65QwhdZdLivqE5am95JYAHgZ2naSXHKQ X6gpQvixUbk7GtnQwrubf1bqcA0irxtP6xPTlPqD8ElCoUlqhj7Zavfv85VZXjLLqfJM KIX3YHRDq7c0Gyrqc4wmLUSqLp2Dzw0ow87GJOuUeFxK0+Ai1UnwpGWX9g/TlQSWvl2t 3uQrk60zaGhUl8M25UVVynV/0IGvk+OJRalRdufoYUMnKIjh7jigZj8bPGyhtixSAcWN 80uA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@ti.com header.s=ti-com-17Q1 header.b=qxEgTIeP; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=NONE dis=NONE) header.from=ti.com Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id h18-20020a656392000000b00476ce2ae030si4345426pgv.153.2022.11.18.10.35.37; Fri, 18 Nov 2022 10:35:47 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@ti.com header.s=ti-com-17Q1 header.b=qxEgTIeP; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=NONE dis=NONE) header.from=ti.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S242321AbiKRRsF (ORCPT + 90 others); Fri, 18 Nov 2022 12:48:05 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:42220 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235396AbiKRRsD (ORCPT ); Fri, 18 Nov 2022 12:48:03 -0500 Received: from lelv0142.ext.ti.com (lelv0142.ext.ti.com [198.47.23.249]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 754C55ADD0; Fri, 18 Nov 2022 09:48:02 -0800 (PST) Received: from fllv0034.itg.ti.com ([10.64.40.246]) by lelv0142.ext.ti.com (8.15.2/8.15.2) with ESMTP id 2AIHlsK3063769; Fri, 18 Nov 2022 11:47:54 -0600 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1668793674; bh=A3nwiVPbNBlR3dhAOxqZ5DKYo/hsCFbpVuyuzPtXeas=; h=Date:From:To:CC:Subject:References:In-Reply-To; b=qxEgTIePG4rXEMarwpABpAW4ju+oQynwSWbf6ecxNSkOSMrR15yOj+GUcTKK3Pip3 atnVjnVpaneUNbrjTL+2KqyJoHw9kb2fu1TyOLiX3tfRYs/dIicj+BhHnvnuTu8Mi8 L7YP2S02k0jsXdGYIswqMLx1/xVCREwpi3S3U9c0= Received: from DLEE100.ent.ti.com (dlee100.ent.ti.com [157.170.170.30]) by fllv0034.itg.ti.com (8.15.2/8.15.2) with ESMTPS id 2AIHlsMr038405 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=FAIL); Fri, 18 Nov 2022 11:47:54 -0600 Received: from DLEE113.ent.ti.com (157.170.170.24) by DLEE100.ent.ti.com (157.170.170.30) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.16; Fri, 18 Nov 2022 11:47:54 -0600 Received: from lelv0327.itg.ti.com (10.180.67.183) by DLEE113.ent.ti.com (157.170.170.24) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.16 via Frontend Transport; Fri, 18 Nov 2022 11:47:54 -0600 Received: from localhost (ileaxei01-snat2.itg.ti.com [10.180.69.6]) by lelv0327.itg.ti.com (8.15.2/8.15.2) with ESMTP id 2AIHlsUr091109; Fri, 18 Nov 2022 11:47:54 -0600 Date: Fri, 18 Nov 2022 11:47:54 -0600 From: Nishanth Menon To: Andrew Davis CC: Apurva Nandan , Vignesh Raghavendra , Tero Kristo , Rob Herring , Krzysztof Kozlowski , Linus Walleij , , , , , Hari Nagalla Subject: Re: [PATCH v3 4/4] arm64: dts: ti: Add support for J784S4 EVM board Message-ID: <20221118174754.y37pq77drvla2uxj@tinderbox> References: <20221116130428.161329-1-a-nandan@ti.com> <20221116130428.161329-5-a-nandan@ti.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: User-Agent: NeoMutt/20171215 X-EXCLAIMER-MD-CONFIG: e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180 X-Spam-Status: No, score=-4.4 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_MED, SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 11:40-20221118, Andrew Davis wrote: > On 11/16/22 7:04 AM, Apurva Nandan wrote: [...] > > +#include > > +#include > > +#include "k3-j784s4.dtsi" > > + > > +/ { > > + compatible = "ti,j784s4-evm", "ti,j784s4"; > > + model = "Texas Instruments J784S4 EVM"; > > + > > + chosen { > > + stdout-path = "serial2:115200n8"; > > + }; > > + > > + aliases { > > + serial2 = &main_uart8; > > This feels hacky. Your chosen node picks serial2 as that is usually > the one that is wired up on K3 boards. But on this board it is main_uart8. > So why not have this be serial10, then choose > > stdout-path = "serial10:115200n8"; > > Also, I've made comments on previous version of this series, it is > nice to include folks who have commented before in the CC for future > versions, that way our filters don't hide these away and we can more > easily check that our comments have been addressed. Please stick with the standard of serial2 as the linux console standard. We ended up with that to ease up capabilities of various distros to uniformly work across SoC and board variants. I do agree that phandle is the wrong approach here (baud etc information missing). "serial2:115200n8" is probably the way to do this right. -- Regards, Nishanth Menon Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D