Hi Masahiro,
On 27 November 2014 at 10:03, Masahiro Yamada yamada.m@jp.panasonic.com wrote:
The common/board_r.c has show_model_r() to display the model name if the DTB has a "model" property. It sounds useful to have a similar function in common/board_f.c too because most of the boards show their board name before relocation.
Instead of implementing the same function in both common/board_f.c and common/board_r.c, let's split it up into common/show_board_info.c.
Signed-off-by: Masahiro Yamada yamada.m@jp.panasonic.com
common/Makefile | 2 ++ common/board_f.c | 2 +- common/board_info.c | 34 ++++++++++++++++++++++++++++++++++ common/board_r.c | 18 +----------------- include/common.h | 13 +++++++------ 5 files changed, 45 insertions(+), 24 deletions(-) create mode 100644 common/board_info.c
This is a change of behaviour in show_model_r() but I think it is OK.
Acked-by: Simon Glass sjg@chromium.org
I'd also suggest adding a comment to checkboard() in the header file - it is only allowed to print the board name, and cannot do any init, etc. That way we can retain the flexibility to drop it one day.
Regards, Simon