From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 221E7BA41; Fri, 28 Oct 2022 18:55:42 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1666983343; x=1698519343; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=mbASiVGvErzkpUdFk2i4WMCyzZgdpNopIv441Gv81DY=; b=aI/YWv9f0gIBVhYbvMcverx06X2jHfG3BOIKRVGU7/n0T+lgABJVUWad 9ePoj4lT4/z1+3SK2g0gpZKHlajVNzDQlclhFaCAhCzU2HzBvYWpnOgl4 oKxwrWVFH7M44G9E1Lc6jyqm71YdbQvhEnWEZOCm5jc1boSd44UR1Cass 6hJKaCMHiiL6dCywzF9A/YfgjjL7yfctnkfrRkXk4LWjI24/swaG7CLMf 1VAtgsxJ7if70hW+D6G//4/SFIl49IybDrlDHWMwhxTfSLwj9syvIbE4h bWWcItqrL6+0U4u7OQOzhO4ZNW0FvVeQV56xBYvo/GYze8d+nuqL5K1zM A==; X-IronPort-AV: E=McAfee;i="6500,9779,10514"; a="310264240" X-IronPort-AV: E=Sophos;i="5.95,222,1661842800"; d="scan'208";a="310264240" Received: from orsmga004.jf.intel.com ([10.7.209.38]) by orsmga103.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 28 Oct 2022 11:55:42 -0700 X-IronPort-AV: E=McAfee;i="6500,9779,10514"; a="758181868" X-IronPort-AV: E=Sophos;i="5.95,222,1661842800"; d="scan'208";a="758181868" Received: from aschofie-mobl2.amr.corp.intel.com (HELO aschofie-mobl2) ([10.212.175.207]) by orsmga004-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 28 Oct 2022 11:55:38 -0700 Date: Fri, 28 Oct 2022 11:55:36 -0700 From: Alison Schofield To: Tanjuate Brunostar , gregkh@linuxfoundation.org Cc: linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, outreachy@lists.linux.dev Subject: Re: [PATCH] staging: vt6655: fix lines of code ending in a '(' Message-ID: References: Precedence: bulk X-Mailing-List: linux-staging@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: On Fri, Oct 28, 2022 at 06:28:30PM +0000, Tanjuate Brunostar wrote: > fix checkpatch errors related to lines ending with a '(' > by joining splitted lines of code and indenting properly to improve > visibility > > Signed-off-by: Tanjuate Brunostar --- > drivers/staging/vt6655/rxtx.c | 165 ++++++++++++++-------------------- > 1 file changed, 69 insertions(+), 96 deletions(-) > > diff --git a/drivers/staging/vt6655/rxtx.c b/drivers/staging/vt6655/rxtx.c > index 1e5036121665..7eb7c6eb5cf0 100644 > --- a/drivers/staging/vt6655/rxtx.c > +++ b/drivers/staging/vt6655/rxtx.c > @@ -139,15 +139,11 @@ static __le16 vnt_time_stamp_off(struct vnt_private *priv, u16 rate) > * PK_TYPE_11GB 2 > * PK_TYPE_11GA 3 > */ > -static > -unsigned int > -s_uGetTxRsvTime( > - struct vnt_private *pDevice, > - unsigned char byPktType, > - unsigned int cbFrameLength, > - unsigned short wRate, > - bool bNeedAck > -) > +static unsigned int s_uGetTxRsvTime(struct vnt_private *pDevice, > + unsigned char byPktType, > + unsigned int cbFrameLength, > + unsigned short wRate, > + bool bNeedAck) > { > unsigned int uDataTime, uAckTime; > > @@ -214,20 +210,16 @@ static __le16 get_rtscts_time(struct vnt_private *priv, > } > > /* byFreqType 0: 5GHz, 1:2.4Ghz */ > -static > -unsigned int > -s_uGetDataDuration( > - struct vnt_private *pDevice, > - unsigned char byDurType, > - unsigned int cbFrameLength, > - unsigned char byPktType, > - unsigned short wRate, > - bool bNeedAck, > - unsigned int uFragIdx, > - unsigned int cbLastFragmentSize, > - unsigned int uMACfragNum, > - unsigned char byFBOption > -) > +static unsigned int s_uGetDataDuration(struct vnt_private *pDevice, > + unsigned char byDurType, > + unsigned int cbFrameLength, > + unsigned char byPktType, > + unsigned short wRate, > + bool bNeedAck, > + unsigned int uFragIdx, > + unsigned int cbLastFragmentSize, > + unsigned int uMACfragNum, > + unsigned char byFBOption) > { > bool bLastFrag = false; > unsigned int uAckTime = 0, uNextPktTime = 0, len; > @@ -316,17 +308,13 @@ s_uGetDataDuration( > } > > /* byFreqType: 0=>5GHZ 1=>2.4GHZ */ > -static > -__le16 > -s_uGetRTSCTSDuration( > - struct vnt_private *pDevice, > - unsigned char byDurType, > - unsigned int cbFrameLength, > - unsigned char byPktType, > - unsigned short wRate, > - bool bNeedAck, > - unsigned char byFBOption > -) > +static __le16 s_uGetRTSCTSDuration(struct vnt_private *pDevice, > + unsigned char byDurType, > + unsigned int cbFrameLength, > + unsigned char byPktType, > + unsigned short wRate, > + bool bNeedAck, > + unsigned char byFBOption) > { > unsigned int uCTSTime = 0, uDurTime = 0; > > @@ -409,22 +397,18 @@ s_uGetRTSCTSDuration( > return cpu_to_le16((u16)uDurTime); > } > > -static > -__le16 > -s_uFillDataHead( > - struct vnt_private *pDevice, > - unsigned char byPktType, > - void *pTxDataHead, > - unsigned int cbFrameLength, > - unsigned int uDMAIdx, > - bool bNeedAck, > - unsigned int uFragIdx, > - unsigned int cbLastFragmentSize, > - unsigned int uMACfragNum, > - unsigned char byFBOption, > - unsigned short wCurrentRate, > - bool is_pspoll > -) > +static __le16 s_uFillDataHead(struct vnt_private *pDevice, > + unsigned char byPktType, > + void *pTxDataHead, > + unsigned int cbFrameLength, > + unsigned int uDMAIdx, > + bool bNeedAck, > + unsigned int uFragIdx, > + unsigned int cbLastFragmentSize, > + unsigned int uMACfragNum, > + unsigned char byFBOption, > + unsigned short wCurrentRate, > + bool is_pspoll) > { > struct vnt_tx_datahead_ab *buf = pTxDataHead; > > @@ -555,19 +539,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; > > @@ -750,19 +730,15 @@ s_vFillRTSHead( > } > } > > -static > -void > -s_vFillCTSHead( > - struct vnt_private *pDevice, > - unsigned int uDMAIdx, > - unsigned char byPktType, > - void *pvCTS, > - unsigned int cbFrameLength, > - bool bNeedAck, > - bool bDisCRC, > - unsigned short wCurrentRate, > - unsigned char byFBOption > -) > +static void s_vFillCTSHead(struct vnt_private *pDevice, > + unsigned int uDMAIdx, > + unsigned char byPktType, > + void *pvCTS, > + unsigned int cbFrameLength, > + bool bNeedAck, > + bool bDisCRC, > + unsigned short wCurrentRate, > + unsigned char byFBOption) > { > unsigned int uCTSFrameLen = 14; > > @@ -868,21 +844,17 @@ s_vFillCTSHead( > - > * unsigned int cbFrameSize, Hdr+Payload+FCS > */ > -static > -void > -s_vGenerateTxParameter( > - struct vnt_private *pDevice, > - unsigned char byPktType, > - struct vnt_tx_fifo_head *tx_buffer_head, > - void *pvRrvTime, > - void *pvRTS, > - void *pvCTS, > - unsigned int cbFrameSize, > - bool bNeedACK, > - unsigned int uDMAIdx, > - void *psEthHeader, > - unsigned short wCurrentRate > -) > +static void s_vGenerateTxParameter(struct vnt_private *pDevice, > + unsigned char byPktType, > + struct vnt_tx_fifo_head *tx_buffer_head, > + void *pvRrvTime, > + void *pvRTS, > + void *pvCTS, > + unsigned int cbFrameSize, > + bool bNeedACK, > + unsigned int uDMAIdx, > + void *psEthHeader, > + unsigned short wCurrentRate) > { > u16 fifo_ctl = le16_to_cpu(tx_buffer_head->fifo_ctl); > bool bDisCRC = false; > @@ -954,11 +926,12 @@ s_vGenerateTxParameter( > } > } > > -static unsigned int > -s_cbFillTxBufHead(struct vnt_private *pDevice, unsigned char byPktType, > - unsigned char *pbyTxBufferAddr, > - unsigned int uDMAIdx, struct vnt_tx_desc *pHeadTD, > - unsigned int is_pspoll) > +static unsigned int s_cbFillTxBufHead(struct vnt_private *pDevice, > + unsigned char byPktType, > + unsigned char *pbyTxBufferAddr, > + unsigned int uDMAIdx, > + struct vnt_tx_desc *pHeadTD, > + unsigned int is_pspoll) > { > struct vnt_td_info *td_info = pHeadTD->td_info; > struct sk_buff *skb = td_info->skb; > -- > 2.34.1 > >