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;
Date:   Tue, 25 Apr 2023 15:46:14 +0800
From:   Chester Lin <clin@suse.com>
To:     Matthias Brugger <mbrugger@suse.com>
Cc:     Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
        Andreas =?iso-8859-1?Q?F=E4rber?= <afaerber@suse.de>,
        NXP S32 Linux Team <s32@nxp.com>,
        Shawn Guo <shawnguo@kernel.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>,
        Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH] arm64: dts: s32: add missing cache properties
Message-ID: <ZEeFRpfL9TTYF1Ow@linux-8mug>
References: <20230421223202.115472-1-krzysztof.kozlowski@linaro.org>
 <aa3c1fda-553b-fa71-21dc-41d1244007a4@suse.com>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <aa3c1fda-553b-fa71-21dc-41d1244007a4@suse.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?zdWKAcfuGz+/elvTp0QbX+4rsVPBrIfMwTeZjtr4h0vf/+XMjpyg+4XfDr0D?=
 =?us-ascii?Q?QmTj8jttORBjHrGGv7mMfQgHQJELmgVw36eE05FAnmLyA3936ovbrGAtv/dF?=
 =?us-ascii?Q?LIf3D/uCAkrHswHmW96qCwOla0X/otV3rxeW3U2voV8NWgX4slYZVoGlJ9s1?=
 =?us-ascii?Q?GJNBm6wv56r5jRiq7hlQQj7cGQTJ4QtUu0UbB3u3iKAru3k8WxA+VrlD7Zhm?=
 =?us-ascii?Q?Ow/sxWXc/0Iimf9yGYvIvawbJceF8ocl3HAg2+YmKsSk3PA361NlbZN2OpIc?=
 =?us-ascii?Q?q6IiAwOIoQa9IcxPipqMslxqPBia10Wpj10+QWPUWBSEKyLO3xW6fREPKFps?=
 =?us-ascii?Q?kPCAGXSDHrwEDJ49osgmG+rU3m7ttwjyYBivbPQsGGUj9OLnG2z3IVeDqhLs?=
 =?us-ascii?Q?gJ14XQpDikV+o5ACtaXOlkAMIMrmjn9lk6evFFvuaSGGdZ+TBrKG7FKCUWQg?=
 =?us-ascii?Q?VJrBvWJwB7lBJ5/wcyR1x0Te8k+z4KR743U5yygvGFhRRKz17lDoCaA8tof/?=
 =?us-ascii?Q?KPw1f3DuoT1WiM7RSltNj69uQp32R1FZzvTR0rO4P9vxFSl80zHR1XwNl5LA?=
 =?us-ascii?Q?DBF9Np6x3pRoo+bgOgvd/OPJxvgKwpTVvtyeVPC5xjwQRDoWVd8T0dM2YSki?=
 =?us-ascii?Q?z56k4AglbdI5TSE0sT6FaiNKz9c1ovzCgzogVdKO1CN5HTLpvysRjFJ/TfrZ?=
 =?us-ascii?Q?Dytzyakc9rvliPMI9FhqUUEWhXDYFhcZ5N9Upha1UGPdjVmJSnLOe3374HlW?=
 =?us-ascii?Q?7nH7BNY5jgetRTFwA7R418hcQP6NDazUqXbLZnMX8571MMF/zr0fxrBV5kAD?=
 =?us-ascii?Q?2LUijiUoPi/2cV894eSVA/97uXN6tG4OGS2JBjZIY2jNMjl0wlKpeO2CQ4j8?=
 =?us-ascii?Q?kanbx+0i9K5UrAXEmu2Alv7AFp7nEHPUQ0r01jj0NBF+IL0EzvguBpUHDhSk?=
 =?us-ascii?Q?sDV+5WXrI3fjfsk+x+8i0J3AOJdGGReJnhieEN+c3WaizAG6g/d/wPYdG/i2?=
 =?us-ascii?Q?Zc+vagE7E41REq63JRjtaYOMt0DFo0Q2ph2+glxRLpDbImcCxyzocZk8UzWf?=
 =?us-ascii?Q?9wI9giPAiciC1wOk1LXD1w7NoLcBe0zxA6DSKuVSJymh1bSvBXHLxUdA82mt?=
 =?us-ascii?Q?E+xadR97a2lmLxJTghGpQVVWl4Xl4zDvT9e6953wQd4F0nOpmTzNRBc8OwA6?=
 =?us-ascii?Q?H4pNllxgVxvYMUH1AqKFxCsuEidlD2oy4wzYCcfT+doE4kUJRKMzKxlG/CcH?=
 =?us-ascii?Q?pFGISd2v6LgPqbMlOYRaMBvClW2m/G8qj/4OdDPyzLXIcP8lb7IR+AsHi0Ha?=
 =?us-ascii?Q?S+aKa/IhVUsI7fjP2zj//b235YRTmlEnm/5NWrjlj8PZpj4LkmVn/JspCd5I?=
 =?us-ascii?Q?+l/JjuKP3ID9eyqhbsAMAna41vPAy4TJ1gKFj0lYnRq4NYM68m8uSzf78wVM?=
 =?us-ascii?Q?Yf0PObZroVB/6cBmqln9fBaxMOKgKSVH92kcRbHEvQiCN3uh+IHe99tjGK+P?=
 =?us-ascii?Q?bi++qlGizy9jDIS94AFx1ZveQhjWqJ3blOX8TH6ARVUmGKMFuMQQg8XVFecw?=
 =?us-ascii?Q?uHPANn8xJb/Vad4hyfo=3D?=
