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
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
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
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
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
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
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/
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
>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 |
>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/
>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-
>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
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
>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/
>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
38 matches
Mail list logo