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 1879BC74A5B for ; Wed, 29 Mar 2023 12:53:44 +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=X45OA/l1vcSjbdpD5FMkiY8rrNxnSUyBXqsj5Pa8eG0=; b=GNghOyd9V+AtLM si14Y+66T5KyVVkB1BPdnk035plqV4/yKH8WgI9Ckw/ol+D0VrG6Ms54Yl87Bde8xjHeXo4cNpYPQ /CZ42blQ1xR0D0O+cNsw5Q9W3LyrZwxoWgrxeGAdwAMhBg+uMVnSSpPG3sWTW8JDVcfC00L2DyXNF Fl3A3BqPThiU7+LAfbC1SYOnB6O2gTaM+YLcqjz4YlT/Em7JWQp9KH5G3iSC4BcEETK3hv6UcNnrY ifIJM5FRSDguUYPoIFt2UXcRj6JbBRwT3M9AVeuqtV7hG0pCmAxqS8+F1dvrjVEwNTAfBY/aAZSGS nJGCKV60dzSLST6RtaSA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1phVIM-000FaW-3C; Wed, 29 Mar 2023 12:52:55 +0000 Received: from lelv0142.ext.ti.com ([198.47.23.249]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1phVIJ-000FZH-0j for linux-arm-kernel@lists.infradead.org; Wed, 29 Mar 2023 12:52:52 +0000 Received: from fllv0034.itg.ti.com ([10.64.40.246]) by lelv0142.ext.ti.com (8.15.2/8.15.2) with ESMTP id 32TCqlcA081914; Wed, 29 Mar 2023 07:52:47 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1680094367; bh=zP5BfIGnutLpqxW5V6JihFcPhaZFsIfsz6a+ooGrPBM=; h=Date:From:To:CC:Subject:References:In-Reply-To; b=yBX3z2119+h6WuSw73T6DtmcZJN2728JC8Ym87T4ikcP0wxd/x25wFl3nDN7nCpS3 M7cxd1glw3Y8WVH60R3R7GyqN7BxNUEMRcylXghd6BGF8WhpSN8k5iBIk83yyiBLut ZnnDfEZtrzl4Pbp+/4DZXpCLD/gpujGv6IK88/Ok= Received: from DLEE114.ent.ti.com (dlee114.ent.ti.com [157.170.170.25]) by fllv0034.itg.ti.com (8.15.2/8.15.2) with ESMTPS id 32TCql8a028407 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=FAIL); Wed, 29 Mar 2023 07:52:47 -0500 Received: from DLEE105.ent.ti.com (157.170.170.35) by DLEE114.ent.ti.com (157.170.170.25) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.16; Wed, 29 Mar 2023 07:52:47 -0500 Received: from lelv0327.itg.ti.com (10.180.67.183) by DLEE105.ent.ti.com (157.170.170.35) 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; Wed, 29 Mar 2023 07:52:47 -0500 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 32TCqlWs007089; Wed, 29 Mar 2023 07:52:47 -0500 Date: Wed, 29 Mar 2023 07:52:47 -0500 From: Nishanth Menon To: Hari Nagalla CC: , , , , , , Subject: Re: [PATCH 1/3] arm64: dts: ti: k3-j784s4-main: Add MAIN domain R5F cluster nodes Message-ID: <20230329125247.w45k5fjzmcgjdyso@delicate> References: <20230329093627.30719-1-hnagalla@ti.com> <20230329093627.30719-2-hnagalla@ti.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20230329093627.30719-2-hnagalla@ti.com> 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-20230329_055251_347296_07F3CCBC X-CRM114-Status: GOOD ( 27.75 ) 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 04:36-20230329, Hari Nagalla wrote: > The J784S4 SoCs have 4 dual-core Arm Cortex-R5F processor (R5FSS) > subsystems/clusters. One R5F cluster (MCU_R5FSS0) is present within > the MCU domain, and the remaining three clusters are present in the > MAIN domain (MAIN_R5FSS0, MAIN_R5FSS1 & MAIN_R5FSS2). The functionality > of the R5FSS is same as the R5FSS functionality on earlier K3 platform > device J721S2. Each of the R5FSS can be configured at boot time to be > either run in a LockStep mode or in an Asymmetric Multi Processing (AMP) > fashion in Split-mode. These subsystems have 64 KB each Tightly-Coupled > Memory (TCM) internal memories for each core split between two banks - > ATCM and BTCM (further interleaved into two banks). There are some IP > integration differences from standard Arm R5 clusters such as the absence > of an ACP port, presence of an additional TI-specific Region Address > Translater (RAT) module for translating 32-bit CPU addresses into > larger system bus addresses etc. > > Add the DT nodes for these three MAIN domain R5F cluster/subsystems, > the two R5F cores are each added as child nodes to the corresponding > main cluster node. The clusters are configured to run in LockStep > mode by default, with the ATCMs enabled to allow the R5 cores to execute > code from DDR with boot-strapping code from ATCM. The inter-processor > communication between the main A72 cores and these processors is > achieved through shared memory and Mailboxes. > > The following firmware names are used by default for these cores, and > can be overridden in a board dts file if needed: > MAIN R5FSS0 Core0: j784s4-main-r5f0_0-fw (both in LockStep and Split modes) > MAIN R5FSS0 Core1: j784s4-main-r5f0_1-fw (needed only in Split mode) > MAIN R5FSS1 Core0: j784s4-main-r5f1_0-fw (both in LockStep and Split modes) > MAIN R5FSS1 Core1: j784s4-main-r5f1_1-fw (needed only in Split mode) > MAIN R5FSS2 Core0: j784s4-main-r5f2_0-fw (both in LockStep and Split modes) > MAIN R5FSS2 Core1: j784s4-main-r5f2_1-fw (needed only in Split mode) Why are the patches split up into main and mcu - if you are adding r5f cores, do them as a single patch. > > Signed-off-by: Hari Nagalla > --- > arch/arm64/boot/dts/ti/k3-j784s4-main.dtsi | 128 +++++++++++++++++++++ > 1 file changed, 128 insertions(+) > > diff --git a/arch/arm64/boot/dts/ti/k3-j784s4-main.dtsi b/arch/arm64/boot/dts/ti/k3-j784s4-main.dtsi > index 8c1474a7bd0f..53d337ea35fb 100644 > --- a/arch/arm64/boot/dts/ti/k3-j784s4-main.dtsi > +++ b/arch/arm64/boot/dts/ti/k3-j784s4-main.dtsi > @@ -1024,4 +1024,132 @@ > bosch,mram-cfg = <0x00 128 64 64 64 64 32 32>; > status = "disabled"; > }; > + > + main_r5fss0: r5fss@5c00000 { > + compatible = "ti,j721s2-r5fss"; > + ti,cluster-mode = <1>; > + #address-cells = <1>; > + #size-cells = <1>; > + ranges = <0x5c00000 0x00 0x5c00000 0x20000>, > + <0x5d00000 0x00 0x5d00000 0x20000>; > + power-domains = <&k3_pds 336 TI_SCI_PD_EXCLUSIVE>; > + > + main_r5fss0_core0: r5f@5c00000 { > + compatible = "ti,j721s2-r5f"; > + reg = <0x5c00000 0x00010000>, > + <0x5c10000 0x00010000>; > + reg-names = "atcm", "btcm"; > + ti,sci = <&sms>; > + ti,sci-dev-id = <339>; > + ti,sci-proc-ids = <0x06 0xff>; > + resets = <&k3_reset 339 1>; > + firmware-name = "j784s4-main-r5f0_0-fw"; > + ti,atcm-enable = <1>; > + ti,btcm-enable = <1>; > + ti,loczrama = <1>; > + status = "disabled"; Why are these disabled by default? > + }; > + > + main_r5fss0_core1: r5f@5d00000 { > + compatible = "ti,j721s2-r5f"; > + reg = <0x5d00000 0x00010000>, > + <0x5d10000 0x00010000>; > + reg-names = "atcm", "btcm"; > + ti,sci = <&sms>; > + ti,sci-dev-id = <340>; > + ti,sci-proc-ids = <0x07 0xff>; > + resets = <&k3_reset 340 1>; > + firmware-name = "j784s4-main-r5f0_1-fw"; > + ti,atcm-enable = <1>; > + ti,btcm-enable = <1>; > + ti,loczrama = <1>; > + status = "disabled"; > + }; > + > + }; > + > + main_r5fss1: r5fss@5e00000 { > + compatible = "ti,j721s2-r5fss"; > + ti,cluster-mode = <1>; > + #address-cells = <1>; > + #size-cells = <1>; > + ranges = <0x5e00000 0x00 0x5e00000 0x20000>, > + <0x5f00000 0x00 0x5f00000 0x20000>; > + power-domains = <&k3_pds 337 TI_SCI_PD_EXCLUSIVE>; > + > + main_r5fss1_core0: r5f@5e00000 { > + compatible = "ti,j721s2-r5f"; > + reg = <0x5e00000 0x00010000>, > + <0x5e10000 0x00010000>; > + reg-names = "atcm", "btcm"; > + ti,sci = <&sms>; > + ti,sci-dev-id = <341>; > + ti,sci-proc-ids = <0x08 0xff>; > + resets = <&k3_reset 341 1>; > + firmware-name = "j784s4-main-r5f1_0-fw"; > + ti,atcm-enable = <1>; > + ti,btcm-enable = <1>; > + ti,loczrama = <1>; > + status = "disabled"; > + }; > + > + main_r5fss1_core1: r5f@5f00000 { > + compatible = "ti,j721s2-r5f"; > + reg = <0x5f00000 0x00010000>, > + <0x5f10000 0x00010000>; > + reg-names = "atcm", "btcm"; > + ti,sci = <&sms>; > + ti,sci-dev-id = <342>; > + ti,sci-proc-ids = <0x09 0xff>; > + resets = <&k3_reset 342 1>; > + firmware-name = "j784s4-main-r5f1_1-fw"; > + ti,atcm-enable = <1>; > + ti,btcm-enable = <1>; > + ti,loczrama = <1>; > + status = "disabled"; > + }; > + }; > + > + main_r5fss2: r5fss@5900000 { > + compatible = "ti,j721s2-r5fss"; > + ti,cluster-mode = <1>; > + #address-cells = <1>; > + #size-cells = <1>; > + ranges = <0x5900000 0x00 0x5900000 0x20000>, > + <0x5a00000 0x00 0x5a00000 0x20000>; > + power-domains = <&k3_pds 338 TI_SCI_PD_EXCLUSIVE>; > + > + main_r5fss2_core0: r5f@5900000 { > + compatible = "ti,j721s2-r5f"; > + reg = <0x5900000 0x00010000>, > + <0x5910000 0x00010000>; > + reg-names = "atcm", "btcm"; > + ti,sci = <&sms>; > + ti,sci-dev-id = <343>; > + ti,sci-proc-ids = <0x0a 0xff>; > + resets = <&k3_reset 343 1>; > + firmware-name = "j784s4-main-r5f2_0-fw"; > + ti,atcm-enable = <1>; > + ti,btcm-enable = <1>; > + ti,loczrama = <1>; > + status = "disabled"; > + }; > + > + main_r5fss2_core1: r5f@5a00000 { > + compatible = "ti,j721s2-r5f"; > + reg = <0x5a00000 0x00010000>, > + <0x5a10000 0x00010000>; > + reg-names = "atcm", "btcm"; > + ti,sci = <&sms>; > + ti,sci-dev-id = <344>; > + ti,sci-proc-ids = <0x0b 0xff>; > + resets = <&k3_reset 344 1>; > + firmware-name = "j784s4-main-r5f2_1-fw"; > + ti,atcm-enable = <1>; > + ti,btcm-enable = <1>; > + ti,loczrama = <1>; > + status = "disabled"; > + > + }; > + }; > }; > -- > 2.17.1 > -- 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