Re: [PATCH 6/7] ddr: altera: arria10: Change %i to %u for printf

On 4/15/20 11:00 AM, Ley Foon Tan wrote:

Tiny printf doesn't support %i, change to %u.

Signed-off-by: Ley Foon Tan ley.foon.tan@intel.com

drivers/ddr/altera/sdram_arria10.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/ddr/altera/sdram_arria10.c b/drivers/ddr/altera/sdram_arria10.c index e3f11984a978..8acf324117af 100644 --- a/drivers/ddr/altera/sdram_arria10.c +++ b/drivers/ddr/altera/sdram_arria10.c @@ -195,7 +195,7 @@ static void sdram_init_ecc_bits(u32 size)

dcache_enable();

• printf("DDRCAL: Scrubbing ECC RAM (%i MiB).\n", size >> 20);

• printf("DDRCAL: Scrubbing ECC RAM (%u MiB).\n", size >> 20); memset((void *)0x8000, 0, size - 0x8000); flush_dcache_all(); printf("DDRCAL: Scrubbing ECC RAM done.\n");

Yes, sadly, tiny printf is broken so we need to patch code to work around that breakage.