diff mbox series

ARM: stm32: Fix DBGMCU macro on STM32MP13xx

Message ID 20250512164456.147764-1-marek.vasut@mailbox.org
State Accepted
Delegated to: Patrice Chotard
Headers show
Series ARM: stm32: Fix DBGMCU macro on STM32MP13xx | expand

Commit Message

Marek Vasut May 12, 2025, 4:44 p.m. UTC
The DBGMCU block is available at address 0x50081000 both on STM32MP13xx
and on STM32MP15xx . There is no reason to limit the DBGMCU macro being
set only on STM32MP15xx , remove the ifdeffery.

Signed-off-by: Marek Vasut <marek.vasut@mailbox.org>
---
Cc: Patrice Chotard <patrice.chotard@foss.st.com>
Cc: Patrick Delaunay <patrick.delaunay@foss.st.com>
Cc: Simon Glass <sjg@chromium.org>
Cc: Sughosh Ganu <sughosh.ganu@linaro.org>
Cc: Tom Rini <trini@konsulko.com>
Cc: u-boot@lists.denx.de
Cc: uboot-stm32@st-md-mailman.stormreply.com
---
 arch/arm/mach-stm32mp/include/mach/stm32.h | 2 --
 1 file changed, 2 deletions(-)

Comments

Patrice CHOTARD May 21, 2025, 1:59 p.m. UTC | #1
On 5/12/25 18:44, Marek Vasut wrote:
> The DBGMCU block is available at address 0x50081000 both on STM32MP13xx
> and on STM32MP15xx . There is no reason to limit the DBGMCU macro being
> set only on STM32MP15xx , remove the ifdeffery.
> 
> Signed-off-by: Marek Vasut <marek.vasut@mailbox.org>
> ---
> Cc: Patrice Chotard <patrice.chotard@foss.st.com>
> Cc: Patrick Delaunay <patrick.delaunay@foss.st.com>
> Cc: Simon Glass <sjg@chromium.org>
> Cc: Sughosh Ganu <sughosh.ganu@linaro.org>
> Cc: Tom Rini <trini@konsulko.com>
> Cc: u-boot@lists.denx.de
> Cc: uboot-stm32@st-md-mailman.stormreply.com
> ---
>  arch/arm/mach-stm32mp/include/mach/stm32.h | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/arch/arm/mach-stm32mp/include/mach/stm32.h b/arch/arm/mach-stm32mp/include/mach/stm32.h
> index 47b459b75d1..dfba57e7dc4 100644
> --- a/arch/arm/mach-stm32mp/include/mach/stm32.h
> +++ b/arch/arm/mach-stm32mp/include/mach/stm32.h
> @@ -78,9 +78,7 @@ enum forced_boot_mode {
>  #define STM32_RCC_BASE			0x50000000
>  #define STM32_PWR_BASE			0x50001000
>  #define STM32_SYSCFG_BASE		0x50020000
> -#ifdef CONFIG_STM32MP15X
>  #define STM32_DBGMCU_BASE		0x50081000
> -#endif
>  #define STM32_FMC2_BASE			0x58002000
>  #define STM32_IWDG2_BASE		0x5A002000
>  #define STM32_DDRCTRL_BASE		0x5A003000

Reviewed-by: Patrice Chotard <patrice.chotard@foss.st.com>

Thanks
Patrice
Patrice CHOTARD June 9, 2025, 8:54 a.m. UTC | #2
On 5/21/25 15:59, Patrice CHOTARD wrote:
> 
> 
> On 5/12/25 18:44, Marek Vasut wrote:
>> The DBGMCU block is available at address 0x50081000 both on STM32MP13xx
>> and on STM32MP15xx . There is no reason to limit the DBGMCU macro being
>> set only on STM32MP15xx , remove the ifdeffery.
>>
>> Signed-off-by: Marek Vasut <marek.vasut@mailbox.org>
>> ---
>> Cc: Patrice Chotard <patrice.chotard@foss.st.com>
>> Cc: Patrick Delaunay <patrick.delaunay@foss.st.com>
>> Cc: Simon Glass <sjg@chromium.org>
>> Cc: Sughosh Ganu <sughosh.ganu@linaro.org>
>> Cc: Tom Rini <trini@konsulko.com>
>> Cc: u-boot@lists.denx.de
>> Cc: uboot-stm32@st-md-mailman.stormreply.com
>> ---
>>  arch/arm/mach-stm32mp/include/mach/stm32.h | 2 --
>>  1 file changed, 2 deletions(-)
>>
>> diff --git a/arch/arm/mach-stm32mp/include/mach/stm32.h b/arch/arm/mach-stm32mp/include/mach/stm32.h
>> index 47b459b75d1..dfba57e7dc4 100644
>> --- a/arch/arm/mach-stm32mp/include/mach/stm32.h
>> +++ b/arch/arm/mach-stm32mp/include/mach/stm32.h
>> @@ -78,9 +78,7 @@ enum forced_boot_mode {
>>  #define STM32_RCC_BASE			0x50000000
>>  #define STM32_PWR_BASE			0x50001000
>>  #define STM32_SYSCFG_BASE		0x50020000
>> -#ifdef CONFIG_STM32MP15X
>>  #define STM32_DBGMCU_BASE		0x50081000
>> -#endif
>>  #define STM32_FMC2_BASE			0x58002000
>>  #define STM32_IWDG2_BASE		0x5A002000
>>  #define STM32_DDRCTRL_BASE		0x5A003000
> 
> Reviewed-by: Patrice Chotard <patrice.chotard@foss.st.com>
> 
> Thanks
> Patrice
Applied to u-boot-stm32/next

Thanks
Patrice
diff mbox series

Patch

diff --git a/arch/arm/mach-stm32mp/include/mach/stm32.h b/arch/arm/mach-stm32mp/include/mach/stm32.h
index 47b459b75d1..dfba57e7dc4 100644
--- a/arch/arm/mach-stm32mp/include/mach/stm32.h
+++ b/arch/arm/mach-stm32mp/include/mach/stm32.h
@@ -78,9 +78,7 @@  enum forced_boot_mode {
 #define STM32_RCC_BASE			0x50000000
 #define STM32_PWR_BASE			0x50001000
 #define STM32_SYSCFG_BASE		0x50020000
-#ifdef CONFIG_STM32MP15X
 #define STM32_DBGMCU_BASE		0x50081000
-#endif
 #define STM32_FMC2_BASE			0x58002000
 #define STM32_IWDG2_BASE		0x5A002000
 #define STM32_DDRCTRL_BASE		0x5A003000