Re: [PATCH] arm: at91: gardena-smart-gateway-at91sam: Adjust to production values

2021-08-18 Thread Eugen.Hristev
On 6/17/21 7:57 PM, Reto Schneider wrote:
> From: Reto Schneider 
> 
> This commit updates the default config with the values that will be used
> soon on the Atmel / Microchip AT91SAM9G25 based GARDENA smart gateway.
> 
> Signed-off-by: Reto Schneider 
> ---
> This patch applies to current u-boot-atmel/next

Applied to u-boot-atmel/next

Sorry for taking so long, somehow I missed this patch as it had 
identical subject with a previous patch which I already applied, but 
this patch does something different.
Anyway, I applied it to next, but will be queued for the next merge window.

Eugen

> 
>   configs/gardena-smart-gateway-at91sam_defconfig | 15 ++-
>   1 file changed, 14 insertions(+), 1 deletion(-)
> 
> diff --git a/configs/gardena-smart-gateway-at91sam_defconfig 
> b/configs/gardena-smart-gateway-at91sam_defconfig
> index 40fe0f150e..c626cd699a 100644
> --- a/configs/gardena-smart-gateway-at91sam_defconfig
> +++ b/configs/gardena-smart-gateway-at91sam_defconfig
> @@ -8,6 +8,7 @@ CONFIG_SPL_LIBCOMMON_SUPPORT=y
>   CONFIG_SPL_LIBGENERIC_SUPPORT=y
>   CONFIG_SYS_MALLOC_F_LEN=0x2000
>   CONFIG_NR_DRAM_BANKS=1
> +CONFIG_SYS_MEMTEST_START=0x0
>   CONFIG_ENV_SIZE=0x1
>   CONFIG_DM_GPIO=y
>   CONFIG_SPL_TEXT_BASE=0x30
> @@ -22,6 +23,8 @@ CONFIG_DEBUG_UART=y
>   CONFIG_FIT=y
>   CONFIG_NAND_BOOT=y
>   CONFIG_BOOTDELAY=0
> +CONFIG_AUTOBOOT_KEYED=y
> +CONFIG_AUTOBOOT_STOP_STR="x"
>   CONFIG_USE_BOOTARGS=y
>   CONFIG_BOOTARGS="console=ttyS0,115200 earlyprintk 
> mtdparts=atmel_nand:256k(bootstrap)ro,768k(uboot)ro,256k(env_redundant),256k(env),512k(dtb),6M(kernel)ro,-(rootfs)
>  rootfstype=ubifs ubi.mtd=6 root=ubi0:rootfs rw"
>   CONFIG_SYS_CONSOLE_IS_IN_ENV=y
> @@ -32,20 +35,26 @@ CONFIG_SPL_NAND_SUPPORT=y
>   CONFIG_SPL_NAND_DRIVERS=y
>   CONFIG_SPL_NAND_BASE=y
>   CONFIG_HUSH_PARSER=y
> +CONFIG_CMD_LICENSE=y
>   CONFIG_CMD_BOOTZ=y
> +CONFIG_CMD_MEMINFO=y
> +CONFIG_CMD_MEMTEST=y
> +CONFIG_SYS_ALT_MEMTEST=y
>   CONFIG_CMD_DM=y
>   # CONFIG_CMD_FLASH is not set
>   CONFIG_CMD_GPIO=y
>   CONFIG_CMD_MTD=y
>   CONFIG_CMD_NAND=y
>   CONFIG_CMD_WDT=y
> -# CONFIG_CMD_SETEXPR is not set
>   CONFIG_CMD_DHCP=y
>   CONFIG_CMD_PING=y
>   CONFIG_CMD_CACHE=y
>   CONFIG_CMD_TIME=y
> +CONFIG_CMD_GETTIME=y
> +CONFIG_CMD_UUID=y
>   CONFIG_CMD_FAT=y
>   CONFIG_CMD_MTDPARTS=y
> +CONFIG_CMD_MTDPARTS_SHOW_NET_SIZES=y
>   CONFIG_MTDIDS_DEFAULT="nand0=atmel_nand"
>   CONFIG_MTDPARTS_DEFAULT="atmel_nand:1536k(uboot),10752k(unused),-(ubi)"
>   CONFIG_CMD_UBI=y
> @@ -58,6 +67,8 @@ CONFIG_ENV_UBI_PART="ubi"
>   CONFIG_ENV_UBI_VOLUME="uboot_env0"
>   CONFIG_ENV_UBI_VOLUME_REDUND="uboot_env1"
>   CONFIG_SYS_RELOC_GD_ENV_ADDR=y
> +CONFIG_ENV_VARS_UBOOT_RUNTIME_CONFIG=y
> +CONFIG_VERSION_VARIABLE=y
>   CONFIG_NET_RANDOM_ETHADDR=y
>   CONFIG_DM=y
>   CONFIG_SPL_DM=y
> @@ -71,6 +82,7 @@ CONFIG_LED_GPIO=y
>   CONFIG_MTD=y
>   CONFIG_NAND_ATMEL=y
>   CONFIG_SPL_GENERATE_ATMEL_PMECC_HEADER=y
> +CONFIG_MTD_UBI_FASTMAP=y
>   CONFIG_DM_ETH=y
>   CONFIG_MACB=y
>   CONFIG_PINCTRL=y
> @@ -84,3 +96,4 @@ CONFIG_ATMEL_PIT_TIMER=y
>   CONFIG_WDT=y
>   CONFIG_WDT_AT91=y
>   # CONFIG_UBIFS_SILENCE_MSG is not set
> +CONFIG_LZMA=y
> --
> 2.30.2
> 



