| 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 |
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
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 --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
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(-)