Commit 6a6f9e3e authored by Anirudh Venkataramanan's avatar Anirudh Venkataramanan Committed by Tony Nguyen

ixgbe: Remove local variable

Remove local variable "match" and directly return evaluated conditional
instead.
Suggested-by: default avatarAlexander Duyck <alexander.duyck@gmail.com>
Signed-off-by: default avatarAnirudh Venkataramanan <anirudh.venkataramanan@intel.com>
Tested-by: Gurucharan <gurucharanx.g@intel.com> (A Contingent worker at Intel)
Signed-off-by: default avatarTony Nguyen <anthony.l.nguyen@intel.com>
parent 864f8888
...@@ -1996,18 +1996,13 @@ static bool ixgbe_check_lbtest_frame(struct ixgbe_rx_buffer *rx_buffer, ...@@ -1996,18 +1996,13 @@ static bool ixgbe_check_lbtest_frame(struct ixgbe_rx_buffer *rx_buffer,
unsigned int frame_size) unsigned int frame_size)
{ {
unsigned char *data; unsigned char *data;
bool match = true;
frame_size >>= 1; frame_size >>= 1;
data = page_address(rx_buffer->page) + rx_buffer->page_offset; data = page_address(rx_buffer->page) + rx_buffer->page_offset;
if (data[3] != 0xFF || return data[3] == 0xFF && data[frame_size + 10] == 0xBE &&
data[frame_size + 10] != 0xBE || data[frame_size + 12] == 0xAF;
data[frame_size + 12] != 0xAF)
match = false;
return match;
} }
static u16 ixgbe_clean_test_rings(struct ixgbe_ring *rx_ring, static u16 ixgbe_clean_test_rings(struct ixgbe_ring *rx_ring,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment