Put the first parameter immediately after the '(' and align the other parameters underneath it. Put static and void on a single and function type together. Code style warnings reported by checkpatch. Signed-off-by: Aliya Rahmani <aliyarahmani786@xxxxxxxxx> --- v1->v2 changes: Put static and void on a single line. v2->v3->v4 changes: Rework commit description. v4->v5 changes: Put storage class and function type on a single line. Rework commit description and subject. --- drivers/staging/vt6655/rxtx.c | 24 ++++++++++-------------- 1 file changed, 10 insertions(+), 14 deletions(-) diff --git a/drivers/staging/vt6655/rxtx.c b/drivers/staging/vt6655/rxtx.c index 53506e242a96..8127807070a7 100644 --- a/drivers/staging/vt6655/rxtx.c +++ b/drivers/staging/vt6655/rxtx.c @@ -567,19 +567,15 @@ s_uFillDataHead( return buf->duration; } -static -void -s_vFillRTSHead( - struct vnt_private *pDevice, - unsigned char byPktType, - void *pvRTS, - unsigned int cbFrameLength, - bool bNeedAck, - bool bDisCRC, - struct ieee80211_hdr *hdr, - unsigned short wCurrentRate, - unsigned char byFBOption -) +static void s_vFillRTSHead(struct vnt_private *pDevice, + unsigned char byPktType, + void *pvRTS, + unsigned int cbFrameLength, + bool bNeedAck, + bool bDisCRC, + struct ieee80211_hdr *hdr, + unsigned short wCurrentRate, + unsigned char byFBOption) { unsigned int uRTSFrameLen = 20; @@ -601,7 +597,7 @@ s_vFillRTSHead( if (byFBOption == AUTO_FB_NONE) { struct vnt_rts_g *buf = pvRTS; /* Get SignalField, ServiceField & Length */ - vnt_get_phy_field(pDevice, uRTSFrameLen, + vnt_get_phy_field(pDevice, uRTSFrameLen pDevice->byTopCCKBasicRate, PK_TYPE_11B, &buf->b); -- 2.25.1