Re: [PATCH] arm: at91: gardena-smart-gateway-at91sam: Adjust to production values

2021-06-17 Thread Stefan Roese

On 17.06.21 18:57, Reto Schneider wrote:

From: Reto Schneider 

This commit updates the default config with the values that will be used
soon on the Atmel / Microchip AT91SAM9G25 based GARDENA smart gateway.

Signed-off-by: Reto Schneider 


Reviewed-by: Stefan Roese 

Thanks,
Stefan


---
This patch applies to current u-boot-atmel/next

  configs/gardena-smart-gateway-at91sam_defconfig | 15 ++-
  1 file changed, 14 insertions(+), 1 deletion(-)

diff --git a/configs/gardena-smart-gateway-at91sam_defconfig 
b/configs/gardena-smart-gateway-at91sam_defconfig
index 40fe0f150e..c626cd699a 100644
--- a/configs/gardena-smart-gateway-at91sam_defconfig
+++ b/configs/gardena-smart-gateway-at91sam_defconfig
@@ -8,6 +8,7 @@ CONFIG_SPL_LIBCOMMON_SUPPORT=y
  CONFIG_SPL_LIBGENERIC_SUPPORT=y
  CONFIG_SYS_MALLOC_F_LEN=0x2000
  CONFIG_NR_DRAM_BANKS=1
+CONFIG_SYS_MEMTEST_START=0x0
  CONFIG_ENV_SIZE=0x1
  CONFIG_DM_GPIO=y
  CONFIG_SPL_TEXT_BASE=0x30
@@ -22,6 +23,8 @@ CONFIG_DEBUG_UART=y
  CONFIG_FIT=y
  CONFIG_NAND_BOOT=y
  CONFIG_BOOTDELAY=0
+CONFIG_AUTOBOOT_KEYED=y
+CONFIG_AUTOBOOT_STOP_STR="x"
  CONFIG_USE_BOOTARGS=y
  CONFIG_BOOTARGS="console=ttyS0,115200 earlyprintk 
