Skip to content

Commit a69e5a0

Browse files
committed
Merge branch 'master' of git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/next-queue
Jeff Kirsher says: ==================== Intel Wired LAN Driver Updates 2015-07-23 This series contains updates to e1000e, igb, ixgbevf, i40e and i40evf. Emil extends the reporting of the RSS key and hash table by adding support for x550 VFs. Jia-Ju Bai fixes a QoS issue in e1000e where the error handling lacked a call to pm_qos_remove_request() to cleanup the QoS request made in e1000_open(). Todd updates igb to report unsupported for ethtool coalesce settings that are not supported. Also updated the driver to use the ARRAY_SIZE() macro. Carolyn fixes and refactors the dynamic ITR code for i40e and i40evf which would never change dynamically. So update the switch() statement to have a default case and switch on "new_latency_range" versus the current ITR setting. Shannon cleans up i40e code, where there were un-needed goto's. Also clean up error status messages that were causing some confusion in PHY and FCoE setup error reports. Mitch updates the virtual channel interface to prepare for the x722 device and other future devices, so that the VF driver can report what its capable of supporting to the PF driver. Updates the i40evf driver to handle resets like Core or EMP resets, where the device is reinitialized and the VF will not get the same VSI. Jesse updates the i40e and i40evf driver to use the kernel BIT() and BIT_ULL() macros. ==================== Signed-off-by: David S. Miller <[email protected]>
2 parents 6ca91c6 + 41a1d04 commit a69e5a0

40 files changed

+1376
-712
lines changed

drivers/net/ethernet/intel/e1000e/netdev.c

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4588,6 +4588,7 @@ static int e1000_open(struct net_device *netdev)
45884588
return 0;
45894589

45904590
err_req_irq:
4591+
pm_qos_remove_request(&adapter->pm_qos_req);
45914592
e1000e_release_hw_control(adapter);
45924593
e1000_power_down_phy(adapter);
45934594
e1000e_free_rx_resources(adapter->rx_ring);

drivers/net/ethernet/intel/i40e/i40e.h

Lines changed: 28 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -98,7 +98,7 @@
9898
#define I40E_INT_NAME_STR_LEN (IFNAMSIZ + 9)
9999

100100
/* Ethtool Private Flags */
101-
#define I40E_PRIV_FLAGS_NPAR_FLAG (1 << 0)
101+
#define I40E_PRIV_FLAGS_NPAR_FLAG BIT(0)
102102

