Hi Tom, On Mon, 2013-12-09 at 16:57 +0400, Alexey Brodkin wrote: > Current implementation works fine if "usec * get_tbclk()" fits in 32 > bits. Otherwise result will be cut down to 32-bit. > > Fix is obvious - first extend data type of either operand. > > Signed-off-by: Alexey Brodkin <[email protected]> > > Cc: Rob Herring <[email protected]> > Cc: Mischa Jonker <[email protected]> > --- > lib/time.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/lib/time.c b/lib/time.c > index 09bb05a..80003c3 100644 > --- a/lib/time.c > +++ b/lib/time.c > @@ -71,7 +71,7 @@ unsigned long __weak notrace timer_get_us(void) > } > static unsigned long long usec_to_tick(unsigned long usec) > { > - uint64_t tick = usec * get_tbclk(); > + uint64_t tick = (uint64_t)usec * get_tbclk(); > usec *= get_tbclk(); > do_div(tick, 1000000); > return tick;
Any chance to get it applied? Regards, Alexey _______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
