169 lines
6.0 KiB
Diff
169 lines
6.0 KiB
Diff
|
From 5f85cbb9ee1c00cec81a848a9e871ad5d1e7f53f Mon Sep 17 00:00:00 2001
|
||
|
From: Michael Brown <mcb30@ipxe.org>
|
||
|
Date: Wed, 29 Mar 2017 10:36:03 +0300
|
||
|
Subject: [PATCH] [build] Avoid implicit-fallthrough warnings on GCC 7
|
||
|
|
||
|
Reported-by: Vinson Lee <vlee@freedesktop.org>
|
||
|
Reported-by: Liang Yan <lyan@suse.com>
|
||
|
Signed-off-by: Michael Brown <mcb30@ipxe.org>
|
||
|
[BR: BSC#1031340]
|
||
|
Signed-off-by: Bruce Rogers <brogers@suse.com>
|
||
|
---
|
||
|
src/arch/x86/image/bzimage.c | 2 ++
|
||
|
src/drivers/infiniband/golan.c | 1 +
|
||
|
src/drivers/net/ath/ath9k/ath9k_ar5008_phy.c | 2 ++
|
||
|
src/drivers/net/ath/ath9k/ath9k_ar9002_phy.c | 1 +
|
||
|
src/drivers/net/ath/ath9k/ath9k_ar9003_phy.c | 1 +
|
||
|
src/drivers/net/igbvf/igbvf_vf.c | 1 +
|
||
|
src/drivers/net/tg3/tg3_hw.c | 12 ++++++++++++
|
||
|
src/tests/setjmp_test.c | 5 +++--
|
||
|
8 files changed, 23 insertions(+), 2 deletions(-)
|
||
|
|
||
|
diff --git a/src/arch/x86/image/bzimage.c b/src/arch/x86/image/bzimage.c
|
||
|
index e3c4cb83..51498bf9 100644
|
||
|
--- a/src/arch/x86/image/bzimage.c
|
||
|
+++ b/src/arch/x86/image/bzimage.c
|
||
|
@@ -282,9 +282,11 @@ static int bzimage_parse_cmdline ( struct image *image,
|
||
|
case 'G':
|
||
|
case 'g':
|
||
|
bzimg->mem_limit <<= 10;
|
||
|
+ /* Fall through */
|
||
|
case 'M':
|
||
|
case 'm':
|
||
|
bzimg->mem_limit <<= 10;
|
||
|
+ /* Fall through */
|
||
|
case 'K':
|
||
|
case 'k':
|
||
|
bzimg->mem_limit <<= 10;
|
||
|
diff --git a/src/drivers/infiniband/golan.c b/src/drivers/infiniband/golan.c
|
||
|
index 30eaabab..61331d4c 100755
|
||
|
--- a/src/drivers/infiniband/golan.c
|
||
|
+++ b/src/drivers/infiniband/golan.c
|
||
|
@@ -1956,6 +1956,7 @@ static inline void golan_handle_port_event(struct golan *golan, struct golan_eqe
|
||
|
case GOLAN_PORT_CHANGE_SUBTYPE_CLIENT_REREG:
|
||
|
case GOLAN_PORT_CHANGE_SUBTYPE_ACTIVE:
|
||
|
golan_ib_update ( ibdev );
|
||
|
+ /* Fall through */
|
||
|
case GOLAN_PORT_CHANGE_SUBTYPE_DOWN:
|
||
|
case GOLAN_PORT_CHANGE_SUBTYPE_LID:
|
||
|
case GOLAN_PORT_CHANGE_SUBTYPE_PKEY:
|
||
|
diff --git a/src/drivers/net/ath/ath9k/ath9k_ar5008_phy.c b/src/drivers/net/ath/ath9k/ath9k_ar5008_phy.c
|
||
|
index 2b6c133c..a98e4bb6 100644
|
||
|
--- a/src/drivers/net/ath/ath9k/ath9k_ar5008_phy.c
|
||
|
+++ b/src/drivers/net/ath/ath9k/ath9k_ar5008_phy.c
|
||
|
@@ -640,12 +640,14 @@ static void ar5008_hw_init_chain_masks(struct ath_hw *ah)
|
||
|
case 0x5:
|
||
|
REG_SET_BIT(ah, AR_PHY_ANALOG_SWAP,
|
||
|
AR_PHY_SWAP_ALT_CHAIN);
|
||
|
+ /* Fall through */
|
||
|
case 0x3:
|
||
|
if (ah->hw_version.macVersion == AR_SREV_REVISION_5416_10) {
|
||
|
REG_WRITE(ah, AR_PHY_RX_CHAINMASK, 0x7);
|
||
|
REG_WRITE(ah, AR_PHY_CAL_CHAINMASK, 0x7);
|
||
|
break;
|
||
|
}
|
||
|
+ /* Fall through */
|
||
|
case 0x1:
|
||
|
case 0x2:
|
||
|
case 0x7:
|
||
|
diff --git a/src/drivers/net/ath/ath9k/ath9k_ar9002_phy.c b/src/drivers/net/ath/ath9k/ath9k_ar9002_phy.c
|
||
|
index 72203ba4..65cfad59 100644
|
||
|
--- a/src/drivers/net/ath/ath9k/ath9k_ar9002_phy.c
|
||
|
+++ b/src/drivers/net/ath/ath9k/ath9k_ar9002_phy.c
|
||
|
@@ -122,6 +122,7 @@ static int ar9002_hw_set_channel(struct ath_hw *ah, struct ath9k_channel *chan)
|
||
|
aModeRefSel = 2;
|
||
|
if (aModeRefSel)
|
||
|
break;
|
||
|
+ /* Fall through */
|
||
|
case 1:
|
||
|
default:
|
||
|
aModeRefSel = 0;
|
||
|
diff --git a/src/drivers/net/ath/ath9k/ath9k_ar9003_phy.c b/src/drivers/net/ath/ath9k/ath9k_ar9003_phy.c
|
||
|
index 2244b775..b66358b9 100644
|
||
|
--- a/src/drivers/net/ath/ath9k/ath9k_ar9003_phy.c
|
||
|
+++ b/src/drivers/net/ath/ath9k/ath9k_ar9003_phy.c
|
||
|
@@ -539,6 +539,7 @@ void ar9003_hw_set_chain_masks(struct ath_hw *ah, u8 rx, u8 tx)
|
||
|
case 0x5:
|
||
|
REG_SET_BIT(ah, AR_PHY_ANALOG_SWAP,
|
||
|
AR_PHY_SWAP_ALT_CHAIN);
|
||
|
+ /* Fall through */
|
||
|
case 0x3:
|
||
|
case 0x1:
|
||
|
case 0x2:
|
||
|
diff --git a/src/drivers/net/igbvf/igbvf_vf.c b/src/drivers/net/igbvf/igbvf_vf.c
|
||
|
index f2dac8be..f841d5e3 100644
|
||
|
--- a/src/drivers/net/igbvf/igbvf_vf.c
|
||
|
+++ b/src/drivers/net/igbvf/igbvf_vf.c
|
||
|
@@ -357,6 +357,7 @@ s32 igbvf_promisc_set_vf(struct e1000_hw *hw, enum e1000_promisc_type type)
|
||
|
break;
|
||
|
case e1000_promisc_enabled:
|
||
|
msgbuf |= E1000_VF_SET_PROMISC_MULTICAST;
|
||
|
+ /* Fall through */
|
||
|
case e1000_promisc_unicast:
|
||
|
msgbuf |= E1000_VF_SET_PROMISC_UNICAST;
|
||
|
case e1000_promisc_disabled:
|
||
|
diff --git a/src/drivers/net/tg3/tg3_hw.c b/src/drivers/net/tg3/tg3_hw.c
|
||
|
index 50353cf3..798f8519 100644
|
||
|
--- a/src/drivers/net/tg3/tg3_hw.c
|
||
|
+++ b/src/drivers/net/tg3/tg3_hw.c
|
||
|
@@ -2518,28 +2518,40 @@ static int tg3_reset_hw(struct tg3 *tp, int reset_phy)
|
||
|
switch (limit) {
|
||
|
case 16:
|
||
|
tw32(MAC_RCV_RULE_15, 0); tw32(MAC_RCV_VALUE_15, 0);
|
||
|
+ /* Fall through */
|
||
|
case 15:
|
||
|
tw32(MAC_RCV_RULE_14, 0); tw32(MAC_RCV_VALUE_14, 0);
|
||
|
+ /* Fall through */
|
||
|
case 14:
|
||
|
tw32(MAC_RCV_RULE_13, 0); tw32(MAC_RCV_VALUE_13, 0);
|
||
|
+ /* Fall through */
|
||
|
case 13:
|
||
|
tw32(MAC_RCV_RULE_12, 0); tw32(MAC_RCV_VALUE_12, 0);
|
||
|
+ /* Fall through */
|
||
|
case 12:
|
||
|
tw32(MAC_RCV_RULE_11, 0); tw32(MAC_RCV_VALUE_11, 0);
|
||
|
+ /* Fall through */
|
||
|
case 11:
|
||
|
tw32(MAC_RCV_RULE_10, 0); tw32(MAC_RCV_VALUE_10, 0);
|
||
|
+ /* Fall through */
|
||
|
case 10:
|
||
|
tw32(MAC_RCV_RULE_9, 0); tw32(MAC_RCV_VALUE_9, 0);
|
||
|
+ /* Fall through */
|
||
|
case 9:
|
||
|
tw32(MAC_RCV_RULE_8, 0); tw32(MAC_RCV_VALUE_8, 0);
|
||
|
+ /* Fall through */
|
||
|
case 8:
|
||
|
tw32(MAC_RCV_RULE_7, 0); tw32(MAC_RCV_VALUE_7, 0);
|
||
|
+ /* Fall through */
|
||
|
case 7:
|
||
|
tw32(MAC_RCV_RULE_6, 0); tw32(MAC_RCV_VALUE_6, 0);
|
||
|
+ /* Fall through */
|
||
|
case 6:
|
||
|
tw32(MAC_RCV_RULE_5, 0); tw32(MAC_RCV_VALUE_5, 0);
|
||
|
+ /* Fall through */
|
||
|
case 5:
|
||
|
tw32(MAC_RCV_RULE_4, 0); tw32(MAC_RCV_VALUE_4, 0);
|
||
|
+ /* Fall through */
|
||
|
case 4:
|
||
|
/* tw32(MAC_RCV_RULE_3, 0); tw32(MAC_RCV_VALUE_3, 0); */
|
||
|
case 3:
|
||
|
diff --git a/src/tests/setjmp_test.c b/src/tests/setjmp_test.c
|
||
|
index 50ad13f3..deafcee0 100644
|
||
|
--- a/src/tests/setjmp_test.c
|
||
|
+++ b/src/tests/setjmp_test.c
|
||
|
@@ -111,8 +111,9 @@ static void setjmp_return_ok ( struct setjmp_test *test, int value ) {
|
||
|
* @v file Test code file
|
||
|
* @v line Test code line
|
||
|
*/
|
||
|
-static void longjmp_okx ( struct setjmp_test *test, int value,
|
||
|
- const char *file, unsigned int line ) {
|
||
|
+static void __attribute__ (( noreturn ))
|
||
|
+longjmp_okx ( struct setjmp_test *test, int value,
|
||
|
+ const char *file, unsigned int line ) {
|
||
|
|
||
|
/* Record expected value. A zero passed to longjmp() should
|
||
|
* result in setjmp() returning a value of one.
|
||
|
--
|
||
|
2.12.2
|
||
|
|