[PATCH] rtl8188eu: Removed Unnecessary ftrace-like logging

2021-03-18 Thread Paul McQuade
prefer using ftrace Signed-off-by: Paul McQuade --- drivers/staging/rtl8188eu/core/rtw_ap.c | 6 -- 1 file changed, 6 deletions(-) diff --git a/drivers/staging/rtl8188eu/core/rtw_ap.c b/drivers/staging/rtl8188eu/core/rtw_ap.c index 182bb944c9b3..008b60e72758 100644 --- a/drivers/staging

[PATCH v2] staging: rtl8188eu: Removed Unnecessary logging

2021-03-19 Thread Paul McQuade
fix the following checkpatch.pl issues: WARNING: Unnecessary ftrace-like logging - prefer using ftrace Signed-off-by: Paul McQuade --- drivers/staging/rtl8188eu/core/rtw_ap.c | 6 -- 1 file changed, 6 deletions(-) diff --git a/drivers/staging/rtl8188eu/core/rtw_ap.c b/drivers/staging

[PATCH] Staging:Octeon-usb:octeon-hcd.c return value

2014-04-08 Thread Paul McQuade
return value instead of function. Signed-off-by: Paul McQuade --- drivers/staging/octeon-usb/octeon-hcd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/octeon-usb/octeon-hcd.c b/drivers/staging/octeon-usb/octeon-hcd.c index 8b8ce72..2d15001 100644 --- a

[PATCH] Staging:Line6:usbdefs.h parenthesis for Marcos

2014-04-08 Thread Paul McQuade
ERROR: Macros with complex values should be enclosed in parenthesis Signed-off-by: Paul McQuade --- drivers/staging/line6/usbdefs.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/line6/usbdefs.h b/drivers/staging/line6/usbdefs.h index 2d1cc47..48958b5 100644

[PATCH 2/2] Staging:keucr:scsiglue.c-no semicolon terminated

2014-04-08 Thread Paul McQuade
WARNING: do {} while (0) macros should not be semicolon terminated Signed-off-by: Paul McQuade --- drivers/staging/keucr/scsiglue.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/keucr/scsiglue.c b/drivers/staging/keucr/scsiglue.c index ac3d34d..fce19a4

[PATCH 3/3] Staging:android:uapi:binder.h __packed

2014-04-08 Thread Paul McQuade
WARNING: __packed is preferred over __attribute__((packed)) Signed-off-by: Paul McQuade --- drivers/staging/android/uapi/binder.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/android/uapi/binder.h b/drivers/staging/android/uapi/binder.h index 904adb7

[PATCH] Staging:dgnc:dgnc_driver open/close braces

2014-04-09 Thread Paul McQuade
er that ',' (ctx:VxV) ERROR: space required before the open parenthesis '(' Signed-off-by: Paul McQuade --- drivers/staging/dgnc/dgnc_driver.c | 22 ++ 1 file changed, 10 insertions(+), 12 deletions(-) diff --git a/drivers/staging/dgnc/dgnc_driver.c b/

[PATCH 2/2] Staging:rtl8192e:rtl819x_BAProc:Remove whitespaces

2014-04-09 Thread Paul McQuade
ERROR: spaces required around that '=' (ctx:OxW) Signed-off-by: Paul McQuade --- drivers/staging/rtl8192e/rtl819x_BAProc.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/rtl8192e/rtl819x_BAProc.c b/drivers/staging/rtl8192e/rtl819x_BAPr

[PATCH] Staging: vt6655: Renamed uRATE to rate

2015-10-11 Thread Paul McQuade
Renamed uRATE to rate to avoid camelcase Signed-off-by: Paul McQuade --- drivers/staging/vt6655/rf.c | 12 ++-- drivers/staging/vt6655/rf.h | 4 ++-- 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/staging/vt6655/rf.c b/drivers/staging/vt6655/rf.c index c537321

[PATCH] Staging:rtl8723au:core brace issue

