configs: remove redundant comment sections
PATCH 1/2 configs: remove redundant comment sections
mv-common.h and mv-plug-common.h still had comments delimiting sections of configuration options that have all been moved to Kconfig by previous treewide efforts. Remove the redundant comment sections.
---
include/configs/mv-common.h | 4 ---- include/configs/mv-plug-common.h | 8 -------- 2 files changed, 12 deletions(-)
include/configs/mv-common.h +0 -4
@@ -28,10 +28,6 @@/* additions for new ARM relocation support */ #define CONFIG_SYS_SDRAM_BASE 0x00000000 -/* -* CLKs configurations -*/ -/* * NS16550 Configuration */
include/configs/mv-plug-common.h +0 -8
@@ -23,14 +23,6 @@#define CONFIG_BZIP2 #endif /* CONFIG_SYS_MVFS */ -/* -* Commands configuration -*/ - -/* -* Extra file system -*/ -/* * mv-common.h should be defined after CMD configs since it used them * to enable certain macros
Signature · 1 line
2.17.0
PATCH 2/2 configs: remove CONFIG_SYS_MVFS
This was being used by some Marvell boards to enable some file system related features (many of which have already been moved to Kconfig). Make the future migration of the final 2 or 3 config options easier by expanding #define CONFIG_SYS_MVFS into the options that it enables and remove CONFIG_SYS_MVFS.
---
README | 5 ----- include/configs/dns325.h | 3 ++- include/configs/ds414.h | 3 ++- include/configs/goflexhome.h | 3 ++- include/configs/guruplug.h | 4 +++- include/configs/ib62x0.h | 3 ++- include/configs/iconnect.h | 3 ++- include/configs/mv-common.h | 8 -------- include/configs/mv-plug-common.h | 7 ------- include/configs/nsa310s.h | 3 ++- include/configs/openrd.h | 3 ++- include/configs/pogo_e02.h | 3 ++- include/configs/sheevaplug.h | 4 +++- scripts/config_whitelist.txt | 1 - 14 files changed, 22 insertions(+), 31 deletions(-)
README +0 -5
@@ -331,11 +331,6 @@ The following options need to be configured: - Board Type: Define exactly one, e.g. CONFIG_MPC8540ADS. -- Marvell Family Member - CONFIG_SYS_MVFS - define it if you want to enable - multiple fs option at one time - for marvell soc family - - 85xx CPU Options: CONFIG_SYS_PPC64
include/configs/dns325.h +2 -1
@@ -27,7 +27,8 @@/* * Commands configuration */ -#define CONFIG_SYS_MVFS +#define CONFIG_MTD_DEVICE /* needed for mtdparts commands */ +#define CONFIG_MTD_PARTITIONS #define CONFIG_NR_DRAM_BANKS 1
include/configs/ds414.h +2 -1
@@ -61,7 +61,8 @@#endif /* why is this only defined in mv-common.h if CONFIG_DM is undefined? */ -#define CONFIG_SYS_MVFS +#define CONFIG_MTD_DEVICE /* needed for mtdparts commands */ +#define CONFIG_MTD_PARTITIONS /* * mv-common.h should be defined after CMD configs since it used them
include/configs/goflexhome.h +2 -1
@@ -41,7 +41,8 @@* Commands configuration */ -#define CONFIG_SYS_MVFS /* Picks up Filesystem from mv-common.h */ +#define CONFIG_MTD_DEVICE /* needed for mtdparts commands */ +#define CONFIG_MTD_PARTITIONS /* * mv-common.h should be defined after CMD configs since it used them
include/configs/guruplug.h +3 -1
@@ -17,7 +17,9 @@/* * Standard filesystems */ -#define CONFIG_SYS_MVFS +#define CONFIG_BZIP2 +#define CONFIG_MTD_DEVICE /* needed for mtdparts commands */ +#define CONFIG_MTD_PARTITIONS /* * mv-plug-common.h should be defined after CMD configs since it used them
include/configs/ib62x0.h +2 -1
@@ -26,7 +26,8 @@/* * Commands configuration */ -#define CONFIG_SYS_MVFS +#define CONFIG_MTD_DEVICE /* needed for mtdparts commands */ +#define CONFIG_MTD_PARTITIONS /* * mv-common.h should be defined after CMD configs since it used them
include/configs/iconnect.h +2 -1
@@ -28,7 +28,8 @@/* * Commands configuration */ -#define CONFIG_SYS_MVFS +#define CONFIG_MTD_DEVICE /* needed for mtdparts commands */ +#define CONFIG_MTD_PARTITIONS /* * mv-common.h should be defined after CMD configs since it used them
include/configs/mv-common.h +0 -8
@@ -97,12 +97,4 @@#define CONFIG_SYS_MAX_NAND_DEVICE 1#endif -/* -* File system -*/ -#ifdef CONFIG_SYS_MVFS -#define CONFIG_MTD_DEVICE /* needed for mtdparts commands */ -#define CONFIG_MTD_PARTITIONS -#endif -#endif /* _MV_COMMON_H */
include/configs/mv-plug-common.h +0 -7
@@ -16,13 +16,6 @@/* Add target to build it automatically upon "make" */ #define CONFIG_BUILD_TARGET "u-boot.kwb" -/* -* Compression configuration -*/ -#ifdef CONFIG_SYS_MVFS -#define CONFIG_BZIP2 -#endif /* CONFIG_SYS_MVFS */ -/* * mv-common.h should be defined after CMD configs since it used them * to enable certain macros
include/configs/nsa310s.h +2 -1
@@ -22,7 +22,8 @@#define CONFIG_BZIP2 /* commands configuration */ -#define CONFIG_SYS_MVFS +#define CONFIG_MTD_DEVICE /* needed for mtdparts commands */ +#define CONFIG_MTD_PARTITIONS /* * mv-common.h should be defined after CMD configs since it used them
include/configs/openrd.h +2 -1
@@ -23,7 +23,8 @@/* * Commands configuration */ -#define CONFIG_SYS_MVFS +#define CONFIG_MTD_DEVICE /* needed for mtdparts commands */ +#define CONFIG_MTD_PARTITIONS /* * mv-common.h should be defined after CMD configs since it used them
include/configs/pogo_e02.h +2 -1
@@ -27,7 +27,8 @@/* * Commands configuration */ -#define CONFIG_SYS_MVFS +#define CONFIG_MTD_DEVICE /* needed for mtdparts commands */ +#define CONFIG_MTD_PARTITIONS /* * mv-common.h should be defined after CMD configs since it used them
include/configs/sheevaplug.h +3 -1
@@ -21,7 +21,9 @@/* * Standard filesystems */ -#define CONFIG_SYS_MVFS +#define CONFIG_BZIP2 +#define CONFIG_MTD_DEVICE /* needed for mtdparts commands */ +#define CONFIG_MTD_PARTITIONS /* * mv-plug-common.h should be defined after CMD configs since it used them
scripts/config_whitelist.txt +0 -1
@@ -3578,7 +3578,6 @@ CONFIG_SYS_MSC0_VAL CONFIG_SYS_MSC1_VAL CONFIG_SYS_MSC2_VAL CONFIG_SYS_MTDPARTS_RUNTIME -CONFIG_SYS_MVFS CONFIG_SYS_MX5_CLK32 CONFIG_SYS_MX5_HCLK CONFIG_SYS_MX6_CLK32
Signature · 1 line
2.17.0
PATCH 2/2 configs: remove CONFIG_SYS_MVFS
Quoted 9 lines · Chris Packham
This was being used by some Marvell boards to enable some file system related features (many of which have already been moved to
Kconfig). Make the future migration of the final 2 or 3 config options easier by expanding #defineCONFIG_SYS_MVFSinto the options that it enables and removeCONFIG_SYS_MVFS.Signed-off-by Chris Packham <[email protected]>---
Quoted 15 lines
README | 5 ----- include/configs/dns325.h | 3 ++- include/configs/ds414.h | 3 ++- include/configs/goflexhome.h | 3 ++- include/configs/guruplug.h | 4 +++- include/configs/ib62x0.h | 3 ++- include/configs/iconnect.h | 3 ++- include/configs/mv-common.h | 8 -------- include/configs/mv-plug-common.h | 7 ------- include/configs/nsa310s.h | 3 ++- include/configs/openrd.h | 3 ++- include/configs/pogo_e02.h | 3 ++- include/configs/sheevaplug.h | 4 +++- scripts/config_whitelist.txt | 1 - 14 files changed, 22 insertions(+), 31 deletions(-)
PATCH 2/2 configs: remove CONFIG_SYS_MVFS
In reply to Chris PackhamQuoted 9 lines · Chris Packham
This was being used by some Marvell boards to enable some file system related features (many of which have already been moved to
Kconfig). Make the future migration of the final 2 or 3 config options easier by expanding #defineCONFIG_SYS_MVFSinto the options that it enables and removeCONFIG_SYS_MVFS.Signed-off-by Chris Packham <[email protected]>---
Quoted 15 lines
README | 5 ----- include/configs/dns325.h | 3 ++- include/configs/ds414.h | 3 ++- include/configs/goflexhome.h | 3 ++- include/configs/guruplug.h | 4 +++- include/configs/ib62x0.h | 3 ++- include/configs/iconnect.h | 3 ++- include/configs/mv-common.h | 8 -------- include/configs/mv-plug-common.h | 7 ------- include/configs/nsa310s.h | 3 ++- include/configs/openrd.h | 3 ++- include/configs/pogo_e02.h | 3 ++- include/configs/sheevaplug.h | 4 +++- scripts/config_whitelist.txt | 1 - 14 files changed, 22 insertions(+), 31 deletions(-)
Thanks,
Stefan
PATCH 2/2 configs: remove CONFIG_SYS_MVFS
In reply to Chris PackhamQuoted 7 lines · Chris Packham
This was being used by some Marvell boards to enable some file system related features (many of which have already been moved to
Kconfig). Make the future migration of the final 2 or 3 config options easier by expanding #defineCONFIG_SYS_MVFSinto the options that it enables and removeCONFIG_SYS_MVFS.Signed-off-by Chris Packham <[email protected]>
Applied to u-boot-marvell/master
Thanks,
Stefan
PATCH 1/2 configs: remove redundant comment sections
In reply to Chris PackhamQuoted 36 lines · Chris Packham
mv-common.handmv-plug-common.hstill had comments delimiting sections of configuration options that have all been moved toKconfigby previous treewide efforts. Remove the redundant comment sections.Signed-off-by Chris Packham <[email protected]>---
include/configs/mv-common.h | 4 ---- include/configs/mv-plug-common.h | 8 -------- 2 files changed, 12 deletions(-)include/configs/mv-common.h +0
-4@@ -28,10 +28,6 @@/* additions for new ARM relocation support */ #define CONFIG_SYS_SDRAM_BASE 0x00000000 -/* -* CLKs configurations -*/ -/* * NS16550 Configuration */ include/configs/mv-plug-common.h +0
-1@@ -23,14 +23,6 @@#define CONFIG_BZIP2 #endif /* CONFIG_SYS_MVFS */ -/*
include/configs/mv-plug-common.h line 28
Quoted 10 lines
include/configs/mv-plug-common.h +0
-7-* Commands configuration -*/ - -/* -* Extra file system -*/ -/* * mv-common.h should be defined after CMD configs since it used them * to enable certain macros
Thanks,
Stefan
PATCH 1/2 configs: remove redundant comment sections
In reply to Chris PackhamQuoted 5 lines · Chris Packham
mv-common.handmv-plug-common.hstill had comments delimiting sections of configuration options that have all been moved toKconfigby previous treewide efforts. Remove the redundant comment sections.Signed-off-by Chris Packham <[email protected]>
Applied to u-boot-marvell/master
Thanks,
Stefan