mtdparts=atmel_nand:256k(bootstrap)ro,768k(uboot)ro,256k(env_redundant),256k(env),512k(dtb),6M(kernel)ro,-(rootfs)
 rootfstype=ubifs ubi.mtd=6 root=ubi0:rootfs rw"
  CONFIG_SYS_CONSOLE_IS_IN_ENV=y
@@ -32,20 +35,26 @@ CONFIG_SPL_NAND_SUPPORT=y
  CONFIG_SPL_NAND_DRIVERS=y
  CONFIG_SPL_NAND_BASE=y
  CONFIG_HUSH_PARSER=y
+CONFIG_CMD_LICENSE=y
  CONFIG_CMD_BOOTZ=y
+CONFIG_CMD_MEMINFO=y
+CONFIG_CMD_MEMTEST=y
+CONFIG_SYS_ALT_MEMTEST=y
  CONFIG_CMD_DM=y
  # CONFIG_CMD_FLASH is not set
  CONFIG_CMD_GPIO=y
  CONFIG_CMD_MTD=y
  CONFIG_CMD_NAND=y
  CONFIG_CMD_WDT=y
-# CONFIG_CMD_SETEXPR is not set
  CONFIG_CMD_DHCP=y
  CONFIG_CMD_PING=y
  CONFIG_CMD_CACHE=y
  CONFIG_CMD_TIME=y
+CONFIG_CMD_GETTIME=y
+CONFIG_CMD_UUID=y
  CONFIG_CMD_FAT=y
  CONFIG_CMD_MTDPARTS=y
+CONFIG_CMD_MTDPARTS_SHOW_NET_SIZES=y
  CONFIG_MTDIDS_DEFAULT="nand0=atmel_nand"
  CONFIG_MTDPARTS_DEFAULT="atmel_nand:1536k(uboot),10752k(unused),-(ubi)"
  CONFIG_CMD_UBI=y
@@ -58,6 +67,8 @@ CONFIG_ENV_UBI_PART="ubi"
  CONFIG_ENV_UBI_VOLUME="uboot_env0"
  CONFIG_ENV_UBI_VOLUME_REDUND="uboot_env1"
  CONFIG_SYS_RELOC_GD_ENV_ADDR=y
+CONFIG_ENV_VARS_UBOOT_RUNTIME_CONFIG=y
+CONFIG_VERSION_VARIABLE=y
  CONFIG_NET_RANDOM_ETHADDR=y
  CONFIG_DM=y
  CONFIG_SPL_DM=y
@@ -71,6 +82,7 @@ CONFIG_LED_GPIO=y
  CONFIG_MTD=y
  CONFIG_NAND_ATMEL=y
  CONFIG_SPL_GENERATE_ATMEL_PMECC_HEADER=y
+CONFIG_MTD_UBI_FASTMAP=y
  CONFIG_DM_ETH=y
  CONFIG_MACB=y
  CONFIG_PINCTRL=y
@@ -84,3 +96,4 @@ CONFIG_ATMEL_PIT_TIMER=y
  CONFIG_WDT=y
  CONFIG_WDT_AT91=y
  # CONFIG_UBIFS_SILENCE_MSG is not set
+CONFIG_LZMA=y




Viele Grüße,
Stefan

--
DENX Software Engineering GmbH,  Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: s...@denx.de


[PATCH] arm: at91: gardena-smart-gateway-at91sam: Adjust to production values

2021-06-17 Thread Reto Schneider
From: Reto Schneider 

This commit updates the default config with the values that will be used
soon on the Atmel / Microchip AT91SAM9G25 based GARDENA smart gateway.

Signed-off-by: Reto Schneider 
---
This patch applies to current u-boot-atmel/next

 configs/gardena-smart-gateway-at91sam_defconfig | 15 ++-
 1 file changed, 14 insertions(+), 1 deletion(-)

