[PATCH v4 1/2] configs: Add support in Kconfig and convert for armada boards

[Michal Simek]

On 5/19/23 06:30, Venkatesh Yadav Abbarapu wrote:
> From: Algapally Santosh Sagar <santoshsagar.algapally at amd.com>
> 
> The DEFAULT_ENV_IS_RW is moved to the Kconfig for easier configuration.
> Hence, the CONFIG_DEFAULT_ENV_IS_RW config is added to the defconfig files
> to allow enabling them for armada boards.
> 
> Signed-off-by: Algapally Santosh Sagar <santoshsagar.algapally at amd.com>
> Signed-off-by: Venkatesh Yadav Abbarapu <venkatesh.abbarapu at amd.com>
> ---
>   configs/mvebu_db-88f3720_defconfig          | 1 +
>   configs/mvebu_espressobin-88f3720_defconfig | 1 +
>   configs/mvebu_mcbin-88f8040_defconfig       | 1 +
>   drivers/serial/Kconfig                      | 7 +++++++
>   include/configs/mvebu_armada-37xx.h         | 1 -
>   5 files changed, 10 insertions(+), 1 deletion(-)
> 
> diff --git a/configs/mvebu_db-88f3720_defconfig b/configs/mvebu_db-88f3720_defconfig
> index 829567014f..e6fb80167a 100644
> --- a/configs/mvebu_db-88f3720_defconfig
> +++ b/configs/mvebu_db-88f3720_defconfig
> @@ -22,6 +22,7 @@ CONFIG_USE_PREBOOT=y
>   CONFIG_SYS_CONSOLE_INFO_QUIET=y
>   # CONFIG_DISPLAY_CPUINFO is not set
>   # CONFIG_DISPLAY_BOARDINFO is not set
> +CONFIG_DEFAULT_ENV_IS_RW=y
>   CONFIG_DISPLAY_BOARDINFO_LATE=y
>   CONFIG_BOARD_EARLY_INIT_F=y
>   CONFIG_SYS_MAXARGS=32
> diff --git a/configs/mvebu_espressobin-88f3720_defconfig b/configs/mvebu_espressobin-88f3720_defconfig
> index fc394a7e9d..64ee99d64b 100644
> --- a/configs/mvebu_espressobin-88f3720_defconfig
> +++ b/configs/mvebu_espressobin-88f3720_defconfig
> @@ -24,6 +24,7 @@ CONFIG_USE_PREBOOT=y
>   CONFIG_SYS_CONSOLE_INFO_QUIET=y
>   # CONFIG_DISPLAY_CPUINFO is not set
>   # CONFIG_DISPLAY_BOARDINFO is not set
> +CONFIG_DEFAULT_ENV_IS_RW=y
>   CONFIG_DISPLAY_BOARDINFO_LATE=y
>   CONFIG_BOARD_EARLY_INIT_F=y
>   CONFIG_BOARD_LATE_INIT=y
> diff --git a/configs/mvebu_mcbin-88f8040_defconfig b/configs/mvebu_mcbin-88f8040_defconfig
> index 3839d7d98c..1655a5ad76 100644
> --- a/configs/mvebu_mcbin-88f8040_defconfig
> +++ b/configs/mvebu_mcbin-88f8040_defconfig
> @@ -23,6 +23,7 @@ CONFIG_USE_PREBOOT=y
>   CONFIG_SYS_CONSOLE_INFO_QUIET=y
>   # CONFIG_DISPLAY_CPUINFO is not set
>   # CONFIG_DISPLAY_BOARDINFO is not set
> +CONFIG_DEFAULT_ENV_IS_RW=y
>   CONFIG_DISPLAY_BOARDINFO_LATE=y
>   CONFIG_BOARD_EARLY_INIT_F=y
>   CONFIG_SYS_MAXARGS=32
> diff --git a/drivers/serial/Kconfig b/drivers/serial/Kconfig
> index f4767c838f..ad5de8b8db 100644
> --- a/drivers/serial/Kconfig
> +++ b/drivers/serial/Kconfig
> @@ -24,6 +24,13 @@ config BAUDRATE
>   	  in the SPL stage (most drivers) or for choosing a default baudrate
>   	  in the absence of an environment setting (serial_mxc.c).
>   
> +config DEFAULT_ENV_IS_RW
> +	bool "Make default environment as writable"
> +	depends on DM_SERIAL

This dependency is not there for armada board. They are using without any 
dependency on serial.

Thanks,
Michal