[U-Boot] [PATCH] cmd_mtdparts.c: allow to omit definitions for default settings

There is actually no good reason to enforce that all board configuations must define default settings for "mtdids" and "mtdparts". Actually this may be difficult to handle, especially on boards where different sizes of flash chips can be fit, so there is no real "default" partition map for all boards.
Lift this arbitrary limitation.
Signed-off-by: Wolfgang Denk wd@denx.de --- common/cmd_mtdparts.c | 8 ++++---- 1 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/common/cmd_mtdparts.c b/common/cmd_mtdparts.c index e7b6acc..bb07fde 100644 --- a/common/cmd_mtdparts.c +++ b/common/cmd_mtdparts.c @@ -137,14 +137,12 @@ #if defined(MTDIDS_DEFAULT) static const char *const mtdids_default = MTDIDS_DEFAULT; #else -#warning "MTDIDS_DEFAULT not defined!" static const char *const mtdids_default = NULL; #endif
#if defined(MTDPARTS_DEFAULT) static const char *const mtdparts_default = MTDPARTS_DEFAULT; #else -#warning "MTDPARTS_DEFAULT not defined!" static const char *const mtdparts_default = NULL; #endif
@@ -1344,8 +1342,10 @@ static void list_partitions(void) }
printf("\ndefaults:\n"); - printf("mtdids : %s\n", mtdids_default); - printf("mtdparts: %s\n", mtdparts_default); + printf("mtdids : %s\n", + mtdids_default ? mtdids_default : "none"); + printf("mtdparts: %s\n", + mtdparts_default ? mtdparts_default : "none"); }
/**

Hello,
In message 1242568914-21978-1-git-send-email-wd@denx.de you wrote:
There is actually no good reason to enforce that all board configuations must define default settings for "mtdids" and "mtdparts". Actually this may be difficult to handle, especially on boards where different sizes of flash chips can be fit, so there is no real "default" partition map for all boards.
Lift this arbitrary limitation.
Signed-off-by: Wolfgang Denk wd@denx.de
common/cmd_mtdparts.c | 8 ++++---- 1 files changed, 4 insertions(+), 4 deletions(-)
Applied to next.
Best regards,
Wolfgang Denk
participants (1)
-
Wolfgang Denk