Re: [U-Boot-Users] [PATCH] bdinfo not showing correct info about memory size

In message 1217414549-28700-1-git-send-email-ricardo.ribalda@uam.es you wrote:

Signed-off-by: Ricardo Ribalda Delgado ricardo.ribalda@uam.es

common/cmd_bdinfo.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/common/cmd_bdinfo.c b/common/cmd_bdinfo.c index caa467d..583bbc2 100644 --- a/common/cmd_bdinfo.c +++ b/common/cmd_bdinfo.c @@ -48,7 +48,7 @@ int do_bdinfo ( cmd_tbl_t *cmdtp, int flag, int argc, char *argv[]) print_num ("bd address", (ulong)bd ); #endif print_num ("memstart", bd->bi_memstart );

• print_lnum ("memsize", bd->bi_memsize );

• print_num ("memsize", bd->bi_memsize ); print_num ("flashstart", bd->bi_flashstart ); print_num ("flashsize", bd->bi_flashsize ); print_num ("flashoffset", bd->bi_flashoffset );

This patch is wrong. print_num() takes an ulong argument, but bi_memsize has type phys_size_t.

Rejected.

Best regards,

Wolfgang Denk