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 C8AB4C4167D for ; Tue, 31 Oct 2023 19:01:38 +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:From:References:Cc:To: Subject:MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=o7dU7gGfcnYQPOAKjjUopQVpjr/Z40tZi74SpnDHIhE=; b=O27Ar/vaf5n7jX XHluQb4QtDe3FHWDyr1BD5L0wKxjRfPOAQMCRIjx8CYioi2MCQymTgMFLW9j4cGULYobxKq42/U5D krVtFPXXo/vTlorWSwKFRnUlE9UETOD/kpIvPsOlJYHUk0NfkSxuqcgggoNxz+JK++cuHDOAHkYcd /VRy39k7SAxoMNa6Pswvw6VS5uzI/UQLB1xTfe60/3MLCf+pybCLOwoKJJKZeR0C33+nf0ebZjKgr FGCE2ZldhCc4CmiE2npXASveTepqIuIPB8U1vZ3NqWqc8AwyaC8RCe+97xzfzS4USess3/sjMZ05t sX0udytjP8RudEX4RS3w==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qxtzU-005zit-0p; Tue, 31 Oct 2023 19:01:28 +0000 Received: from madras.collabora.co.uk ([46.235.227.172]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qxtzQ-005zgH-00; Tue, 31 Oct 2023 19:01:25 +0000 Received: from [100.116.17.117] (cola.collaboradmins.com [195.201.22.229]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: cristicc) by madras.collabora.co.uk (Postfix) with ESMTPSA id 49CD3660739C; Tue, 31 Oct 2023 19:01:17 +0000 (GMT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1698778879; bh=BsAPLmxB3DURCesPkkALL1Ia9odxLqJSAjKdGFe9DFk=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=WCZxeK3sLNVm9MIkijRBlhmkwy+YNEGi5hmdI3xqpapX61FnUrjRkL+OVx3VFmVRS drh0mK9vLQP+hiWGTMupWxpUK3dHW3AsVKlU0EwqPDsnW3mLKoKyOONwGZMpq8HWJz URc2exjGqu2/Y5ywgAZ3362kvT+5yZn7GlgWUx6EerBjLP2j+CTXlTQpy+Gph1EqW8 xdk7eBBSBJaQlo1MDw8Ft/ZYBEpGJl4NnoHL3cFKCgnapNzn0knWhG6LPvdOaE5KHV vOBY59kZBj9096AbhYoDMrII3me6aa9boP5HBZtK6eIpo8fHs1vTeb4aHcbNZpukU5 FKOXKuH1JbEVw== Message-ID: <4e71c2ff-6189-4a2d-8ec0-fb9fe4a9971f@collabora.com> Date: Tue, 31 Oct 2023 21:01:14 +0200 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v2 07/12] riscv: dts: starfive: jh7100: Add ccache DT node Content-Language: en-US To: Emil Renner Berthing , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Emil Renner Berthing , Samin Guo , Paul Walmsley , Palmer Dabbelt , Albert Ou , Alexandre Torgue , Jose Abreu , Maxime Coquelin , Richard Cochran , Giuseppe Cavallaro Cc: netdev@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-riscv@lists.infradead.org, linux-stm32@st-md-mailman.stormreply.com, linux-arm-kernel@lists.infradead.org, kernel@collabora.com References: <20231029042712.520010-1-cristian.ciocaltea@collabora.com> <20231029042712.520010-8-cristian.ciocaltea@collabora.com> From: Cristian Ciocaltea In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231031_120124_199632_DBC5D150 X-CRM114-Status: GOOD ( 15.78 ) X-BeenThere: linux-riscv@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-riscv" Errors-To: linux-riscv-bounces+linux-riscv=archiver.kernel.org@lists.infradead.org On 10/31/23 16:38, Emil Renner Berthing wrote: > Cristian Ciocaltea wrote: >> Provide a DT node for the SiFive Composable Cache controller found on >> the StarFive JH7100 SoC. >> >> Note this is also used to support non-coherent DMA, via the >> sifive,cache-ops cache flushing operations. > > This property is no longer needed: > https://lore.kernel.org/linux-riscv/20231031141444.53426-1-emil.renner.berthing@canonical.com/ Thanks for the heads up! I actually noticed that from v1 reviews and was just waiting for v2. :) > Also it would be nice to mention that these nodes are copied from my > visionfive patches ;) Ups, sorry about that! Those were initially taken from a patch adding a full DT (the repo is mentioned in the cover letter) with many contributors mentioned, without being clear who did what. That's why I didn't provide a Co-developed-by tag and, unfortunately, I also missed to add it in v2 (will handle this in v3 and also provide the link to the new repo), but I'm still not sure about the gmac stuff. Thanks, Cristian >> >> Signed-off-by: Cristian Ciocaltea >> --- >> arch/riscv/boot/dts/starfive/jh7100.dtsi | 14 ++++++++++++++ >> 1 file changed, 14 insertions(+) >> >> diff --git a/arch/riscv/boot/dts/starfive/jh7100.dtsi b/arch/riscv/boot/dts/starfive/jh7100.dtsi >> index 06bb157ce111..a8a5bb00b0d8 100644 >> --- a/arch/riscv/boot/dts/starfive/jh7100.dtsi >> +++ b/arch/riscv/boot/dts/starfive/jh7100.dtsi >> @@ -32,6 +32,7 @@ U74_0: cpu@0 { >> i-tlb-sets = <1>; >> i-tlb-size = <32>; >> mmu-type = "riscv,sv39"; >> + next-level-cache = <&ccache>; >> riscv,isa = "rv64imafdc"; >> riscv,isa-base = "rv64i"; >> riscv,isa-extensions = "i", "m", "a", "f", "d", "c", "zicntr", "zicsr", >> @@ -60,6 +61,7 @@ U74_1: cpu@1 { >> i-tlb-sets = <1>; >> i-tlb-size = <32>; >> mmu-type = "riscv,sv39"; >> + next-level-cache = <&ccache>; >> riscv,isa = "rv64imafdc"; >> riscv,isa-base = "rv64i"; >> riscv,isa-extensions = "i", "m", "a", "f", "d", "c", "zicntr", "zicsr", >> @@ -147,6 +149,18 @@ soc { >> dma-noncoherent; >> ranges; >> >> + ccache: cache-controller@2010000 { >> + compatible = "starfive,jh7100-ccache", "sifive,ccache0", "cache"; >> + reg = <0x0 0x2010000 0x0 0x1000>; >> + interrupts = <128>, <130>, <131>, <129>; >> + cache-block-size = <64>; >> + cache-level = <2>; >> + cache-sets = <2048>; >> + cache-size = <2097152>; >> + cache-unified; >> + sifive,cache-ops; >> + }; >> + >> clint: clint@2000000 { >> compatible = "starfive,jh7100-clint", "sifive,clint0"; >> reg = <0x0 0x2000000 0x0 0x10000>; >> -- >> 2.42.0 >> _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv