From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Wed, 15 Apr 2020 10:56:47 -0400 Subject: [PATCH 6/7] ddr: altera: arria10: Change %i to %u for printf In-Reply-To: <8f4b3545-c159-dd24-ed26-8e0b47c759e8@denx.de> References: <20200415090030.128489-1-ley.foon.tan@intel.com> <20200415090030.128489-7-ley.foon.tan@intel.com> <8f4b3545-c159-dd24-ed26-8e0b47c759e8@denx.de> Message-ID: <20200415145647.GA12111@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 > > --- > > 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. Reviewed-by: Tom Rini -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 659 bytes Desc: not available URL: