]> www.infradead.org Git - users/jedix/linux-maple.git/commit
e1000: clean up the checking logic
authorJean Sacren <sakiwit@gmail.com>
Sat, 19 Sep 2015 11:08:40 +0000 (05:08 -0600)
committerChuck Anderson <chuck.anderson@oracle.com>
Thu, 22 Jun 2017 06:24:38 +0000 (23:24 -0700)
commitd2c139591faff57c9c32ecf553decd1bcdaae4ed
tree9e8a7180034560610ee6231f60865cc0f49c1ed4
parent75939d9a542d0f777d50fd9e84ecd8c2b04b9dfa
e1000: clean up the checking logic

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.

CC: Mark Rustad <mark.d.rustad@intel.com>
CC: Alex Duyck <aduyck@mirantis.com>
Signed-off-by: Jean Sacren <sakiwit@gmail.com>
Tested-by: Aaron Brown <aaron.f.brown@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Orabug: 26243014
(cherry picked from commit 4e01f3a802b5910b25814e1d0fd05907edffed6f)
Signed-off-by: Jack Vogel <jack.vogel@oracle.com>
Reviewed-by: Ethan Zhao <ethan.zhao@oracle.com>
drivers/net/ethernet/intel/e1000/e1000_main.c