* [PATCH v6 0/2] staging: rtl8192e: Clean up patchset for style issues in rtl819x_HTProc.c
@ 2021-04-14 7:21 Mitali Borkar
2021-04-14 7:21 ` [PATCH v6 1/2] staging: rtl8192e: remove parentheses around boolean expression Mitali Borkar
2021-04-14 7:22 ` [PATCH v6 2/2] staging: rtl8192e: remove casts and parentheses Mitali Borkar
0 siblings, 2 replies; 4+ messages in thread
From: Mitali Borkar @ 2021-04-14 7:21 UTC (permalink / raw)
To: gregkh; +Cc: linux-staging, linux-kernel, outreachy-kernel, mitali_s
Changes from v5:- Rebased these patches and dropped 2/3 and made this
a patchset of 2.
[PATCH v5 1/2]:- No changes.
[PATCH V5 2/2]:- No changes.
Changes from v4:-
[PATCH v4 1/2]:- No changes.
[PATCH V4 2/2]:- Removed casts and parentheses.
Changes from v3:- Changed subject line to match prefix on the patches.
[PATCH v3 1/2]:- No changes.
[PATCH V3 2/2]:- No changes.
Changes from v2:-
[PATCH v2 1/2]:- Modified subject description. Changes has been made in
v3.
[PATCH v2 2/2]:- Rectified spelling mistake in subject description.
Changes has been made in v3.
Changes from v1:-
[PATCH 1/2]:- Removed unnecessary parentheses around boolean expression.
Changes has been made in v2.
[PATCH 2/2]:- No changes.
Mitali Borkar (2):
staging: rtl8192e: remove parentheses around boolean expression
staging: rtl8192e: remove casts and parentheses
drivers/staging/rtl8192e/rtl819x_HTProc.c | 14 +--
1 file changed, 5 insertions(+), 9 deletions(-)
--
2.30.2
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH v6 1/2] staging: rtl8192e: remove parentheses around boolean expression
2021-04-14 7:21 [PATCH v6 0/2] staging: rtl8192e: Clean up patchset for style issues in rtl819x_HTProc.c Mitali Borkar
@ 2021-04-14 7:21 ` Mitali Borkar
2021-04-14 8:31 ` Greg KH
2021-04-14 7:22 ` [PATCH v6 2/2] staging: rtl8192e: remove casts and parentheses Mitali Borkar
1 sibling, 1 reply; 4+ messages in thread
From: Mitali Borkar @ 2021-04-14 7:21 UTC (permalink / raw)
To: gregkh; +Cc: linux-staging, linux-kernel, outreachy-kernel, mitali_s
Removed unnecessary parentheses around '!xyz' boolean expression as '!'
has higher precedance than '||'
Signed-off-by: Mitali Borkar <mitaliborkar810@gmail.com>
---
Changes from v5:- No changes.
Changes from v4:- No changes.
Changes from v3:- No changes.
Changes from v2:- Modified subject description. Changes has been made in
v3.
Changes from v1:- Removed unnecessary parentheses around boolean
expression. Changes has been made in v2.
drivers/staging/rtl8192e/rtl819x_HTProc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8192e/rtl819x_HTProc.c b/drivers/staging/rtl8192e/rtl819x_HTProc.c
index b1fa8e9a4f28..431202927036 100644
--- a/drivers/staging/rtl8192e/rtl819x_HTProc.c
+++ b/drivers/staging/rtl8192e/rtl819x_HTProc.c
@@ -276,7 +276,7 @@ void HTConstructCapabilityElement(struct rtllib_device *ieee, u8 *posHTCap,
struct rt_hi_throughput *pHT = ieee->pHTInfo;
struct ht_capab_ele *pCapELE = NULL;
- if ((!posHTCap) || (!pHT)) {
+ if (!posHTCap || !pHT) {
netdev_warn(ieee->dev,
"%s(): posHTCap and pHTInfo are null\n", __func__);
return;
--
2.30.2
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH v6 2/2] staging: rtl8192e: remove casts and parentheses
2021-04-14 7:21 [PATCH v6 0/2] staging: rtl8192e: Clean up patchset for style issues in rtl819x_HTProc.c Mitali Borkar
2021-04-14 7:21 ` [PATCH v6 1/2] staging: rtl8192e: remove parentheses around boolean expression Mitali Borkar
@ 2021-04-14 7:22 ` Mitali Borkar
1 sibling, 0 replies; 4+ messages in thread
From: Mitali Borkar @ 2021-04-14 7:22 UTC (permalink / raw)
To: gregkh; +Cc: linux-staging, linux-kernel, outreachy-kernel, mitali_s
Removed unnecessary (void *) cast and parentheses to meet linux kernel
coding style.
Signed-off-by: Mitali Borkar <mitaliborkar810@gmail.com>
---
Changes from v5:- No changes.
Changes from v4:- Removed unnecessary casts and parentheses.
Changes from v3:- No changes.
Changes from v2:- Rectified spelling mistake in subject description.
Changes has been made in v3.
Changes from v1:- No changes.
drivers/staging/rtl8192e/rtl819x_HTProc.c | 12 ++++--------
1 file changed, 4 insertions(+), 8 deletions(-)
diff --git a/drivers/staging/rtl8192e/rtl819x_HTProc.c b/drivers/staging/rtl8192e/rtl819x_HTProc.c
index 431202927036..ec6b46166e84 100644
--- a/drivers/staging/rtl8192e/rtl819x_HTProc.c
+++ b/drivers/staging/rtl8192e/rtl819x_HTProc.c
@@ -646,14 +646,10 @@ void HTInitializeHTInfo(struct rtllib_device *ieee)
pHTInfo->CurrentMPDUDensity = pHTInfo->MPDU_Density;
pHTInfo->CurrentAMPDUFactor = pHTInfo->AMPDU_Factor;
- memset((void *)(&pHTInfo->SelfHTCap), 0,
- sizeof(pHTInfo->SelfHTCap));
- memset((void *)(&pHTInfo->SelfHTInfo), 0,
- sizeof(pHTInfo->SelfHTInfo));
- memset((void *)(&pHTInfo->PeerHTCapBuf), 0,
- sizeof(pHTInfo->PeerHTCapBuf));
- memset((void *)(&pHTInfo->PeerHTInfoBuf), 0,
- sizeof(pHTInfo->PeerHTInfoBuf));
+ memset(&pHTInfo->SelfHTCap, 0, sizeof(pHTInfo->SelfHTCap));
+ memset(&pHTInfo->SelfHTInfo, 0, sizeof(pHTInfo->SelfHTInfo));
+ memset(&pHTInfo->PeerHTCapBuf, 0, sizeof(pHTInfo->PeerHTCapBuf));
+ memset(&pHTInfo->PeerHTInfoBuf, 0, sizeof(pHTInfo->PeerHTInfoBuf));
pHTInfo->bSwBwInProgress = false;
--
2.30.2
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v6 1/2] staging: rtl8192e: remove parentheses around boolean expression
2021-04-14 7:21 ` [PATCH v6 1/2] staging: rtl8192e: remove parentheses around boolean expression Mitali Borkar
@ 2021-04-14 8:31 ` Greg KH
0 siblings, 0 replies; 4+ messages in thread
From: Greg KH @ 2021-04-14 8:31 UTC (permalink / raw)
To: Mitali Borkar; +Cc: linux-staging, linux-kernel, outreachy-kernel, mitali_s
On Wed, Apr 14, 2021 at 12:51:55PM +0530, Mitali Borkar wrote:
> Removed unnecessary parentheses around '!xyz' boolean expression as '!'
> has higher precedance than '||'
>
> Signed-off-by: Mitali Borkar <mitaliborkar810@gmail.com>
> ---
This series does not apply to my tree :(
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2021-04-14 8:31 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-14 7:21 [PATCH v6 0/2] staging: rtl8192e: Clean up patchset for style issues in rtl819x_HTProc.c Mitali Borkar
2021-04-14 7:21 ` [PATCH v6 1/2] staging: rtl8192e: remove parentheses around boolean expression Mitali Borkar
2021-04-14 8:31 ` Greg KH
2021-04-14 7:22 ` [PATCH v6 2/2] staging: rtl8192e: remove casts and parentheses Mitali Borkar
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).