diff --git a/configs/gardena-smart-gateway-at91sam_defconfig 
b/configs/gardena-smart-gateway-at91sam_defconfig
index 40fe0f150e..c626cd699a 100644
--- a/configs/gardena-smart-gateway-at91sam_defconfig
+++ b/configs/gardena-smart-gateway-at91sam_defconfig
@@ -8,6 +8,7 @@ CONFIG_SPL_LIBCOMMON_SUPPORT=y
 CONFIG_SPL_LIBGENERIC_SUPPORT=y
 CONFIG_SYS_MALLOC_F_LEN=0x2000
 CONFIG_NR_DRAM_BANKS=1
+CONFIG_SYS_MEMTEST_START=0x0
 CONFIG_ENV_SIZE=0x1
 CONFIG_DM_GPIO=y
 CONFIG_SPL_TEXT_BASE=0x30
@@ -22,6 +23,8 @@ CONFIG_DEBUG_UART=y
 CONFIG_FIT=y
 CONFIG_NAND_BOOT=y
 CONFIG_BOOTDELAY=0
+CONFIG_AUTOBOOT_KEYED=y
+CONFIG_AUTOBOOT_STOP_STR="x"
 CONFIG_USE_BOOTARGS=y
 CONFIG_BOOTARGS="console=ttyS0,115200 earlyprintk 
mtdparts=atmel_nand:256k(bootstrap)ro,768k(uboot)ro,256k(env_redundant),256k(env),512k(dtb),6M(kernel)ro,-(rootfs)
 rootfstype=ubifs ubi.mtd=6 root=ubi0:rootfs rw"
 CONFIG_SYS_CONSOLE_IS_IN_ENV=y
@@ -32,20 +35,26 @@ CONFIG_SPL_NAND_SUPPORT=y
 CONFIG_SPL_NAND_DRIVERS=y
 CONFIG_SPL_NAND_BASE=y
 CONFIG_HUSH_PARSER=y
+CONFIG_CMD_LICENSE=y
 CONFIG_CMD_BOOTZ=y
+CONFIG_CMD_MEMINFO=y
+CONFIG_CMD_MEMTEST=y
+CONFIG_SYS_ALT_MEMTEST=y
 CONFIG_CMD_DM=y
 # CONFIG_CMD_FLASH is not set
 CONFIG_CMD_GPIO=y
 CONFIG_CMD_MTD=y
 CONFIG_CMD_NAND=y
 CONFIG_CMD_WDT=y
-# CONFIG_CMD_SETEXPR is not set
 CONFIG_CMD_DHCP=y
 CONFIG_CMD_PING=y
 CONFIG_CMD_CACHE=y
 CONFIG_CMD_TIME=y
+CONFIG_CMD_GETTIME=y
+CONFIG_CMD_UUID=y
 CONFIG_CMD_FAT=y
 CONFIG_CMD_MTDPARTS=y
+CONFIG_CMD_MTDPARTS_SHOW_NET_SIZES=y
 CONFIG_MTDIDS_DEFAULT="nand0=atmel_nand"
 CONFIG_MTDPARTS_DEFAULT="atmel_nand:1536k(uboot),10752k(unused),-(ubi)"
 CONFIG_CMD_UBI=y
@@ -58,6 +67,8 @@ CONFIG_ENV_UBI_PART="ubi"
 CONFIG_ENV_UBI_VOLUME="uboot_env0"
 CONFIG_ENV_UBI_VOLUME_REDUND="uboot_env1"
 CONFIG_SYS_RELOC_GD_ENV_ADDR=y
+CONFIG_ENV_VARS_UBOOT_RUNTIME_CONFIG=y
+CONFIG_VERSION_VARIABLE=y
 CONFIG_NET_RANDOM_ETHADDR=y
 CONFIG_DM=y
 CONFIG_SPL_DM=y
@@ -71,6 +82,7 @@ CONFIG_LED_GPIO=y
 CONFIG_MTD=y
 CONFIG_NAND_ATMEL=y
 CONFIG_SPL_GENERATE_ATMEL_PMECC_HEADER=y
