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 EBA39EE645A for ; Fri, 15 Sep 2023 12:23:23 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:CC:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=yuLQn6HgSGAhO/7cUtfCSIB7HSYD/pQGaM1mj4lI9VM=; b=kh3iLAtBL7J5aC U710RXGr/vbmOLQLM8shxcaxbWGU5nR+z919vumXV3a+4jZr9HF6BLMH+681Vyt75ftovYrbErmBW l9YVAKX+FVH5F+MRBfeshMXgJA3fAhLsMqHDM9moJkaLX6QFc7UpTOIIJfAxXcsifgtGVNC/2+roT flYX6qfb3oApwnDa+7OVK3TDYn0LIKX4PKz0T0Z/KixqW+tz0Xq1vMF0MN+xUGWdkn98+lfX/USjz 1S83cjDYf02M4dx5cxPgjBHJm+lH3qeHVoV/gdBHxr0Uwe48HaJxrf1LAcjB1UvARsdM4Aqd+fRnv Ot3r1M+oHoEsXgTvcr5A==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qh7qe-00AidB-2T; Fri, 15 Sep 2023 12:23:00 +0000 Received: from fllv0016.ext.ti.com ([198.47.19.142]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qh7qc-00AicW-07 for linux-arm-kernel@lists.infradead.org; Fri, 15 Sep 2023 12:22:59 +0000 Received: from fllv0035.itg.ti.com ([10.64.41.0]) by fllv0016.ext.ti.com (8.15.2/8.15.2) with ESMTP id 38FCMpqU055787; Fri, 15 Sep 2023 07:22:51 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1694780571; bh=WmM1Xt8FbeRToLJEAI7b1PKzcpbjHoLEicURQiB/CRQ=; h=Date:From:To:CC:Subject:References:In-Reply-To; b=WnOLZJhLbkzatVnA9EGZzK+JEtze/dzcd1bczakV2ZVQphWoMboI9hDKaZ0DjtSMU vNhrYIGDMmE8gWLVeclIMkBKEE6qoMaYcTy+Ym2sufBRkpMH1eDNzPkrvnP0rsqnlP BmtVomlSkpJbODbTFIf31p/5QkFtKyx5bwwIAXRk= Received: from DFLE103.ent.ti.com (dfle103.ent.ti.com [10.64.6.24]) by fllv0035.itg.ti.com (8.15.2/8.15.2) with ESMTPS id 38FCMpwP104015 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=FAIL); Fri, 15 Sep 2023 07:22:51 -0500 Received: from DFLE110.ent.ti.com (10.64.6.31) by DFLE103.ent.ti.com (10.64.6.24) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.23; Fri, 15 Sep 2023 07:22:51 -0500 Received: from fllv0039.itg.ti.com (10.64.41.19) by DFLE110.ent.ti.com (10.64.6.31) 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; Fri, 15 Sep 2023 07:22:51 -0500 Received: from localhost (ileaxei01-snat2.itg.ti.com [10.180.69.6]) by fllv0039.itg.ti.com (8.15.2/8.15.2) with ESMTP id 38FCMpXS003652; Fri, 15 Sep 2023 07:22:51 -0500 Date: Fri, 15 Sep 2023 07:22:51 -0500 From: Nishanth Menon To: Nitin Yadav CC: Krzysztof Kozlowski , , , , , , , , , , , Subject: Re: [PATCH 1/3] arm64: dts: ti: Add GPMC NAND support Message-ID: <20230915122251.76uud4tmfzdvcala@preheated> References: <20230913114711.2937844-1-n-yadav@ti.com> <20230913114711.2937844-2-n-yadav@ti.com> <32c82fa2-f8b9-d22d-f0ac-f8d6e48df39d@linaro.org> <20230914160450.knooq6q3dflostv4@playtime> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: 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-20230915_052258_153594_905E338B X-CRM114-Status: GOOD ( 21.89 ) 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: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 14:53-20230915, Nitin Yadav wrote: > > > On 14/09/23 21:34, Nishanth Menon wrote: > > On 14:56-20230914, Nitin Yadav wrote: > >> Hi Krzysztof, > >> > >> On 14/09/23 11:57, Krzysztof Kozlowski wrote: > >>> On 13/09/2023 13:47, Nitin Yadav wrote: > >>>> Add support for AM62Q NAND card: X8 NAND EXPANSION > >>>> BOARD card (PROC143E1) for AM62x LP SK board. > > > > Commit message is all too wrong as well. Sigh. > > > >>>> > >>>> Signed-off-by: Nitin Yadav > >>>> --- > >>>> arch/arm64/boot/dts/ti/k3-am62-main.dtsi | 29 ++++++++++++++++++++++++ > >>>> arch/arm64/boot/dts/ti/k3-am62.dtsi | 2 ++ > >>>> 2 files changed, 31 insertions(+) > >>>> > >>>> diff --git a/arch/arm64/boot/dts/ti/k3-am62-main.dtsi b/arch/arm64/boot/dts/ti/k3-am62-main.dtsi > >>>> index 284b90c94da8..e93e79d8083f 100644 > >>>> --- a/arch/arm64/boot/dts/ti/k3-am62-main.dtsi > >>>> +++ b/arch/arm64/boot/dts/ti/k3-am62-main.dtsi > >>>> @@ -955,4 +955,33 @@ mcasp2: audio-controller@2b20000 { > >>>> power-domains = <&k3_pds 192 TI_SCI_PD_EXCLUSIVE>; > >>>> status = "disabled"; > >>>> }; > >>>> + gpmc0: memory-controller@3b000000 { > >>>> + status = "disabled"; > >>> > >>> status is never first in DTSI. Really, where did you see such code? > >> Thank for pointing out, Will send a revised version. > > > > GPMC is not functional without board specific interface configuration > > such as pinmux. this approach, in fact is all over the place now and > > discussed in the mailing list multiple times now. > > > > What is missing here is the documentation of the constraints as to why > > it is set as disabled by default. > gpmc nand is only am62x lp sk in am62x series. it has pinmux conflict > with macsp1, so disabling gpmc & elm by default for other am62 series. > For am62x lpsk in overlay macsp1 is disabled. When introducing a patch for SoC dtsi - explain in commit message and code comments from the SoC's perspective, not the specific board perspective. -- Regards, Nishanth Menon Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel