diff mbox series

ARM: dts: stm32: Drop stm32mp15xc.dtsi from Avenger96

Message ID 20220923023745.272076-1-marex@denx.de (mailing list archive)
State New, archived
Headers show
Series ARM: dts: stm32: Drop stm32mp15xc.dtsi from Avenger96 | expand

Commit Message

Marek Vasut Sept. 23, 2022, 2:37 a.m. UTC 
The Avenger96 is populated with STM32MP157A DHCOR SoM, drop the
stm32mp15xc.dtsi which should only be included in DTs of devices
which are populated with STM32MP15xC/F SoC as the stm32mp15xc.dtsi
enables CRYP block not present in the STM32MP15xA/D SoC .

Fixes: 7e76f82acd9e1 ("ARM: dts: stm32: Split Avenger96 into DHCOR SoM and Avenger96 board")
Signed-off-by: Marek Vasut <marex@denx.de>
---
Cc: Alexandre Torgue <alexandre.torgue@foss.st.com>
Cc: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
Cc: Patrice Chotard <patrice.chotard@foss.st.com>
Cc: Patrick Delaunay <patrick.delaunay@foss.st.com>
Cc: linux-stm32@st-md-mailman.stormreply.com
To: linux-arm-kernel@lists.infradead.org
---
 arch/arm/boot/dts/stm32mp157a-dhcor-avenger96.dts | 1 -
 1 file changed, 1 deletion(-)

Comments

Patrice CHOTARD Sept. 23, 2022, 6:13 a.m. UTC | #1 
Hi Marek

On 9/23/22 04:37, Marek Vasut wrote:
> The Avenger96 is populated with STM32MP157A DHCOR SoM, drop the
> stm32mp15xc.dtsi which should only be included in DTs of devices
> which are populated with STM32MP15xC/F SoC as the stm32mp15xc.dtsi
> enables CRYP block not present in the STM32MP15xA/D SoC .
> 
> Fixes: 7e76f82acd9e1 ("ARM: dts: stm32: Split Avenger96 into DHCOR SoM and Avenger96 board")
> Signed-off-by: Marek Vasut <marex@denx.de>
> ---
> Cc: Alexandre Torgue <alexandre.torgue@foss.st.com>
> Cc: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
> Cc: Patrice Chotard <patrice.chotard@foss.st.com>
> Cc: Patrick Delaunay <patrick.delaunay@foss.st.com>
> Cc: linux-stm32@st-md-mailman.stormreply.com
> To: linux-arm-kernel@lists.infradead.org
> ---
>  arch/arm/boot/dts/stm32mp157a-dhcor-avenger96.dts | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/arch/arm/boot/dts/stm32mp157a-dhcor-avenger96.dts b/arch/arm/boot/dts/stm32mp157a-dhcor-avenger96.dts
> index 2e3c9fbb4eb36..275167f26fd9d 100644
> --- a/arch/arm/boot/dts/stm32mp157a-dhcor-avenger96.dts
> +++ b/arch/arm/boot/dts/stm32mp157a-dhcor-avenger96.dts
> @@ -13,7 +13,6 @@
>  /dts-v1/;
>  
>  #include "stm32mp157.dtsi"
> -#include "stm32mp15xc.dtsi"
>  #include "stm32mp15xx-dhcor-som.dtsi"
>  #include "stm32mp15xx-dhcor-avenger96.dtsi"
>  
Reviewed-by: Patrice Chotard <patrice.chotard@foss.st.com>

Patrice
Alexandre TORGUE Oct. 20, 2022, 3:40 p.m. UTC | #2 
Hi Marek

