From 96372bd9b42e917cb83add7d43bcfbe60307da56 Mon Sep 17 00:00:00 2001 From: Malcolm Priestley Date: Sat, 30 Aug 2014 22:25:48 +0100 Subject: [PATCH] staging: vt6655: s_uGetRTSCTSDuration always return __le16 This function should always return __le16, move all endian correction to here. Signed-off-by: Malcolm Priestley Signed-off-by: Greg Kroah-Hartman --- drivers/staging/vt6655/rxtx.c | 149 +++++++++++++++++++--------------- 1 file changed, 84 insertions(+), 65 deletions(-) diff --git a/drivers/staging/vt6655/rxtx.c b/drivers/staging/vt6655/rxtx.c index 3c8c4998fe26..0c69193a1d5b 100644 --- a/drivers/staging/vt6655/rxtx.c +++ b/drivers/staging/vt6655/rxtx.c @@ -578,7 +578,7 @@ s_uGetDataDuration( //byFreqType: 0=>5GHZ 1=>2.4GHZ static -unsigned int +__le16 s_uGetRTSCTSDuration( struct vnt_private *pDevice, unsigned char byDurType, @@ -667,7 +667,7 @@ s_uGetRTSCTSDuration( break; } - return uDurTime; + return cpu_to_le16((u16)uDurTime); } static @@ -829,17 +829,20 @@ s_vFillRTSHead( byPktType, &buf->a); /* Get Duration */ buf->duration_bb = - cpu_to_le16((u16)s_uGetRTSCTSDuration(pDevice, RTSDUR_BB, cbFrameLength, - PK_TYPE_11B, pDevice->byTopCCKBasicRate, - bNeedAck, byFBOption)); + s_uGetRTSCTSDuration(pDevice, RTSDUR_BB, + cbFrameLength, PK_TYPE_11B, + pDevice->byTopCCKBasicRate, + bNeedAck, byFBOption); buf->duration_aa = - cpu_to_le16((u16)s_uGetRTSCTSDuration(pDevice, RTSDUR_AA, cbFrameLength, - byPktType, wCurrentRate, bNeedAck, - byFBOption)); + s_uGetRTSCTSDuration(pDevice, RTSDUR_AA, + cbFrameLength, byPktType, + wCurrentRate, bNeedAck, + byFBOption); buf->duration_ba = - cpu_to_le16((u16)s_uGetRTSCTSDuration(pDevice, RTSDUR_BA, cbFrameLength, - byPktType, wCurrentRate, bNeedAck, - byFBOption)); + s_uGetRTSCTSDuration(pDevice, RTSDUR_BA, + cbFrameLength, byPktType, + wCurrentRate, bNeedAck, + byFBOption); buf->data.duration = buf->duration_aa; /* Get RTS Frame body */ @@ -871,33 +874,40 @@ s_vFillRTSHead( byPktType, &buf->a); /* Get Duration */ buf->duration_bb = - cpu_to_le16((u16)s_uGetRTSCTSDuration(pDevice, RTSDUR_BB, cbFrameLength, - PK_TYPE_11B, pDevice->byTopCCKBasicRate, - bNeedAck, byFBOption)); + s_uGetRTSCTSDuration(pDevice, RTSDUR_BB, + cbFrameLength, PK_TYPE_11B, + pDevice->byTopCCKBasicRate, + bNeedAck, byFBOption); buf->duration_aa = - cpu_to_le16((u16)s_uGetRTSCTSDuration(pDevice, RTSDUR_AA, cbFrameLength, - byPktType, wCurrentRate, bNeedAck, - byFBOption)); + s_uGetRTSCTSDuration(pDevice, RTSDUR_AA, + cbFrameLength, byPktType, + wCurrentRate, bNeedAck, + byFBOption); buf->duration_ba = - cpu_to_le16((u16)s_uGetRTSCTSDuration(pDevice, RTSDUR_BA, cbFrameLength, - byPktType, wCurrentRate, bNeedAck, - byFBOption)); + s_uGetRTSCTSDuration(pDevice, RTSDUR_BA, + cbFrameLength, byPktType, + wCurrentRate, bNeedAck, + byFBOption); buf->rts_duration_ba_f0 = - cpu_to_le16((u16)s_uGetRTSCTSDuration(pDevice, RTSDUR_BA_F0, cbFrameLength, - byPktType, wCurrentRate, bNeedAck, - byFBOption)); + s_uGetRTSCTSDuration(pDevice, RTSDUR_BA_F0, + cbFrameLength, byPktType, + wCurrentRate, bNeedAck, + byFBOption); buf->rts_duration_aa_f0 = - cpu_to_le16((u16)s_uGetRTSCTSDuration(pDevice, RTSDUR_AA_F0, cbFrameLength, - byPktType, wCurrentRate, bNeedAck, - byFBOption)); + s_uGetRTSCTSDuration(pDevice, RTSDUR_AA_F0, + cbFrameLength, byPktType, + wCurrentRate, bNeedAck, + byFBOption); buf->rts_duration_ba_f1 = - cpu_to_le16((u16)s_uGetRTSCTSDuration(pDevice, RTSDUR_BA_F1, cbFrameLength, - byPktType, wCurrentRate, bNeedAck, - byFBOption)); + s_uGetRTSCTSDuration(pDevice, RTSDUR_BA_F1, + cbFrameLength, byPktType, + wCurrentRate, bNeedAck, + byFBOption); buf->rts_duration_aa_f1 = - cpu_to_le16((u16)s_uGetRTSCTSDuration(pDevice, RTSDUR_AA_F1, cbFrameLength, - byPktType, wCurrentRate, bNeedAck, - byFBOption)); + s_uGetRTSCTSDuration(pDevice, RTSDUR_AA_F1, + cbFrameLength, byPktType, + wCurrentRate, bNeedAck, + byFBOption); buf->data.duration = buf->duration_aa; /* Get RTS Frame body */ buf->data.frame_control = @@ -927,9 +937,10 @@ s_vFillRTSHead( byPktType, &buf->ab); /* Get Duration */ buf->duration = - cpu_to_le16((u16)s_uGetRTSCTSDuration(pDevice, RTSDUR_AA, cbFrameLength, - byPktType, wCurrentRate, bNeedAck, - byFBOption)); + s_uGetRTSCTSDuration(pDevice, RTSDUR_AA, + cbFrameLength, byPktType, + wCurrentRate, bNeedAck, + byFBOption); buf->data.duration = buf->duration; /* Get RTS Frame body */ buf->data.frame_control = @@ -957,17 +968,20 @@ s_vFillRTSHead( byPktType, &buf->a); /* Get Duration */ buf->duration = - cpu_to_le16((u16)s_uGetRTSCTSDuration(pDevice, RTSDUR_AA, cbFrameLength, - byPktType, wCurrentRate, - bNeedAck, byFBOption)); + s_uGetRTSCTSDuration(pDevice, RTSDUR_AA, + cbFrameLength, byPktType, + wCurrentRate, bNeedAck, + byFBOption); buf->rts_duration_f0 = - cpu_to_le16((u16)s_uGetRTSCTSDuration(pDevice, RTSDUR_AA_F0, cbFrameLength, - byPktType, wCurrentRate, - bNeedAck, byFBOption)); + s_uGetRTSCTSDuration(pDevice, RTSDUR_AA_F0, + cbFrameLength, byPktType, + wCurrentRate, bNeedAck, + byFBOption); buf->rts_duration_f1 = - cpu_to_le16((u16)s_uGetRTSCTSDuration(pDevice, RTSDUR_AA_F1, cbFrameLength, - byPktType, wCurrentRate, - bNeedAck, byFBOption)); + s_uGetRTSCTSDuration(pDevice, RTSDUR_AA_F1, + cbFrameLength, byPktType, + wCurrentRate, bNeedAck, + byFBOption); buf->data.duration = buf->duration; /* Get RTS Frame body */ buf->data.frame_control = @@ -993,9 +1007,10 @@ s_vFillRTSHead( PK_TYPE_11B, &buf->ab); /* Get Duration */ buf->duration = - cpu_to_le16((u16)s_uGetRTSCTSDuration(pDevice, RTSDUR_BB, cbFrameLength, - byPktType, wCurrentRate, bNeedAck, - byFBOption)); + s_uGetRTSCTSDuration(pDevice, RTSDUR_BB, cbFrameLength, + byPktType, wCurrentRate, bNeedAck, + byFBOption); + buf->data.duration = buf->duration; /* Get RTS Frame body */ buf->data.frame_control = @@ -1049,21 +1064,26 @@ s_vFillCTSHead( pDevice->byTopCCKBasicRate, PK_TYPE_11B, &buf->b); - buf->duration_ba = (u16)s_uGetRTSCTSDuration(pDevice, CTSDUR_BA, cbFrameLength, - byPktType, wCurrentRate, bNeedAck, byFBOption); - buf->duration_ba = cpu_to_le16(buf->duration_ba); - /* Get CTSDuration_ba_f0 */ - buf->cts_duration_ba_f0 = (u16)s_uGetRTSCTSDuration(pDevice, CTSDUR_BA_F0, cbFrameLength, - byPktType, wCurrentRate, - bNeedAck, byFBOption); + buf->duration_ba = + s_uGetRTSCTSDuration(pDevice, CTSDUR_BA, + cbFrameLength, byPktType, + wCurrentRate, bNeedAck, + byFBOption); - buf->cts_duration_ba_f0 = cpu_to_le16(buf->cts_duration_ba_f0); - //Get CTSDuration_ba_f1 - buf->cts_duration_ba_f1 = (u16)s_uGetRTSCTSDuration(pDevice, CTSDUR_BA_F1, cbFrameLength, - byPktType, wCurrentRate, - bNeedAck, byFBOption); + /* Get CTSDuration_ba_f0 */ + buf->cts_duration_ba_f0 = + s_uGetRTSCTSDuration(pDevice, CTSDUR_BA_F0, + cbFrameLength, byPktType, + wCurrentRate, bNeedAck, + byFBOption); + + /* Get CTSDuration_ba_f1 */ + buf->cts_duration_ba_f1 = + s_uGetRTSCTSDuration(pDevice, CTSDUR_BA_F1, + cbFrameLength, byPktType, + wCurrentRate, bNeedAck, + byFBOption); - buf->cts_duration_ba_f1 = cpu_to_le16(buf->cts_duration_ba_f1); /* Get CTS Frame body */ buf->data.duration = buf->duration_ba; @@ -1083,11 +1103,10 @@ s_vFillCTSHead( /* Get CTSDuration_ba */ buf->duration_ba = - cpu_to_le16((u16)s_uGetRTSCTSDuration(pDevice, CTSDUR_BA, cbFrameLength, - byPktType, wCurrentRate, bNeedAck, - byFBOption)); - - buf->duration_ba = cpu_to_le16(buf->duration_ba); + s_uGetRTSCTSDuration(pDevice, CTSDUR_BA, + cbFrameLength, byPktType, + wCurrentRate, bNeedAck, + byFBOption); /* Get CTS Frame body */ buf->data.duration = buf->duration_ba; -- 2.20.1