e1000e: use msleep instead of mdelay

The e1000_link_stall_workaround_lv() function is always called in non-
atomic context so it should use msleep instead of mdelay.  Also, remove
unnecessary #include <linux/delay.h>.

(backported from commit bb9c5ee15b560f944a8877f60e7d3d77ad542286)
Signed-off-by: Bruce Allan <bruce.w.allan@intel.com>
Tested-by: Aaron Brown <aaron.f.brown@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Signed-off-by: Joe Jin <joe.jin@oracle.com>

diff --git a/drivers/net/e1000e/phy.c b/drivers/net/e1000e/phy.c
index d4dbbe7412469227d02ce920b6a9c44e667804af..d2cc188b35eb0fc8d5112df17d9a3131145e9b5b 100644 (file)
--- a/drivers/net/e1000e/phy.c
+++ b/drivers/net/e1000e/phy.c
@@ -26,8 +26,6 @@
 
 *******************************************************************************/
 
-#include <linux/delay.h>
-
 #include "e1000.h"
 
 static s32 e1000_get_phy_cfg_done(struct e1000_hw *hw);
@@ -3164,7 +3162,7 @@ s32 e1000_link_stall_workaround_hv(struct e1000_hw *hw)
                     BM_CS_STATUS_SPEED_1000))
                return 0;
 
-       mdelay(200);
+       msleep(200);
 
        /* flush the packets in the fifo buffer */
        ret_val = e1e_wphy(hw, HV_MUX_DATA_CTRL, HV_MUX_DATA_CTRL_GEN_TO_MAC |