+CONFIG_MTD_UBI_FASTMAP=y
 CONFIG_DM_ETH=y
 CONFIG_MACB=y
 CONFIG_PINCTRL=y
@@ -84,3 +96,4 @@ CONFIG_ATMEL_PIT_TIMER=y
 CONFIG_WDT=y
 CONFIG_WDT_AT91=y
 # CONFIG_UBIFS_SILENCE_MSG is not set
+CONFIG_LZMA=y
-- 
2.30.2



Re: [PATCH] arm: at91: gardena-smart-gateway-at91sam: Adjust to production values

2021-04-14 Thread Stefan Roese

Hi Reto,

On 14.04.21 16:49, Reto Schneider wrote:

From: Reto Schneider 

This commit updates the default config with the values that are actually
used "in the wild" and which are close to what is used on the MediaTek
MT7688 based, 2nd generation of the GARDENA smart gateway:
  - Reduce startup time by setting bootdelay to 0 (still allows accessing
the shell, one just has to send a key press quicker)
  - Adjusting U-Boot environment volume names and MTD partitions to
the actual layout

Signed-off-by: Reto Schneider 
---

  configs/gardena-smart-gateway-at91sam_defconfig | 9 +
  1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/configs/gardena-smart-gateway-at91sam_defconfig 
b/configs/gardena-smart-gateway-at91sam_defconfig
index e3d5bc47d6..76a1c42ce0 100644
--- a/configs/gardena-smart-gateway-at91sam_defconfig
+++ b/configs/gardena-smart-gateway-at91sam_defconfig
@@ -24,6 +24,7 @@ CONFIG_NAND_BOOT=y
  CONFIG_BOOTDELAY=3


Please see below...


  CONFIG_USE_BOOTARGS=y
  CONFIG_BOOTARGS="console=ttyS0,115200 earlyprintk 
mtdparts=atmel_nand:256k(bootstrap)ro,768k(uboot)ro,256k(env_redundant),256k(env),512k(dtb),6M(kernel)ro,-(rootfs)
 rootfstype=ubifs ubi.mtd=6 root=ubi0:rootfs rw"
+CONFIG_BOOTDELAY=0


I'm wondering if this defconfig was generated via make savedefconfig? As
there are multiple configurations for BOOTDELAY. Could you please
double-check again?

Other than that:

Reviewed-by: Stefan Roese 

Thanks,
Stefan


  CONFIG_SYS_CONSOLE_IS_IN_ENV=y
  CONFIG_SYS_CONSOLE_INFO_QUIET=y
  CONFIG_SPL_SYS_MALLOC_SIMPLE=y
@@ -46,8 +47,8 @@ CONFIG_CMD_CACHE=y
  CONFIG_CMD_TIME=y
  CONFIG_CMD_FAT=y
  CONFIG_CMD_MTDPARTS=y
-CONFIG_MTDIDS_DEFAULT="nand0=nand0"
-CONFIG_MTDPARTS_DEFAULT="nand0:1536k(uboot),1024k(unused),512k(dtb_old),4608k(kernel_old),86528k(ubi),-(rootfs_old)"
+CONFIG_MTDIDS_DEFAULT="nand0=atmel_nand"
+CONFIG_MTDPARTS_DEFAULT="atmel_nand:1536k(uboot),10752k(unused),-(ubi)"
  CONFIG_CMD_UBI=y
  CONFIG_OF_CONTROL=y
  CONFIG_SPL_OF_CONTROL=y
@@ -55,8 +56,8 @@ CONFIG_OF_SPL_REMOVE_PROPS="pinctrl-0 pinctrl-names clocks 
clock-names interrupt
  CONFIG_ENV_IS_IN_UBI=y
  CONFIG_SYS_REDUNDAND_ENVIRONMENT=y
  CONFIG_ENV_UBI_PART="ubi"
