[U-Boot] [PATCH 1/2] [v2] fix print_size printing fractional gigabyte numbers on 32-bit platforms
Timur Tabi
timur at freescale.com
Tue Apr 13 20:16:02 CEST 2010
In print_size(), the math that calculates the fractional remainder of a number
used the same integer size as a physical address. However, the "10 *" factor
of the algorithm means that a large number (e.g. 1.5GB) can overflow the
integer if we're running on a 32-bit system. Therefore, we need to
disassociate this function from the size of a physical address.
Signed-off-by: Timur Tabi <timur at freescale.com>
---
lib/display_options.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/lib/display_options.c b/lib/display_options.c
index 2dc2567..08a7914 100644
--- a/lib/display_options.c
+++ b/lib/display_options.c
@@ -45,8 +45,8 @@ int display_options (void)
*/
void print_size (phys_size_t size, const char *s)
{
- ulong m = 0, n;
- phys_size_t d = 1 << 30; /* 1 GB */
+ unsigned long m = 0, n;
+ unsigned long long d = 1 << 30; /* 1 GB */
char c = 'G';
if (size < d) { /* try MB */
--
1.6.5
More information about the U-Boot
mailing list