From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=foss.st.com header.i=@foss.st.com header.b="e6f2Z6kr" Received: from mx07-00178001.pphosted.com (mx08-00178001.pphosted.com [91.207.212.93]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 4671C10F; Thu, 14 Dec 2023 07:37:50 -0800 (PST) Received: from pps.filterd (m0046661.ppops.net [127.0.0.1]) by mx07-00178001.pphosted.com (8.17.1.22/8.17.1.22) with ESMTP id 3BEFRSLY006650; Thu, 14 Dec 2023 16:37:34 +0100 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=foss.st.com; h= message-id:date:mime-version:subject:to:cc:references:from :in-reply-to:content-type:content-transfer-encoding; s= selector1; bh=z+Okecx9m3lcQPbiL7ZlZzQneMvRP6AYBDxySkUnCxA=; b=e6 f2Z6krzrutBAiwdO7Z2D4b7/heMdokJJ/UEjc9gVA1NIYUflmISkKwznYUtUSVQ1 eCOHF321XRe+RyBRHoJe8zDnLkaloW7HUfOhSjHnuf2f9+3MGJszOQXMvtLHDF85 U2HwpVw0ig8p9GCHPP6RKouLZyyHjPQPY8i6VeN8LrtXO0jVSTAIBa8KGlnKFcLZ WqIZ2qlyFqavLQ169t/eM4VOljjW2ZBnSSAYOT9z+0WbQ6zZ5Tpgah9JMgkWwpPS 6XVxDd53fW/lYADzwYau6pCL+lJS/6lPFoDziDe/bs2C5TUQ2alurFqOqWuKenvy tVC7dsnPq69aRo8u5NSg== Received: from beta.dmz-eu.st.com (beta.dmz-eu.st.com [164.129.1.35]) by mx07-00178001.pphosted.com (PPS) with ESMTPS id 3uvg0h5m92-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Thu, 14 Dec 2023 16:37:34 +0100 (CET) Received: from euls16034.sgp.st.com (euls16034.sgp.st.com [10.75.44.20]) by beta.dmz-eu.st.com (STMicroelectronics) with ESMTP id 1A19D100053; Thu, 14 Dec 2023 16:37:33 +0100 (CET) Received: from Webmail-eu.st.com (shfdag1node1.st.com [10.75.129.69]) by euls16034.sgp.st.com (STMicroelectronics) with ESMTP id 107ED26C06D; Thu, 14 Dec 2023 16:37:33 +0100 (CET) Received: from [10.201.21.122] (10.201.21.122) by SHFDAG1NODE1.st.com (10.75.129.69) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.27; Thu, 14 Dec 2023 16:37:32 +0100 Message-ID: <8adec7b4-bdd9-4ad0-a302-326ed99ea732@foss.st.com> Date: Thu, 14 Dec 2023 16:37:31 +0100 Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 2/2] ARM: dts: stm32: don't mix SCMI and non-SCMI board compatibles Content-Language: en-US To: Ahmad Fatoum , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Maxime Coquelin CC: , , , , References: <20231122185235.2017642-1-a.fatoum@pengutronix.de> <20231122185235.2017642-2-a.fatoum@pengutronix.de> From: Alexandre TORGUE In-Reply-To: <20231122185235.2017642-2-a.fatoum@pengutronix.de> Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 7bit X-ClientProxiedBy: EQNCAS1NODE4.st.com (10.75.129.82) To SHFDAG1NODE1.st.com (10.75.129.69) X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.272,Aquarius:18.0.997,Hydra:6.0.619,FMLib:17.11.176.26 definitions=2023-12-14_10,2023-12-14_01,2023-05-22_02 Hi Ahmad On 11/22/23 19:52, Ahmad Fatoum wrote: > The binding erroneously decreed that the SCMI variants of the ST > evaluation kits are compatible with the non-SCMI variants. > > This is not correct, as a kernel or bootloader compatible with the non-SCMI > variant is not necessarily able to function, when direct access > to resources is replaced by having to talk SCMI to the secure monitor. > > The binding has been adjusted to reflect thus, so synchronize the device > trees now. > > Fixes: 5b7e58313a77 ("ARM: dts: stm32: Add SCMI version of STM32 boards (DK1/DK2/ED1/EV1)") > Signed-off-by: Ahmad Fatoum > --- Thanks to have fix my error. Both patches applied on stm32-next. Cheers Alex > arch/arm/boot/dts/st/stm32mp157a-dk1-scmi.dts | 2 +- > arch/arm/boot/dts/st/stm32mp157c-dk2-scmi.dts | 2 +- > arch/arm/boot/dts/st/stm32mp157c-ed1-scmi.dts | 2 +- > arch/arm/boot/dts/st/stm32mp157c-ev1-scmi.dts | 3 +-- > 4 files changed, 4 insertions(+), 5 deletions(-) > > diff --git a/arch/arm/boot/dts/st/stm32mp157a-dk1-scmi.dts b/arch/arm/boot/dts/st/stm32mp157a-dk1-scmi.dts > index afcd6285890c..c27963898b5e 100644 > --- a/arch/arm/boot/dts/st/stm32mp157a-dk1-scmi.dts > +++ b/arch/arm/boot/dts/st/stm32mp157a-dk1-scmi.dts > @@ -11,7 +11,7 @@ > > / { > model = "STMicroelectronics STM32MP157A-DK1 SCMI Discovery Board"; > - compatible = "st,stm32mp157a-dk1-scmi", "st,stm32mp157a-dk1", "st,stm32mp157"; > + compatible = "st,stm32mp157a-dk1-scmi", "st,stm32mp157"; > > reserved-memory { > optee@de000000 { > diff --git a/arch/arm/boot/dts/st/stm32mp157c-dk2-scmi.dts b/arch/arm/boot/dts/st/stm32mp157c-dk2-scmi.dts > index 39358d902000..622618943134 100644 > --- a/arch/arm/boot/dts/st/stm32mp157c-dk2-scmi.dts > +++ b/arch/arm/boot/dts/st/stm32mp157c-dk2-scmi.dts > @@ -11,7 +11,7 @@ > > / { > model = "STMicroelectronics STM32MP157C-DK2 SCMI Discovery Board"; > - compatible = "st,stm32mp157c-dk2-scmi", "st,stm32mp157c-dk2", "st,stm32mp157"; > + compatible = "st,stm32mp157c-dk2-scmi", "st,stm32mp157"; > > reserved-memory { > optee@de000000 { > diff --git a/arch/arm/boot/dts/st/stm32mp157c-ed1-scmi.dts b/arch/arm/boot/dts/st/stm32mp157c-ed1-scmi.dts > index 07ea765a4553..c7c4d7e89d61 100644 > --- a/arch/arm/boot/dts/st/stm32mp157c-ed1-scmi.dts > +++ b/arch/arm/boot/dts/st/stm32mp157c-ed1-scmi.dts > @@ -11,7 +11,7 @@ > > / { > model = "STMicroelectronics STM32MP157C-ED1 SCMI eval daughter"; > - compatible = "st,stm32mp157c-ed1-scmi", "st,stm32mp157c-ed1", "st,stm32mp157"; > + compatible = "st,stm32mp157c-ed1-scmi", "st,stm32mp157"; > > reserved-memory { > optee@fe000000 { > diff --git a/arch/arm/boot/dts/st/stm32mp157c-ev1-scmi.dts b/arch/arm/boot/dts/st/stm32mp157c-ev1-scmi.dts > index 813086ec2489..2ab77e64f1bb 100644 > --- a/arch/arm/boot/dts/st/stm32mp157c-ev1-scmi.dts > +++ b/arch/arm/boot/dts/st/stm32mp157c-ev1-scmi.dts > @@ -11,8 +11,7 @@ > > / { > model = "STMicroelectronics STM32MP157C-EV1 SCMI eval daughter on eval mother"; > - compatible = "st,stm32mp157c-ev1-scmi", "st,stm32mp157c-ev1", "st,stm32mp157c-ed1", > - "st,stm32mp157"; > + compatible = "st,stm32mp157c-ev1-scmi", "st,stm32mp157c-ed1", "st,stm32mp157"; > > reserved-memory { > optee@fe000000 {