[U-Boot] [PATCH v4 08/12] samsung: board: enable support of multiple board types
This change adds declaration of functions: - set_board_type() - called at checkboard() - get_board_type() - called at checkboard() - get_board_name() For supporting multiple board types in a one config - it is welcome to display the current board model. This is what get_board_type() should return. Signed-off-by: Przemyslaw Marczak Cc: Piotr Wilczek Cc: Minkyu Kang --- Changes v3: - change the commit order - include/samsung/misc.h: add new functions declaration - update a commit message Changes v4: - checkboard(): remove redundant variable --- board/samsung/common/board.c | 12 +--- include/samsung/misc.h | 5 + 2 files changed, 14 insertions(+), 3 deletions(-) diff --git a/board/samsung/common/board.c b/board/samsung/common/board.c index a46e87b..3e27130 100644 --- a/board/samsung/common/board.c +++ b/board/samsung/common/board.c @@ -280,11 +280,17 @@ int board_mmc_init(bd_t *bis) #ifdef CONFIG_DISPLAY_BOARDINFO int checkboard(void) { - const char *board_name; + const char *board_info; - board_name = fdt_getprop(gd->fdt_blob, 0, "model", NULL); - printf("Board: %s\n", board_name ? board_name : "unknown"); + board_info = fdt_getprop(gd->fdt_blob, 0, "model", NULL); + printf("Board: %s\n", board_info ? board_info : "unknown"); +#ifdef CONFIG_BOARD_TYPES + set_board_type(); + board_info = get_board_type(); + + printf("Model: %s\n", board_info ? board_info : "unknown"); +#endif return 0; } #endif diff --git a/include/samsung/misc.h b/include/samsung/misc.h index e82bf32..cc41997 100644 --- a/include/samsung/misc.h +++ b/include/samsung/misc.h @@ -33,5 +33,10 @@ char *get_dfu_alt_system(void); char *get_dfu_alt_boot(void); void set_dfu_alt_info(void); #endif +#ifdef CONFIG_BOARD_TYPES +void set_board_type(void); +const char *get_board_type(void); +const char *get_board_name(void); +#endif #endif /* __SAMSUNG_MISC_COMMON_H__ */ -- 1.9.1 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v4 08/12] samsung: board: enable support of multiple board types
On 07/02/2014 08:50 PM, Przemyslaw Marczak wrote: > This change adds declaration of functions: > - set_board_type() - called at checkboard() > - get_board_type() - called at checkboard() > - get_board_name() > > For supporting multiple board types in a one config - it is welcome > to display the current board model. This is what get_board_type() > should return. > > Signed-off-by: Przemyslaw Marczak > Cc: Piotr Wilczek > Cc: Minkyu Kang > > --- > Changes v3: > - change the commit order > - include/samsung/misc.h: add new functions declaration > - update a commit message > > Changes v4: > - checkboard(): remove redundant variable > --- > board/samsung/common/board.c | 12 +--- > include/samsung/misc.h | 5 + > 2 files changed, 14 insertions(+), 3 deletions(-) > > diff --git a/board/samsung/common/board.c b/board/samsung/common/board.c > index a46e87b..3e27130 100644 > --- a/board/samsung/common/board.c > +++ b/board/samsung/common/board.c > @@ -280,11 +280,17 @@ int board_mmc_init(bd_t *bis) > #ifdef CONFIG_DISPLAY_BOARDINFO > int checkboard(void) > { > - const char *board_name; > + const char *board_info; Is there a reason that it's renaming? Best Regards, Jaehoon Chung > > - board_name = fdt_getprop(gd->fdt_blob, 0, "model", NULL); > - printf("Board: %s\n", board_name ? board_name : "unknown"); > + board_info = fdt_getprop(gd->fdt_blob, 0, "model", NULL); > + printf("Board: %s\n", board_info ? board_info : "unknown"); > +#ifdef CONFIG_BOARD_TYPES > + set_board_type(); > > + board_info = get_board_type(); > + > + printf("Model: %s\n", board_info ? board_info : "unknown"); > +#endif > return 0; > } > #endif > diff --git a/include/samsung/misc.h b/include/samsung/misc.h > index e82bf32..cc41997 100644 > --- a/include/samsung/misc.h > +++ b/include/samsung/misc.h > @@ -33,5 +33,10 @@ char *get_dfu_alt_system(void); > char *get_dfu_alt_boot(void); > void set_dfu_alt_info(void); > #endif > +#ifdef CONFIG_BOARD_TYPES > +void set_board_type(void); > +const char *get_board_type(void); > +const char *get_board_name(void); > +#endif > > #endif /* __SAMSUNG_MISC_COMMON_H__ */ > ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v4 08/12] samsung: board: enable support of multiple board types
On 07/04/2014 07:49 AM, Jaehoon Chung wrote: On 07/02/2014 08:50 PM, Przemyslaw Marczak wrote: This change adds declaration of functions: - set_board_type() - called at checkboard() - get_board_type() - called at checkboard() - get_board_name() For supporting multiple board types in a one config - it is welcome to display the current board model. This is what get_board_type() should return. Signed-off-by: Przemyslaw Marczak Cc: Piotr Wilczek Cc: Minkyu Kang --- Changes v3: - change the commit order - include/samsung/misc.h: add new functions declaration - update a commit message Changes v4: - checkboard(): remove redundant variable --- board/samsung/common/board.c | 12 +--- include/samsung/misc.h | 5 + 2 files changed, 14 insertions(+), 3 deletions(-) diff --git a/board/samsung/common/board.c b/board/samsung/common/board.c index a46e87b..3e27130 100644 --- a/board/samsung/common/board.c +++ b/board/samsung/common/board.c @@ -280,11 +280,17 @@ int board_mmc_init(bd_t *bis) #ifdef CONFIG_DISPLAY_BOARDINFO int checkboard(void) { - const char *board_name; + const char *board_info; Is there a reason that it's renaming? Best Regards, Jaehoon Chung Yes, I think that the board_info is more suitable as a common for "name" and "type". - board_name = fdt_getprop(gd->fdt_blob, 0, "model", NULL); - printf("Board: %s\n", board_name ? board_name : "unknown"); + board_info = fdt_getprop(gd->fdt_blob, 0, "model", NULL); + printf("Board: %s\n", board_info ? board_info : "unknown"); +#ifdef CONFIG_BOARD_TYPES + set_board_type(); + board_info = get_board_type(); + + printf("Model: %s\n", board_info ? board_info : "unknown"); +#endif return 0; } #endif diff --git a/include/samsung/misc.h b/include/samsung/misc.h index e82bf32..cc41997 100644 --- a/include/samsung/misc.h +++ b/include/samsung/misc.h @@ -33,5 +33,10 @@ char *get_dfu_alt_system(void); char *get_dfu_alt_boot(void); void set_dfu_alt_info(void); #endif +#ifdef CONFIG_BOARD_TYPES +void set_board_type(void); +const char *get_board_type(void); +const char *get_board_name(void); +#endif #endif /* __SAMSUNG_MISC_COMMON_H__ */ Thanks, -- Przemyslaw Marczak Samsung R&D Institute Poland Samsung Electronics p.marc...@samsung.com ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot