Commit 27e40255 authored by Gustavo A. R. Silva's avatar Gustavo A. R. Silva Committed by Tony Nguyen
Browse files

ixgbe: Fix fall-through warnings for Clang 

In preparation to enable -Wimplicit-fallthrough for Clang, fix multiple
warnings by explicitly adding multiple break statements instead of just
letting the code fall through to the next case.

Link: https://github.com/KSPP/linux/issues/115


Signed-off-by: default avatarGustavo A. R. Silva <gustavoars@kernel.org>
Signed-off-by: default avatarTony Nguyen <anthony.l.nguyen@intel.com>
parent f83a0d0a

drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c

+2 −0
Original line number Diff line number Diff line
@@ -1542,6 +1542,7 @@ s32 ixgbe_fdir_set_input_mask_82599(struct ixgbe_hw *hw, 
	switch (input_mask->formatted.vm_pool & 0x7F) {

	case 0x0:

		fdirm |= IXGBE_FDIRM_POOL;

		break;

	case 0x7F:

		break;

	default:
@@ -1557,6 +1558,7 @@ s32 ixgbe_fdir_set_input_mask_82599(struct ixgbe_hw *hw, 
			hw_dbg(hw, " Error on src/dst port mask\n");

			return IXGBE_ERR_CONFIG;

		}

		break;

	case IXGBE_ATR_L4TYPE_MASK:

		break;

	default:

drivers/net/ethernet/intel/ixgbe/ixgbe_common.c

+1 −0
Original line number Diff line number Diff line
@@ -93,6 +93,7 @@ bool ixgbe_device_supports_autoneg_fc(struct ixgbe_hw *hw) 
		default:

			break;

		}

		break;

	default:

		break;

	}

drivers/net/ethernet/intel/ixgbe/ixgbe_lib.c

+1 −0
Original line number Diff line number Diff line
@@ -132,6 +132,7 @@ static void ixgbe_get_first_reg_idx(struct ixgbe_adapter *adapter, u8 tc, 
			else

				*tx = (tc + 4) << 4;	/* 96, 112 */

		}

		break;

	default:

		break;

	}

drivers/net/ethernet/intel/ixgbe/ixgbe_ptp.c

+1 −0
Original line number Diff line number Diff line
@@ -999,6 +999,7 @@ static int ixgbe_ptp_set_timestamp_mode(struct ixgbe_adapter *adapter, 
	switch (config->tx_type) {

	case HWTSTAMP_TX_OFF:

		tsync_tx_ctl = 0;

		break;

	case HWTSTAMP_TX_ON:

		break;

	default: