rockchip: move CONFIG_ENV_SIZE and CONFIG_ENV_OFFSET to Kconfig
authorKlaus Goger <klaus.goger@theobroma-systems.com>
Mon, 11 Dec 2017 16:56:08 +0000 (17:56 +0100)
committerPhilipp Tomsich <philipp.tomsich@theobroma-systems.com>
Sun, 17 Dec 2017 23:26:16 +0000 (00:26 +0100)
This commit adds ENV_SIZE and ENV_OFFSET configuration items for
ARCH_ROCKCHIP, but keeps these non-visible (i.e. not prompt is given).
With these new items present, the configuration from the header files
is moved to Kconfig.

Keeping these non-visible is necessary to have the possibility to
select new default values if CONFIG_IS_IN_* is changed (interactively
or with oldconfig). Otherwise it will always be set to a previous
value if used with a prompt.  As an example if we do a defconfig with
CONFIG_IS_IN_MMC and change it to CONFIG_IS_IN_SPI_FLASH via
menuconfig, ENV_SIZE and ENV_OFFSET will not be changed to the correct
values as defconfig will already have set them to the default values
of CONFIG_IS_IN_MMC in .config.

Signed-off-by: Klaus Goger <klaus.goger@theobroma-systems.com>
Reviewed-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
board/theobroma-systems/puma_rk3399/Kconfig
env/Kconfig
include/configs/puma_rk3399.h
include/configs/rockchip-common.h

index a645590d7874fbdd77263ffec57795484db25e6a..80b3460d4cd5a6369f53d87057f98ef9a7225866 100644 (file)
@@ -12,4 +12,10 @@ config SYS_CONFIG_NAME
 config BOARD_SPECIFIC_OPTIONS # dummy
        def_bool y
 
+config ENV_SIZE
+       default 0x2000 if ENV_IS_IN_SPI_FLASH
+
+config ENV_OFFSET
+       default 0x3c000 if ENV_IS_IN_SPI_FLASH
+
 endif
index 2477bf85309b20d6aba37ce72b34d7584154a5fc..bef6e89bfc3cc670077c5fb4cd80cbf5e29974bb 100644 (file)
@@ -427,4 +427,22 @@ 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
+
 endmenu
index 39d07862669ac773a2785e51d21a04b2d4e24b7e..6523a7c34a64b6706873a4450f3c07f7f766bcc9 100644 (file)
@@ -9,14 +9,6 @@
 
 #include <configs/rk3399_common.h>
 
-/*
- * SPL @ 32kB for ~130kB
- * ENV @ 240KB for 8kB
- * FIT payload (ATF, U-Boot, FDT) @ 256kB
- */
-#undef CONFIG_ENV_OFFSET
-#define CONFIG_ENV_OFFSET (240 * 1024)
-
 #if defined(CONFIG_ENV_IS_IN_MMC)
 #define CONFIG_SYS_MMC_ENV_DEV 1
 #elif defined(CONFIG_ENV_IS_IN_SPI_FLASH)
index 4139a7b73d340f758cefd4e0deeeb883acc300f5..f465d35330bc1e0b87e8a858e6bdc404968b5e36 100644 (file)
 
 #endif
 
-/*
- * Rockchip SoCs use fixed ENV 32KB@(4MB-32KB)
- */
-#define CONFIG_ENV_OFFSET      (SZ_4M - SZ_32K)
-#define CONFIG_ENV_SIZE                SZ_32K
-
 #define CONFIG_DISPLAY_BOARDINFO_LATE
 
 #endif /* _ROCKCHIP_COMMON_H_ */