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 15F14C5475B for ; Fri, 8 Mar 2024 14:26:22 +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=qRAZRmTupqRC6tHHZhYDaVOiYycOuu3zk29vyaJK68g=; b=K66bpo3rNqIKws Le3MY0PGoIN3RRvcCToEDuwQA8KScqqXD/kNv3ue+OqizlRfogsjdvk4V44A13VoiiRYB2kXB8szY MS+7T8Znw7OQAp3wtQlfSPJiL1aCqkrgI0rjsHtpNA5Xw8yXvECC1qM4VPyFPEIwOsPvSwwLzKJ92 MgHxw5Cf7rKPQeiqRsPVQdtcCRsmV6ev3dCRuW/yWeksHsxAorg9T+FAm/mkESCUcW5WsmWq/yUOi n8sTcCOdoaJBbDyIXbEHRwIsRaDAJLz0XlcWuOTB1wEfqyO2ThPOQAJzuNB4v9G6ImxVtI/sYeDQv YnkKBeM4scxuxvYk2x0g==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1ribAc-00000009Yq0-2Osj; Fri, 08 Mar 2024 14:25:58 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1ribAa-00000009Yos-0POU for linux-arm-kernel@lists.infradead.org; Fri, 08 Mar 2024 14:25:57 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id B8EDFC15; Fri, 8 Mar 2024 06:26:28 -0800 (PST) Received: from e130802.arm.com (unknown [10.57.14.181]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id E68CD3F762; Fri, 8 Mar 2024 06:25:48 -0800 (PST) Date: Fri, 8 Mar 2024 14:25:45 +0000 From: Abdellatif El Khlifi To: Sudeep Holla Cc: Bjorn Andersson , Mathieu Poirier , Rob Herring , Liviu Dudau , Lorenzo Pieralisi , Krzysztof Kozlowski , Conor Dooley , Drew.Reed@arm.com, Adam.Johnston@arm.com, linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-remoteproc@vger.kernel.org Subject: Re: [PATCH 2/3] arm64: dts: Add corstone1000 external system device node Message-ID: <20240308142545.GA199646@e130802.arm.com> References: <20240301164227.339208-1-abdellatif.elkhlifi@arm.com> <20240301164227.339208-3-abdellatif.elkhlifi@arm.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240308_062556_361354_8DE1F2EF X-CRM114-Status: GOOD ( 11.70 ) 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 Hi Sudeep, > > + extsys0: remoteproc@1a010310 { > > + compatible = "arm,corstone1000-extsys"; > > + reg = <0x1a010310 0x4>, > > + <0x1a010314 0X4>; > > > As per [1], this is just a few registers within the 64kB block. > Not sure if it should be represented as a whole on just couple > of registers like this for reset. > > [1] https://developer.arm.com/documentation/101418/0100/Programmers-model/Register-descriptions/Host-Base-System-Control-register-summary The Host Base System Control registers are not specific to the External System processors. They are various registers with different purposes. Only 4 registers matter for the remoteproc feature: - The External system 0 reset control and status registers: EXT_SYS0_RST_CTRL, EXT_SYS0_RST_ST - Same for the the External system 1: EXT_SYS1_RST_CTRL, EXT_SYS1_RST_ST So, mapping the whole Host Base System Control area doesn't make sense for the remoteproc feature and exposes registers that are not related to the External Systems to the driver. By the way, the latest document we are referring to is [1]. [1]: https://developer.arm.com/documentation/102342/0000/Programmers-model/Register-descriptions/Host-Base-System-Control-register-summary Cheers, Abdellatif _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel