[U-Boot] [PATCH] net: Fix netretry condition

[U-Boot] [PATCH] net: Fix netretry condition

[Leonid Iziumtsev]

The "net_try_count" counter starts from "1".
And the "retrycnt" contains requested amount of retries.

With current logic, that means that the actual retry amount
will be one time less then what we set in "netretry" env.
For example setting "netretry" to "once" will make "retrycnt"
equal "1", so no retries will be triggered at all.

Fix the logic by changing the statement of "if" condition.

Signed-off-by: Leonid Iziumtsev <leonid.iziumtsev@se.atlascopco.com>
---
 net/net.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/net/net.c b/net/net.c
index 4259c9e..8a9b69c 100644
--- a/net/net.c
+++ b/net/net.c
@@ -683,7 +683,7 @@ int net_start_again(void)
                retry_forever = 0;
        }

-       if ((!retry_forever) && (net_try_count >= retrycnt)) {
+       if ((!retry_forever) && (net_try_count > retrycnt)) {
                eth_halt();
                net_set_state(NETLOOP_FAIL);
                /*
-- 
2.7.4

[U-Boot] [PATCH] net: Fix netretry condition

[Joe Hershberger]

On Fri, Mar 9, 2018 at 8:29 AM, Leonid Iziumtsev
<leonid.iziumtsev@gmail.com> wrote:
> The "net_try_count" counter starts from "1".
> And the "retrycnt" contains requested amount of retries.
>
> With current logic, that means that the actual retry amount
> will be one time less then what we set in "netretry" env.
> For example setting "netretry" to "once" will make "retrycnt"
> equal "1", so no retries will be triggered at all.
>
> Fix the logic by changing the statement of "if" condition.
>
> Signed-off-by: Leonid Iziumtsev <leonid.iziumtsev@se.atlascopco.com>

Acked-by: Joe Hershberger <joe.hershberger@ni.com>

[U-Boot] net: Fix netretry condition

[Joe Hershberger]

Hi Leonid,

https://patchwork.ozlabs.org/patch/883696/ was applied to http://git.denx.de/?p=u-boot/u-boot-net.git

Thanks!
-Joe