103103
#define I40E_NVM_VERSION_LO_SHIFT 0
104104
#define I40E_NVM_VERSION_LO_MASK (0xff << I40E_NVM_VERSION_LO_SHIFT)
@@ -289,35 +289,35 @@ struct i40e_pf {
289289
struct work_struct service_task;
290290

291291
u64 flags;
292-
#define I40E_FLAG_RX_CSUM_ENABLED (u64)(1 << 1)
293-
#define I40E_FLAG_MSI_ENABLED (u64)(1 << 2)
294-
#define I40E_FLAG_MSIX_ENABLED (u64)(1 << 3)
295-
#define I40E_FLAG_RX_1BUF_ENABLED (u64)(1 << 4)
296-
#define I40E_FLAG_RX_PS_ENABLED (u64)(1 << 5)
297-
#define I40E_FLAG_RSS_ENABLED (u64)(1 << 6)
298-
#define I40E_FLAG_VMDQ_ENABLED (u64)(1 << 7)
299-
#define I40E_FLAG_FDIR_REQUIRES_REINIT (u64)(1 << 8)
300-
#define I40E_FLAG_NEED_LINK_UPDATE (u64)(1 << 9)
292+
#define I40E_FLAG_RX_CSUM_ENABLED BIT_ULL(1)
293+
#define I40E_FLAG_MSI_ENABLED BIT_ULL(2)
294+
#define I40E_FLAG_MSIX_ENABLED BIT_ULL(3)
295+
#define I40E_FLAG_RX_1BUF_ENABLED BIT_ULL(4)
296+
#define I40E_FLAG_RX_PS_ENABLED BIT_ULL(5)
297+
#define I40E_FLAG_RSS_ENABLED BIT_ULL(6)
298+
#define I40E_FLAG_VMDQ_ENABLED BIT_ULL(7)
299+
#define I40E_FLAG_FDIR_REQUIRES_REINIT BIT_ULL(8)
300+
#define I40E_FLAG_NEED_LINK_UPDATE BIT_ULL(9)
301301
#ifdef I40E_FCOE
302-
#define I40E_FLAG_FCOE_ENABLED (u64)(1 << 11)
302+
#define I40E_FLAG_FCOE_ENABLED BIT_ULL(11)
303303
#endif /* I40E_FCOE */
304-
#define I40E_FLAG_IN_NETPOLL (u64)(1 << 12)
305-
#define I40E_FLAG_16BYTE_RX_DESC_ENABLED (u64)(1 << 13)
306-
#define I40E_FLAG_CLEAN_ADMINQ (u64)(1 << 14)
307-
#define I40E_FLAG_FILTER_SYNC (u64)(1 << 15)
308-
#define I40E_FLAG_PROCESS_MDD_EVENT (u64)(1 << 17)
309-
#define I40E_FLAG_PROCESS_VFLR_EVENT (u64)(1 << 18)
310-
#define I40E_FLAG_SRIOV_ENABLED (u64)(1 << 19)
311-
#define I40E_FLAG_DCB_ENABLED (u64)(1 << 20)
312-
#define I40E_FLAG_FD_SB_ENABLED (u64)(1 << 21)
313-
#define I40E_FLAG_FD_ATR_ENABLED (u64)(1 << 22)
314-
#define I40E_FLAG_PTP (u64)(1 << 25)
315-
#define I40E_FLAG_MFP_ENABLED (u64)(1 << 26)
304+
#define I40E_FLAG_IN_NETPOLL BIT_ULL(12)
305+
#define I40E_FLAG_16BYTE_RX_DESC_ENABLED BIT_ULL(13)
306+
#define I40E_FLAG_CLEAN_ADMINQ BIT_ULL(14)
307+
#define I40E_FLAG_FILTER_SYNC BIT_ULL(15)
308+
#define I40E_FLAG_PROCESS_MDD_EVENT BIT_ULL(17)
309+
#define I40E_FLAG_PROCESS_VFLR_EVENT BIT_ULL(18)
310+
#define I40E_FLAG_SRIOV_ENABLED BIT_ULL(19)
311+
#define I40E_FLAG_DCB_ENABLED BIT_ULL(20)
312+
#define I40E_FLAG_FD_SB_ENABLED BIT_ULL(21)
313+
#define I40E_FLAG_FD_ATR_ENABLED BIT_ULL(22)
314+
#define I40E_FLAG_PTP BIT_ULL(25)
315+
#define I40E_FLAG_MFP_ENABLED BIT_ULL(26)
316316
#ifdef CONFIG_I40E_VXLAN
317-
#define I40E_FLAG_VXLAN_FILTER_SYNC (u64)(1 << 27)
317+
#define I40E_FLAG_VXLAN_FILTER_SYNC BIT_ULL(27)
318318
#endif
319-
#define I40E_FLAG_PORT_ID_VALID (u64)(1 << 28)
320-
#define I40E_FLAG_DCB_CAPABLE (u64)(1 << 29)
319+
#define I40E_FLAG_PORT_ID_VALID BIT_ULL(28)
320+
#define I40E_FLAG_DCB_CAPABLE BIT_ULL(29)
321321
#define I40E_FLAG_VEB_MODE_ENABLED BIT_ULL(40)
322322

323323
/* tracks features that get auto disabled by errors */
@@ -443,8 +443,8 @@ struct i40e_vsi {
443443

444444
u32 current_netdev_flags;
445445
unsigned long state;
446-
#define I40E_VSI_FLAG_FILTER_CHANGED (1<<0)
447-
#define I40E_VSI_FLAG_VEB_OWNER (1<<1)
446+
#define I40E_VSI_FLAG_FILTER_CHANGED BIT(0)
447+
#define I40E_VSI_FLAG_VEB_OWNER BIT(1)
448448
unsigned long flags;
449449

450450
struct list_head mac_filter_list;

drivers/net/ethernet/intel/i40e/i40e_common.c

Lines changed: 208 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -71,6 +71,212 @@ static i40e_status i40e_set_mac_type(struct i40e_hw *hw)
7171
return status;
7272
}
7373

74+
/**
75+
* i40e_aq_str - convert AQ err code to a string
76+
* @hw: pointer to the HW structure
77+
* @aq_err: the AQ error code to convert
78+
**/
79+
char *i40e_aq_str(struct i40e_hw *hw, enum i40e_admin_queue_err aq_err)
80+
{
81+
switch (aq_err) {
82+
case I40E_AQ_RC_OK:
83+
return "OK";
84+
case I40E_AQ_RC_EPERM:
85+
return "I40E_AQ_RC_EPERM";
86+
case I40E_AQ_RC_ENOENT:
87+
return "I40E_AQ_RC_ENOENT";
88+
case I40E_AQ_RC_ESRCH:
89+
return "I40E_AQ_RC_ESRCH";
90+
case I40E_AQ_RC_EINTR:
91+
return "I40E_AQ_RC_EINTR";
92+
case I40E_AQ_RC_EIO:
93+
return "I40E_AQ_RC_EIO";
94+
case I40E_AQ_RC_ENXIO:
95+
return "I40E_AQ_RC_ENXIO";
96+
case I40E_AQ_RC_E2BIG:
97+
return "I40E_AQ_RC_E2BIG";
98+
case I40E_AQ_RC_EAGAIN:
99+
return "I40E_AQ_RC_EAGAIN";
100+
case I40E_AQ_RC_ENOMEM:
101+
return "I40E_AQ_RC_ENOMEM";
102+
case I40E_AQ_RC_EACCES:
103+
return "I40E_AQ_RC_EACCES";
104+
case I40E_AQ_RC_EFAULT:
105+
return "I40E_AQ_RC_EFAULT";
106+
case I40E_AQ_RC_EBUSY:
107+
return "I40E_AQ_RC_EBUSY";
108+
case I40E_AQ_RC_EEXIST:
109+
return "I40E_AQ_RC_EEXIST";
110+
case I40E_AQ_RC_EINVAL:
111+
return "I40E_AQ_RC_EINVAL";
112+
case I40E_AQ_RC_ENOTTY:
113+
return "I40E_AQ_RC_ENOTTY";
114+
case I40E_AQ_RC_ENOSPC:
115+
return "I40E_AQ_RC_ENOSPC";
116+
case I40E_AQ_RC_ENOSYS:
117+
return "I40E_AQ_RC_ENOSYS";
118+
case I40E_AQ_RC_ERANGE:
119+
return "I40E_AQ_RC_ERANGE";
120+
case I40E_AQ_RC_EFLUSHED:
121+
return "I40E_AQ_RC_EFLUSHED";
122+
case I40E_AQ_RC_BAD_ADDR:
123+
return "I40E_AQ_RC_BAD_ADDR";
124+
case I40E_AQ_RC_EMODE:
125+
return "I40E_AQ_RC_EMODE";
126+
case I40E_AQ_RC_EFBIG:
127+
return "I40E_AQ_RC_EFBIG";
128+
}
129+
130+
snprintf(hw->err_str, sizeof(hw->err_str), "%d", aq_err);
131+
return hw->err_str;
132+
}
133+
134+
/**
135+
* i40e_stat_str - convert status err code to a string
136+
* @hw: pointer to the HW structure
137+
* @stat_err: the status error code to convert
138+
**/
139+
char *i40e_stat_str(struct i40e_hw *hw, i40e_status stat_err)
140+
{
141+
switch (stat_err) {
142+
case 0:
143+
return "OK";
144+
case I40E_ERR_NVM:
145+
return "I40E_ERR_NVM";
146+
case I40E_ERR_NVM_CHECKSUM:
147+
return "I40E_ERR_NVM_CHECKSUM";
148+
case I40E_ERR_PHY:
149+
return "I40E_ERR_PHY";
150+
case I40E_ERR_CONFIG:
151+
return "I40E_ERR_CONFIG";
152+
case I40E_ERR_PARAM:
153+
return "I40E_ERR_PARAM";
154+
case I40E_ERR_MAC_TYPE:
155+
return "I40E_ERR_MAC_TYPE";
156+
case I40E_ERR_UNKNOWN_PHY:
157+
return "I40E_ERR_UNKNOWN_PHY";
158+
case I40E_ERR_LINK_SETUP:
159+
return "I40E_ERR_LINK_SETUP";
160+
case I40E_ERR_ADAPTER_STOPPED:
161+
return "I40E_ERR_ADAPTER_STOPPED";
162+
case I40E_ERR_INVALID_MAC_ADDR:
163+
return "I40E_ERR_INVALID_MAC_ADDR";
164+
case I40E_ERR_DEVICE_NOT_SUPPORTED:
165+
return "I40E_ERR_DEVICE_NOT_SUPPORTED";
166+
case I40E_ERR_MASTER_REQUESTS_PENDING:
167+
return "I40E_ERR_MASTER_REQUESTS_PENDING";
168+
case I40E_ERR_INVALID_LINK_SETTINGS:
169+
return "I40E_ERR_INVALID_LINK_SETTINGS";
170+
case I40E_ERR_AUTONEG_NOT_COMPLETE:
171+
return "I40E_ERR_AUTONEG_NOT_COMPLETE";
172+
case I40E_ERR_RESET_FAILED:
173+
return "I40E_ERR_RESET_FAILED";
174+
case I40E_ERR_SWFW_SYNC:
175+
return "I40E_ERR_SWFW_SYNC";
176+
case I40E_ERR_NO_AVAILABLE_VSI:
177+
return "I40E_ERR_NO_AVAILABLE_VSI";
178+
case I40E_ERR_NO_MEMORY:
179+
return "I40E_ERR_NO_MEMORY";
180+
case I40E_ERR_BAD_PTR:
181+
return "I40E_ERR_BAD_PTR";
182+
case I40E_ERR_RING_FULL:
183+
return "I40E_ERR_RING_FULL";
184+
case I40E_ERR_INVALID_PD_ID:
185+
return "I40E_ERR_INVALID_PD_ID";
186+
case I40E_ERR_INVALID_QP_ID:
187+
return "I40E_ERR_INVALID_QP_ID";
188+
case I40E_ERR_INVALID_CQ_ID:
189+
return "I40E_ERR_INVALID_CQ_ID";
190+
case I40E_ERR_INVALID_CEQ_ID:
191+
return "I40E_ERR_INVALID_CEQ_ID";
192+
case I40E_ERR_INVALID_AEQ_ID:
193+
return "I40E_ERR_INVALID_AEQ_ID";
194+
case I40E_ERR_INVALID_SIZE:
195+
return "I40E_ERR_INVALID_SIZE";
196+
case I40E_ERR_INVALID_ARP_INDEX:
197+
return "I40E_ERR_INVALID_ARP_INDEX";
198+
case I40E_ERR_INVALID_FPM_FUNC_ID:
199+
return "I40E_ERR_INVALID_FPM_FUNC_ID";
200+
case I40E_ERR_QP_INVALID_MSG_SIZE:
201+
return "I40E_ERR_QP_INVALID_MSG_SIZE";
202+
case I40E_ERR_QP_TOOMANY_WRS_POSTED:
203+
return "I40E_ERR_QP_TOOMANY_WRS_POSTED";
204+
case I40E_ERR_INVALID_FRAG_COUNT:
205+
return "I40E_ERR_INVALID_FRAG_COUNT";
206+
case I40E_ERR_QUEUE_EMPTY:
207+
return "I40E_ERR_QUEUE_EMPTY";
208+
case I40E_ERR_INVALID_ALIGNMENT:
209+
return "I40E_ERR_INVALID_ALIGNMENT";
210+
case I40E_ERR_FLUSHED_QUEUE:
211+
return "I40E_ERR_FLUSHED_QUEUE";
212+
case I40E_ERR_INVALID_PUSH_PAGE_INDEX:
213+
return "I40E_ERR_INVALID_PUSH_PAGE_INDEX";
214+
case I40E_ERR_INVALID_IMM_DATA_SIZE:
215+
return "I40E_ERR_INVALID_IMM_DATA_SIZE";
216+
case I40E_ERR_TIMEOUT:
217+
return "I40E_ERR_TIMEOUT";
218+
case I40E_ERR_OPCODE_MISMATCH:
219+
return "I40E_ERR_OPCODE_MISMATCH";
220+
case I40E_ERR_CQP_COMPL_ERROR:
221+
return "I40E_ERR_CQP_COMPL_ERROR";
222+
case I40E_ERR_INVALID_VF_ID:
223+
return "I40E_ERR_INVALID_VF_ID";
224+
case I40E_ERR_INVALID_HMCFN_ID:
225+
return "I40E_ERR_INVALID_HMCFN_ID";
226+
case I40E_ERR_BACKING_PAGE_ERROR:
227+
return "I40E_ERR_BACKING_PAGE_ERROR";
228+
case I40E_ERR_NO_PBLCHUNKS_AVAILABLE:
229+
return "I40E_ERR_NO_PBLCHUNKS_AVAILABLE";
230+
case I40E_ERR_INVALID_PBLE_INDEX:
231+
return "I40E_ERR_INVALID_PBLE_INDEX";
232+
case I40E_ERR_INVALID_SD_INDEX:
233+
return "I40E_ERR_INVALID_SD_INDEX";
234+
case I40E_ERR_INVALID_PAGE_DESC_INDEX:
235+
return "I40E_ERR_INVALID_PAGE_DESC_INDEX";
236+
case I40E_ERR_INVALID_SD_TYPE:
237+
return "I40E_ERR_INVALID_SD_TYPE";
238+
case I40E_ERR_MEMCPY_FAILED:
239+
return "I40E_ERR_MEMCPY_FAILED";
240+
case I40E_ERR_INVALID_HMC_OBJ_INDEX:
241+
return "I40E_ERR_INVALID_HMC_OBJ_INDEX";
242+
case I40E_ERR_INVALID_HMC_OBJ_COUNT:
243+
return "I40E_ERR_INVALID_HMC_OBJ_COUNT";
244+
case I40E_ERR_INVALID_SRQ_ARM_LIMIT:
245+
return "I40E_ERR_INVALID_SRQ_ARM_LIMIT";
246+
case I40E_ERR_SRQ_ENABLED:
247+
return "I40E_ERR_SRQ_ENABLED";
248+
case I40E_ERR_ADMIN_QUEUE_ERROR:
249+
return "I40E_ERR_ADMIN_QUEUE_ERROR";
250+
case I40E_ERR_ADMIN_QUEUE_TIMEOUT:
251+
return "I40E_ERR_ADMIN_QUEUE_TIMEOUT";
252+
case I40E_ERR_BUF_TOO_SHORT:
253+
return "I40E_ERR_BUF_TOO_SHORT";
254+
case I40E_ERR_ADMIN_QUEUE_FULL:
255+
return "I40E_ERR_ADMIN_QUEUE_FULL";
256+
case I40E_ERR_ADMIN_QUEUE_NO_WORK:
257+
return "I40E_ERR_ADMIN_QUEUE_NO_WORK";
258+
case I40E_ERR_BAD_IWARP_CQE:
259+
return "I40E_ERR_BAD_IWARP_CQE";
260+
case I40E_ERR_NVM_BLANK_MODE:
261+
return "I40E_ERR_NVM_BLANK_MODE";
262+
case I40E_ERR_NOT_IMPLEMENTED:
263+
return "I40E_ERR_NOT_IMPLEMENTED";
264+
case I40E_ERR_PE_DOORBELL_NOT_ENABLED:
265+
return "I40E_ERR_PE_DOORBELL_NOT_ENABLED";
266+
case I40E_ERR_DIAG_TEST_FAILED:
267+
return "I40E_ERR_DIAG_TEST_FAILED";
268+
case I40E_ERR_NOT_READY:
269+
return "I40E_ERR_NOT_READY";
270+
case I40E_NOT_SUPPORTED:
271+
return "I40E_NOT_SUPPORTED";
272+
case I40E_ERR_FIRMWARE_API_VERSION:
273+
return "I40E_ERR_FIRMWARE_API_VERSION";
274+
}
275+
276+
snprintf(hw->err_str, sizeof(hw->err_str), "%d", stat_err);
277+
return hw->err_str;
278+
}
279+
74280
/**
75281
* i40e_debug_aq
76282
* @hw: debug mask related to admin queue
@@ -1187,9 +1393,9 @@ void i40e_led_set(struct i40e_hw *hw, u32 mode, bool blink)
11871393
blink = false;
11881394

11891395
if (blink)
1190-
gpio_val |= (1 << I40E_GLGEN_GPIO_CTL_LED_BLINK_SHIFT);
1396+
gpio_val |= BIT(I40E_GLGEN_GPIO_CTL_LED_BLINK_SHIFT);
11911397
else
1192-
gpio_val &= ~(1 << I40E_GLGEN_GPIO_CTL_LED_BLINK_SHIFT);
1398+
gpio_val &= ~BIT(I40E_GLGEN_GPIO_CTL_LED_BLINK_SHIFT);
11931399

11941400
wr32(hw, I40E_GLGEN_GPIO_CTL(i), gpio_val);
11951401
break;

drivers/net/ethernet/intel/i40e/i40e_dcb.h

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -58,9 +58,9 @@
5858
#define I40E_IEEE_ETS_MAXTC_SHIFT 0
5959
#define I40E_IEEE_ETS_MAXTC_MASK (0x7 << I40E_IEEE_ETS_MAXTC_SHIFT)
6060
#define I40E_IEEE_ETS_CBS_SHIFT 6
61-
#define I40E_IEEE_ETS_CBS_MASK (0x1 << I40E_IEEE_ETS_CBS_SHIFT)
61+
#define I40E_IEEE_ETS_CBS_MASK BIT(I40E_IEEE_ETS_CBS_SHIFT)
6262
#define I40E_IEEE_ETS_WILLING_SHIFT 7
63-
#define I40E_IEEE_ETS_WILLING_MASK (0x1 << I40E_IEEE_ETS_WILLING_SHIFT)
63+
#define I40E_IEEE_ETS_WILLING_MASK BIT(I40E_IEEE_ETS_WILLING_SHIFT)
6464
#define I40E_IEEE_ETS_PRIO_0_SHIFT 0
6565
#define I40E_IEEE_ETS_PRIO_0_MASK (0x7 << I40E_IEEE_ETS_PRIO_0_SHIFT)
6666
#define I40E_IEEE_ETS_PRIO_1_SHIFT 4
@@ -79,9 +79,9 @@
7979
#define I40E_IEEE_PFC_CAP_SHIFT 0
8080
#define I40E_IEEE_PFC_CAP_MASK (0xF << I40E_IEEE_PFC_CAP_SHIFT)
8181
#define I40E_IEEE_PFC_MBC_SHIFT 6
82-
#define I40E_IEEE_PFC_MBC_MASK (0x1 << I40E_IEEE_PFC_MBC_SHIFT)
82+
#define I40E_IEEE_PFC_MBC_MASK BIT(I40E_IEEE_PFC_MBC_SHIFT)
8383
#define I40E_IEEE_PFC_WILLING_SHIFT 7
84-
#define I40E_IEEE_PFC_WILLING_MASK (0x1 << I40E_IEEE_PFC_WILLING_SHIFT)
84+
#define I40E_IEEE_PFC_WILLING_MASK BIT(I40E_IEEE_PFC_WILLING_SHIFT)
8585

8686
/* Defines for IEEE APP TLV */
8787
#define I40E_IEEE_APP_SEL_SHIFT 0

drivers/net/ethernet/intel/i40e/i40e_dcb_nl.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -187,7 +187,7 @@ void i40e_dcbnl_set_all(struct i40e_vsi *vsi)
187187
/* Set up all the App TLVs if DCBx is negotiated */
188188
for (i = 0; i < dcbxcfg->numapps; i++) {
189189
prio = dcbxcfg->app[i].priority;
190-
tc_map = (1 << dcbxcfg->etscfg.prioritytable[prio]);
190+
tc_map = BIT(dcbxcfg->etscfg.prioritytable[prio]);
191191

192192
/* Add APP only if the TC is enabled for this VSI */
193193
if (tc_map & vsi->tc_config.enabled_tc) {

drivers/net/ethernet/intel/i40e/i40e_debugfs.c

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -964,7 +964,7 @@ static void i40e_dbg_cmd_fd_ctrl(struct i40e_pf *pf, u64 flag, bool enable)
964964
pf->auto_disable_flags |= flag;
965965
}
966966
dev_info(&pf->pdev->dev, "requesting a PF reset\n");
967-
i40e_do_reset_safe(pf, (1 << __I40E_PF_RESET_REQUESTED));
967+
i40e_do_reset_safe(pf, BIT(__I40E_PF_RESET_REQUESTED));
968968
}
969969

970970
#define I40E_MAX_DEBUG_OUT_BUFFER (4096*4)
@@ -1471,19 +1471,19 @@ static ssize_t i40e_dbg_command_write(struct file *filp,
14711471
}
14721472
} else if (strncmp(cmd_buf, "pfr", 3) == 0) {
14731473
dev_info(&pf->pdev->dev, "debugfs: forcing PFR\n");
1474-
i40e_do_reset_safe(pf, (1 << __I40E_PF_RESET_REQUESTED));
1474+
i40e_do_reset_safe(pf, BIT(__I40E_PF_RESET_REQUESTED));
14751475

14761476
} else if (strncmp(cmd_buf, "corer", 5) == 0) {
14771477
dev_info(&pf->pdev->dev, "debugfs: forcing CoreR\n");
1478-
i40e_do_reset_safe(pf, (1 << __I40E_CORE_RESET_REQUESTED));
1478+
i40e_do_reset_safe(pf, BIT(__I40E_CORE_RESET_REQUESTED));
14791479

14801480
} else if (strncmp(cmd_buf, "globr", 5) == 0) {
14811481
dev_info(&pf->pdev->dev, "debugfs: forcing GlobR\n");
1482-
i40e_do_reset_safe(pf, (1 << __I40E_GLOBAL_RESET_REQUESTED));
1482+
i40e_do_reset_safe(pf, BIT(__I40E_GLOBAL_RESET_REQUESTED));
14831483

14841484
} else if (strncmp(cmd_buf, "empr", 4) == 0) {
14851485
dev_info(&pf->pdev->dev, "debugfs: forcing EMPR\n");
1486-
i40e_do_reset_safe(pf, (1 << __I40E_EMP_RESET_REQUESTED));
1486+
i40e_do_reset_safe(pf, BIT(__I40E_EMP_RESET_REQUESTED));
14871487

14881488
} else if (strncmp(cmd_buf, "read", 4) == 0) {
14891489
u32 address;

drivers/net/ethernet/intel/i40e/i40e_diag.c

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -144,11 +144,8 @@ i40e_status i40e_diag_eeprom_test(struct i40e_hw *hw)
144144
ret_code = i40e_read_nvm_word(hw, I40E_SR_NVM_CONTROL_WORD, &reg_val);
145145
if (!ret_code &&
146146
((reg_val & I40E_SR_CONTROL_WORD_1_MASK) ==
147-
(0x01 << I40E_SR_CONTROL_WORD_1_SHIFT))) {
148-
ret_code = i40e_validate_nvm_checksum(hw, NULL);
149-
} else {
150-
ret_code = I40E_ERR_DIAG_TEST_FAILED;
151-
}
152-
153-
return ret_code;
147+
BIT(I40E_SR_CONTROL_WORD_1_SHIFT)))
148+
return i40e_validate_nvm_checksum(hw, NULL);
149+
else
150+
return I40E_ERR_DIAG_TEST_FAILED;
154151
}

0 commit comments

Comments
 (0)