[Intel-wired-lan] [next-queue, v2, 1/8] e1000: clean up the checking logic

[Brown, Aaron F]

Resending with subject line (hopefully) de-corrupted so patchwork will pick it up.

> The checking logic needed some clean-up work, so we rewrite it by
> checking for break first. With that change in place, we can even move
> the second check for goto statement outside of the loop.
>
> As this is merely a cleanup, no functional change is involved. The
> questionable 'tmp != 0xFF' is intentionally left alone.
>
> Mark Rustad and Alexander Duyck contributed to this patch.
>
> Signed-off-by: Jean Sacren <sakiwit at gmail.com>
> Cc: "Rustad, Mark D" <mark.d.rustad at intel.com>
> Cc: Alexander Duyck <alexander.duyck at gmail.com>
> ---
> v2:
>
> 1) Update the changelog to reflect the rewrite of the checking logic and
>    others' contributions;
> 2) Update the changelog title accordingly;
> 3) Add credits and cc the pertinent parties.
> 
>  drivers/net/ethernet/intel/e1000/e1000_main.c | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)

Tested-by: Aaron Brown <aaron.f.brown at intel.com>