[PATCH 4/6] nitrogen6x: Migrate DDR_MB to Kconfig

[Troy Kisky]

On 8/24/2021 5:41 PM, Tom Rini wrote:
> Move the CONFIG_DDR_MB symbol to Kconfig.  A later clean-up would be to
> make dynamic memory size detection work based on how this is done on
> other i.MX6 platforms.
> 
> Cc: Troy Kisky <troy.kisky at boundarydevices.com>
> Signed-off-by: Tom Rini <trini at konsulko.com>
> ---
> For example, mx6cuboxi does dynamic memory size detection and that code
> could be adapted for this platform and perhaps moved to a more generic
> imx6 location.
> ---
>  board/boundary/nitrogen6x/Kconfig | 3 +++
>  configs/marsboard_defconfig       | 1 -
>  configs/mx6qsabrelite_defconfig   | 3 ++-
>  configs/nitrogen6dl2g_defconfig   | 2 +-
>  configs/nitrogen6dl_defconfig     | 2 +-
>  configs/nitrogen6q2g_defconfig    | 2 +-
>  configs/nitrogen6q_defconfig      | 2 +-
>  configs/nitrogen6s1g_defconfig    | 2 +-
>  configs/nitrogen6s_defconfig      | 2 +-
>  configs/riotboard_defconfig       | 1 -
>  10 files changed, 11 insertions(+), 9 deletions(-)
> 
> diff --git a/board/boundary/nitrogen6x/Kconfig b/board/boundary/nitrogen6x/Kconfig
> index f4db56d49601..9c176c071e72 100644
> --- a/board/boundary/nitrogen6x/Kconfig
> +++ b/board/boundary/nitrogen6x/Kconfig
> @@ -9,4 +9,7 @@ config SYS_VENDOR
>  config SYS_CONFIG_NAME
>  	default "nitrogen6x"
>  
> +config DDR_MB
> +	int "Memory size in MB"
> +
>  endif



> diff --git a/configs/marsboard_defconfig b/configs/marsboard_defconfig
> index bcb83b5d73de..8fcb16c108d1 100644
> --- a/configs/marsboard_defconfig
> +++ b/configs/marsboard_defconfig
> @@ -11,7 +11,6 @@ CONFIG_TARGET_EMBESTMX6BOARDS=y
>  CONFIG_DM_GPIO=y
>  CONFIG_DEFAULT_DEVICE_TREE="imx6q-marsboard"
>  CONFIG_DISTRO_DEFAULTS=y
> -CONFIG_SYS_EXTRA_OPTIONS="DDR_MB=1024"
>  CONFIG_BOOTCOMMAND="run finduuid; run distro_bootcmd"
>  # CONFIG_CONSOLE_MUX is not set
>  CONFIG_SYS_CONSOLE_IS_IN_ENV=y

> +++ b/configs/riotboard_defconfig
> @@ -18,7 +18,6 @@ CONFIG_SPL_SERIAL_SUPPORT=y
>  CONFIG_SPL=y
>  CONFIG_SPL_LIBDISK_SUPPORT=y
>  CONFIG_DISTRO_DEFAULTS=y
> -CONFIG_SYS_EXTRA_OPTIONS="DDR_MB=1024"
>  CONFIG_BOOTCOMMAND="run finduuid; run distro_bootcmd"
>  # CONFIG_CONSOLE_MUX is not set
>  CONFIG_SYS_CONSOLE_IS_IN_ENV=y
> 


I guess marsboard_defconfig/riotboard_defconfig was a copy/paste issue, and never needed  DDR_MB=1024 ?


For  boundary boards
Acked-by: Troy Kisky <troy.kisky at boundarydevices.com>



Thanks