On 4/15/20 4:56 PM, Tom Rini wrote:
On Wed, Apr 15, 2020 at 02:45:08PM +0200, Marek Vasut wrote:
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.
Yes, limited by design, thanks for changing.
This code could be used without tiny printf, so this change is unnecessary.