* [PATCH] staging: rtl8723bs: include: Fix identation issue
@ 2019-03-28 18:45 Vatsala Narang
2019-03-29 16:13 ` Greg KH
0 siblings, 1 reply; 4+ messages in thread
From: Vatsala Narang @ 2019-03-28 18:45 UTC (permalink / raw)
To: outreachy-kernel; +Cc: gregkh, Vatsala Narang
Corrected the identation of switch case to get rid of checkpatch error
Signed-off-by: Vatsala Narang <vatsalanarang@gmail.com>
---
drivers/staging/rtl8723bs/include/wifi.h | 72 ++++++++++++------------
1 file changed, 36 insertions(+), 36 deletions(-)
diff --git a/drivers/staging/rtl8723bs/include/wifi.h b/drivers/staging/rtl8723bs/include/wifi.h
index 559bf2606fb7..ee1994ecbf18 100644
--- a/drivers/staging/rtl8723bs/include/wifi.h
+++ b/drivers/staging/rtl8723bs/include/wifi.h
@@ -374,18 +374,18 @@ __inline static unsigned char * get_da(unsigned char *pframe)
unsigned int to_fr_ds = (GetToDs(pframe) << 1) | GetFrDs(pframe);
switch (to_fr_ds) {
- case 0x00: /* ToDs = 0, FromDs = 0 */
- da = GetAddr1Ptr(pframe);
- break;
- case 0x01: /* ToDs = 0, FromDs = 1 */
- da = GetAddr1Ptr(pframe);
- break;
- case 0x02: /* ToDs = 1, FromDs = 0 */
- da = GetAddr3Ptr(pframe);
- break;
- default: /* ToDs = 1, FromDs = 1 */
- da = GetAddr3Ptr(pframe);
- break;
+ case 0x00: /* ToDs = 0, FromDs = 0 */
+ da = GetAddr1Ptr(pframe);
+ break;
+ case 0x01: /* ToDs = 0, FromDs = 1 */
+ da = GetAddr1Ptr(pframe);
+ break;
+ case 0x02: /* ToDs = 1, FromDs = 0 */
+ da = GetAddr3Ptr(pframe);
+ break;
+ default: /* ToDs = 1, FromDs = 1 */
+ da = GetAddr3Ptr(pframe);
+ break;
}
return da;
@@ -398,18 +398,18 @@ __inline static unsigned char * get_sa(unsigned char *pframe)
unsigned int to_fr_ds = (GetToDs(pframe) << 1) | GetFrDs(pframe);
switch (to_fr_ds) {
- case 0x00: /* ToDs = 0, FromDs = 0 */
- sa = GetAddr2Ptr(pframe);
- break;
- case 0x01: /* ToDs = 0, FromDs = 1 */
- sa = GetAddr3Ptr(pframe);
- break;
- case 0x02: /* ToDs = 1, FromDs = 0 */
- sa = GetAddr2Ptr(pframe);
- break;
- default: /* ToDs = 1, FromDs = 1 */
- sa = GetAddr4Ptr(pframe);
- break;
+ case 0x00: /* ToDs = 0, FromDs = 0 */
+ sa = GetAddr2Ptr(pframe);
+ break;
+ case 0x01: /* ToDs = 0, FromDs = 1 */
+ sa = GetAddr3Ptr(pframe);
+ break;
+ case 0x02: /* ToDs = 1, FromDs = 0 */
+ sa = GetAddr2Ptr(pframe);
+ break;
+ default: /* ToDs = 1, FromDs = 1 */
+ sa = GetAddr4Ptr(pframe);
+ break;
}
return sa;
@@ -421,18 +421,18 @@ __inline static unsigned char * get_hdr_bssid(unsigned char *pframe)
unsigned int to_fr_ds = (GetToDs(pframe) << 1) | GetFrDs(pframe);
switch (to_fr_ds) {
- case 0x00: /* ToDs = 0, FromDs = 0 */
- sa = GetAddr3Ptr(pframe);
- break;
- case 0x01: /* ToDs = 0, FromDs = 1 */
- sa = GetAddr2Ptr(pframe);
- break;
- case 0x02: /* ToDs = 1, FromDs = 0 */
- sa = GetAddr1Ptr(pframe);
- break;
- case 0x03: /* ToDs = 1, FromDs = 1 */
- sa = GetAddr1Ptr(pframe);
- break;
+ case 0x00: /* ToDs = 0, FromDs = 0 */
+ sa = GetAddr3Ptr(pframe);
+ break;
+ case 0x01: /* ToDs = 0, FromDs = 1 */
+ sa = GetAddr2Ptr(pframe);
+ break;
+ case 0x02: /* ToDs = 1, FromDs = 0 */
+ sa = GetAddr1Ptr(pframe);
+ break;
+ case 0x03: /* ToDs = 1, FromDs = 1 */
+ sa = GetAddr1Ptr(pframe);
+ break;
}
return sa;
--
2.17.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] staging: rtl8723bs: include: Fix identation issue
2019-03-28 18:45 [PATCH] staging: rtl8723bs: include: Fix identation issue Vatsala Narang
@ 2019-03-29 16:13 ` Greg KH
2019-03-29 16:17 ` Vatsala Narang
0 siblings, 1 reply; 4+ messages in thread
From: Greg KH @ 2019-03-29 16:13 UTC (permalink / raw)
To: Vatsala Narang; +Cc: outreachy-kernel
On Fri, Mar 29, 2019 at 12:15:23AM +0530, Vatsala Narang wrote:
> Corrected the identation of switch case to get rid of checkpatch error
>
> Signed-off-by: Vatsala Narang <vatsalanarang@gmail.com>
> ---
> drivers/staging/rtl8723bs/include/wifi.h | 72 ++++++++++++------------
> 1 file changed, 36 insertions(+), 36 deletions(-)
>
> diff --git a/drivers/staging/rtl8723bs/include/wifi.h b/drivers/staging/rtl8723bs/include/wifi.h
> index 559bf2606fb7..ee1994ecbf18 100644
> --- a/drivers/staging/rtl8723bs/include/wifi.h
> +++ b/drivers/staging/rtl8723bs/include/wifi.h
> @@ -374,18 +374,18 @@ __inline static unsigned char * get_da(unsigned char *pframe)
> unsigned int to_fr_ds = (GetToDs(pframe) << 1) | GetFrDs(pframe);
>
> switch (to_fr_ds) {
> - case 0x00: /* ToDs = 0, FromDs = 0 */
> - da = GetAddr1Ptr(pframe);
> - break;
> - case 0x01: /* ToDs = 0, FromDs = 1 */
> - da = GetAddr1Ptr(pframe);
> - break;
> - case 0x02: /* ToDs = 1, FromDs = 0 */
> - da = GetAddr3Ptr(pframe);
> - break;
> - default: /* ToDs = 1, FromDs = 1 */
> - da = GetAddr3Ptr(pframe);
> - break;
> + case 0x00: /* ToDs = 0, FromDs = 0 */
> + da = GetAddr1Ptr(pframe);
> + break;
> + case 0x01: /* ToDs = 0, FromDs = 1 */
> + da = GetAddr1Ptr(pframe);
> + break;
> + case 0x02: /* ToDs = 1, FromDs = 0 */
> + da = GetAddr3Ptr(pframe);
> + break;
> + default: /* ToDs = 1, FromDs = 1 */
> + da = GetAddr3Ptr(pframe);
> + break;
> }
>
> return da;
> @@ -398,18 +398,18 @@ __inline static unsigned char * get_sa(unsigned char *pframe)
> unsigned int to_fr_ds = (GetToDs(pframe) << 1) | GetFrDs(pframe);
>
> switch (to_fr_ds) {
> - case 0x00: /* ToDs = 0, FromDs = 0 */
> - sa = GetAddr2Ptr(pframe);
> - break;
> - case 0x01: /* ToDs = 0, FromDs = 1 */
> - sa = GetAddr3Ptr(pframe);
> - break;
> - case 0x02: /* ToDs = 1, FromDs = 0 */
> - sa = GetAddr2Ptr(pframe);
> - break;
> - default: /* ToDs = 1, FromDs = 1 */
> - sa = GetAddr4Ptr(pframe);
> - break;
> + case 0x00: /* ToDs = 0, FromDs = 0 */
> + sa = GetAddr2Ptr(pframe);
> + break;
> + case 0x01: /* ToDs = 0, FromDs = 1 */
> + sa = GetAddr3Ptr(pframe);
> + break;
> + case 0x02: /* ToDs = 1, FromDs = 0 */
> + sa = GetAddr2Ptr(pframe);
> + break;
> + default: /* ToDs = 1, FromDs = 1 */
> + sa = GetAddr4Ptr(pframe);
> + break;
> }
>
> return sa;
> @@ -421,18 +421,18 @@ __inline static unsigned char * get_hdr_bssid(unsigned char *pframe)
> unsigned int to_fr_ds = (GetToDs(pframe) << 1) | GetFrDs(pframe);
>
> switch (to_fr_ds) {
> - case 0x00: /* ToDs = 0, FromDs = 0 */
> - sa = GetAddr3Ptr(pframe);
> - break;
> - case 0x01: /* ToDs = 0, FromDs = 1 */
> - sa = GetAddr2Ptr(pframe);
> - break;
> - case 0x02: /* ToDs = 1, FromDs = 0 */
> - sa = GetAddr1Ptr(pframe);
> - break;
> - case 0x03: /* ToDs = 1, FromDs = 1 */
> - sa = GetAddr1Ptr(pframe);
> - break;
> + case 0x00: /* ToDs = 0, FromDs = 0 */
> + sa = GetAddr3Ptr(pframe);
> + break;
> + case 0x01: /* ToDs = 0, FromDs = 1 */
> + sa = GetAddr2Ptr(pframe);
> + break;
> + case 0x02: /* ToDs = 1, FromDs = 0 */
> + sa = GetAddr1Ptr(pframe);
> + break;
> + case 0x03: /* ToDs = 1, FromDs = 1 */
> + sa = GetAddr1Ptr(pframe);
> + break;
> }
>
> return sa;
> --
> 2.17.1
>
Patch does not apply to my staging-next branch. Please rebase and
resend.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH] staging: rtl8723bs: include: Fix identation issue
@ 2019-03-23 22:09 Vatsala Narang
0 siblings, 0 replies; 4+ messages in thread
From: Vatsala Narang @ 2019-03-23 22:09 UTC (permalink / raw)
To: outreachy-kernel; +Cc: gregkh, Vatsala Narang
Corrected the identation of switch case to get rid of checkpatch error
Signed-off-by: Vatsala Narang <vatsalanarang@gmail.com>
---
drivers/staging/rtl8723bs/include/wifi.h | 72 ++++++++++++------------
1 file changed, 36 insertions(+), 36 deletions(-)
diff --git a/drivers/staging/rtl8723bs/include/wifi.h b/drivers/staging/rtl8723bs/include/wifi.h
index 559bf2606fb7..ee1994ecbf18 100644
--- a/drivers/staging/rtl8723bs/include/wifi.h
+++ b/drivers/staging/rtl8723bs/include/wifi.h
@@ -374,18 +374,18 @@ __inline static unsigned char * get_da(unsigned char *pframe)
unsigned int to_fr_ds = (GetToDs(pframe) << 1) | GetFrDs(pframe);
switch (to_fr_ds) {
- case 0x00: /* ToDs = 0, FromDs = 0 */
- da = GetAddr1Ptr(pframe);
- break;
- case 0x01: /* ToDs = 0, FromDs = 1 */
- da = GetAddr1Ptr(pframe);
- break;
- case 0x02: /* ToDs = 1, FromDs = 0 */
- da = GetAddr3Ptr(pframe);
- break;
- default: /* ToDs = 1, FromDs = 1 */
- da = GetAddr3Ptr(pframe);
- break;
+ case 0x00: /* ToDs = 0, FromDs = 0 */
+ da = GetAddr1Ptr(pframe);
+ break;
+ case 0x01: /* ToDs = 0, FromDs = 1 */
+ da = GetAddr1Ptr(pframe);
+ break;
+ case 0x02: /* ToDs = 1, FromDs = 0 */
+ da = GetAddr3Ptr(pframe);
+ break;
+ default: /* ToDs = 1, FromDs = 1 */
+ da = GetAddr3Ptr(pframe);
+ break;
}
return da;
@@ -398,18 +398,18 @@ __inline static unsigned char * get_sa(unsigned char *pframe)
unsigned int to_fr_ds = (GetToDs(pframe) << 1) | GetFrDs(pframe);
switch (to_fr_ds) {
- case 0x00: /* ToDs = 0, FromDs = 0 */
- sa = GetAddr2Ptr(pframe);
- break;
- case 0x01: /* ToDs = 0, FromDs = 1 */
- sa = GetAddr3Ptr(pframe);
- break;
- case 0x02: /* ToDs = 1, FromDs = 0 */
- sa = GetAddr2Ptr(pframe);
- break;
- default: /* ToDs = 1, FromDs = 1 */
- sa = GetAddr4Ptr(pframe);
- break;
+ case 0x00: /* ToDs = 0, FromDs = 0 */
+ sa = GetAddr2Ptr(pframe);
+ break;
+ case 0x01: /* ToDs = 0, FromDs = 1 */
+ sa = GetAddr3Ptr(pframe);
+ break;
+ case 0x02: /* ToDs = 1, FromDs = 0 */
+ sa = GetAddr2Ptr(pframe);
+ break;
+ default: /* ToDs = 1, FromDs = 1 */
+ sa = GetAddr4Ptr(pframe);
+ break;
}
return sa;
@@ -421,18 +421,18 @@ __inline static unsigned char * get_hdr_bssid(unsigned char *pframe)
unsigned int to_fr_ds = (GetToDs(pframe) << 1) | GetFrDs(pframe);
switch (to_fr_ds) {
- case 0x00: /* ToDs = 0, FromDs = 0 */
- sa = GetAddr3Ptr(pframe);
- break;
- case 0x01: /* ToDs = 0, FromDs = 1 */
- sa = GetAddr2Ptr(pframe);
- break;
- case 0x02: /* ToDs = 1, FromDs = 0 */
- sa = GetAddr1Ptr(pframe);
- break;
- case 0x03: /* ToDs = 1, FromDs = 1 */
- sa = GetAddr1Ptr(pframe);
- break;
+ case 0x00: /* ToDs = 0, FromDs = 0 */
+ sa = GetAddr3Ptr(pframe);
+ break;
+ case 0x01: /* ToDs = 0, FromDs = 1 */
+ sa = GetAddr2Ptr(pframe);
+ break;
+ case 0x02: /* ToDs = 1, FromDs = 0 */
+ sa = GetAddr1Ptr(pframe);
+ break;
+ case 0x03: /* ToDs = 1, FromDs = 1 */
+ sa = GetAddr1Ptr(pframe);
+ break;
}
return sa;
--
2.17.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-03-29 16:17 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-03-28 18:45 [PATCH] staging: rtl8723bs: include: Fix identation issue Vatsala Narang
2019-03-29 16:13 ` Greg KH
2019-03-29 16:17 ` Vatsala Narang
-- strict thread matches above, loose matches on Subject: below --
2019-03-23 22:09 Vatsala Narang
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.