From mboxrd@z Thu Jan 1 00:00:00 1970 From: Pavel Machek Date: Sun, 14 Dec 2014 18:56:48 +0100 Subject: [U-Boot] [PATCH] disk/part.c: use unsigned format when printing capacity In-Reply-To: <20141208000411.GA10845@makrotopia.org> References: <20141208000411.GA10845@makrotopia.org> Message-ID: <20141214175648.GA10222@amd> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de I'm afraid I'm not the right contact to merge the patch. It should probably go to Tom Rini and u-boot list. (I put them in Cc). Pavel On Mon 2014-12-08 01:04:16, Daniel Golle wrote: > Large disks otherwise produce highly unplausible output such as > Capacity: 1907729.0 MB = 1863.0 GB (-387938128 x 512) > > As supposedly all size-related decimals are unsigned, use unsigned > format in printf statement, resulting in a correct capacity being > displayed: > Capacity: 1907729.0 MB = 1863.0 GB (3907029168 x 512) > > Signed-off-by: Daniel Golle > --- > disk/part.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/disk/part.c b/disk/part.c > index 43485c9..7c67ea6 100644 > --- a/disk/part.c > +++ b/disk/part.c > @@ -229,13 +229,13 @@ void dev_print (block_dev_desc_t *dev_desc) > printf (" Supports 48-bit addressing\n"); > #endif > #if defined(CONFIG_SYS_64BIT_LBA) > - printf (" Capacity: %ld.%ld MB = %ld.%ld GB (%Ld x %ld)\n", > + printf (" Capacity: %lu.%lu MB = %lu.%lu GB (%Lu x %lu)\n", > mb_quot, mb_rem, > gb_quot, gb_rem, > lba, > dev_desc->blksz); > #else > - printf (" Capacity: %ld.%ld MB = %ld.%ld GB (%ld x %ld)\n", > + printf (" Capacity: %lu.%lu MB = %lu.%lu GB (%lu x %lu)\n", > mb_quot, mb_rem, > gb_quot, gb_rem, > (ulong)lba, -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html