Message ID | 20221031091956.531935-1-pierre.gondois@arm.com |
---|---|
State | New |
Headers |
Return-Path: <linux-kernel-owner@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:a5d:6687:0:0:0:0:0 with SMTP id l7csp2203794wru; Mon, 31 Oct 2022 02:23:38 -0700 (PDT) X-Google-Smtp-Source: AMsMyM5CCIZ2ps+6qCWT97+pVI6nITSigmrwGztE1LOLzGLKwSFblDTmfjqbA0OIbXYTS6F7pNp3 X-Received: by 2002:aa7:cb59:0:b0:461:7378:7be0 with SMTP id w25-20020aa7cb59000000b0046173787be0mr12725955edt.60.1667208218528; Mon, 31 Oct 2022 02:23:38 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1667208218; cv=none; d=google.com; s=arc-20160816; b=wBm2Ap8tq0Kf2x1NVaJUVzdx3XBDkUfAASgzZMAsbI3dOaWWXBmz3+XpawYhKXInRn BY9686Klk6YtIm3qprzRonMrl7wywHJyWP1rU02AHcHWaJIs9hUe0K/YwZo+oNma+WzS tCeNEqao2kmuQyOd8boUZmHYvw7TehYGkU3uQwdUGfBpVV/DTkgS89byz7Kjy4nHjeYY 4q9nHow9jrqb/8chvMUcZ0FHIrQ0P1Jjod/YWz5sgMrO9lCv31uv6MJfsXqDWgp+KInC 8KIjIugNV4qjm5xixKAVnNOjby1buFPDh9setcgpnNighCaa/5UbyStE/Lbkb7LimWbA YKJA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from; bh=J5Gh/+TJbWcfiHg6s6TXC5P+U55OtwGVHvUGb54vcIw=; b=QSqUiAe6CPaJqRmO3uHevqYz0DketWm3ofNut7IQd0m/40wzxtnkXOe++iigYF9/kO Jl2WoG9LWrZJDX3q03vM0Vlb2feT3/8u3d29yyON+X3rsnTqKxMHY770IsgOM0vL+8e4 DqolcYDTNFDuOg2PFY7rKayxw0fEjpZ0CD+UN5F67c7dNvGguCWUTTpB2tQ/Yz8heEKQ Cj6qj6y+7ZbA+TFkSMwhap7isAyjuS6isoGOqH/8oWCPBq7k6tuPqkRhho5IpIkogpZu l4LDuKemRUCVY1FS1OvCKVCt0xvBaOjEXSViBUtvsVIymOn3U/hszQ3+VEBbCf03abTu vY7Q== ARC-Authentication-Results: i=1; mx.google.com; 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=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id ce12-20020a170906b24c00b007ab34c9d9d3si5842258ejb.733.2022.10.31.02.23.15; Mon, 31 Oct 2022 02:23:38 -0700 (PDT) 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; 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=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230175AbiJaJU2 (ORCPT <rfc822;kartikey406@gmail.com> + 99 others); Mon, 31 Oct 2022 05:20:28 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:50804 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230150AbiJaJUL (ORCPT <rfc822;linux-kernel@vger.kernel.org>); Mon, 31 Oct 2022 05:20:11 -0400 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id 88315DF31; Mon, 31 Oct 2022 02:20:00 -0700 (PDT) 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 8DD451FB; Mon, 31 Oct 2022 02:20:06 -0700 (PDT) Received: from pierre123.arm.com (unknown [10.57.7.107]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id 44AFA3F703; Mon, 31 Oct 2022 02:19:56 -0700 (PDT) From: Pierre Gondois <pierre.gondois@arm.com> To: linux-kernel@vger.kernel.org Cc: pierre.gondois@arm.com, Rob.Herring@arm.com, Shawn Guo <shawnguo@kernel.org>, Li Yang <leoyang.li@nxp.com>, Rob Herring <robh+dt@kernel.org>, Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>, Sascha Hauer <s.hauer@pengutronix.de>, Pengutronix Kernel Team <kernel@pengutronix.de>, Fabio Estevam <festevam@gmail.com>, NXP Linux Team <linux-imx@nxp.com>, Chester Lin <clin@suse.com>, =?utf-8?q?Andreas_F=C3=A4rber?= <afaerber@suse.de>, Matthias Brugger <mbrugger@suse.com>, NXP S32 Linux Team <s32@nxp.com>, Peng Fan <peng.fan@nxp.com>, Jacky Bai <ping.bai@nxp.com>, Sudeep Holla <sudeep.holla@arm.com>, linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org Subject: [PATCH 07/20] arm64: dts: Update cache properties for freescale Date: Mon, 31 Oct 2022 10:19:51 +0100 Message-Id: <20221031091956.531935-1-pierre.gondois@arm.com> X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-4.2 required=5.0 tests=BAYES_00,RCVD_IN_DNSWL_MED, SPF_HELO_NONE,SPF_NONE 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: <linux-kernel.vger.kernel.org> X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1748194524557769136?= X-GMAIL-MSGID: =?utf-8?q?1748194524557769136?= |
Series |
Update cache properties for arm64 DTS
|
|
Commit Message
Pierre Gondois
Oct. 31, 2022, 9:19 a.m. UTC
The DeviceTree Specification v0.3 specifies that the cache node
'compatible' and 'cache-level' properties are 'required'. Cf.
s3.8 Multi-level and Shared Cache Nodes
The recently added init_of_cache_level() function checks
these properties. Add them if missing.
Signed-off-by: Pierre Gondois <pierre.gondois@arm.com>
---
arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi | 1 +
arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi | 1 +
arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi | 1 +
arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi | 4 ++++
arch/arm64/boot/dts/freescale/fsl-ls2088a.dtsi | 4 ++++
arch/arm64/boot/dts/freescale/imx8ulp.dtsi | 1 +
arch/arm64/boot/dts/freescale/s32g2.dtsi | 2 ++
arch/arm64/boot/dts/freescale/s32v234.dtsi | 2 ++
8 files changed, 16 insertions(+)
Comments
On Mon, Oct 31, 2022 at 10:19:51AM +0100, Pierre Gondois wrote: > The DeviceTree Specification v0.3 specifies that the cache node > 'compatible' and 'cache-level' properties are 'required'. Cf. > s3.8 Multi-level and Shared Cache Nodes > > The recently added init_of_cache_level() function checks > these properties. Add them if missing. > > Signed-off-by: Pierre Gondois <pierre.gondois@arm.com> Looks good to me. Should I pick it up on IMX tree, or it needs to go as part of the series (via other tree)? Shawn > --- > arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi | 1 + > arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi | 1 + > arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi | 1 + > arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi | 4 ++++ > arch/arm64/boot/dts/freescale/fsl-ls2088a.dtsi | 4 ++++ > arch/arm64/boot/dts/freescale/imx8ulp.dtsi | 1 + > arch/arm64/boot/dts/freescale/s32g2.dtsi | 2 ++ > arch/arm64/boot/dts/freescale/s32v234.dtsi | 2 ++ > 8 files changed, 16 insertions(+) > > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi > index 5627dd7734f3..ed0cc1a5d17e 100644 > --- a/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi > +++ b/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi > @@ -46,6 +46,7 @@ cpu1: cpu@1 { > > l2: l2-cache { > compatible = "cache"; > + cache-level = <2>; > }; > }; > > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi > index ca3d5a90d6d4..c8b1202d2584 100644 > --- a/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi > +++ b/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi > @@ -83,6 +83,7 @@ cpu3: cpu@3 { > > l2: l2-cache { > compatible = "cache"; > + cache-level = <2>; > }; > }; > > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi > index feab604322cf..4590bdc076b7 100644 > --- a/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi > +++ b/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi > @@ -78,6 +78,7 @@ cpu3: cpu@3 { > > l2: l2-cache { > compatible = "cache"; > + cache-level = <2>; > }; > }; > > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi > index 6f6667b70028..2a7e13b6ef8a 100644 > --- a/arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi > +++ b/arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi > @@ -95,18 +95,22 @@ cpu7: cpu@301 { > > cluster0_l2: l2-cache0 { > compatible = "cache"; > + cache-level = <2>; > }; > > cluster1_l2: l2-cache1 { > compatible = "cache"; > + cache-level = <2>; > }; > > cluster2_l2: l2-cache2 { > compatible = "cache"; > + cache-level = <2>; > }; > > cluster3_l2: l2-cache3 { > compatible = "cache"; > + cache-level = <2>; > }; > > CPU_PW20: cpu-pw20 { > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls2088a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls2088a.dtsi > index c3dc38188c17..c12c86915ec8 100644 > --- a/arch/arm64/boot/dts/freescale/fsl-ls2088a.dtsi > +++ b/arch/arm64/boot/dts/freescale/fsl-ls2088a.dtsi > @@ -95,18 +95,22 @@ cpu7: cpu@301 { > > cluster0_l2: l2-cache0 { > compatible = "cache"; > + cache-level = <2>; > }; > > cluster1_l2: l2-cache1 { > compatible = "cache"; > + cache-level = <2>; > }; > > cluster2_l2: l2-cache2 { > compatible = "cache"; > + cache-level = <2>; > }; > > cluster3_l2: l2-cache3 { > compatible = "cache"; > + cache-level = <2>; > }; > > CPU_PW20: cpu-pw20 { > diff --git a/arch/arm64/boot/dts/freescale/imx8ulp.dtsi b/arch/arm64/boot/dts/freescale/imx8ulp.dtsi > index 60c1b018bf03..187353458673 100644 > --- a/arch/arm64/boot/dts/freescale/imx8ulp.dtsi > +++ b/arch/arm64/boot/dts/freescale/imx8ulp.dtsi > @@ -50,6 +50,7 @@ A35_1: cpu@1 { > > A35_L2: l2-cache0 { > compatible = "cache"; > + cache-level = <2>; > }; > }; > > diff --git a/arch/arm64/boot/dts/freescale/s32g2.dtsi b/arch/arm64/boot/dts/freescale/s32g2.dtsi > index 824d401e7a2c..d8c82da88ca0 100644 > --- a/arch/arm64/boot/dts/freescale/s32g2.dtsi > +++ b/arch/arm64/boot/dts/freescale/s32g2.dtsi > @@ -52,10 +52,12 @@ cpu3: cpu@101 { > > cluster0_l2: l2-cache0 { > compatible = "cache"; > + cache-level = <2>; > }; > > cluster1_l2: l2-cache1 { > compatible = "cache"; > + cache-level = <2>; > }; > }; > > diff --git a/arch/arm64/boot/dts/freescale/s32v234.dtsi b/arch/arm64/boot/dts/freescale/s32v234.dtsi > index ba0b5305d481..3e306218d533 100644 > --- a/arch/arm64/boot/dts/freescale/s32v234.dtsi > +++ b/arch/arm64/boot/dts/freescale/s32v234.dtsi > @@ -61,10 +61,12 @@ cpu3: cpu@101 { > > cluster0_l2_cache: l2-cache0 { > compatible = "cache"; > + cache-level = <2>; > }; > > cluster1_l2_cache: l2-cache1 { > compatible = "cache"; > + cache-level = <2>; > }; > }; > > -- > 2.25.1 >
Hi Shawn, On Fri, Nov 11, 2022 at 10:55:53AM +0800, Shawn Guo wrote: > On Mon, Oct 31, 2022 at 10:19:51AM +0100, Pierre Gondois wrote: > > The DeviceTree Specification v0.3 specifies that the cache node > > 'compatible' and 'cache-level' properties are 'required'. Cf. > > s3.8 Multi-level and Shared Cache Nodes > > > > The recently added init_of_cache_level() function checks > > these properties. Add them if missing. > > > > Signed-off-by: Pierre Gondois <pierre.gondois@arm.com> > > Looks good to me. Should I pick it up on IMX tree, or it needs to go as > part of the series (via other tree)? > > Shawn > The change for s32g2 looks good to me. I would be grateful if you could pick this patch to the IMX tree. Thanks! Reviewed-by: Chester Lin <clin@suse.com> > > --- > > arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi | 1 + > > arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi | 1 + > > arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi | 1 + > > arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi | 4 ++++ > > arch/arm64/boot/dts/freescale/fsl-ls2088a.dtsi | 4 ++++ > > arch/arm64/boot/dts/freescale/imx8ulp.dtsi | 1 + > > arch/arm64/boot/dts/freescale/s32g2.dtsi | 2 ++ > > arch/arm64/boot/dts/freescale/s32v234.dtsi | 2 ++ > > 8 files changed, 16 insertions(+) > > > > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi > > index 5627dd7734f3..ed0cc1a5d17e 100644 > > --- a/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi > > +++ b/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi > > @@ -46,6 +46,7 @@ cpu1: cpu@1 { > > > > l2: l2-cache { > > compatible = "cache"; > > + cache-level = <2>; > > }; > > }; > > > > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi > > index ca3d5a90d6d4..c8b1202d2584 100644 > > --- a/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi > > +++ b/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi > > @@ -83,6 +83,7 @@ cpu3: cpu@3 { > > > > l2: l2-cache { > > compatible = "cache"; > > + cache-level = <2>; > > }; > > }; > > > > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi > > index feab604322cf..4590bdc076b7 100644 > > --- a/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi > > +++ b/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi > > @@ -78,6 +78,7 @@ cpu3: cpu@3 { > > > > l2: l2-cache { > > compatible = "cache"; > > + cache-level = <2>; > > }; > > }; > > > > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi > > index 6f6667b70028..2a7e13b6ef8a 100644 > > --- a/arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi > > +++ b/arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi > > @@ -95,18 +95,22 @@ cpu7: cpu@301 { > > > > cluster0_l2: l2-cache0 { > > compatible = "cache"; > > + cache-level = <2>; > > }; > > > > cluster1_l2: l2-cache1 { > > compatible = "cache"; > > + cache-level = <2>; > > }; > > > > cluster2_l2: l2-cache2 { > > compatible = "cache"; > > + cache-level = <2>; > > }; > > > > cluster3_l2: l2-cache3 { > > compatible = "cache"; > > + cache-level = <2>; > > }; > > > > CPU_PW20: cpu-pw20 { > > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls2088a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls2088a.dtsi > > index c3dc38188c17..c12c86915ec8 100644 > > --- a/arch/arm64/boot/dts/freescale/fsl-ls2088a.dtsi > > +++ b/arch/arm64/boot/dts/freescale/fsl-ls2088a.dtsi > > @@ -95,18 +95,22 @@ cpu7: cpu@301 { > > > > cluster0_l2: l2-cache0 { > > compatible = "cache"; > > + cache-level = <2>; > > }; > > > > cluster1_l2: l2-cache1 { > > compatible = "cache"; > > + cache-level = <2>; > > }; > > > > cluster2_l2: l2-cache2 { > > compatible = "cache"; > > + cache-level = <2>; > > }; > > > > cluster3_l2: l2-cache3 { > > compatible = "cache"; > > + cache-level = <2>; > > }; > > > > CPU_PW20: cpu-pw20 { > > diff --git a/arch/arm64/boot/dts/freescale/imx8ulp.dtsi b/arch/arm64/boot/dts/freescale/imx8ulp.dtsi > > index 60c1b018bf03..187353458673 100644 > > --- a/arch/arm64/boot/dts/freescale/imx8ulp.dtsi > > +++ b/arch/arm64/boot/dts/freescale/imx8ulp.dtsi > > @@ -50,6 +50,7 @@ A35_1: cpu@1 { > > > > A35_L2: l2-cache0 { > > compatible = "cache"; > > + cache-level = <2>; > > }; > > }; > > > > diff --git a/arch/arm64/boot/dts/freescale/s32g2.dtsi b/arch/arm64/boot/dts/freescale/s32g2.dtsi > > index 824d401e7a2c..d8c82da88ca0 100644 > > --- a/arch/arm64/boot/dts/freescale/s32g2.dtsi > > +++ b/arch/arm64/boot/dts/freescale/s32g2.dtsi > > @@ -52,10 +52,12 @@ cpu3: cpu@101 { > > > > cluster0_l2: l2-cache0 { > > compatible = "cache"; > > + cache-level = <2>; > > }; > > > > cluster1_l2: l2-cache1 { > > compatible = "cache"; > > + cache-level = <2>; > > }; > > }; > > > > diff --git a/arch/arm64/boot/dts/freescale/s32v234.dtsi b/arch/arm64/boot/dts/freescale/s32v234.dtsi > > index ba0b5305d481..3e306218d533 100644 > > --- a/arch/arm64/boot/dts/freescale/s32v234.dtsi > > +++ b/arch/arm64/boot/dts/freescale/s32v234.dtsi > > @@ -61,10 +61,12 @@ cpu3: cpu@101 { > > > > cluster0_l2_cache: l2-cache0 { > > compatible = "cache"; > > + cache-level = <2>; > > }; > > > > cluster1_l2_cache: l2-cache1 { > > compatible = "cache"; > > + cache-level = <2>; > > }; > > }; > > > > -- > > 2.25.1 > >
On Fri, Nov 11, 2022 at 12:49:08PM +0800, Chester Lin wrote: > Hi Shawn, > > On Fri, Nov 11, 2022 at 10:55:53AM +0800, Shawn Guo wrote: > > On Mon, Oct 31, 2022 at 10:19:51AM +0100, Pierre Gondois wrote: > > > The DeviceTree Specification v0.3 specifies that the cache node > > > 'compatible' and 'cache-level' properties are 'required'. Cf. > > > s3.8 Multi-level and Shared Cache Nodes > > > > > > The recently added init_of_cache_level() function checks > > > these properties. Add them if missing. > > > > > > Signed-off-by: Pierre Gondois <pierre.gondois@arm.com> > > > > Looks good to me. Should I pick it up on IMX tree, or it needs to go as > > part of the series (via other tree)? > > > > Shawn > > > > The change for s32g2 looks good to me. I would be grateful if you could pick > this patch to the IMX tree. Thanks! > > Reviewed-by: Chester Lin <clin@suse.com> Ok, I will wait a confirmation from Pierre that the intention is indeed for platform maintainer to pick up individual patch. Shawn
On 11/11/22 05:57, Shawn Guo wrote: > On Fri, Nov 11, 2022 at 12:49:08PM +0800, Chester Lin wrote: >> Hi Shawn, >> >> On Fri, Nov 11, 2022 at 10:55:53AM +0800, Shawn Guo wrote: >>> On Mon, Oct 31, 2022 at 10:19:51AM +0100, Pierre Gondois wrote: >>>> The DeviceTree Specification v0.3 specifies that the cache node >>>> 'compatible' and 'cache-level' properties are 'required'. Cf. >>>> s3.8 Multi-level and Shared Cache Nodes >>>> >>>> The recently added init_of_cache_level() function checks >>>> these properties. Add them if missing. >>>> >>>> Signed-off-by: Pierre Gondois <pierre.gondois@arm.com> >>> >>> Looks good to me. Should I pick it up on IMX tree, or it needs to go as >>> part of the series (via other tree)? >>> >>> Shawn >>> >> >> The change for s32g2 looks good to me. I would be grateful if you could pick >> this patch to the IMX tree. Thanks! >> >> Reviewed-by: Chester Lin <clin@suse.com> > > Ok, I will wait a confirmation from Pierre that the intention is indeed > for platform maintainer to pick up individual patch. > > Shawn Hello Shawn, Chester, Yes please pick the patch individually, Regards, Pierre
> -----Original Message----- > From: Pierre Gondois <pierre.gondois@arm.com> > Sent: Monday, October 31, 2022 4:20 AM > To: linux-kernel@vger.kernel.org > Cc: pierre.gondois@arm.com; Rob.Herring@arm.com; Shawn Guo > <shawnguo@kernel.org>; Leo Li <leoyang.li@nxp.com>; Rob Herring > <robh+dt@kernel.org>; Krzysztof Kozlowski > <krzysztof.kozlowski+dt@linaro.org>; Sascha Hauer > <s.hauer@pengutronix.de>; Pengutronix Kernel Team > <kernel@pengutronix.de>; Fabio Estevam <festevam@gmail.com>; dl-linux- > imx <linux-imx@nxp.com>; Chester Lin <clin@suse.com>; Andreas Färber > <afaerber@suse.de>; Matthias Brugger <mbrugger@suse.com>; dl-S32 > <S32@nxp.com>; Peng Fan <peng.fan@nxp.com>; Jacky Bai > <ping.bai@nxp.com>; Sudeep Holla <sudeep.holla@arm.com>; linux-arm- > kernel@lists.infradead.org; devicetree@vger.kernel.org > Subject: [PATCH 07/20] arm64: dts: Update cache properties for freescale > > The DeviceTree Specification v0.3 specifies that the cache node 'compatible' > and 'cache-level' properties are 'required'. Cf. > s3.8 Multi-level and Shared Cache Nodes > > The recently added init_of_cache_level() function checks these properties. > Add them if missing. > > Signed-off-by: Pierre Gondois <pierre.gondois@arm.com> Acked-by: Li Yang <leoyang.li@nxp.com> > --- > arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi | 1 + > arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi | 1 + > arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi | 1 + > arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi | 4 ++++ > arch/arm64/boot/dts/freescale/fsl-ls2088a.dtsi | 4 ++++ > arch/arm64/boot/dts/freescale/imx8ulp.dtsi | 1 + > arch/arm64/boot/dts/freescale/s32g2.dtsi | 2 ++ > arch/arm64/boot/dts/freescale/s32v234.dtsi | 2 ++ > 8 files changed, 16 insertions(+) > > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi > b/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi > index 5627dd7734f3..ed0cc1a5d17e 100644 > --- a/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi > +++ b/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi > @@ -46,6 +46,7 @@ cpu1: cpu@1 { > > l2: l2-cache { > compatible = "cache"; > + cache-level = <2>; > }; > }; > > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi > b/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi > index ca3d5a90d6d4..c8b1202d2584 100644 > --- a/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi > +++ b/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi > @@ -83,6 +83,7 @@ cpu3: cpu@3 { > > l2: l2-cache { > compatible = "cache"; > + cache-level = <2>; > }; > }; > > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi > b/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi > index feab604322cf..4590bdc076b7 100644 > --- a/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi > +++ b/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi > @@ -78,6 +78,7 @@ cpu3: cpu@3 { > > l2: l2-cache { > compatible = "cache"; > + cache-level = <2>; > }; > }; > > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi > b/arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi > index 6f6667b70028..2a7e13b6ef8a 100644 > --- a/arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi > +++ b/arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi > @@ -95,18 +95,22 @@ cpu7: cpu@301 { > > cluster0_l2: l2-cache0 { > compatible = "cache"; > + cache-level = <2>; > }; > > cluster1_l2: l2-cache1 { > compatible = "cache"; > + cache-level = <2>; > }; > > cluster2_l2: l2-cache2 { > compatible = "cache"; > + cache-level = <2>; > }; > > cluster3_l2: l2-cache3 { > compatible = "cache"; > + cache-level = <2>; > }; > > CPU_PW20: cpu-pw20 { > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls2088a.dtsi > b/arch/arm64/boot/dts/freescale/fsl-ls2088a.dtsi > index c3dc38188c17..c12c86915ec8 100644 > --- a/arch/arm64/boot/dts/freescale/fsl-ls2088a.dtsi > +++ b/arch/arm64/boot/dts/freescale/fsl-ls2088a.dtsi > @@ -95,18 +95,22 @@ cpu7: cpu@301 { > > cluster0_l2: l2-cache0 { > compatible = "cache"; > + cache-level = <2>; > }; > > cluster1_l2: l2-cache1 { > compatible = "cache"; > + cache-level = <2>; > }; > > cluster2_l2: l2-cache2 { > compatible = "cache"; > + cache-level = <2>; > }; > > cluster3_l2: l2-cache3 { > compatible = "cache"; > + cache-level = <2>; > }; > > CPU_PW20: cpu-pw20 { > diff --git a/arch/arm64/boot/dts/freescale/imx8ulp.dtsi > b/arch/arm64/boot/dts/freescale/imx8ulp.dtsi > index 60c1b018bf03..187353458673 100644 > --- a/arch/arm64/boot/dts/freescale/imx8ulp.dtsi > +++ b/arch/arm64/boot/dts/freescale/imx8ulp.dtsi > @@ -50,6 +50,7 @@ A35_1: cpu@1 { > > A35_L2: l2-cache0 { > compatible = "cache"; > + cache-level = <2>; > }; > }; > > diff --git a/arch/arm64/boot/dts/freescale/s32g2.dtsi > b/arch/arm64/boot/dts/freescale/s32g2.dtsi > index 824d401e7a2c..d8c82da88ca0 100644 > --- a/arch/arm64/boot/dts/freescale/s32g2.dtsi > +++ b/arch/arm64/boot/dts/freescale/s32g2.dtsi > @@ -52,10 +52,12 @@ cpu3: cpu@101 { > > cluster0_l2: l2-cache0 { > compatible = "cache"; > + cache-level = <2>; > }; > > cluster1_l2: l2-cache1 { > compatible = "cache"; > + cache-level = <2>; > }; > }; > > diff --git a/arch/arm64/boot/dts/freescale/s32v234.dtsi > b/arch/arm64/boot/dts/freescale/s32v234.dtsi > index ba0b5305d481..3e306218d533 100644 > --- a/arch/arm64/boot/dts/freescale/s32v234.dtsi > +++ b/arch/arm64/boot/dts/freescale/s32v234.dtsi > @@ -61,10 +61,12 @@ cpu3: cpu@101 { > > cluster0_l2_cache: l2-cache0 { > compatible = "cache"; > + cache-level = <2>; > }; > > cluster1_l2_cache: l2-cache1 { > compatible = "cache"; > + cache-level = <2>; > }; > }; > > -- > 2.25.1
diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi index 5627dd7734f3..ed0cc1a5d17e 100644 --- a/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi +++ b/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi @@ -46,6 +46,7 @@ cpu1: cpu@1 { l2: l2-cache { compatible = "cache"; + cache-level = <2>; }; }; diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi index ca3d5a90d6d4..c8b1202d2584 100644 --- a/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi +++ b/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi @@ -83,6 +83,7 @@ cpu3: cpu@3 { l2: l2-cache { compatible = "cache"; + cache-level = <2>; }; }; diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi index feab604322cf..4590bdc076b7 100644 --- a/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi +++ b/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi @@ -78,6 +78,7 @@ cpu3: cpu@3 { l2: l2-cache { compatible = "cache"; + cache-level = <2>; }; }; diff --git a/arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi index 6f6667b70028..2a7e13b6ef8a 100644 --- a/arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi +++ b/arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi @@ -95,18 +95,22 @@ cpu7: cpu@301 { cluster0_l2: l2-cache0 { compatible = "cache"; + cache-level = <2>; }; cluster1_l2: l2-cache1 { compatible = "cache"; + cache-level = <2>; }; cluster2_l2: l2-cache2 { compatible = "cache"; + cache-level = <2>; }; cluster3_l2: l2-cache3 { compatible = "cache"; + cache-level = <2>; }; CPU_PW20: cpu-pw20 { diff --git a/arch/arm64/boot/dts/freescale/fsl-ls2088a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls2088a.dtsi index c3dc38188c17..c12c86915ec8 100644 --- a/arch/arm64/boot/dts/freescale/fsl-ls2088a.dtsi +++ b/arch/arm64/boot/dts/freescale/fsl-ls2088a.dtsi @@ -95,18 +95,22 @@ cpu7: cpu@301 { cluster0_l2: l2-cache0 { compatible = "cache"; + cache-level = <2>; }; cluster1_l2: l2-cache1 { compatible = "cache"; + cache-level = <2>; }; cluster2_l2: l2-cache2 { compatible = "cache"; + cache-level = <2>; }; cluster3_l2: l2-cache3 { compatible = "cache"; + cache-level = <2>; }; CPU_PW20: cpu-pw20 { diff --git a/arch/arm64/boot/dts/freescale/imx8ulp.dtsi b/arch/arm64/boot/dts/freescale/imx8ulp.dtsi index 60c1b018bf03..187353458673 100644 --- a/arch/arm64/boot/dts/freescale/imx8ulp.dtsi +++ b/arch/arm64/boot/dts/freescale/imx8ulp.dtsi @@ -50,6 +50,7 @@ A35_1: cpu@1 { A35_L2: l2-cache0 { compatible = "cache"; + cache-level = <2>; }; }; diff --git a/arch/arm64/boot/dts/freescale/s32g2.dtsi b/arch/arm64/boot/dts/freescale/s32g2.dtsi index 824d401e7a2c..d8c82da88ca0 100644 --- a/arch/arm64/boot/dts/freescale/s32g2.dtsi +++ b/arch/arm64/boot/dts/freescale/s32g2.dtsi @@ -52,10 +52,12 @@ cpu3: cpu@101 { cluster0_l2: l2-cache0 { compatible = "cache"; + cache-level = <2>; }; cluster1_l2: l2-cache1 { compatible = "cache"; + cache-level = <2>; }; }; diff --git a/arch/arm64/boot/dts/freescale/s32v234.dtsi b/arch/arm64/boot/dts/freescale/s32v234.dtsi index ba0b5305d481..3e306218d533 100644 --- a/arch/arm64/boot/dts/freescale/s32v234.dtsi +++ b/arch/arm64/boot/dts/freescale/s32v234.dtsi @@ -61,10 +61,12 @@ cpu3: cpu@101 { cluster0_l2_cache: l2-cache0 { compatible = "cache"; + cache-level = <2>; }; cluster1_l2_cache: l2-cache1 { compatible = "cache"; + cache-level = <2>; }; };