-CONFIG_ENV_UBI_VOLUME="env"
-CONFIG_ENV_UBI_VOLUME_REDUND="env_r"
+CONFIG_ENV_UBI_VOLUME="uboot_env0"
+CONFIG_ENV_UBI_VOLUME_REDUND="uboot_env1"
  CONFIG_SYS_RELOC_GD_ENV_ADDR=y
  CONFIG_NET_RANDOM_ETHADDR=y
  CONFIG_DM=y




Viele Grüße,
Stefan

--
DENX Software Engineering GmbH,  Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: s...@denx.de


[PATCH] arm: at91: gardena-smart-gateway-at91sam: Adjust to production values

2021-04-14 Thread Reto Schneider
From: Reto Schneider 

This commit updates the default config with the values that are actually
used "in the wild" and which are close to what is used on the MediaTek
MT7688 based, 2nd generation of the GARDENA smart gateway:
 - Reduce startup time by setting bootdelay to 0 (still allows accessing
   the shell, one just has to send a key press quicker)
 - Adjusting U-Boot environment volume names and MTD partitions to
   the actual layout

Signed-off-by: Reto Schneider 
---

 configs/gardena-smart-gateway-at91sam_defconfig | 9 +
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/configs/gardena-smart-gateway-at91sam_defconfig 
b/configs/gardena-smart-gateway-at91sam_defconfig
index e3d5bc47d6..76a1c42ce0 100644
--- a/configs/gardena-smart-gateway-at91sam_defconfig
+++ b/configs/gardena-smart-gateway-at91sam_defconfig
@@ -24,6 +24,7 @@ CONFIG_NAND_BOOT=y
 CONFIG_BOOTDELAY=3
 CONFIG_USE_BOOTARGS=y
 CONFIG_BOOTARGS="console=ttyS0,115200 earlyprintk 
mtdparts=atmel_nand:256k(bootstrap)ro,768k(uboot)ro,256k(env_redundant),256k(env),512k(dtb),6M(kernel)ro,-(rootfs)
 rootfstype=ubifs ubi.mtd=6 root=ubi0:rootfs rw"
+CONFIG_BOOTDELAY=0
 CONFIG_SYS_CONSOLE_IS_IN_ENV=y
 CONFIG_SYS_CONSOLE_INFO_QUIET=y
 CONFIG_SPL_SYS_MALLOC_SIMPLE=y
@@ -46,8 +47,8 @@ CONFIG_CMD_CACHE=y
 CONFIG_CMD_TIME=y
 CONFIG_CMD_FAT=y
 CONFIG_CMD_MTDPARTS=y
-CONFIG_MTDIDS_DEFAULT="nand0=nand0"
-CONFIG_MTDPARTS_DEFAULT="nand0:1536k(uboot),1024k(unused),512k(dtb_old),4608k(kernel_old),86528k(ubi),-(rootfs_old)"
+CONFIG_MTDIDS_DEFAULT="nand0=atmel_nand"
+CONFIG_MTDPARTS_DEFAULT="atmel_nand:1536k(uboot),10752k(unused),-(ubi)"
 CONFIG_CMD_UBI=y
 CONFIG_OF_CONTROL=y
 CONFIG_SPL_OF_CONTROL=y
@@ -55,8 +56,8 @@ CONFIG_OF_SPL_REMOVE_PROPS="pinctrl-0 pinctrl-names clocks 
clock-names interrupt
 CONFIG_ENV_IS_IN_UBI=y
 CONFIG_SYS_REDUNDAND_ENVIRONMENT=y
 CONFIG_ENV_UBI_PART="ubi"
-CONFIG_ENV_UBI_VOLUME="env"
-CONFIG_ENV_UBI_VOLUME_REDUND="env_r"
+CONFIG_ENV_UBI_VOLUME="uboot_env0"
+CONFIG_ENV_UBI_VOLUME_REDUND="uboot_env1"
 CONFIG_SYS_RELOC_GD_ENV_ADDR=y
 CONFIG_NET_RANDOM_ETHADDR=y
 CONFIG_DM=y
-- 
2.29.2