Re: [U-Boot] [PATCH 8/9] board: cm_fx6: use PRIx64 macros for printing ram size

2018-07-26 22:59 GMT+09:00 Philipp Tomsich philipp.tomsich@theobroma-systems.com:

With the recent changes of the underlying types for the ram size, we need to adjust the formatting. Use PRIx64 to print the (now) u64 field.

Signed-off-by: Philipp Tomsich philipp.tomsich@theobroma-systems.com

As I commented in 3/9, using PRIx64 is wrong.

board/compulab/cm_fx6/cm_fx6.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/board/compulab/cm_fx6/cm_fx6.c b/board/compulab/cm_fx6/cm_fx6.c index c114cdc..b163abf 100644 --- a/board/compulab/cm_fx6/cm_fx6.c +++ b/board/compulab/cm_fx6/cm_fx6.c @@ -704,7 +704,8 @@ int dram_init(void) gd->ram_size -= 0x100000; break; default:

•           printf("ERROR: Unsupported DRAM size 0x%lx\n", gd->ram_size);
  

•           printf("ERROR: Unsupported DRAM size 0x%" PRIx64 "\n",
  

•                  gd->ram_size);
            return -1;
    }
  
  

-- 2.1.4

---

U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot