There is no reason to have the same Kconfig options for different SoCs
separately. The patch is merging them together.

Signed-off-by: Michal Simek <michal.si...@xilinx.com>
---

Patch is based on
https://lists.denx.de/pipermail/u-boot/2018-July/335126.html

I have ENV_SECT_SIZE just for zynq/zynqmp because rockchip and sunxi
have this in their configs. When they decide to move then can enable
that option for them too.
I expect when more platforms extend this we will have less constrain
Kconfig setup.

---
 env/Kconfig | 66 ++++++++++++++++---------------------------------------------
 1 file changed, 17 insertions(+), 49 deletions(-)

diff --git a/env/Kconfig b/env/Kconfig
index b37dcd78eb75..0ded003d7d41 100644
--- a/env/Kconfig
+++ b/env/Kconfig
@@ -431,23 +431,37 @@ config ENV_EXT4_FILE
          It's a string of the EXT4 file name. This file use to store the
          environment (explicit path to the file)
 
-if ARCH_SUNXI
+if ARCH_ROCKCHIP || ARCH_SUNXI || ARCH_ZYNQ || ARCH_ZYNQMP
 
 config ENV_OFFSET
        hex "Environment Offset"
        depends on !ENV_IS_IN_UBI
        depends on !ENV_IS_NOWHERE
+       default 0x3f8000 if ARCH_ROCKCHIP
        default 0x88000 if ARCH_SUNXI
+       default 0xE0000 if ARCH_ZYNQ
+       default 0x1E00000 if ARCH_ZYNQMP
        help
          Offset from the start of the device (or partition)
 
 config ENV_SIZE
        hex "Environment Size"
-       depends on !ENV_IS_NOWHERE
-       default 0x20000 if ARCH_SUNXI
+       default 0x8000 if ARCH_ROCKCHIP && !ENV_IS_NOWHERE
+       default 0x20000 if ARCH_SUNXI && !ENV_IS_NOWHERE
+       default 0x20000 if ARCH_ZYNQ
+       default 0x40000 if ENV_IS_IN_SPI_FLASH && ARCH_ZYNQMP
+       default 0x8000 if ARCH_ZYNQMP
        help
          Size of the environment storage area
 
+config ENV_SECT_SIZE
+       hex "Environment Sector-Size"
+       depends on !ENV_IS_NOWHERE && (ARCH_ZYNQ || ARCH_ZYNQMP)
+       default 0x40000 if ARCH_ZYNQMP
+       default 0x20000 if ARCH_ZYNQ
+       help
+         Size of the sector containing the environment.
+
 config ENV_UBI_PART
        string "UBI partition name"
        depends on ENV_IS_IN_UBI
@@ -462,52 +476,6 @@ config ENV_UBI_VOLUME
 
 endif
 
-if ARCH_ROCKCHIP
-
-config ENV_OFFSET
-       hex
-       depends on !ENV_IS_IN_UBI
-       depends on !ENV_IS_NOWHERE
-       default 0x3f8000
-       help
-         Offset from the start of the device (or partition)
-
-config ENV_SIZE
-       hex
-       default 0x8000
-       help
-         Size of the environment storage area
-
-endif
-
-if ARCH_ZYNQMP || ARCH_ZYNQ
-
-config ENV_OFFSET
-       hex "Environment Offset"
-       depends on !ENV_IS_NOWHERE
-       default 0x1E00000 if ARCH_ZYNQMP
-       default 0xE0000 if ARCH_ZYNQ
-       help
-         Offset from the start of the device (or partition)
-
-config ENV_SIZE
-       hex "Environment Size"
-       default 0x40000 if ENV_IS_IN_SPI_FLASH && ARCH_ZYNQMP
-       default 0x8000 if ARCH_ZYNQMP
-       default 0x20000 if ARCH_ZYNQ
-       help
-         Size of the environment storage area.
-
-config ENV_SECT_SIZE
-       hex "Environment Sector-Size"
-       depends on !ENV_IS_NOWHERE
-       default 0x40000 if ARCH_ZYNQMP
-       default 0x20000 if ARCH_ZYNQ
-       help
-         Size of the sector containing the environment.
-
-endif
-
 config USE_DEFAULT_ENV_FILE
        bool "Create default environment from file"
        help
-- 
1.9.1

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to