[PATCH] configs: stm32mp: cleanup the stm32mp15 file

[PATCH] configs: stm32mp: cleanup the stm32mp15 file

[Patrick Delaunay]

Clean the comments in stm32mp15_common.h file
after move CONFIG_ to Kconfig:
- CONFIG_SYS_CBSIZE
- CONFIG_SYS_SPL_MALLOC_START and CONFIG_SYS_SPL_MALLOC_SIZE

Signed-off-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
---

 include/configs/stm32mp15_common.h | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/include/configs/stm32mp15_common.h b/include/configs/stm32mp15_common.h
index 08a72483bf..c7b200d4b1 100644
--- a/include/configs/stm32mp15_common.h
+++ b/include/configs/stm32mp15_common.h
@@ -15,10 +15,6 @@
  */
 #define CONFIG_SYS_SDRAM_BASE			STM32_DDR_BASE
 
-/*
- * Console I/O buffer size
- */
-
 /*
  * For booting Linux, use the first 256 MB of memory, since this is
  * the maximum mapped by the Linux kernel during initialization.
@@ -30,11 +26,10 @@
 
 /* SPL support */
 #ifdef CONFIG_SPL
-/* SPL use DDR */
-
 /* Restrict SPL to fit within SYSRAM */
 #define STM32_SYSRAM_END		(STM32_SYSRAM_BASE + STM32_SYSRAM_SIZE)
 #endif /* #ifdef CONFIG_SPL */
+
 /*MMC SD*/
 #define CONFIG_SYS_MMC_MAX_DEVICE	3
 
-- 
2.25.1

Re: [PATCH] configs: stm32mp: cleanup the stm32mp15 file

[Patrick DELAUNAY]

Hi,

On 6/15/22 17:48, Patrick Delaunay wrote:
> Clean the comments in stm32mp15_common.h file
> after move CONFIG_ to Kconfig:
> - CONFIG_SYS_CBSIZE
> - CONFIG_SYS_SPL_MALLOC_START and CONFIG_SYS_SPL_MALLOC_SIZE
>
> Signed-off-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
> ---
>
>   include/configs/stm32mp15_common.h | 7 +------
>   1 file changed, 1 insertion(+), 6 deletions(-)
>
> diff --git a/include/configs/stm32mp15_common.h b/include/configs/stm32mp15_common.h
> index 08a72483bf..c7b200d4b1 100644
> --- a/include/configs/stm32mp15_common.h
> +++ b/include/configs/stm32mp15_common.h
> @@ -15,10 +15,6 @@
>    */
>   #define CONFIG_SYS_SDRAM_BASE			STM32_DDR_BASE
>   
> -/*
> - * Console I/O buffer size
> - */
> -
>   /*
>    * For booting Linux, use the first 256 MB of memory, since this is
>    * the maximum mapped by the Linux kernel during initialization.
> @@ -30,11 +26,10 @@
>   
>   /* SPL support */
>   #ifdef CONFIG_SPL
> -/* SPL use DDR */
> -
>   /* Restrict SPL to fit within SYSRAM */
>   #define STM32_SYSRAM_END		(STM32_SYSRAM_BASE + STM32_SYSRAM_SIZE)
>   #endif /* #ifdef CONFIG_SPL */
> +

after check the define STM32_SYSRAM_END is no more used => to remove

i was only used in a define removed in config file after migration to Kconfig

#define CONFIG_SPL_MAX_FOOTPRINT    (STM32_SYSRAM_END - 
CONFIG_SPL_TEXT_BASE)


>   /*MMC SD*/
>   #define CONFIG_SYS_MMC_MAX_DEVICE	3
>   


Regards

Patrick