* [PATCH] staging: rtl8712: Fixed multiple parenthesis alignment warnings in ieee80211.c
@ 2016-06-08 11:27 Parth Sane
2016-06-08 11:31 ` Luis de Bethencourt
0 siblings, 1 reply; 5+ messages in thread
From: Parth Sane @ 2016-06-08 11:27 UTC (permalink / raw)
To: gregkh
Cc: Larry.Finger, florian.c.schilhabel, luisbg, punitvara,
stillcompiling, bhaktipriya96, devel, linux-kernel, Parth Sane
Multiple parenthesis alignment warnings were thrown by checkpatch in ieee80211.c
This patch effectively fixes that.
Signed-off-by: Parth Sane <laerdevstudios@gmail.com>
---
drivers/staging/rtl8712/ieee80211.c | 25 ++++++++++++-------------
1 file changed, 12 insertions(+), 13 deletions(-)
diff --git a/drivers/staging/rtl8712/ieee80211.c b/drivers/staging/rtl8712/ieee80211.c
index 8918654..5dc3b5b 100644
--- a/drivers/staging/rtl8712/ieee80211.c
+++ b/drivers/staging/rtl8712/ieee80211.c
@@ -145,7 +145,7 @@ static void set_supported_rate(u8 *rates, uint mode)
case WIRELESS_11BG:
memcpy(rates, WIFI_CCKRATES, IEEE80211_CCK_RATE_LEN);
memcpy(rates + IEEE80211_CCK_RATE_LEN, WIFI_OFDMRATES,
- IEEE80211_NUM_OFDM_RATESLEN);
+ IEEE80211_NUM_OFDM_RATESLEN);
break;
}
}
@@ -188,24 +188,24 @@ int r8712_generate_ie(struct registry_priv *pregistrypriv)
ie += 2;
/*SSID*/
ie = r8712_set_ie(ie, _SSID_IE_, pdev_network->Ssid.SsidLength,
- pdev_network->Ssid.Ssid, &sz);
+ pdev_network->Ssid.Ssid, &sz);
/*supported rates*/
set_supported_rate(pdev_network->rates, pregistrypriv->wireless_mode);
rateLen = r8712_get_rateset_len(pdev_network->rates);
if (rateLen > 8) {
ie = r8712_set_ie(ie, _SUPPORTEDRATES_IE_, 8,
- pdev_network->rates, &sz);
+ pdev_network->rates, &sz);
ie = r8712_set_ie(ie, _EXT_SUPPORTEDRATES_IE_, (rateLen - 8),
- (pdev_network->rates + 8), &sz);
+ (pdev_network->rates + 8), &sz);
} else
ie = r8712_set_ie(ie, _SUPPORTEDRATES_IE_,
- rateLen, pdev_network->rates, &sz);
+ rateLen, pdev_network->rates, &sz);
/*DS parameter set*/
ie = r8712_set_ie(ie, _DSSET_IE_, 1,
- (u8 *)&(pdev_network->Configuration.DSConfig), &sz);
+ (u8 *)&(pdev_network->Configuration.DSConfig), &sz);
/*IBSS Parameter Set*/
ie = r8712_set_ie(ie, _IBSS_PARA_IE_, 2,
- (u8 *)&(pdev_network->Configuration.ATIMWindow), &sz);
+ (u8 *)&(pdev_network->Configuration.ATIMWindow), &sz);
return sz;
}
@@ -220,8 +220,7 @@ unsigned char *r8712_get_wpa_ie(unsigned char *pie, int *wpa_ie_len, int limit)
pbuf = r8712_get_ie(pbuf, _WPA_IE_ID_, &len, limit);
if (pbuf) {
/*check if oui matches...*/
- if (memcmp((pbuf + 2), wpa_oui_type,
- sizeof(wpa_oui_type)))
+ if (memcmp((pbuf + 2), wpa_oui_type, sizeof(wpa_oui_type)))
goto check_next_ie;
/*check version...*/
memcpy((u8 *)&val16, (pbuf + 6), sizeof(val16));
@@ -279,7 +278,7 @@ static int r8712_get_wpa2_cipher_suite(u8 *s)
}
int r8712_parse_wpa_ie(u8 *wpa_ie, int wpa_ie_len, int *group_cipher,
- int *pairwise_cipher)
+ int *pairwise_cipher)
{
int i;
int left, count;
@@ -322,7 +321,7 @@ int r8712_parse_wpa_ie(u8 *wpa_ie, int wpa_ie_len, int *group_cipher,
}
int r8712_parse_wpa2_ie(u8 *rsn_ie, int rsn_ie_len, int *group_cipher,
- int *pairwise_cipher)
+ int *pairwise_cipher)
{
int i;
int left, count;
@@ -365,7 +364,7 @@ int r8712_parse_wpa2_ie(u8 *rsn_ie, int rsn_ie_len, int *group_cipher,
}
int r8712_get_sec_ie(u8 *in_ie, uint in_len, u8 *rsn_ie, u16 *rsn_len,
- u8 *wpa_ie, u16 *wpa_len)
+ u8 *wpa_ie, u16 *wpa_len)
{
u8 authmode;
u8 wpa_oui[4] = {0x0, 0x50, 0xf2, 0x01};
@@ -383,7 +382,7 @@ int r8712_get_sec_ie(u8 *in_ie, uint in_len, u8 *rsn_ie, u16 *rsn_len,
} else {
if (authmode == _WPA2_IE_ID_) {
memcpy(rsn_ie, &in_ie[cnt],
- in_ie[cnt + 1] + 2);
+ in_ie[cnt + 1] + 2);
*rsn_len = in_ie[cnt + 1] + 2;
cnt += in_ie[cnt + 1] + 2; /*get next*/
} else {
--
1.9.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] staging: rtl8712: Fixed multiple parenthesis alignment warnings in ieee80211.c
2016-06-08 11:27 [PATCH] staging: rtl8712: Fixed multiple parenthesis alignment warnings in ieee80211.c Parth Sane
@ 2016-06-08 11:31 ` Luis de Bethencourt
2016-06-08 11:37 ` Parth Sane
0 siblings, 1 reply; 5+ messages in thread
From: Luis de Bethencourt @ 2016-06-08 11:31 UTC (permalink / raw)
To: Parth Sane, gregkh
Cc: Larry.Finger, florian.c.schilhabel, punitvara, stillcompiling,
bhaktipriya96, devel, linux-kernel
On 08/06/16 12:27, Parth Sane wrote:
> Multiple parenthesis alignment warnings were thrown by checkpatch in ieee80211.c
> This patch effectively fixes that.
>
> Signed-off-by: Parth Sane <laerdevstudios@gmail.com>
>
> ---
> drivers/staging/rtl8712/ieee80211.c | 25 ++++++++++++-------------
> 1 file changed, 12 insertions(+), 13 deletions(-)
Some of the lines you change are over 80 characters. You could fix those as well in
a second patch :)
Thanks,
Luis
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] staging: rtl8712: Fixed multiple parenthesis alignment warnings in ieee80211.c
2016-06-08 11:31 ` Luis de Bethencourt
@ 2016-06-08 11:37 ` Parth Sane
[not found] ` <5758064E.6080709@osg.samsung.com>
0 siblings, 1 reply; 5+ messages in thread
From: Parth Sane @ 2016-06-08 11:37 UTC (permalink / raw)
To: Luis de Bethencourt
Cc: Greg KH, Larry.Finger, florian.c.schilhabel, punitvara,
stillcompiling, bhaktipriya96, open list:STAGING SUBSYSTEM,
linux-kernel
[-- Attachment #1: Type: text/plain, Size: 803 bytes --]
> On 08-Jun-2016, at 5:01 PM, Luis de Bethencourt <luisbg@osg.samsung.com> wrote:
>
> On 08/06/16 12:27, Parth Sane wrote:
>> Multiple parenthesis alignment warnings were thrown by checkpatch in ieee80211.c
>> This patch effectively fixes that.
>>
>> Signed-off-by: Parth Sane <laerdevstudios@gmail.com>
>>
>> ---
>> drivers/staging/rtl8712/ieee80211.c | 25 ++++++++++++-------------
>> 1 file changed, 12 insertions(+), 13 deletions(-)
>
> Some of the lines you change are over 80 characters. You could fix those as well in
> a second patch :)
>
> Thanks,
> Luis
>
You’re right on that regard. Maybe once this is accepted I can come back to other warnings. I will submit other patches meanwhile.
I seriously want to see this driver in the next release!
Regards,
Parth
[-- Attachment #2: Message signed with OpenPGP using GPGMail --]
[-- Type: application/pgp-signature, Size: 842 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] staging: rtl8712: Fixed multiple parenthesis alignment warnings in ieee80211.c
[not found] ` <5758064E.6080709@osg.samsung.com>
@ 2016-06-08 11:58 ` Parth Sane
[not found] ` <57580C8B.6040204@osg.samsung.com>
0 siblings, 1 reply; 5+ messages in thread
From: Parth Sane @ 2016-06-08 11:58 UTC (permalink / raw)
To: Luis de Bethencourt
Cc: Greg KH, Larry.Finger, florian.c.schilhabel, punitvara,
stillcompiling, bhaktipriya96, open list:STAGING SUBSYSTEM,
linux-kernel
[-- Attachment #1: Type: text/plain, Size: 1365 bytes --]
I’m trying to fix line 224 but its simply not happening. Can you help me with this?
Regards,
Parth
> On 08-Jun-2016, at 5:19 PM, Luis de Bethencourt <luisbg@osg.samsung.com> wrote:
>
> On 08/06/16 12:37, Parth Sane wrote:
>>
>>> On 08-Jun-2016, at 5:01 PM, Luis de Bethencourt <luisbg@osg.samsung.com> wrote:
>>>
>>> On 08/06/16 12:27, Parth Sane wrote:
>>>> Multiple parenthesis alignment warnings were thrown by checkpatch in ieee80211.c
>>>> This patch effectively fixes that.
>>>>
>>>> Signed-off-by: Parth Sane <laerdevstudios@gmail.com>
>>>>
>>>> ---
>>>> drivers/staging/rtl8712/ieee80211.c | 25 ++++++++++++-------------
>>>> 1 file changed, 12 insertions(+), 13 deletions(-)
>>>
>>> Some of the lines you change are over 80 characters. You could fix those as well in
>>> a second patch :)
>>>
>>> Thanks,
>>> Luis
>>>
>>
>> You’re right on that regard. Maybe once this is accepted I can come back to other warnings. I will submit other patches meanwhile.
>> I seriously want to see this driver in the next release!
>> Regards,
>> Parth
>>
>
> You can resend this patch and the one fixing the 80 char limit with the subject
> prefix [RESEND PATCH v2 x/2].
>
> AFAIK this driver is already in the release, check menuconfig under
> "Device Drivers > Staging drivers > Realtek RTL8172"
>
> Thanks,
> Luis
[-- Attachment #2: Message signed with OpenPGP using GPGMail --]
[-- Type: application/pgp-signature, Size: 842 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] staging: rtl8712: Fixed multiple parenthesis alignment warnings in ieee80211.c
[not found] ` <57580C8B.6040204@osg.samsung.com>
@ 2016-06-08 12:25 ` Parth Sane
0 siblings, 0 replies; 5+ messages in thread
From: Parth Sane @ 2016-06-08 12:25 UTC (permalink / raw)
To: Luis de Bethencourt
Cc: Greg KH, Larry.Finger, florian.c.schilhabel, punitvara,
stillcompiling, bhaktipriya96, open list:STAGING SUBSYSTEM,
linux-kernel
[-- Attachment #1: Type: text/plain, Size: 2189 bytes --]
No mate, that aint just cutting it.
I tried putting it, maybe i’m doing something wrong?
Regards,
Parth
> On 08-Jun-2016, at 5:46 PM, Luis de Bethencourt <luisbg@osg.samsung.com> wrote:
>
> On 08/06/16 12:58, Parth Sane wrote:
>> I’m trying to fix line 224 but its simply not happening. Can you help me with this?
>> Regards,
>> Parth
>>> On 08-Jun-2016, at 5:19 PM, Luis de Bethencourt <luisbg@osg.samsung.com> wrote:
>>>
>>> On 08/06/16 12:37, Parth Sane wrote:
>>>>
>>>>> On 08-Jun-2016, at 5:01 PM, Luis de Bethencourt <luisbg@osg.samsung.com> wrote:
>>>>>
>>>>> On 08/06/16 12:27, Parth Sane wrote:
>>>>>> Multiple parenthesis alignment warnings were thrown by checkpatch in ieee80211.c
>>>>>> This patch effectively fixes that.
>>>>>>
>>>>>> Signed-off-by: Parth Sane <laerdevstudios@gmail.com>
>>>>>>
>>>>>> ---
>>>>>> drivers/staging/rtl8712/ieee80211.c | 25 ++++++++++++-------------
>>>>>> 1 file changed, 12 insertions(+), 13 deletions(-)
>>>>>
>>>>> Some of the lines you change are over 80 characters. You could fix those as well in
>>>>> a second patch :)
>>>>>
>>>>> Thanks,
>>>>> Luis
>>>>>
>>>>
>>>> You’re right on that regard. Maybe once this is accepted I can come back to other warnings. I will submit other patches meanwhile.
>>>> I seriously want to see this driver in the next release!
>>>> Regards,
>>>> Parth
>>>>
>>>
>>> You can resend this patch and the one fixing the 80 char limit with the subject
>>> prefix [RESEND PATCH v2 x/2].
>>>
>>> AFAIK this driver is already in the release, check menuconfig under
>>> "Device Drivers > Staging drivers > Realtek RTL8172"
>>>
>>> Thanks,
>>> Luis
>>
>
> Sure.
>
> You want something like this:
> - if (memcmp((pbuf + 2), wpa_oui_type, sizeof(wpa_oui_type)))
> + if (memcmp((pbuf + 2), wpa_oui_type,
> + sizeof(wpa_oui_type)))
>
> You could check for references of how other parts of the code have solved it
> by doing:
> git grep memcpy * | grep -v ";"
>
> I hope that helps.
> Luis
>
>
>
> Reviewed-by: Luis de Bethencourt <luisbg@osg.samsung.com>
[-- Attachment #2: Message signed with OpenPGP using GPGMail --]
[-- Type: application/pgp-signature, Size: 842 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2016-06-08 12:25 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-06-08 11:27 [PATCH] staging: rtl8712: Fixed multiple parenthesis alignment warnings in ieee80211.c Parth Sane
2016-06-08 11:31 ` Luis de Bethencourt
2016-06-08 11:37 ` Parth Sane
[not found] ` <5758064E.6080709@osg.samsung.com>
2016-06-08 11:58 ` Parth Sane
[not found] ` <57580C8B.6040204@osg.samsung.com>
2016-06-08 12:25 ` Parth Sane
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).