attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross
ARCH=ia64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot
All warnings (new ones prefixed by >>):
drivers/staging/rtl8192u/iee
From: Colin Ian King
A statement is indented one level too deeply; clean this up by
removing a tab.
Signed-off-by: Colin Ian King
---
drivers/staging/rtl8192u/r8192U_dm.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8192u/r8192U_dm.c
b/drivers
one in this patch series.
v3 contains the same patch again, but this time with this changelog appended.
drivers/staging/rtl8192u/r8192U_dm.c | 69 ++--
1 file changed, 23 insertions(+), 46 deletions(-)
diff --git a/drivers/staging/rtl8192u/r8192U_dm.c
b/drivers/staging
-by: Felix Trommer
---
v1 contains the initial patch.
v2 contains the same patch, but with a different subject to better distinguish
this patch from the other one in this patch series.
v3 contains the same patch again, but this time with this changelog appended.
drivers/staging/rtl8192u/r
omments in this file initially looked like the first example.
> But since this file is part of a networking driver and thus should
> be moved to drivers/net/ one day, this patch adjusts the comments
> such that they are fitting to the style imposed for drivers/net/.
>
> Signed-off-by:
-by: Felix Trommer
---
drivers/staging/rtl8192u/r8192U_dm.c | 28
1 file changed, 16 insertions(+), 12 deletions(-)
diff --git a/drivers/staging/rtl8192u/r8192U_dm.c
b/drivers/staging/rtl8192u/r8192U_dm.c
index 86215fee8f0b..cb6c653e624d 100644
--- a/drivers/stagin
adjusts the comments
such that they are fitting to the style imposed for drivers/net/.
Signed-off-by: Christian Müller
Signed-off-by: Felix Trommer
---
drivers/staging/rtl8192u/r8192U_dm.c | 69 ++--
1 file changed, 23 insertions(+), 46 deletions(-)
diff --git a/drivers
On Mon, Jun 24, 2019 at 11:46:39AM +0200, Christian Müller wrote:
> As stated in coding-styles.rst multiline comments should be structured in a
> way,
> that the actual comment starts on the second line of the commented portion.
> E.g:
You sent 2 patches that did different things, yet have the
-by: Felix Trommer
---
drivers/staging/rtl8192u/r8192U_dm.c | 28
1 file changed, 16 insertions(+), 12 deletions(-)
diff --git a/drivers/staging/rtl8192u/r8192U_dm.c
b/drivers/staging/rtl8192u/r8192U_dm.c
index 86215fee8f0b..cb6c653e624d 100644
--- a/drivers/stagin
adjusts the comments
such that they are fitting to the style imposed for drivers/net/.
Signed-off-by: Christian Müller
Signed-off-by: Felix Trommer
---
drivers/staging/rtl8192u/r8192U_dm.c | 69 ++--
1 file changed, 23 insertions(+), 46 deletions(-)
diff --git a/drivers
such that they obey these standards.
All multiline-comments in this file now look like the following example:
/* Multiline comments
* in r8192U_dm.c
* now look like this.
*/
Christian Müller (2):
drivers/staging/rtl8192u: adjust block comments
drivers/staging/rtl8192u: adjust block comments
drivers/staging/rtl8192u/r819xU_phy.c | 39 ++-
1 file changed, 26 insertions(+), 13 deletions(-)
--
2.17.1
block of this patch doesn't look like it could
ever compile properly.
> diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211.h
> b/drivers/staging/rtl8192u/ieee80211/ieee80211.h
[]
> @@ -446,26 +446,26 @@ typedef enum _InitialGainOpType {
[]
> #else
> -#define IEEE80
ot;pseudo-code"-like nature of the
> code that helped to clarify certain functionalities.
>
> Signed-off-by: Felix Trommer
> Signed-off-by: Christian Müller
> ---
> drivers/staging/rtl8192u/ieee80211/dot11d.h | 2 +-
> .../staging/rtl8192u/ieee80211/ieee802
On Mon, Jun 03, 2019 at 02:21:04PM +0200, Christian Müller wrote:
> Fix issues that lead to multiple checkpatch warnings and errors, most of
> them regarding formatting of code and comments.
> Comments that contain only commented out code are removed as well.
>
> Signed-off-by: Felix Trommer
> Si
On Mon, Jun 03, 2019 at 02:21:02PM +0200, Christian Müller wrote:
> From: Felix Trommer
>
> Replace C99-Style comments with C89-Style comments.
What does that mean?
> if (MaxChnlNum >= pTriple->first_channel) {
> - /* It is not in a monotonically increasing ord
s.
Signed-off-by: Felix Trommer
Signed-off-by: Christian Müller
---
drivers/staging/rtl8192u/ieee80211/dot11d.h | 2 +-
.../staging/rtl8192u/ieee80211/ieee80211.h| 35 ++-
.../rtl8192u/ieee80211/ieee80211_crypt_ccmp.c | 1 -
.../rtl8192u/ieee80211/ieee80211_crypt_tkip.c | 1 -
..
/ieee80211_wx.c | 59 +-
.../rtl8192u/ieee80211/rtl819x_BAProc.c | 18 +-
.../staging/rtl8192u/ieee80211/rtl819x_HT.h | 15 +-
.../rtl8192u/ieee80211/rtl819x_HTProc.c | 16 +-
.../rtl8192u/ieee80211/rtl819x_TSProc.c | 14 +-
drivers/staging/rtl8192u/r8192U_dm.c | 53
ed check
>> before kfree().
>>
>> Reported-by: John Whitmore
>> Signed-off-by: Valdis Kletnieks
>> ---
>> diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_module.c
>> b/drivers/staging/rtl8192u/ieee80211/ieee80211_module.c
>> index 90a097f2cd4
e
> Signed-off-by: Valdis Kletnieks
> ---
> diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_module.c
> b/drivers/staging/rtl8192u/ieee80211/ieee80211_module.c
> index 90a097f2cd4e..97ff0371b5bb 100644
> --- a/drivers/staging/rtl8192u/ieee80211/ieee80211_module.c
> +
On Mon, May 14, 2018 at 10:24:08PM +0100, John Whitmore wrote:
> Coding style edit to clear the checkpatch.pl errors of the type:
> ERROR: that open brace { should be on the previous line
>
> Signed-off-by: John Whitmore
> ---
Any reason why your subject line has the full path and not the shorte
Second attempt at this commit, apologies for that.
Coding style changes to resolve some of the scripts/checkpatch.pl issues with
the file drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c.
There remain line length and CamelCase issues unresolved in the file.
Coding style edit to clear the checkpatch.pl errors of the type:
ERROR: space prohibited before that ','
Signed-off-by: John Whitmore
---
drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/stagin
Coding style edit to clear the checkpatch.pl errors of the type:
ERROR: space prohibited before that close parenthesis ')'
Signed-off-by: John Whitmore
---
drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c | 16
1 file changed, 8 insertions(+), 8 deletions(-)
di
Coding style edit to clear the checkpatch.pl error:
ERROR: space required after that close brace '}'
And balanced the brace of the same else statement:
CHECK: Unbalanced braces around else statement
Signed-off-by: John Whitmore
---
drivers/staging/rtl8192u/ieee80211/rtl819x_HT
ff --git a/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c
b/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c
index f66f181e24f2..ed3e3acd78a3 100644
--- a/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c
+++ b/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c
@@ -67,7 +67
Coding style edit to clear the remaining checkpatch.pl errors:
ERROR: trailing whitespace
ERROR: space prohibited after that open parenthesis '('
ERROR: space required before the open brace '{'
Signed-off-by: John Whitmore
---
drivers/staging/rtl8192u/ieee80211/rtl819x_HT
/staging/rtl8192u/ieee80211/rtl819x_HTProc.c
b/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c
index beecdbfe401b..5591cde8a5a0 100644
--- a/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c
+++ b/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c
@@ -177,35 +177,35 @@ void HTDebugHTInfo(u8 *InfoIE
Coding style edit to clear the checkpatch.pl warning:
WARNING: void function return statements are not generally useful
Signed-off-by: John Whitmore
---
drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/staging/rtl8192u/ieee80211
Coding style edit to clear the checkpatch.pl warning:
WARNING: please, no spaces at the start of a line
Signed-off-by: John Whitmore
---
drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8192u/ieee80211
Coding style edit to clear the checkpatch.pl warnings:
WARNING: Missing a blank line after declarations
Signed-off-by: John Whitmore
---
drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/drivers/staging/rtl8192u/ieee80211
Coding style edit to clear the checkpatch.pl warning:
WARNING: braces {} are not necessary for single statement blocks
Signed-off-by: John Whitmore
---
drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c | 10 +++---
1 file changed, 3 insertions(+), 7 deletions(-)
diff --git a/drivers
Coding style edit to clear the checkpatch.pl check:
CHECK: Blank lines aren't necessary before a close brace '}'
Signed-off-by: John Whitmore
---
drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c | 13 -
1 file changed, 13 deletions(-)
diff --git a/drivers/s
Coding style edit to clear the checkpatch.pl warning:
WARNING: Block comments use * on subsequent lines
Signed-off-by: John Whitmore
---
drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c | 20 +---
1 file changed, 9 insertions(+), 11 deletions(-)
diff --git a/drivers/staging
Coding style edit to clear the checkpatch.pl warnings:
WARNING: Prefer using '"%s...", __func__' to using '', this function's
name, in a string
Signed-off-by: John Whitmore
---
drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c | 12 ++--
Coding style edit to clear the checkpatch.pl check:
CHECK: Please don't use multiple blank lines
Signed-off-by: John Whitmore
---
.../staging/rtl8192u/ieee80211/rtl819x_HTProc.c| 24 --
1 file changed, 24 deletions(-)
diff --git a/drivers/staging/rtl8192u/ieee
Coding style edit to clear the checkpatch.pl check:
CHECK: Please use a blank line after function/struct/union/enum declarations
Signed-off-by: John Whitmore
---
drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c | 7 +++
1 file changed, 7 insertions(+)
diff --git a/drivers/staging
Coding style edit to clear the checkpatch.pl check:
CHECK: Lines should not end with a '('
Signed-off-by: John Whitmore
---
drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/staging/rtl8192u
Coding style edit to clear the checkpatch.pl check:
CHECK: Unnecessary parentheses around ...
Signed-off-by: John Whitmore
---
drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/drivers/staging/rtl8192u/ieee80211
Coding style edit to clear the checkpatch.pl check:
CHECK: spaces preferred around that 'x'
Signed-off-by: John Whitmore
---
drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c | 16
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/drivers/stagin
Coding style edit to clear the checkpatch.pl check:
CHECK: Comparison to NULL could be written "!..."
Signed-off-by: John Whitmore
---
drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/stagin
Coding style edit to clear the checkpatch.pl warnings:
WARNING: void function return statements are not generally useful
Signed-off-by: John Whitmore
---
drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/staging/rtl8192u/ieee80211
Spelling correction in comment: singal -> signal
Signed-off-by: John Whitmore
---
drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c
b/drivers/staging/rtl8192u/ieee80
/staging/rtl8192u/ieee80211/rtl819x_HTProc.c
b/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c
index 3342c9ccec3f..425ea1279b87 100644
--- a/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c
+++ b/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c
@@ -213,8 +213,8 @@ void HTDebugHTInfo(u8
--git a/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c
b/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c
index bf7b7122d042..c116bd895b50 100644
--- a/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c
+++ b/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c
@@ -7,28 +7,28 @@ u8
Coding style edit to clear the checkpatch.pl check:
CHECK: Alignment should match open parenthesis
Signed-off-by: John Whitmore
---
drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c | 16
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/drivers/staging/rtl8192u
Coding style edit to clear the checkpatch.pl check:
CHECK: Blank lines aren't necessary after an open brace '{'
Signed-off-by: John Whitmore
---
drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c | 5 -
1 file changed, 5 deletions(-)
diff --git a/drivers/staging/rtl
/rtl8192u/ieee80211/rtl819x_HTProc.c
b/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c
index d4ab8481ee30..f66f181e24f2 100644
--- a/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c
+++ b/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c
@@ -60,8 +60,8 @@ void HTUpdateDefaultSetting(struct
There were a lot of coding style issues with this file, but I think I've
reduced the number down to mainly line length and CamelCase issues. I've put
it in 13 patches to, hopefully, make it easier to review, and see that I've
made no changes to functionality of the code, (I hope).
John
Hi,
Static analysis via CoverityScan picked up an issue in
cmpk_handle_query_config_rx where the following operation on an u8 is
clearly not correct:
rx_query_cfg.cfg_action = (pmsg[4] & 0x8000) >> 31;
The result of this operation is always zero. I suspect this should be:
On Fri, Jul 17, 2015 at 03:59:16PM +0200, Joseph-Eugene Winzer wrote:
> Fixing several coding style issues, like
> C99 Comment Style
> Trailing whitespaces
> Inconsistent spacing of operators
> Started to reformat comments/expressions for 80 character limit
These are multiple thing
On Sat, Jul 18, 2015 at 02:29:19PM +0200, m...@openmailbox.org wrote:
> Rather than polluting the main mailing list I should look at the kernelnewbies
> mailing list for further criticism on my patch attemps.
Use the de...@driverdev.osuosl.org it's fine. No one reads
linux-kernel@vger.kernel.org
t; Very few of these seem to be improvements and many
> of them should likely be in separate patches.
>
> > diff --git a/drivers/staging/rtl8192u/r8192U_core.c
> > b/drivers/staging/rtl8192u/r8192U_core.c
> []
> > @@ -143,17 +143,35 @@ struct CHANNEL_LIST {
> > };
> &
On Fri, 2015-07-17 at 15:59 +0200, Joseph-Eugene Winzer wrote:
> Fixed a few lines that were longer than 80 chars.
[]
> diff --git a/drivers/staging/rtl8192u/r8192U_core.c
> b/drivers/staging/rtl8192u/r8192U_core.c
[]
> @@ -212,7 +212,9 @@ static void rtl819x_set_channel_map(u8
patches.
> diff --git a/drivers/staging/rtl8192u/r8192U_core.c
> b/drivers/staging/rtl8192u/r8192U_core.c
[]
> @@ -143,17 +143,35 @@ struct CHANNEL_LIST {
> };
>
> static struct CHANNEL_LIST ChannelPlan[] = {
> - {{1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 36, 40, 44, 48, 52, 5
Checkpatch didn't stop complaining about it and as linux/uaccess.h also
includes asm/uaccess.h, there shouldn't be a problem.
Signed-off-by: Joseph-Eugene Winzer
---
drivers/staging/rtl8192u/r8192U_core.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drive
Fixed a few lines that were longer than 80 chars.
Signed-off-by: Joseph-Eugene Winzer
---
drivers/staging/rtl8192u/r8192U_core.c | 61 +++---
1 file changed, 41 insertions(+), 20 deletions(-)
diff --git a/drivers/staging/rtl8192u/r8192U_core.c
b/drivers/staging
Reformatting the code without introducing other warnings
like 'Avoid unnecessary line continuations' or breaking strings.
Signed-off-by: Joseph-Eugene Winzer
---
drivers/staging/rtl8192u/r8192U_core.c | 809 ++---
1 file changed, 536 insertions(+), 273
Fixing several coding style issues, like
C99 Comment Style
Trailing whitespaces
Inconsistent spacing of operators
Started to reformat comments/expressions for 80 character limit
Signed-off-by: Joseph-Eugene Winzer
---
drivers/staging/rtl8192u/r8192U_core.c | 1323
kfree() can handle NULL pointers itself, so the check is unnecessary.
Signed-off-by: Joseph-Eugene Winzer
---
drivers/staging/rtl8192u/r8192U_core.c | 13 -
1 file changed, 4 insertions(+), 9 deletions(-)
diff --git a/drivers/staging/rtl8192u/r8192U_core.c
b/drivers/staging
-
> 1 file changed, 44 insertions(+), 53 deletions(-)
>
> diff --git a/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c
> b/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c
> index c2588f8..df20979 100644
> --- a/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c
>
On Tue, Jun 23, 2015 at 3:01 AM, Dan Carpenter wrote:
> Actually, Greg complains about these all the time.
All I know is he's yet to turn away a single one of my patches.
> And here is a useless URL that you can look at: http://slashdot.org
> I'm sure it supports my argument.
>
> But seriously,
On Mon, Jun 22, 2015 at 06:25:11AM -0500, Greg Donald wrote:
> Umm.. you have to fix more than one error if there's more than one
> error on or very near the same line you are already fixing, else
> checkpatch.pl complains that your patch has errors. Not to mention
> Greg KH has never complained:
Donald
>> ---
>> .../staging/rtl8192u/ieee80211/rtl819x_HTProc.c| 97
>> ++
>> 1 file changed, 44 insertions(+), 53 deletions(-)
>>
>> diff --git a/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c
>> b/drivers/staging/rtl8192u/i
-
> 1 file changed, 44 insertions(+), 53 deletions(-)
>
> diff --git a/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c
> b/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c
> index c2588f8..df20979 100644
> --- a/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c
>
Fix checkpatch.pl "space required before the open parenthesis '('" errors
Signed-off-by: Greg Donald
---
.../staging/rtl8192u/ieee80211/rtl819x_HTProc.c| 97 ++
1 file changed, 44 insertions(+), 53 deletions(-)
diff --git a/drivers/stagin
I appreciate the suggestions.
I simply grepped this piece of code by chance while looking at an
actual sparse warning somewhere else. There's no
symptoms and I'm not sure under what conditions this debug statement
is printed nor why/when. But this is an obviously buggy
code only impacting debug/tr
Good advice. This was actually found by Sparse and the changelog should
say:
Sparse complains about this:
ieee80211_softmac.c:1561:16: warning: cast from restricted __le16
ieee80211_softmac.c:1561:16: warning: incorrect type in return expression
(different base types)
ieee80211_softmac.c:1561:1
On 15-05-23 10:53 PM, tolga ceylan wrote:
> On 05/12/2015 08:55 PM, Tolga Ceylan wrote:
>> Error code returned from auth_parse() should in cpu byte order.
>>
>> Signed-off-by: Tolga Ceylan
>> ---
>> drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c | 2 +-
On Sat, May 23, 2015 at 07:53:28PM -0700, tolga ceylan wrote:
> On 05/12/2015 08:55 PM, Tolga Ceylan wrote:
> >
> >
>
> I haven't gotten a response for this patch. Just checking back.
no response means no one is having anything to say about it. Final say
will be Greg's when he reviews that when h
On 05/12/2015 08:55 PM, Tolga Ceylan wrote:
Error code returned from auth_parse() should in cpu byte order.
Signed-off-by: Tolga Ceylan
---
drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8192u
Error code returned from auth_parse() should in cpu byte order.
Signed-off-by: Tolga Ceylan
---
drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c
b/drivers/staging
Correct redundant initialization reported by sparse
Signed-off-by: Rene Kolarik
---
drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c
b/drivers/staging/rtl8192u/ieee80211
...@driverdev.osuosl.org; linux-kernel@vger.kernel.org
Cc: Greg Donald
Subject: [PATCH] drivers: staging: rtl8192u: Fix "space prohibited after that
open parenthesis '('" errors
Fix checkpatch.pl "space prohibited after that open parenthesis '('" errors
Signed-off-by: Gr
Fix checkpatch.pl "space prohibited after that open parenthesis '('" errors
Signed-off-by: Greg Donald
---
drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c | 22 +-
.../staging/rtl8192u/ieee80211/ieee80211_softmac.c | 11 ---
..
Fix checkpatch.pl "space required before that '*'" errors
Signed-off-by: Greg Donald
---
drivers/staging/rtl8192u/ieee80211/ieee80211.h | 6 +++---
drivers/staging/rtl8192u/ieee80211/rtl819x_HT.h | 2 +-
drivers/staging/rtl8192u/ieee80211/rtl819x_Qos.h | 8 ----
driv
Fix checkpatch.pl "space prohibited after that open parenthesis '('" errors
Signed-off-by: Greg Donald
---
drivers/staging/rtl8192u/ieee80211/ieee80211.h | 18 +++-
.../staging/rtl8192u/ieee80211/ieee80211_softmac.c | 4 +--
drivers/staging/rtl8192u/ieee80211/
Fix checkpatch.pl switch and case should be at the same indent errors
Signed-off-by: Greg Donald
---
drivers/staging/rtl8192u/r8192U_core.c | 510 -
1 file changed, 252 insertions(+), 258 deletions(-)
diff --git a/drivers/staging/rtl8192u/r8192U_core.c
b
Fix checkpatch.pl space required after that ',' errors
Signed-off-by: Greg Donald
---
drivers/staging/rtl8192u/ieee80211/ieee80211.h | 4 +-
.../staging/rtl8192u/ieee80211/ieee80211_module.c | 2 +-
drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c | 20 ++---
.../stagin
ends on !ok && unchanged@
position r.p2;
expression x;
@@
*kfree@p2(x);
//
Signed-off-by: Peter Senna Tschudin
---
drivers/staging/rtl8192u/r8192U_core.c | 23 +++
1 file changed, 7 insertions(+), 16 deletions(-)
diff --git a/drivers/staging/rtl8192u/r8192U_co
80 matches
Mail list logo
