| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Tue, 15 Nov 2022 07:23:38 +0100
From: Philipp Hortmann <philipp.g.hortmann@...il.com>
To: Tanju Brunostar <tanjubrunostar0@...il.com>
Cc: gregkh@...uxfoundation.org, linux-staging@...ts.linux.dev,
linux-kernel@...r.kernel.org, outreachy@...ts.linux.dev
Subject: Re: [PATCH v3] staging: vt6655: change the function name
s_vFillRTSHead
On 11/14/22 23:59, Tanju Brunostar wrote:
> On Mon, Nov 14, 2022 at 9:19 PM Philipp Hortmann
> <philipp.g.hortmann@...il.com> wrote:
>>
>> On 11/14/22 19:29, Tanjuate Brunostar wrote:
>>> Remove the use of Hungarian notation, which is not used in the Linux
>>> kernel. Reported by checkpatch
>>> Add indentation to the affected function to follow the Linux kernel
>>> coding style. This improves visibility
>>>
>>> Signed-off-by: Tanjuate Brunostar <tanjubrunostar0@...il.com>
>>> ---
>>> v2: corrected confusing changlog message on this patch
>>> v3: removed unnecessary function prototype
>>>
>>> drivers/staging/vt6655/rxtx.c | 40 ++++++++++++-----------------------
>>> 1 file changed, 13 insertions(+), 27 deletions(-)
>>>
>>> diff --git a/drivers/staging/vt6655/rxtx.c b/drivers/staging/vt6655/rxtx.c
>>> index 31ae99b3cb35..715b323cd2a8 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,16 +85,6 @@ 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 s_vGenerateTxParameter(struct vnt_private *pDevice,
>>> unsigned char byPktType,
>>> struct vnt_tx_fifo_head *,
>>> @@ -555,19 +545,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 +898,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 +917,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 +931,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;
>>>
>>
>> Hi,
>>
>> I cannot apply your patch.
>>
>> Here my git infos:
>> git remote show origin
>> * remote origin
>> Fetch URL:
>> git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git
>> ...
>> git branch -a
>> my branch: staging-testing
>>
>> Bye Philipp
>
> I was recently asked to rebase to the staging-next branch. I think
> this is where there is conflict
I do not think so.
between this commit:
commit 335140116703920ddcbd9a09ae9edfb02902f3d1 (HEAD ->
staging-testing, origin/staging-testing)
and this one:
commit a614e753e8e7f3322b560dcf6eaf44468ec22b3a (origin/staging-next)
are only 4 commits and non of them is regarding the vt6655.
So this should work.
kernel@...rix-ESPRIMO-P710:~/Documents/git/kernels/staging$ git status
On branch staging-testing
Your branch is up to date with 'origin/staging-testing'.
nothing to commit, working tree clean
kernel@...rix-ESPRIMO-P710:~/Documents/git/kernels/staging$ git pull
Already up to date.
kernel@...rix-ESPRIMO-P710:~/Documents/git/kernels/staging$ cat
~/Downloads/20221114-\[PATCH\ v3\]\ staging_\ vt6655_\ change\ the\
function\ name\ s_vFillRTSHead-5397.txt | git am
Applying: staging: vt6655: change the function name s_vFillRTSHead
error: patch failed: drivers/staging/vt6655/rxtx.c:85
error: drivers/staging/vt6655/rxtx.c: patch does not apply
Patch failed at 0001 staging: vt6655: change the function name
s_vFillRTSHead
hint: Use 'git am --show-current-patch=diff' to see the failed patch
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".
kernel@...rix-ESPRIMO-P710:~/Documents/git/kernels/staging$
Bye Philipp
Powered by blists - more mailing lists