2014-10-24 Thread Paul McQuade
ERROR: that open brace { should be on the previous line Signed-off-by: Paul McQuade --- drivers/staging/rtl8723au/core/rtw_ap.c | 137 ++-- 1 file changed, 43 insertions(+), 94 deletions(-) diff --git a/drivers/staging/rtl8723au/core/rtw_ap.c b/drivers/staging

[PATCH] Staging:rtl8723au: open/closed brace issue

2014-10-24 Thread Paul McQuade
ERROR: that open brace { should be on the previous line Signed-off-by: Paul McQuade --- drivers/staging/rtl8723au/core/rtw_ap.c | 130 ++-- 1 file changed, 40 insertions(+), 90 deletions(-) diff --git a/drivers/staging/rtl8723au/core/rtw_ap.c b/drivers/staging

[PATCH][V2] Staging:rtl8723au: open/closed brace issue

2014-10-26 Thread Paul McQuade
ERROR: that open brace { should be on the previous line Signed-off-by: Paul McQuade --- drivers/staging/rtl8723au/core/rtw_ap.c | 130 ++-- 1 file changed, 40 insertions(+), 90 deletions(-) diff --git a/drivers/staging/rtl8723au/core/rtw_ap.c b/drivers/staging

[PATCH] staging: rtl8723au pointer issue

2014-10-26 Thread Paul McQuade
ERROR: "foo*bar" should be "foo *bar" Signed-off-by: Paul McQuade --- drivers/staging/rtl8723au/core/rtw_cmd.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/rtl8723au/core/rtw_cmd.c b/drivers/staging/rtl8723au/core/rtw

[PATCH 1/2] staging: rtl8723au: core: pointer issue

2014-10-26 Thread Paul McQuade
ERROR: "foo * bar" should be "foo *bar" Signed-off-by: Paul McQuade --- drivers/staging/rtl8723au/core/rtw_efuse.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/rtl8723au/core/rtw_efuse.c b/drivers/staging/rtl8723au/co

[PATCH 2/2] staging: rtl8723au:core

2014-10-26 Thread Paul McQuade
ERROR: spaces required around that ':' (ctx:VxE) Signed-off-by: Paul McQuade --- drivers/staging/rtl8723au/core/rtw_ieee80211.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/staging/rtl8723au/core/rtw_ieee80211.c b/drivers/staging/rtl8

[PATCH] staging: rtl8723au: create macro get_max_rate

2014-10-26 Thread Paul McQuade
create marco for max_rate values Signed-off-by: Paul McQuade --- drivers/staging/rtl8723au/core/rtw_ieee80211.c | 75 -- 1 file changed, 34 insertions(+), 41 deletions(-) diff --git a/drivers/staging/rtl8723au/core/rtw_ieee80211.c b/drivers/staging/rtl8723au/core

[PATCH][V2] staging: rtl8723au: create macro get_max_rate

2014-10-26 Thread Paul McQuade
create marco for max_rate values Signed-off-by: Paul McQuade --- drivers/staging/rtl8723au/core/rtw_ieee80211.c | 58 -- 1 file changed, 26 insertions(+), 32 deletions(-) diff --git a/drivers/staging/rtl8723au/core/rtw_ieee80211.c b/drivers/staging/rtl8723au/core

[PATCH] staging: rtl8723au: create macro get_max_rate

2014-10-26 Thread Paul McQuade
create marco for max_rate values Signed-off-by: Paul McQuade --- drivers/staging/rtl8723au/core/rtw_ieee80211.c | 58 -- 1 file changed, 26 insertions(+), 32 deletions(-) diff --git a/drivers/staging/rtl8723au/core/rtw_ieee80211.c b/drivers/staging/rtl8723au/core

[PATCH 2/2] staging: rtl8723au: core style issue

2014-10-26 Thread Paul McQuade
spaces required around % Signed-off-by: Paul McQuade --- drivers/staging/rtl8723au/core/rtw_recv.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/rtl8723au/core/rtw_recv.c b/drivers/staging/rtl8723au/core/rtw_recv.c index 5bc7734..6343d49a 100644 --- a

[PATCH] staging: rtl8188eu: hal: space required

2014-10-27 Thread Paul McQuade
ERROR: space required before the open brace '{' Signed-off-by: Paul McQuade --- drivers/staging/rtl8188eu/hal/bb_cfg.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8188eu/hal/bb_cfg.c b/drivers/staging/rtl8188eu/hal/bb_cfg.c index 80e8cc

[PATCH 3/5] Staging:rtl8723bs:os_dep braces not necessary

2018-03-19 Thread Paul McQuade
WARNING: braces {} are not necessary for any arm of this statement Signed-off-by: Paul McQuade --- drivers/staging/rtl8723bs/os_dep/recv_linux.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/rtl8723bs/os_dep/recv_linux.c b/drivers/staging/rtl8723bs

[PATCH 1/5] Staging: rtl8723bs: os_dep

2018-03-19 Thread Paul McQuade
ERROR: do not initialise statics to NULL Signed-off-by: Paul McQuade --- drivers/staging/rtl8723bs/os_dep/rtw_proc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8723bs/os_dep/rtw_proc.c b/drivers/staging/rtl8723bs/os_dep/rtw_proc.c index 9a885e626d1c

[PATCH 4/5] Staging:rtl8723bs:os_dep blank line needed

2018-03-19 Thread Paul McQuade
WARNING: Missing a blank line after declarations Signed-off-by: Paul McQuade --- drivers/staging/rtl8723bs/os_dep/recv_linux.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/rtl8723bs/os_dep/recv_linux.c b/drivers/staging/rtl8723bs/os_dep/recv_linux.c index 117fda4fa1bf

[PATCH 2/5] Staging: rtl8723bs: os_dep Spacing issue

2018-03-19 Thread Paul McQuade
ERROR: spaces required around that '<' (ctx:VxV) Signed-off-by: Paul McQuade --- drivers/staging/rtl8723bs/os_dep/rtw_proc.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/staging/rtl8723bs/os_dep/rtw_proc.c b/drivers/staging/rtl

[PATCH 5/5] Staging:rtl8723bs:os_dep Fixed braces

2018-03-19 Thread Paul McQuade
ERROR: that open brace { should be on the previous line Signed-off-by: Paul McQuade --- drivers/staging/rtl8723bs/os_dep/recv_linux.c | 9 +++-- 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/drivers/staging/rtl8723bs/os_dep/recv_linux.c b/drivers/staging/rtl8723bs/os_dep

[v2 PATCH 5/5] Staging:rtl8723bs clean spaces

2018-03-20 Thread Paul McQuade
Used checkpatch.pl to clean up spaces around if and for statements to make it easier to read Signed-off-by: Paul McQuade --- drivers/staging/rtl8723bs/os_dep/recv_linux.c | 9 +++-- 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/drivers/staging/rtl8723bs/os_dep/recv_linux.c

[v2 PATCH 1/5] Staging:rtl8723bs static variables are always 0

2018-03-20 Thread Paul McQuade
C standard guarantees that: global and static variables will be implicitly initialized to 0 or NULL if no explicit initializer is given. Signed-off-by: Paul McQuade --- drivers/staging/rtl8723bs/os_dep/rtw_proc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers

[v2 PATCH 2/5] Staging:rtl8723bs clean up spaces

2018-03-20 Thread Paul McQuade
Used checkpatch.pl to clean up spaces around for statements to make it easier to read Signed-off-by: Paul McQuade --- drivers/staging/rtl8723bs/os_dep/rtw_proc.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/staging/rtl8723bs/os_dep/rtw_proc.c b

[v2 PATCH 3/5] Staging:rtl8723bs Remove unnecessary braces

2018-03-20 Thread Paul McQuade
Remove unnecessary parentheses highlighted by checkpatch.pl Signed-off-by: Paul McQuade --- drivers/staging/rtl8723bs/os_dep/recv_linux.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/rtl8723bs/os_dep/recv_linux.c b/drivers/staging/rtl8723bs/os_dep

[v2 PATCH 4/5] Staging:rtl8723bs:Add blank line after declaration

2018-03-20 Thread Paul McQuade
missing a blank line after declaration checkpatch warnings. Issue found by checkpatch.pl Signed-off-by: Paul McQuade --- drivers/staging/rtl8723bs/os_dep/recv_linux.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/rtl8723bs/os_dep/recv_linux.c b/drivers/staging/rtl8723bs

[PATCH] Staging: Comedi: Use Preferred kernel type

2018-03-27 Thread Paul McQuade
Change to kenel type u8 or u16 instead of uint8_t or uint16_t Signed-off-by: Paul McQuade --- drivers/staging/comedi/drivers/s626.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/comedi/drivers/s626.c b/drivers/staging/comedi/drivers/s626.c index

[PATCH] Staging:BCM:DDRInit.c:Replacing __FUNCTION__

2013-08-07 Thread Paul McQuade
>From 5b6fbb6496ddc4e428f5dd0950b0617faed6596b Mon Sep 17 00:00:00 2001 From: Paul McQuade Date: Wed, 7 Aug 2013 16:21:16 +0100 Subject: [PATCH] Staging:BCM:DDRInit.c:Replacing __FUNCTION__ __func__ replaces __Function__. Signed-off-by: Paul McQuade --- drivers/staging/bcm/DDRInit.c |

[PATCH] Staging:BCM:DDRInit.c:Replacing __FUNCTION__

2013-08-07 Thread Paul McQuade
>From 31cc3c00d2699a846a8bb54d0831ab6480c8bd12 Mon Sep 17 00:00:00 2001 From: Paul McQuade Date: Wed, 7 Aug 2013 20:29:13 +0100 Subject: [PATCH] Subject: [PATCH] Staging:BCM:DDRInit.c:Replacing __FUNCTION__ __func__ replaces __Function__. Signed-off-by: Paul McQuade --- drivers/staging/

[PATCH 1/2] Staging: BCM: removed whitespaces and Errors.

2013-08-08 Thread Paul McQuade
>From 6e70cd8754968f37e82f6a4d8b9da75ff6e0 Mon Sep 17 00:00:00 2001 From: Paul McQuade Date: Thu, 8 Aug 2013 16:09:46 +0100 Subject: [PATCH 1/2] Staging: BCM: removed whitespaces and Errors. BCM Driver had a lot of whitespaces and errors. Some of them are removed by this patch. Signed-

[PATCH 2/2] Staging: BCM: Removed more whitespace/Errors

2013-08-08 Thread Paul McQuade
>From 9f6231ce16996e35545e20ce1d91abc058f2a6fb Mon Sep 17 00:00:00 2001 From: Paul McQuade Date: Thu, 8 Aug 2013 17:49:22 +0100 Subject: [PATCH 2/2] Staging: BCM: Removed more whitespace/Errors Interface Whitespace/Errors Removed with the help of checkpatch.pl Signed-off-by: Paul McQu

Review [2/2] Staging: BCM: Removed more whitespace/Errors

2013-08-08 Thread Paul McQuade
From 9f6231ce16996e35545e20ce1d91abc058f2a6fb Mon Sep 17 00:00:00 2001 From: Paul McQuade Date: Thu, 8 Aug 2013 17:49:22 +0100 Subject: [PATCH 2/2] Staging: BCM: Removed more whitespace/Errors Interface Whitespace/Errors Removed with the help of checkpatch.pl Signed-off-by: Paul McQuade

[PATCH] Staging:BCM:DDRInit.c:Replacing __FUNCTION__

2013-08-18 Thread Paul McQuade
>From c21d0da84c92d351f37b70c0d9c01a66fcb84e88 Mon Sep 17 00:00:00 2001 From: Paul McQuade Date: Thu, 15 Aug 2013 20:00:50 +0100 Subject: [PATCH] Staging:BCM:DDRInit.c:Replacing __FUNCTION__ __Function__ gets replaced with __func__ Signed-Off-By: Paul McQuade --- drivers/staging/

[PATCH] Staging:BCM:DDRInit.c:Renaming __FUNCTION__

2013-08-30 Thread Paul McQuade
>From c21d0da84c92d351f37b70c0d9c01a66fcb84e88 Mon Sep 17 00:00:00 2001 From: Paul McQuade Date: Thu, 15 Aug 2013 20:00:50 +0100 Subject: [PATCH] Staging:BCM:DDRInit.c:Renaming __FUNCTION__ __Function__ gets renamed with __func__ Signed-Off-By: Paul McQuade --- drivers/staging/bcm/DDRIni