X-OriginatorOrg: suse.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 5247e237-1c3a-4b0d-247e-08db45612c4d
X-MS-Exchange-CrossTenant-AuthSource: VI1PR0402MB3439.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 25 Apr 2023 07:46:26.3141
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: f7a17af6-1c5c-4a36-aa8b-f5be247aa4ba
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: X7z5mYlWJq5/+DvUhGtlaAmtPzT0VyieVYmOJq4RU4Kem4P6HUvJWiPS3AOuZdtM
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PA4PR04MB9224
X-Spam-Status: No, score=-1.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FORGED_SPF_HELO,
        RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,
        T_SCC_BODY_TEXT_LINE,T_SPF_TEMPERROR,URIBL_BLOCKED autolearn=no
        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

On Mon, Apr 24, 2023 at 12:37:34PM +0200, Matthias Brugger wrote:
> 
> 
> On 22/04/2023 00:32, Krzysztof Kozlowski wrote:
> > As all level 2 and level 3 caches are unified, add required
> > cache-unified properties to fix warnings like:
> > 
> >    s32g274a-evb.dtb: l2-cache1: 'cache-unified' is a required property
> > 
> > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> 
> Reviewed-by: Matthias Brugger <mbrugger@suse.com>
> 
> > 
> > ---
> > 
> > Please take the patch via sub-arch SoC tree.
> > ---
> >   arch/arm64/boot/dts/freescale/s32g2.dtsi   | 2 ++
> >   arch/arm64/boot/dts/freescale/s32v234.dtsi | 2 ++
> >   2 files changed, 4 insertions(+)
> > 
> > diff --git a/arch/arm64/boot/dts/freescale/s32g2.dtsi b/arch/arm64/boot/dts/freescale/s32g2.dtsi
> > index d8c82da88ca0..5ac1cc9ff50e 100644
> > --- a/arch/arm64/boot/dts/freescale/s32g2.dtsi
> > +++ b/arch/arm64/boot/dts/freescale/s32g2.dtsi
> > @@ -53,11 +53,13 @@ cpu3: cpu@101 {
> >   		cluster0_l2: l2-cache0 {
> >   			compatible = "cache";
> >   			cache-level = <2>;
> > +			cache-unified;
> >   		};
> >   		cluster1_l2: l2-cache1 {
> >   			compatible = "cache";
> >   			cache-level = <2>;
> > +			cache-unified;
> >   		};
> >   	};
> > diff --git a/arch/arm64/boot/dts/freescale/s32v234.dtsi b/arch/arm64/boot/dts/freescale/s32v234.dtsi
> > index 3e306218d533..42409ec56792 100644
> > --- a/arch/arm64/boot/dts/freescale/s32v234.dtsi
> > +++ b/arch/arm64/boot/dts/freescale/s32v234.dtsi
> > @@ -62,11 +62,13 @@ cpu3: cpu@101 {
> >   		cluster0_l2_cache: l2-cache0 {
> >   			compatible = "cache";
> >   			cache-level = <2>;
> > +			cache-unified;
> >   		};
> >   		cluster1_l2_cache: l2-cache1 {
> >   			compatible = "cache";
> >   			cache-level = <2>;
> > +			cache-unified;
> >   		};
> >   	};

Applied to https://github.com/chesterlintw/linux-s32g/tree/s32g2/next

Thanks,
Chester