[PATCH] Staging: fix coding style in rtl8188eu/core

Set constant on the left of the test, and jump a new line to avoid to exceed the 80 char length limit. Signed-off-by: Colin Vidal --- drivers/staging/rtl8188eu/core/rtw_iol.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/staging/rtl8188eu/core/rtw_iol.c b

Re: [PATCH] Staging: fix coding style in rtl8188eu/core

On Tue, 2016-02-02 at 13:14 -0800, Joe Perches wrote: > On Tue, 2016-02-02 at 21:57 +0100, Colin Vidal wrote: > > Set constant on the left of the test, and jump a new line to avoid > > to > > exceed the 80 char length limit. > [] > > diff --git a/drivers/staging/

[PATCH] Staging: fix coding style in rtl8188eu/core

Set constant operand on right of test, and refactor the code in a more compact and readable way. Signed-off-by: Colin Vidal --- drivers/staging/rtl8188eu/core/rtw_iol.c | 13 + 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/drivers/staging/rtl8188eu/core/rtw_iol.c b

Re: [PATCH] Staging: fix coding style in rtl8188eu/core

On Tue, 2016-02-02 at 23:54 +0100, Colin Vidal wrote: > Set constant operand on right of test, and refactor the code in a > more > compact and readable way. > > Signed-off-by: Colin Vidal > --- >  drivers/staging/rtl8188eu/core/rtw_iol.c | 13 + >  1 file cha

[PATCH v2] Staging: fix coding style in rtl8188eu/core

Set constant operand on right of test, and refactor the code in a more compact and readable way. Signed-off-by: Colin Vidal --- drivers/staging/rtl8188eu/core/rtw_iol.c | 13 + 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/drivers/staging/rtl8188eu/core/rtw_iol.c b

[PATCH v3 1/5] Staging: rtl8188eu/core: remove paragraph which mention FSF address in comment header

Signed-off-by: Colin Vidal --- drivers/staging/rtl8188eu/core/rtw_iol.c | 4 1 file changed, 4 deletions(-) diff --git a/drivers/staging/rtl8188eu/core/rtw_iol.c b/drivers/staging/rtl8188eu/core/rtw_iol.c index cdcf0ea..000a81c 100644 --- a/drivers/staging/rtl8188eu/core/rtw_iol.c +++ b

[PATCH v3 3/5] Staging: rtl8188eu/core: Coding style fix, set constant operand on right in tests

Signed-off-by: Colin Vidal --- drivers/staging/rtl8188eu/core/rtw_iol.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/rtl8188eu/core/rtw_iol.c b/drivers/staging/rtl8188eu/core/rtw_iol.c index 7642c22..3dd1a9a 100644 --- a/drivers/staging/rtl8188eu/core

[PATCH v3 5/5] Staging: rtl8188eu/core: make core more readable

Signed-off-by: Colin Vidal --- drivers/staging/rtl8188eu/core/rtw_iol.c | 10 +++--- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/drivers/staging/rtl8188eu/core/rtw_iol.c b/drivers/staging/rtl8188eu/core/rtw_iol.c index 2e2145c..ba6c492 100644 --- a/drivers/staging

[PATCH v3 4/5] Staging: rtl8188eu/core: Coding style fix, avoid line over 80 characters

Signed-off-by: Colin Vidal --- drivers/staging/rtl8188eu/core/rtw_iol.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/rtl8188eu/core/rtw_iol.c b/drivers/staging/rtl8188eu/core/rtw_iol.c index 3dd1a9a..2e2145c 100644 --- a/drivers/staging/rtl8188eu/core

[PATCH v3 2/5] Staging: rtl8188eu/core: Coding style fix, set conform spaces between identifiers

Signed-off-by: Colin Vidal --- drivers/staging/rtl8188eu/core/rtw_iol.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/rtl8188eu/core/rtw_iol.c b/drivers/staging/rtl8188eu/core/rtw_iol.c index 000a81c..7642c22 100644 --- a/drivers/staging/rtl8188eu/core

Re: [PATCH v3 5/5] Staging: rtl8188eu/core: make core more readable

On Sun, 2016-02-14 at 16:39 -0800, Greg Kroah-Hartman wrote: > On Fri, Feb 12, 2016 at 07:05:53PM +0100, Colin Vidal wrote: > > Signed-off-by: Colin Vidal > > --- > >  drivers/staging/rtl8188eu/core/rtw_iol.c | 10 +++--- > >  1 file changed, 3 insertions(+), 7 delet

[PATCH v4 1/4] Staging: rtl8188eu/core: remove paragraph which mention FSF address in comment header

As FSF address changed in the past, and can change in the future, remove the address paragraph in the comment header, and avoid a warning of checkpatch. Signed-off-by: Colin Vidal --- drivers/staging/rtl8188eu/core/rtw_iol.c | 4 1 file changed, 4 deletions(-) diff --git a/drivers/staging

[PATCH v4 3/4] Staging: rtl8188eu/core: Coding style fix, set constant operand on right in tests

Remove a checkpatch warning, putting constant operant on right of two tests. Signed-off-by: Colin Vidal --- drivers/staging/rtl8188eu/core/rtw_iol.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/rtl8188eu/core/rtw_iol.c b/drivers/staging/rtl8188eu/core

[PATCH v4 4/4] Staging: rtl8188eu/core: Coding style fix, avoid line over 80 characters

Jump a new line after and operator of the test. It avoids to exceed 80 chars line, and remove a checkpatch warning. Signed-off-by: Colin Vidal --- drivers/staging/rtl8188eu/core/rtw_iol.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/rtl8188eu/core

[PATCH v4 2/4] Staging: rtl8188eu/core: Coding style fix, set conform spaces between identifiers

And a space after include keyword, making the preprocessor more readable, and remove an unexpected space between a type and variable name. Signed-off-by: Colin Vidal --- drivers/staging/rtl8188eu/core/rtw_iol.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers

[PATCH] staging/rtl8192u: add endianness conversions

Fields frag_size and playload_size of struct ieee80211_txb are encoded as short little-endian. This patch adds conversions to / from cpu byte order when copy / write these values in variables of architecture independent byte order. It also avoid a sparse type warning. Signed-off-by: Colin Vidal