From: Philipp Hortmann <philipp.g.hortmann@gmail.com>
To: Gary Rookard <garyrookard@fastmail.org>, gregkh@linuxfoundation.org
Cc: linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v3 0/5] staging: rtl8192e: renaming 5 different variables patch series
Date: Thu, 23 Nov 2023 22:11:54 +0100 [thread overview]
Message-ID: <b98d32fa-6860-4ef0-9c45-6602cfc5880e@gmail.com> (raw)
In-Reply-To: <20231123144337.13112-1-garyrookard@fastmail.org>
On 11/23/23 15:43, Gary Rookard wrote:
> Hi,
>
> This patch series renames (5) different variables from Pascal/CamelCase to
> Snake case.
>
> Patch 1/5) renamed variable HTMcsToDataRate
> Patch 2/5) renamed variable TxCountToDataRate
> Patch 3/5) renamed variable IsHtHalfNmodeAps
> Patch 4/5) renamed variable HTIOTPeerDetermine
> Patch 5/5) renamed variable HTIOTActIsMgntUseCCK6M
>
> Linux kernel coding style (cleanup), checkpatch Avoid CamelCase.
> Driver/module rtl8192e compiles.
>
> Regards,
> Gary
>
> ---
> v3: Corrected versioning from v1 -> v3, includes change history.
> v2: v2 was skipped over..v3 corrects this.
> v1: Style and compile issues, asked to redo.
>
> Gary Rookard (5):
> staging: rtl8192e; renamed variable HTMcsToDataRate
> staging: rtl8192e: renamed variable TXCountToDataRate
> staging: rtl8192e: renamed variable IsHTHalfNmodeAPs
> staging: rtl8192e: renamed variable HTIOTPeerDetermine
> staging: rtl8192e: renamed variable HTIOTActIsMgntUseCCK6M
>
> drivers/staging/rtl8192e/rtl819x_HTProc.c | 18 +++++++++---------
> drivers/staging/rtl8192e/rtllib.h | 4 ++--
> drivers/staging/rtl8192e/rtllib_softmac.c | 2 +-
> drivers/staging/rtl8192e/rtllib_softmac_wx.c | 2 +-
> 4 files changed, 13 insertions(+), 13 deletions(-)
>
So beside the two
CHECK: Alignment should match open parenthesis issues everything else is OK.
Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
prev parent reply other threads:[~2023-11-23 21:11 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-11-23 14:43 [PATCH v3 0/5] staging: rtl8192e: renaming 5 different variables patch series Gary Rookard
2023-11-23 14:43 ` [PATCH v3 1/5] staging: rtl8192e: renamed variable HTMcsToDataRate Gary Rookard
2023-11-23 14:43 ` [PATCH v3 2/5] staging: rtl8192e: renamed variable TXCountToDataRate Gary Rookard
2023-11-23 20:56 ` Philipp Hortmann
2023-11-23 23:19 ` Gary Rookard
2023-11-24 11:30 ` Greg KH
2023-11-23 14:43 ` [PATCH v3 3/5] staging: rtl8192e: renamed variable IsHTHalfNmodeAPs Gary Rookard
2023-11-23 14:43 ` [PATCH v3 4/5] staging: rtl8192e: renamed variable HTIOTPeerDetermine Gary Rookard
2023-11-23 14:43 ` [PATCH v3 5/5] staging: rtl8192e: renamed variable HTIOTActIsMgntUseCCK6M Gary Rookard
2023-11-23 21:05 ` Philipp Hortmann
2023-11-23 21:11 ` Philipp Hortmann [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=b98d32fa-6860-4ef0-9c45-6602cfc5880e@gmail.com \
--to=philipp.g.hortmann@gmail.com \
--cc=garyrookard@fastmail.org \
--cc=gregkh@linuxfoundation.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-staging@lists.linux.dev \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.