On 9/23/22 04:37, Marek Vasut wrote:
> The Avenger96 is populated with STM32MP157A DHCOR SoM, drop the
> stm32mp15xc.dtsi which should only be included in DTs of devices
> which are populated with STM32MP15xC/F SoC as the stm32mp15xc.dtsi
> enables CRYP block not present in the STM32MP15xA/D SoC .
> 
> Fixes: 7e76f82acd9e1 ("ARM: dts: stm32: Split Avenger96 into DHCOR SoM and Avenger96 board")
> Signed-off-by: Marek Vasut <marex@denx.de>
> ---
> Cc: Alexandre Torgue <alexandre.torgue@foss.st.com>
> Cc: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
> Cc: Patrice Chotard <patrice.chotard@foss.st.com>
> Cc: Patrick Delaunay <patrick.delaunay@foss.st.com>
> Cc: linux-stm32@st-md-mailman.stormreply.com
> To: linux-arm-kernel@lists.infradead.org
> ---
>   arch/arm/boot/dts/stm32mp157a-dhcor-avenger96.dts | 1 -
>   1 file changed, 1 deletion(-)
> 
> diff --git a/arch/arm/boot/dts/stm32mp157a-dhcor-avenger96.dts b/arch/arm/boot/dts/stm32mp157a-dhcor-avenger96.dts
> index 2e3c9fbb4eb36..275167f26fd9d 100644
> --- a/arch/arm/boot/dts/stm32mp157a-dhcor-avenger96.dts
> +++ b/arch/arm/boot/dts/stm32mp157a-dhcor-avenger96.dts
> @@ -13,7 +13,6 @@
>   /dts-v1/;
>   
>   #include "stm32mp157.dtsi"
> -#include "stm32mp15xc.dtsi"
>   #include "stm32mp15xx-dhcor-som.dtsi"
>   #include "stm32mp15xx-dhcor-avenger96.dtsi"
>  

Applied on stm32-next.

Thanks.
Alex
Manivannan Sadhasivam Oct. 20, 2022, 4:25 p.m. UTC | #3 
On Fri, Sep 23, 2022 at 04:37:45AM +0200, Marek Vasut wrote:
> The Avenger96 is populated with STM32MP157A DHCOR SoM, drop the
> stm32mp15xc.dtsi which should only be included in DTs of devices
> which are populated with STM32MP15xC/F SoC as the stm32mp15xc.dtsi
> enables CRYP block not present in the STM32MP15xA/D SoC .
> 
> Fixes: 7e76f82acd9e1 ("ARM: dts: stm32: Split Avenger96 into DHCOR SoM and Avenger96 board")
> Signed-off-by: Marek Vasut <marex@denx.de>

Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>

Thanks,
Mani

> ---
> Cc: Alexandre Torgue <alexandre.torgue@foss.st.com>
> Cc: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
> Cc: Patrice Chotard <patrice.chotard@foss.st.com>
> Cc: Patrick Delaunay <patrick.delaunay@foss.st.com>
> Cc: linux-stm32@st-md-mailman.stormreply.com
> To: linux-arm-kernel@lists.infradead.org
> ---
>  arch/arm/boot/dts/stm32mp157a-dhcor-avenger96.dts | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/arch/arm/boot/dts/stm32mp157a-dhcor-avenger96.dts b/arch/arm/boot/dts/stm32mp157a-dhcor-avenger96.dts
> index 2e3c9fbb4eb36..275167f26fd9d 100644
> --- a/arch/arm/boot/dts/stm32mp157a-dhcor-avenger96.dts
> +++ b/arch/arm/boot/dts/stm32mp157a-dhcor-avenger96.dts
> @@ -13,7 +13,6 @@
>  /dts-v1/;
>  
>  #include "stm32mp157.dtsi"
> -#include "stm32mp15xc.dtsi"
>  #include "stm32mp15xx-dhcor-som.dtsi"
>  #include "stm32mp15xx-dhcor-avenger96.dtsi"
>  
> -- 
> 2.35.1
>
diff mbox series

Patch

diff --git a/arch/arm/boot/dts/stm32mp157a-dhcor-avenger96.dts b/arch/arm/boot/dts/stm32mp157a-dhcor-avenger96.dts
index 2e3c9fbb4eb36..275167f26fd9d 100644
--- a/arch/arm/boot/dts/stm32mp157a-dhcor-avenger96.dts
+++ b/arch/arm/boot/dts/stm32mp157a-dhcor-avenger96.dts
@@ -13,7 +13,6 @@ 
 /dts-v1/;
 
 #include "stm32mp157.dtsi"
-#include "stm32mp15xc.dtsi"
 #include "stm32mp15xx-dhcor-som.dtsi"
 #include "stm32mp15xx-dhcor-avenger96.dtsi"