[PATCH 1/3] staging: rtl8192u: Remove unnecessary scope

2017-05-01 Thread Tuomo Rinne
Remove scope unnecessary scope that is already enforced by the if statements scope. Signed-off-by: Tuomo Rinne <tuomo.ri...@gmail.com> --- drivers/staging/rtl8192u/r8192U_dm.c | 66 +--- 1 file changed, 32 insertions(+), 34 deletions(-) diff --git a/d

[PATCH 0/3] staging: rtl8192u: Refactor u4bAcParam construction

2017-05-01 Thread Tuomo Rinne
of the last patch of the series for better explanation. Tuomo Rinne (3): staging: rtl8192u: Remove unnecessary scope staging: rtl8192u: Improve code readability staging: rtl8192u: Convert u4bAcParam to little-endian drivers/staging/rtl8192u/r8192U_dm.c | 77 1

[PATCH 3/3] staging: rtl8192u: Convert u4bAcParam to little-endian

2017-05-01 Thread Tuomo Rinne
ted. Hence on big-endian architectures the parameter will remain as little-endian. Signed-off-by: Tuomo Rinne <tuomo.ri...@gmail.com> --- drivers/staging/rtl8192u/r8192U_dm.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/rtl8192u/r8192U_dm.c b/drivers/staging/rtl8

[PATCH 2/3] staging: rtl8192u: Improve code readability

2017-05-01 Thread Tuomo Rinne
Split the u4bAcParam parameter construction to multiple lines for easier readability. Signed-off-by: Tuomo Rinne <tuomo.ri...@gmail.com> --- drivers/staging/rtl8192u/r8192U_dm.c | 28 +++- 1 file changed, 19 insertions(+), 9 deletions(-) diff --git a/drivers/s

[PATCH 1/1] staging: rtl8192u: Convert u4bAcParam back to little-endian after construction

2017-05-01 Thread Tuomo Rinne
The commit 9304b5b0d4fe ("staging: rtl8192u: Fix sparse warnings in r8192U_dm.c") adds casting of le16 from cpu endianness. This patch converts the u4bAcParam parameter back to little-endian after it has been constructed. Signed-off-by: Tuomo Rinne <tuomo.ri...@gmail.com> ---

Fix kbuild warnings

2017-05-01 Thread Tuomo Rinne
Fix kbuild warnings by passing in variable using address rather than making a copy. Tuomo Rinne (1): staging: rtl8192u: Convert u4bAcParam back to little-endian after construction drivers/staging/rtl8192u/r8192U_dm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) -- 2.1.4

[PATCH 1/3] staging: rtl8192u: Remove unnecessary scope

2017-04-29 Thread Tuomo Rinne
Remove scope unnecessary scope that is already enforced by the if statements scope. Signed-off-by: Tuomo Rinne <tuomo.ri...@gmail.com> --- drivers/staging/rtl8192u/r8192U_dm.c | 66 +--- 1 file changed, 32 insertions(+), 34 deletions(-) diff --git a/d

[PATCH 2/3] staging: rtl8192u: Improve code readability

2017-04-29 Thread Tuomo Rinne
Split the u4bAcParam parameter construction to multiple lines for easier readability. Signed-off-by: Tuomo Rinne <tuomo.ri...@gmail.com> --- drivers/staging/rtl8192u/r8192U_dm.c | 28 +++- 1 file changed, 19 insertions(+), 9 deletions(-) diff --git a/drivers/s

[PATCH 3/3] staging: rtl8192u: Convert u4bAcParam back to little-endian after construction

2017-04-29 Thread Tuomo Rinne
. Unfortunately I don't have the hardware to test this change. Signed-off-by: Tuomo Rinne <tuomo.ri...@gmail.com> --- 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/staging/rtl8192u/r8192U_dm.c

[PATCH] staging: rtl8192u: Improve readability and fix sparse warnings: cast from restricted __le16

2017-04-22 Thread Tuomo Rinne
scope and splitting complex variable construction to multiple lines. Signed-off-by: Tuomo Rinne <tuomo.ri...@gmail.com> --- drivers/staging/rtl8192u/r8192U_dm.c | 77 1 file changed, 42 insertions(+), 35 deletions(-) diff --git a/drivers/staging/rt

[PATCH] staging: vt6655: Copy argument names from function definition to declaration to fix checkpatch warnings

2017-03-15 Thread Tuomo Rinne
Copied function argument names from definition to delcaration. This fixes some checkpatch warnings. Signed-off-by: Tuomo Rinne <tuomo.ri...@gmail.com> --- drivers/staging/vt6655/rxtx.h | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/vt6655/rx

[PATCH] staging: vt6655: Fixed some coding style issues

2017-03-04 Thread Tuomo Rinne
Copied function argument names from definition to delcaration. This fixes some checkpatch warnings. Signed-off-by: Tuomo Rinne <tuomo.ri...@gmail.com> --- drivers/staging/vt6655/rxtx.h | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/vt6655/rx

[PATCH] staging: speakup: Fixed coding style errors and aligned indents

2017-03-02 Thread Tuomo Rinne
Fixed indentation to use tabs and aligned all the fields to same level. Signed-off-by: Tuomo Rinne <tuomo.ri...@gmail.com> --- drivers/staging/speakup/speakup_decpc.c | 90 - 1 file changed, 44 insertions(+), 46 deletions(-) diff --git a/drivers/staging/s