On Thu, Dec 05, 2013 at 12:08:09PM -0700, Stephen Warren wrote:
> From: Stephen Warren <swar...@nvidia.com>
>
> Commit 8dfafdde88eb ("Introduce common timer functions") created a
> common definition of usec_to_tick() which had a couple problems:
>
> static unsigned long long usec_to_tick(unsigned long usec)
> {
> uint64_t tick = usec * get_tbclk();
>
> That likely overflows.
>
> usec *= get_tbclk();
>
> That was an attempt to fix it by performing the multiply after the
> promotion of usec to 64-bit, but was applied to the wrong variable,
> which was never used.
>
> This patch fixes these issues. A user-visible symptom of the problem was
> the e.g. "dhcp zImage" using an ASIX USB Ethernet dongle would print:
>
> Waiting for Ethernet connection... unable to connect.
>
> ... with no delay before "unable to connect". There are likely other
> symptoms.
>
> Signed-off-by: Stephen Warren <swar...@nvidia.com>
> Acked-by: Rob Herring <rob.herr...@calxeda.com>Applied to u-boot/master, thanks! -- Tom
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
