From 0bddd303f416c4abcfd64cedfa46880163c1d9c6 Mon Sep 17 00:00:00 2001 From: Malcolm Priestley Date: Tue, 27 Aug 2013 09:56:50 +0100 Subject: [PATCH] staging: vt6656: rxtx.c: s_vFillRTSHead change if/else to switch on byPktype Allowing fall through where necessary. Signed-off-by: Malcolm Priestley Signed-off-by: Greg Kroah-Hartman --- drivers/staging/vt6656/rxtx.c | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/drivers/staging/vt6656/rxtx.c b/drivers/staging/vt6656/rxtx.c index e97bc4098f89..e049a34ad91f 100644 --- a/drivers/staging/vt6656/rxtx.c +++ b/drivers/staging/vt6656/rxtx.c @@ -730,7 +730,9 @@ static void s_vFillRTSHead(struct vnt_private *pDevice, u8 byPktType, * into account. * Otherwise, we need to modified codes for them. */ - if (byPktType == PK_TYPE_11GB || byPktType == PK_TYPE_11GA) { + switch (byPktType) { + case PK_TYPE_11GB: + case PK_TYPE_11GA: if (byFBOption == AUTO_FB_NONE) vnt_rxtx_rts_g_head(pDevice, &head->rts_g, psEthHeader, byPktType, cbFrameLength, @@ -739,7 +741,8 @@ static void s_vFillRTSHead(struct vnt_private *pDevice, u8 byPktType, vnt_rxtx_rts_g_fb_head(pDevice, &head->rts_g_fb, psEthHeader, byPktType, cbFrameLength, bNeedAck, wCurrentRate, byFBOption); - } else if (byPktType == PK_TYPE_11A) { + break; + case PK_TYPE_11A: if (byFBOption == AUTO_FB_NONE) vnt_rxtx_rts_ab_head(pDevice, &head->rts_ab, psEthHeader, byPktType, cbFrameLength, @@ -748,7 +751,8 @@ static void s_vFillRTSHead(struct vnt_private *pDevice, u8 byPktType, vnt_rxtx_rts_a_fb_head(pDevice, &head->rts_a_fb, psEthHeader, byPktType, cbFrameLength, bNeedAck, wCurrentRate, byFBOption); - } else if (byPktType == PK_TYPE_11B) { + break; + case PK_TYPE_11B: vnt_rxtx_rts_ab_head(pDevice, &head->rts_ab, psEthHeader, byPktType, cbFrameLength, bNeedAck, wCurrentRate, byFBOption); -- 2.30.2