Message ID | Y2OPStg4jXtWpJcd@elroy-temp-vm.gaiao0uenmiufjlowqgp5yxwdh.gvxx.internal.cloudapp.net |
---|---|
State | New |
Headers |
Return-Path: <linux-kernel-owner@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:a5d:6687:0:0:0:0:0 with SMTP id l7csp424472wru; Thu, 3 Nov 2022 02:55:15 -0700 (PDT) X-Google-Smtp-Source: AMsMyM6caD0Gy0w7P8XyAczQtMu4aOc/iKBbGusj22MljKXWtSFhRa6TIDxgKK1Lh4xphqw3iKdS X-Received: by 2002:a17:90a:2acd:b0:213:e2a8:39be with SMTP id i13-20020a17090a2acd00b00213e2a839bemr20359797pjg.205.1667469315297; Thu, 03 Nov 2022 02:55:15 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1667469315; cv=none; d=google.com; s=arc-20160816; b=K2Z+C4H90axyRkz6L3ba3n8N+y3D4ltPdFGJGi9OV6Y7g4aiEWIpWlPT6hC24OGf2v SL4XCdc8jfKNw7uXvvEOwI4kJfcz2FTQB203QmVP4OaXAhUsynEaB84qdVbHgRdB/5MU NF5uEztxDVZNJq6ZFjeZTmeQmofS1ndI2I0RJkuaOYvD9WhWlwfHETdP4Y7BOkWzptWA lAlVA2jvl/0O6AH2X3NoEZBUqYpcFxlkLPiyhOSrWn5Wpa5Yir5yiliyK1vBrjYuimgv awNCUfKWvPVso1o3HZd4nzOxsCH1JRDonNvVYsaFWplEathrTcAIfRgDOcTm/PCZ+h6V lw/A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-disposition:mime-version:message-id :subject:to:from:date:dkim-signature; bh=dwDKMymvmRecMVv+Yu1YP/lKTyHvR7o4RJWcrXGZQkU=; b=0Hs5kpP0tQb0cfhXfeHxjH7r8WSkuYH3uyVftQtX+1iVJBi/Pw6Lso+vHKYK5QaEeB 0QHwh0ZA2Q5ELhdoegbo//KgLKglt4xXByzMBqdwS7Om4ffv9Z29E2rIOo3+Ddm4F/uf +EhToBEzkaXvRti0S11wyLzaA4qiZRXDQBwzP1gmmdLe5jVBLdsZsfN2YWXqbNfHrfnp DLoNKuqXGa/NK4uEBNsyIjHRpulwUqqGaY6r47Iz8acLoUg0xI3cirui8ZVyYE2U9Dhm dYqBraez9gTEpikmP6FquieUxTSM19EqAIAmfqUBA+aEbnRqL5xGGlt7Qb3GHytYJduH byuw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20210112 header.b=Fsb+tFo1; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id t34-20020a632262000000b0046ec7bec2b2si560902pgm.159.2022.11.03.02.55.02; Thu, 03 Nov 2022 02:55:15 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20210112 header.b=Fsb+tFo1; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230507AbiKCJwQ (ORCPT <rfc822;yves.mi.zy@gmail.com> + 99 others); Thu, 3 Nov 2022 05:52:16 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:60922 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229551AbiKCJwN (ORCPT <rfc822;linux-kernel@vger.kernel.org>); Thu, 3 Nov 2022 05:52:13 -0400 Received: from mail-lf1-x136.google.com (mail-lf1-x136.google.com [IPv6:2a00:1450:4864:20::136]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id DBCDF64E3 for <linux-kernel@vger.kernel.org>; Thu, 3 Nov 2022 02:52:12 -0700 (PDT) Received: by mail-lf1-x136.google.com with SMTP id b3so1934749lfv.2 for <linux-kernel@vger.kernel.org>; Thu, 03 Nov 2022 02:52:12 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=content-disposition:mime-version:message-id:subject:to:from:date :from:to:cc:subject:date:message-id:reply-to; bh=dwDKMymvmRecMVv+Yu1YP/lKTyHvR7o4RJWcrXGZQkU=; b=Fsb+tFo1Mh+LJNBrbfqJfBMoJ5E9/H6dHJ4VWiP8OZYHAgHW/s6QawW3a/J9EvVHaP r3qGufzumL2LpzHJdti2X0Eg+3hSfVMOSa9cBj2ajbJA4zU2cSYlxoyLl9UnaRTZ5LGt dMKVobvn7TiN51VHTZDTOnRnqBcHHcIndH2R5tnNc8fmPAth5AgPCXHa0NbMJUxhtSxR hl1cuC4Q0I7NAevpiGBHvTXMj8RUvIStA9cO8uYSfKCRAvjIRtpvUsP1+mKEx5TcxsnA KDZfDYnSomVE04Zfq0+18s43vbM8/6yW471A6x4ZU3IFNpd+/liAErS3eVkGH4nEdFgi H9nQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-disposition:mime-version:message-id:subject:to:from:date :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=dwDKMymvmRecMVv+Yu1YP/lKTyHvR7o4RJWcrXGZQkU=; b=VfUuCTGPMZ7byyHMrwsNQucjOo9GDsDcC4cfJ59TDDBz9Opqbl9b/4ABzLsnnfAzbX npdZ5ynyVAB6JDcyWWC2P15oVro1tlhgYWc2PIiToqNMHcKNRhFh36DrSrb01gM+Oqy5 SvjjkIfWZDQhyBnzx2Avzh8qdMbaBDUsbSmiFZiOID9N3tvLFTDnH/fHhUJWsFPoZdoX NHzQ1NUAHLPX/96ybm+V+luFz0f8s7TKB4pSVobBgM8bV+UGPEPi94Ot3z74+jfQGR0j LJ289Tl7F4MFBjdWThjegZzjwlyBWVlt0UAlqtXx2L7wIxF0tWVazqpWh/OGEjz4ji8+ hBbg== X-Gm-Message-State: ACrzQf0yp73dQx8dYx3tsGTWjkm1eQfwu3j/tmX3+SJluSX4oBOVmx+x J1UVZ/TQrGd4LwtkS/vtYfoyi8LbgznMQw== X-Received: by 2002:a05:6512:1108:b0:4a2:504f:b3f with SMTP id l8-20020a056512110800b004a2504f0b3fmr10980794lfg.169.1667469131064; Thu, 03 Nov 2022 02:52:11 -0700 (PDT) Received: from elroy-temp-vm ([20.240.130.248]) by smtp.gmail.com with ESMTPSA id x26-20020a05651c105a00b002770eafaafbsm30792ljm.99.2022.11.03.02.52.10 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 03 Nov 2022 02:52:10 -0700 (PDT) Date: Thu, 3 Nov 2022 09:52:10 +0000 From: Tanjuate Brunostar <tanjubrunostar0@gmail.com> To: gregkh@linuxfoundation.org, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, outreachy@lists.linux.dev Subject: [PATCH] staging: vt6655: change the function name s_vFillRTSHead Message-ID: <Y2OPStg4jXtWpJcd@elroy-temp-vm.gaiao0uenmiufjlowqgp5yxwdh.gvxx.internal.cloudapp.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline X-Spam-Status: No, score=-1.8 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FREEMAIL_ENVFROM_END_DIGIT, FREEMAIL_FROM,RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: <linux-kernel.vger.kernel.org> X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1748393911824348006?= X-GMAIL-MSGID: =?utf-8?q?1748468304812533244?= |
Series |
staging: vt6655: change the function name s_vFillRTSHead
|
|
Commit Message
Tanjuate Brunostar
Nov. 3, 2022, 9:52 a.m. UTC
Remove the use of Hungarian notation, which is not used in the Linux
kernel. Reported by checkpatch
Fix checkpatch error related to code line ends with a '(', by joining
some lines and indenting correctly. This improves visibility
Signed-off-by: Tanjuate Brunostar <tanjubrunostar0@gmail.com>
---
drivers/staging/vt6655/rxtx.c | 48 ++++++++++++++++-------------------
1 file changed, 22 insertions(+), 26 deletions(-)
Comments
On Thu, 3 Nov 2022, Tanjuate Brunostar wrote: > Remove the use of Hungarian notation, which is not used in the Linux > kernel. Reported by checkpatch > > Fix checkpatch error related to code line ends with a '(', by joining > some lines and indenting correctly. This improves visibility I don't think it will be appreciated to do two things at once. > > Signed-off-by: Tanjuate Brunostar <tanjubrunostar0@gmail.com> > --- > drivers/staging/vt6655/rxtx.c | 48 ++++++++++++++++------------------- > 1 file changed, 22 insertions(+), 26 deletions(-) > > diff --git a/drivers/staging/vt6655/rxtx.c b/drivers/staging/vt6655/rxtx.c > index 31ae99b3cb35..debc5d5daede 100644 > --- a/drivers/staging/vt6655/rxtx.c > +++ b/drivers/staging/vt6655/rxtx.c > @@ -23,7 +23,7 @@ > * s_uGetTxRsvTime- get frame reserved time > * s_vFillCTSHead- fulfill CTS ctl header > * s_vFillFragParameter- Set fragment ctl parameter. > - * s_vFillRTSHead- fulfill RTS ctl header > + * fill_rts_header- fulfill RTS ctl header > * s_vFillTxKey- fulfill tx encrypt key > * s_vSWencryption- Software encrypt header > * vDMA0_tx_80211- tx 802.11 frame via dma0 > @@ -85,15 +85,15 @@ static const unsigned short fb_opt1[2][5] = { > #define DATADUR_A_F1 13 > > /*--------------------- Static Functions --------------------------*/ > -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 fill_rts_header(struct vnt_private *pDevice, > + unsigned char byPktType, > + void *pvRTS, > + unsigned int cbFrameLength, Strange spacing in the above parameter declaration. Maybe there is a tab that should be a space? julia > + bool bNeedAck, > + bool bDisCRC, > + struct ieee80211_hdr *hdr, > + unsigned short wCurrentRate, > + unsigned char byFBOption); > > static void s_vGenerateTxParameter(struct vnt_private *pDevice, > unsigned char byPktType, > @@ -555,19 +555,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 fill_rts_header(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; > > @@ -912,7 +908,7 @@ s_vGenerateTxParameter( > buf->rrv_time_a = vnt_rxtx_rsvtime_le16(pDevice, byPktType, cbFrameSize, wCurrentRate, bNeedACK); > buf->rrv_time_b = vnt_rxtx_rsvtime_le16(pDevice, PK_TYPE_11B, cbFrameSize, pDevice->byTopCCKBasicRate, bNeedACK); > > - s_vFillRTSHead(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > + fill_rts_header(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > } else {/* RTS_needless, PCF mode */ > struct vnt_rrv_time_cts *buf = pvRrvTime; > > @@ -931,7 +927,7 @@ s_vGenerateTxParameter( > buf->rrv_time = vnt_rxtx_rsvtime_le16(pDevice, byPktType, cbFrameSize, wCurrentRate, bNeedACK); > > /* Fill RTS */ > - s_vFillRTSHead(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > + fill_rts_header(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > } else if (!pvRTS) {/* RTS_needless, non PCF mode */ > struct vnt_rrv_time_ab *buf = pvRrvTime; > > @@ -945,7 +941,7 @@ s_vGenerateTxParameter( > buf->rrv_time = vnt_rxtx_rsvtime_le16(pDevice, PK_TYPE_11B, cbFrameSize, wCurrentRate, bNeedACK); > > /* Fill RTS */ > - s_vFillRTSHead(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > + fill_rts_header(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > } else { /* RTS_needless, non PCF mode */ > struct vnt_rrv_time_ab *buf = pvRrvTime; > > -- > 2.34.1 > > >
On Thu, Nov 3, 2022 at 1:38 PM Julia Lawall <julia.lawall@inria.fr> wrote: > > > > On Thu, 3 Nov 2022, Tanjuate Brunostar wrote: > > > Remove the use of Hungarian notation, which is not used in the Linux > > kernel. Reported by checkpatch > > > > Fix checkpatch error related to code line ends with a '(', by joining > > some lines and indenting correctly. This improves visibility > > I don't think it will be appreciated to do two things at once. > I sent it earlier as two patches but it seemed to break and fix the kernel which was not appreciated by one of the mentors > > > > Signed-off-by: Tanjuate Brunostar <tanjubrunostar0@gmail.com> > > --- > > drivers/staging/vt6655/rxtx.c | 48 ++++++++++++++++------------------- > > 1 file changed, 22 insertions(+), 26 deletions(-) > > > > diff --git a/drivers/staging/vt6655/rxtx.c b/drivers/staging/vt6655/rxtx.c > > index 31ae99b3cb35..debc5d5daede 100644 > > --- a/drivers/staging/vt6655/rxtx.c > > +++ b/drivers/staging/vt6655/rxtx.c > > @@ -23,7 +23,7 @@ > > * s_uGetTxRsvTime- get frame reserved time > > * s_vFillCTSHead- fulfill CTS ctl header > > * s_vFillFragParameter- Set fragment ctl parameter. > > - * s_vFillRTSHead- fulfill RTS ctl header > > + * fill_rts_header- fulfill RTS ctl header > > * s_vFillTxKey- fulfill tx encrypt key > > * s_vSWencryption- Software encrypt header > > * vDMA0_tx_80211- tx 802.11 frame via dma0 > > @@ -85,15 +85,15 @@ static const unsigned short fb_opt1[2][5] = { > > #define DATADUR_A_F1 13 > > > > /*--------------------- Static Functions --------------------------*/ > > -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 fill_rts_header(struct vnt_private *pDevice, > > + unsigned char byPktType, > > + void *pvRTS, > > + unsigned int cbFrameLength, > > Strange spacing in the above parameter declaration. Maybe there is a tab > that should be a space? > > julia > I was also shocked when I noticed that. It seemed like a tab but when I removed the space, the two words were joined and when I added the space, it became this lage space once again. checkpatch does not complain about it though > > + bool bNeedAck, > > + bool bDisCRC, > > + struct ieee80211_hdr *hdr, > > + unsigned short wCurrentRate, > > + unsigned char byFBOption); > > > > static void s_vGenerateTxParameter(struct vnt_private *pDevice, > > unsigned char byPktType, > > @@ -555,19 +555,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 fill_rts_header(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; > > > > @@ -912,7 +908,7 @@ s_vGenerateTxParameter( > > buf->rrv_time_a = vnt_rxtx_rsvtime_le16(pDevice, byPktType, cbFrameSize, wCurrentRate, bNeedACK); > > buf->rrv_time_b = vnt_rxtx_rsvtime_le16(pDevice, PK_TYPE_11B, cbFrameSize, pDevice->byTopCCKBasicRate, bNeedACK); > > > > - s_vFillRTSHead(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > > + fill_rts_header(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > > } else {/* RTS_needless, PCF mode */ > > struct vnt_rrv_time_cts *buf = pvRrvTime; > > > > @@ -931,7 +927,7 @@ s_vGenerateTxParameter( > > buf->rrv_time = vnt_rxtx_rsvtime_le16(pDevice, byPktType, cbFrameSize, wCurrentRate, bNeedACK); > > > > /* Fill RTS */ > > - s_vFillRTSHead(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > > + fill_rts_header(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > > } else if (!pvRTS) {/* RTS_needless, non PCF mode */ > > struct vnt_rrv_time_ab *buf = pvRrvTime; > > > > @@ -945,7 +941,7 @@ s_vGenerateTxParameter( > > buf->rrv_time = vnt_rxtx_rsvtime_le16(pDevice, PK_TYPE_11B, cbFrameSize, wCurrentRate, bNeedACK); > > > > /* Fill RTS */ > > - s_vFillRTSHead(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > > + fill_rts_header(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > > } else { /* RTS_needless, non PCF mode */ > > struct vnt_rrv_time_ab *buf = pvRrvTime; > > > > -- > > 2.34.1 > > > > > >
On Thu, 3 Nov 2022, Tanju Brunostar wrote: > On Thu, Nov 3, 2022 at 1:38 PM Julia Lawall <julia.lawall@inria.fr> wrote: > > > > > > > > On Thu, 3 Nov 2022, Tanjuate Brunostar wrote: > > > > > Remove the use of Hungarian notation, which is not used in the Linux > > > kernel. Reported by checkpatch > > > > > > Fix checkpatch error related to code line ends with a '(', by joining > > > some lines and indenting correctly. This improves visibility > > > > I don't think it will be appreciated to do two things at once. > > > > I sent it earlier as two patches but it seemed to break and fix the > kernel which was not appreciated by one of the mentors OK, then it is a presentation problem. You have presented the changes as unrelated. Rewrite the message to make clear what is being done. julia > > > > > > > Signed-off-by: Tanjuate Brunostar <tanjubrunostar0@gmail.com> > > > --- > > > drivers/staging/vt6655/rxtx.c | 48 ++++++++++++++++------------------- > > > 1 file changed, 22 insertions(+), 26 deletions(-) > > > > > > diff --git a/drivers/staging/vt6655/rxtx.c b/drivers/staging/vt6655/rxtx.c > > > index 31ae99b3cb35..debc5d5daede 100644 > > > --- a/drivers/staging/vt6655/rxtx.c > > > +++ b/drivers/staging/vt6655/rxtx.c > > > @@ -23,7 +23,7 @@ > > > * s_uGetTxRsvTime- get frame reserved time > > > * s_vFillCTSHead- fulfill CTS ctl header > > > * s_vFillFragParameter- Set fragment ctl parameter. > > > - * s_vFillRTSHead- fulfill RTS ctl header > > > + * fill_rts_header- fulfill RTS ctl header > > > * s_vFillTxKey- fulfill tx encrypt key > > > * s_vSWencryption- Software encrypt header > > > * vDMA0_tx_80211- tx 802.11 frame via dma0 > > > @@ -85,15 +85,15 @@ static const unsigned short fb_opt1[2][5] = { > > > #define DATADUR_A_F1 13 > > > > > > /*--------------------- Static Functions --------------------------*/ > > > -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 fill_rts_header(struct vnt_private *pDevice, > > > + unsigned char byPktType, > > > + void *pvRTS, > > > + unsigned int cbFrameLength, > > > > Strange spacing in the above parameter declaration. Maybe there is a tab > > that should be a space? > > > > julia > > > > I was also shocked when I noticed that. It seemed like a tab but when > I removed the space, the two words were joined and when I added the > space, it became this lage space once again. checkpatch does not > complain about it though > > > > + bool bNeedAck, > > > + bool bDisCRC, > > > + struct ieee80211_hdr *hdr, > > > + unsigned short wCurrentRate, > > > + unsigned char byFBOption); > > > > > > static void s_vGenerateTxParameter(struct vnt_private *pDevice, > > > unsigned char byPktType, > > > @@ -555,19 +555,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 fill_rts_header(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; > > > > > > @@ -912,7 +908,7 @@ s_vGenerateTxParameter( > > > buf->rrv_time_a = vnt_rxtx_rsvtime_le16(pDevice, byPktType, cbFrameSize, wCurrentRate, bNeedACK); > > > buf->rrv_time_b = vnt_rxtx_rsvtime_le16(pDevice, PK_TYPE_11B, cbFrameSize, pDevice->byTopCCKBasicRate, bNeedACK); > > > > > > - s_vFillRTSHead(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > > > + fill_rts_header(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > > > } else {/* RTS_needless, PCF mode */ > > > struct vnt_rrv_time_cts *buf = pvRrvTime; > > > > > > @@ -931,7 +927,7 @@ s_vGenerateTxParameter( > > > buf->rrv_time = vnt_rxtx_rsvtime_le16(pDevice, byPktType, cbFrameSize, wCurrentRate, bNeedACK); > > > > > > /* Fill RTS */ > > > - s_vFillRTSHead(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > > > + fill_rts_header(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > > > } else if (!pvRTS) {/* RTS_needless, non PCF mode */ > > > struct vnt_rrv_time_ab *buf = pvRrvTime; > > > > > > @@ -945,7 +941,7 @@ s_vGenerateTxParameter( > > > buf->rrv_time = vnt_rxtx_rsvtime_le16(pDevice, PK_TYPE_11B, cbFrameSize, wCurrentRate, bNeedACK); > > > > > > /* Fill RTS */ > > > - s_vFillRTSHead(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > > > + fill_rts_header(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > > > } else { /* RTS_needless, non PCF mode */ > > > struct vnt_rrv_time_ab *buf = pvRrvTime; > > > > > > -- > > > 2.34.1 > > > > > > > > > >
On Thu, Nov 3, 2022 at 4:58 PM Julia Lawall <julia.lawall@inria.fr> wrote: > > > > On Thu, 3 Nov 2022, Tanju Brunostar wrote: > > > On Thu, Nov 3, 2022 at 1:38 PM Julia Lawall <julia.lawall@inria.fr> wrote: > > > > > > > > > > > > On Thu, 3 Nov 2022, Tanjuate Brunostar wrote: > > > > > > > Remove the use of Hungarian notation, which is not used in the Linux > > > > kernel. Reported by checkpatch > > > > > > > > Fix checkpatch error related to code line ends with a '(', by joining > > > > some lines and indenting correctly. This improves visibility > > > > > > I don't think it will be appreciated to do two things at once. > > > > > > > I sent it earlier as two patches but it seemed to break and fix the > > kernel which was not appreciated by one of the mentors > > OK, then it is a presentation problem. You have presented the changes as > unrelated. Rewrite the message to make clear what is being done. > > julia > okay > > > > > > > > > > Signed-off-by: Tanjuate Brunostar <tanjubrunostar0@gmail.com> > > > > --- > > > > drivers/staging/vt6655/rxtx.c | 48 ++++++++++++++++------------------- > > > > 1 file changed, 22 insertions(+), 26 deletions(-) > > > > > > > > diff --git a/drivers/staging/vt6655/rxtx.c b/drivers/staging/vt6655/rxtx.c > > > > index 31ae99b3cb35..debc5d5daede 100644 > > > > --- a/drivers/staging/vt6655/rxtx.c > > > > +++ b/drivers/staging/vt6655/rxtx.c > > > > @@ -23,7 +23,7 @@ > > > > * s_uGetTxRsvTime- get frame reserved time > > > > * s_vFillCTSHead- fulfill CTS ctl header > > > > * s_vFillFragParameter- Set fragment ctl parameter. > > > > - * s_vFillRTSHead- fulfill RTS ctl header > > > > + * fill_rts_header- fulfill RTS ctl header > > > > * s_vFillTxKey- fulfill tx encrypt key > > > > * s_vSWencryption- Software encrypt header > > > > * vDMA0_tx_80211- tx 802.11 frame via dma0 > > > > @@ -85,15 +85,15 @@ static const unsigned short fb_opt1[2][5] = { > > > > #define DATADUR_A_F1 13 > > > > > > > > /*--------------------- Static Functions --------------------------*/ > > > > -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 fill_rts_header(struct vnt_private *pDevice, > > > > + unsigned char byPktType, > > > > + void *pvRTS, > > > > + unsigned int cbFrameLength, > > > > > > Strange spacing in the above parameter declaration. Maybe there is a tab > > > that should be a space? > > > > > > julia > > > > > > > I was also shocked when I noticed that. It seemed like a tab but when > > I removed the space, the two words were joined and when I added the > > space, it became this lage space once again. checkpatch does not > > complain about it though > > > > > > + bool bNeedAck, > > > > + bool bDisCRC, > > > > + struct ieee80211_hdr *hdr, > > > > + unsigned short wCurrentRate, > > > > + unsigned char byFBOption); > > > > > > > > static void s_vGenerateTxParameter(struct vnt_private *pDevice, > > > > unsigned char byPktType, > > > > @@ -555,19 +555,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 fill_rts_header(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; > > > > > > > > @@ -912,7 +908,7 @@ s_vGenerateTxParameter( > > > > buf->rrv_time_a = vnt_rxtx_rsvtime_le16(pDevice, byPktType, cbFrameSize, wCurrentRate, bNeedACK); > > > > buf->rrv_time_b = vnt_rxtx_rsvtime_le16(pDevice, PK_TYPE_11B, cbFrameSize, pDevice->byTopCCKBasicRate, bNeedACK); > > > > > > > > - s_vFillRTSHead(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > > > > + fill_rts_header(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > > > > } else {/* RTS_needless, PCF mode */ > > > > struct vnt_rrv_time_cts *buf = pvRrvTime; > > > > > > > > @@ -931,7 +927,7 @@ s_vGenerateTxParameter( > > > > buf->rrv_time = vnt_rxtx_rsvtime_le16(pDevice, byPktType, cbFrameSize, wCurrentRate, bNeedACK); > > > > > > > > /* Fill RTS */ > > > > - s_vFillRTSHead(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > > > > + fill_rts_header(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > > > > } else if (!pvRTS) {/* RTS_needless, non PCF mode */ > > > > struct vnt_rrv_time_ab *buf = pvRrvTime; > > > > > > > > @@ -945,7 +941,7 @@ s_vGenerateTxParameter( > > > > buf->rrv_time = vnt_rxtx_rsvtime_le16(pDevice, PK_TYPE_11B, cbFrameSize, wCurrentRate, bNeedACK); > > > > > > > > /* Fill RTS */ > > > > - s_vFillRTSHead(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > > > > + fill_rts_header(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > > > > } else { /* RTS_needless, non PCF mode */ > > > > struct vnt_rrv_time_ab *buf = pvRrvTime; > > > > > > > > -- > > > > 2.34.1 > > > > > > > > > > > > > > >
On Thu, Nov 3, 2022 at 5:04 PM Tanju Brunostar <tanjubrunostar0@gmail.com> wrote: > > On Thu, Nov 3, 2022 at 4:58 PM Julia Lawall <julia.lawall@inria.fr> wrote: > > > > > > > > On Thu, 3 Nov 2022, Tanju Brunostar wrote: > > > > > On Thu, Nov 3, 2022 at 1:38 PM Julia Lawall <julia.lawall@inria.fr> wrote: > > > > > > > > > > > > > > > > On Thu, 3 Nov 2022, Tanjuate Brunostar wrote: > > > > > > > > > Remove the use of Hungarian notation, which is not used in the Linux > > > > > kernel. Reported by checkpatch > > > > > > > > > > Fix checkpatch error related to code line ends with a '(', by joining > > > > > some lines and indenting correctly. This improves visibility > > > > > > > > I don't think it will be appreciated to do two things at once. > > > > > > > > > > I sent it earlier as two patches but it seemed to break and fix the > > > kernel which was not appreciated by one of the mentors > > > > OK, then it is a presentation problem. You have presented the changes as > > unrelated. Rewrite the message to make clear what is being done. > > > > julia > > Hi Julia, I am having a hard time figuring out exactly how to write this, since I made both changes in one patch and there is no way to send them in separate patches. should I just report just one change and leave the other? I would appreciate your help here Thanks tanju > okay > > > > > > > > > > > > > Signed-off-by: Tanjuate Brunostar <tanjubrunostar0@gmail.com> > > > > > --- > > > > > drivers/staging/vt6655/rxtx.c | 48 ++++++++++++++++------------------- > > > > > 1 file changed, 22 insertions(+), 26 deletions(-) > > > > > > > > > > diff --git a/drivers/staging/vt6655/rxtx.c b/drivers/staging/vt6655/rxtx.c > > > > > index 31ae99b3cb35..debc5d5daede 100644 > > > > > --- a/drivers/staging/vt6655/rxtx.c > > > > > +++ b/drivers/staging/vt6655/rxtx.c > > > > > @@ -23,7 +23,7 @@ > > > > > * s_uGetTxRsvTime- get frame reserved time > > > > > * s_vFillCTSHead- fulfill CTS ctl header > > > > > * s_vFillFragParameter- Set fragment ctl parameter. > > > > > - * s_vFillRTSHead- fulfill RTS ctl header > > > > > + * fill_rts_header- fulfill RTS ctl header > > > > > * s_vFillTxKey- fulfill tx encrypt key > > > > > * s_vSWencryption- Software encrypt header > > > > > * vDMA0_tx_80211- tx 802.11 frame via dma0 > > > > > @@ -85,15 +85,15 @@ static const unsigned short fb_opt1[2][5] = { > > > > > #define DATADUR_A_F1 13 > > > > > > > > > > /*--------------------- Static Functions --------------------------*/ > > > > > -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 fill_rts_header(struct vnt_private *pDevice, > > > > > + unsigned char byPktType, > > > > > + void *pvRTS, > > > > > + unsigned int cbFrameLength, > > > > > > > > Strange spacing in the above parameter declaration. Maybe there is a tab > > > > that should be a space? > > > > > > > > julia > > > > > > > > > > I was also shocked when I noticed that. It seemed like a tab but when > > > I removed the space, the two words were joined and when I added the > > > space, it became this lage space once again. checkpatch does not > > > complain about it though > > > > > > > > + bool bNeedAck, > > > > > + bool bDisCRC, > > > > > + struct ieee80211_hdr *hdr, > > > > > + unsigned short wCurrentRate, > > > > > + unsigned char byFBOption); > > > > > > > > > > static void s_vGenerateTxParameter(struct vnt_private *pDevice, > > > > > unsigned char byPktType, > > > > > @@ -555,19 +555,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 fill_rts_header(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; > > > > > > > > > > @@ -912,7 +908,7 @@ s_vGenerateTxParameter( > > > > > buf->rrv_time_a = vnt_rxtx_rsvtime_le16(pDevice, byPktType, cbFrameSize, wCurrentRate, bNeedACK); > > > > > buf->rrv_time_b = vnt_rxtx_rsvtime_le16(pDevice, PK_TYPE_11B, cbFrameSize, pDevice->byTopCCKBasicRate, bNeedACK); > > > > > > > > > > - s_vFillRTSHead(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > > > > > + fill_rts_header(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > > > > > } else {/* RTS_needless, PCF mode */ > > > > > struct vnt_rrv_time_cts *buf = pvRrvTime; > > > > > > > > > > @@ -931,7 +927,7 @@ s_vGenerateTxParameter( > > > > > buf->rrv_time = vnt_rxtx_rsvtime_le16(pDevice, byPktType, cbFrameSize, wCurrentRate, bNeedACK); > > > > > > > > > > /* Fill RTS */ > > > > > - s_vFillRTSHead(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > > > > > + fill_rts_header(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > > > > > } else if (!pvRTS) {/* RTS_needless, non PCF mode */ > > > > > struct vnt_rrv_time_ab *buf = pvRrvTime; > > > > > > > > > > @@ -945,7 +941,7 @@ s_vGenerateTxParameter( > > > > > buf->rrv_time = vnt_rxtx_rsvtime_le16(pDevice, PK_TYPE_11B, cbFrameSize, wCurrentRate, bNeedACK); > > > > > > > > > > /* Fill RTS */ > > > > > - s_vFillRTSHead(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > > > > > + fill_rts_header(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > > > > > } else { /* RTS_needless, non PCF mode */ > > > > > struct vnt_rrv_time_ab *buf = pvRrvTime; > > > > > > > > > > -- > > > > > 2.34.1 > > > > > > > > > > > > > > > > > > > >
On Thu, 3 Nov 2022, Tanju Brunostar wrote: > On Thu, Nov 3, 2022 at 5:04 PM Tanju Brunostar > <tanjubrunostar0@gmail.com> wrote: > > > > On Thu, Nov 3, 2022 at 4:58 PM Julia Lawall <julia.lawall@inria.fr> wrote: > > > > > > > > > > > > On Thu, 3 Nov 2022, Tanju Brunostar wrote: > > > > > > > On Thu, Nov 3, 2022 at 1:38 PM Julia Lawall <julia.lawall@inria.fr> wrote: > > > > > > > > > > > > > > > > > > > > On Thu, 3 Nov 2022, Tanjuate Brunostar wrote: > > > > > > > > > > > Remove the use of Hungarian notation, which is not used in the Linux > > > > > > kernel. Reported by checkpatch > > > > > > > > > > > > Fix checkpatch error related to code line ends with a '(', by joining > > > > > > some lines and indenting correctly. This improves visibility > > > > > > > > > > I don't think it will be appreciated to do two things at once. > > > > > > > > > > > > > I sent it earlier as two patches but it seemed to break and fix the > > > > kernel which was not appreciated by one of the mentors > > > > > > OK, then it is a presentation problem. You have presented the changes as > > > unrelated. Rewrite the message to make clear what is being done. > > > > > > julia > > > > > Hi Julia, > I am having a hard time figuring out exactly how to write this, since > I made both changes in one patch and there is no way to send them in > separate patches. should I just report just one change and leave the > other? I would appreciate your help here Add indentation to the affected function headers to follow the Linux kernel coding style? julia > Thanks > > tanju > > > okay > > > > > > > > > > > > > > > > Signed-off-by: Tanjuate Brunostar <tanjubrunostar0@gmail.com> > > > > > > --- > > > > > > drivers/staging/vt6655/rxtx.c | 48 ++++++++++++++++------------------- > > > > > > 1 file changed, 22 insertions(+), 26 deletions(-) > > > > > > > > > > > > diff --git a/drivers/staging/vt6655/rxtx.c b/drivers/staging/vt6655/rxtx.c > > > > > > index 31ae99b3cb35..debc5d5daede 100644 > > > > > > --- a/drivers/staging/vt6655/rxtx.c > > > > > > +++ b/drivers/staging/vt6655/rxtx.c > > > > > > @@ -23,7 +23,7 @@ > > > > > > * s_uGetTxRsvTime- get frame reserved time > > > > > > * s_vFillCTSHead- fulfill CTS ctl header > > > > > > * s_vFillFragParameter- Set fragment ctl parameter. > > > > > > - * s_vFillRTSHead- fulfill RTS ctl header > > > > > > + * fill_rts_header- fulfill RTS ctl header > > > > > > * s_vFillTxKey- fulfill tx encrypt key > > > > > > * s_vSWencryption- Software encrypt header > > > > > > * vDMA0_tx_80211- tx 802.11 frame via dma0 > > > > > > @@ -85,15 +85,15 @@ static const unsigned short fb_opt1[2][5] = { > > > > > > #define DATADUR_A_F1 13 > > > > > > > > > > > > /*--------------------- Static Functions --------------------------*/ > > > > > > -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 fill_rts_header(struct vnt_private *pDevice, > > > > > > + unsigned char byPktType, > > > > > > + void *pvRTS, > > > > > > + unsigned int cbFrameLength, > > > > > > > > > > Strange spacing in the above parameter declaration. Maybe there is a tab > > > > > that should be a space? > > > > > > > > > > julia > > > > > > > > > > > > > I was also shocked when I noticed that. It seemed like a tab but when > > > > I removed the space, the two words were joined and when I added the > > > > space, it became this lage space once again. checkpatch does not > > > > complain about it though > > > > > > > > > > + bool bNeedAck, > > > > > > + bool bDisCRC, > > > > > > + struct ieee80211_hdr *hdr, > > > > > > + unsigned short wCurrentRate, > > > > > > + unsigned char byFBOption); > > > > > > > > > > > > static void s_vGenerateTxParameter(struct vnt_private *pDevice, > > > > > > unsigned char byPktType, > > > > > > @@ -555,19 +555,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 fill_rts_header(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; > > > > > > > > > > > > @@ -912,7 +908,7 @@ s_vGenerateTxParameter( > > > > > > buf->rrv_time_a = vnt_rxtx_rsvtime_le16(pDevice, byPktType, cbFrameSize, wCurrentRate, bNeedACK); > > > > > > buf->rrv_time_b = vnt_rxtx_rsvtime_le16(pDevice, PK_TYPE_11B, cbFrameSize, pDevice->byTopCCKBasicRate, bNeedACK); > > > > > > > > > > > > - s_vFillRTSHead(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > > > > > > + fill_rts_header(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > > > > > > } else {/* RTS_needless, PCF mode */ > > > > > > struct vnt_rrv_time_cts *buf = pvRrvTime; > > > > > > > > > > > > @@ -931,7 +927,7 @@ s_vGenerateTxParameter( > > > > > > buf->rrv_time = vnt_rxtx_rsvtime_le16(pDevice, byPktType, cbFrameSize, wCurrentRate, bNeedACK); > > > > > > > > > > > > /* Fill RTS */ > > > > > > - s_vFillRTSHead(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > > > > > > + fill_rts_header(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > > > > > > } else if (!pvRTS) {/* RTS_needless, non PCF mode */ > > > > > > struct vnt_rrv_time_ab *buf = pvRrvTime; > > > > > > > > > > > > @@ -945,7 +941,7 @@ s_vGenerateTxParameter( > > > > > > buf->rrv_time = vnt_rxtx_rsvtime_le16(pDevice, PK_TYPE_11B, cbFrameSize, wCurrentRate, bNeedACK); > > > > > > > > > > > > /* Fill RTS */ > > > > > > - s_vFillRTSHead(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > > > > > > + fill_rts_header(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > > > > > > } else { /* RTS_needless, non PCF mode */ > > > > > > struct vnt_rrv_time_ab *buf = pvRrvTime; > > > > > > > > > > > > -- > > > > > > 2.34.1 > > > > > > > > > > > > > > > > > > > > > > > > > >
On Thu, Nov 3, 2022 at 6:51 PM Julia Lawall <julia.lawall@inria.fr> wrote: > > > > On Thu, 3 Nov 2022, Tanju Brunostar wrote: > > > On Thu, Nov 3, 2022 at 5:04 PM Tanju Brunostar > > <tanjubrunostar0@gmail.com> wrote: > > > > > > On Thu, Nov 3, 2022 at 4:58 PM Julia Lawall <julia.lawall@inria.fr> wrote: > > > > > > > > > > > > > > > > On Thu, 3 Nov 2022, Tanju Brunostar wrote: > > > > > > > > > On Thu, Nov 3, 2022 at 1:38 PM Julia Lawall <julia.lawall@inria.fr> wrote: > > > > > > > > > > > > > > > > > > > > > > > > On Thu, 3 Nov 2022, Tanjuate Brunostar wrote: > > > > > > > > > > > > > Remove the use of Hungarian notation, which is not used in the Linux > > > > > > > kernel. Reported by checkpatch > > > > > > > > > > > > > > Fix checkpatch error related to code line ends with a '(', by joining > > > > > > > some lines and indenting correctly. This improves visibility > > > > > > > > > > > > I don't think it will be appreciated to do two things at once. > > > > > > > > > > > > > > > > I sent it earlier as two patches but it seemed to break and fix the > > > > > kernel which was not appreciated by one of the mentors > > > > > > > > OK, then it is a presentation problem. You have presented the changes as > > > > unrelated. Rewrite the message to make clear what is being done. > > > > > > > > julia > > > > > > > > Hi Julia, > > I am having a hard time figuring out exactly how to write this, since > > I made both changes in one patch and there is no way to send them in > > separate patches. should I just report just one change and leave the > > other? I would appreciate your help here > > Add indentation to the affected function headers to follow the Linux > kernel coding style? > > julia > Thank you > > > Thanks > > > > tanju > > > > > okay > > > > > > > > > > > > > > > > > > > Signed-off-by: Tanjuate Brunostar <tanjubrunostar0@gmail.com> > > > > > > > --- > > > > > > > drivers/staging/vt6655/rxtx.c | 48 ++++++++++++++++------------------- > > > > > > > 1 file changed, 22 insertions(+), 26 deletions(-) > > > > > > > > > > > > > > diff --git a/drivers/staging/vt6655/rxtx.c b/drivers/staging/vt6655/rxtx.c > > > > > > > index 31ae99b3cb35..debc5d5daede 100644 > > > > > > > --- a/drivers/staging/vt6655/rxtx.c > > > > > > > +++ b/drivers/staging/vt6655/rxtx.c > > > > > > > @@ -23,7 +23,7 @@ > > > > > > > * s_uGetTxRsvTime- get frame reserved time > > > > > > > * s_vFillCTSHead- fulfill CTS ctl header > > > > > > > * s_vFillFragParameter- Set fragment ctl parameter. > > > > > > > - * s_vFillRTSHead- fulfill RTS ctl header > > > > > > > + * fill_rts_header- fulfill RTS ctl header > > > > > > > * s_vFillTxKey- fulfill tx encrypt key > > > > > > > * s_vSWencryption- Software encrypt header > > > > > > > * vDMA0_tx_80211- tx 802.11 frame via dma0 > > > > > > > @@ -85,15 +85,15 @@ static const unsigned short fb_opt1[2][5] = { > > > > > > > #define DATADUR_A_F1 13 > > > > > > > > > > > > > > /*--------------------- Static Functions --------------------------*/ > > > > > > > -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 fill_rts_header(struct vnt_private *pDevice, > > > > > > > + unsigned char byPktType, > > > > > > > + void *pvRTS, > > > > > > > + unsigned int cbFrameLength, > > > > > > > > > > > > Strange spacing in the above parameter declaration. Maybe there is a tab > > > > > > that should be a space? > > > > > > > > > > > > julia > > > > > > > > > > > > > > > > I was also shocked when I noticed that. It seemed like a tab but when > > > > > I removed the space, the two words were joined and when I added the > > > > > space, it became this lage space once again. checkpatch does not > > > > > complain about it though > > > > > > > > > > > > + bool bNeedAck, > > > > > > > + bool bDisCRC, > > > > > > > + struct ieee80211_hdr *hdr, > > > > > > > + unsigned short wCurrentRate, > > > > > > > + unsigned char byFBOption); > > > > > > > > > > > > > > static void s_vGenerateTxParameter(struct vnt_private *pDevice, > > > > > > > unsigned char byPktType, > > > > > > > @@ -555,19 +555,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 fill_rts_header(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; > > > > > > > > > > > > > > @@ -912,7 +908,7 @@ s_vGenerateTxParameter( > > > > > > > buf->rrv_time_a = vnt_rxtx_rsvtime_le16(pDevice, byPktType, cbFrameSize, wCurrentRate, bNeedACK); > > > > > > > buf->rrv_time_b = vnt_rxtx_rsvtime_le16(pDevice, PK_TYPE_11B, cbFrameSize, pDevice->byTopCCKBasicRate, bNeedACK); > > > > > > > > > > > > > > - s_vFillRTSHead(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > > > > > > > + fill_rts_header(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > > > > > > > } else {/* RTS_needless, PCF mode */ > > > > > > > struct vnt_rrv_time_cts *buf = pvRrvTime; > > > > > > > > > > > > > > @@ -931,7 +927,7 @@ s_vGenerateTxParameter( > > > > > > > buf->rrv_time = vnt_rxtx_rsvtime_le16(pDevice, byPktType, cbFrameSize, wCurrentRate, bNeedACK); > > > > > > > > > > > > > > /* Fill RTS */ > > > > > > > - s_vFillRTSHead(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > > > > > > > + fill_rts_header(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > > > > > > > } else if (!pvRTS) {/* RTS_needless, non PCF mode */ > > > > > > > struct vnt_rrv_time_ab *buf = pvRrvTime; > > > > > > > > > > > > > > @@ -945,7 +941,7 @@ s_vGenerateTxParameter( > > > > > > > buf->rrv_time = vnt_rxtx_rsvtime_le16(pDevice, PK_TYPE_11B, cbFrameSize, wCurrentRate, bNeedACK); > > > > > > > > > > > > > > /* Fill RTS */ > > > > > > > - s_vFillRTSHead(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > > > > > > > + fill_rts_header(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); > > > > > > > } else { /* RTS_needless, non PCF mode */ > > > > > > > struct vnt_rrv_time_ab *buf = pvRrvTime; > > > > > > > > > > > > > > -- > > > > > > > 2.34.1 > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
diff --git a/drivers/staging/vt6655/rxtx.c b/drivers/staging/vt6655/rxtx.c index 31ae99b3cb35..debc5d5daede 100644 --- a/drivers/staging/vt6655/rxtx.c +++ b/drivers/staging/vt6655/rxtx.c @@ -23,7 +23,7 @@ * s_uGetTxRsvTime- get frame reserved time * s_vFillCTSHead- fulfill CTS ctl header * s_vFillFragParameter- Set fragment ctl parameter. - * s_vFillRTSHead- fulfill RTS ctl header + * fill_rts_header- fulfill RTS ctl header * s_vFillTxKey- fulfill tx encrypt key * s_vSWencryption- Software encrypt header * vDMA0_tx_80211- tx 802.11 frame via dma0 @@ -85,15 +85,15 @@ static const unsigned short fb_opt1[2][5] = { #define DATADUR_A_F1 13 /*--------------------- Static Functions --------------------------*/ -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 fill_rts_header(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_vGenerateTxParameter(struct vnt_private *pDevice, unsigned char byPktType, @@ -555,19 +555,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 fill_rts_header(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; @@ -912,7 +908,7 @@ s_vGenerateTxParameter( buf->rrv_time_a = vnt_rxtx_rsvtime_le16(pDevice, byPktType, cbFrameSize, wCurrentRate, bNeedACK); buf->rrv_time_b = vnt_rxtx_rsvtime_le16(pDevice, PK_TYPE_11B, cbFrameSize, pDevice->byTopCCKBasicRate, bNeedACK); - s_vFillRTSHead(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); + fill_rts_header(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); } else {/* RTS_needless, PCF mode */ struct vnt_rrv_time_cts *buf = pvRrvTime; @@ -931,7 +927,7 @@ s_vGenerateTxParameter( buf->rrv_time = vnt_rxtx_rsvtime_le16(pDevice, byPktType, cbFrameSize, wCurrentRate, bNeedACK); /* Fill RTS */ - s_vFillRTSHead(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); + fill_rts_header(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); } else if (!pvRTS) {/* RTS_needless, non PCF mode */ struct vnt_rrv_time_ab *buf = pvRrvTime; @@ -945,7 +941,7 @@ s_vGenerateTxParameter( buf->rrv_time = vnt_rxtx_rsvtime_le16(pDevice, PK_TYPE_11B, cbFrameSize, wCurrentRate, bNeedACK); /* Fill RTS */ - s_vFillRTSHead(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); + fill_rts_header(pDevice, byPktType, pvRTS, cbFrameSize, bNeedACK, bDisCRC, psEthHeader, wCurrentRate, byFBOption); } else { /* RTS_needless, non PCF mode */ struct vnt_rrv_time_ab *buf = pvRrvTime;