[PATCH] staging: rtl8192u: r8192U_core: Fix compile issue when CONFIG_WIRELESS_EXT is not defined

‘wireless_handlers’ dev->wireless_handlers = (struct iw_handler_def *) &r8192_wx_handlers_def; ^ Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/rtl8192u/r8192U_core.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/staging/rtl8192u/r8192U_core.c b/drivers/staging/r

[PATCH 1/2] staging: olpc_dcon: check for CONFIG_OLPC before calling olpc_board_at_least()

7; This patch fixes these errors. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/olpc_dcon/olpc_dcon.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/drivers/staging/olpc_dcon/olpc_dcon.c b/drivers/staging/olpc_dcon/olpc_dcon.c index 6a9a881..3708f1e 100644 --- a/drive

[PATCH 0/2] staging: olpc_dcon: fix sparse warnings and compile errors

These two patches fix sparse warnings and make olpc_dcon.c build again when CONFIG_OLPC is not set. Murilo Opsfelder Araujo (2): staging: olpc_dcon: check for CONFIG_OLPC before calling olpc_board_at_least() staging: olpc_dcon: fix sparse symbol not declared warning drivers/staging

[PATCH 2/2] staging: olpc_dcon: fix sparse symbol not declared warning

This patch gets rid of the following sparse warning: drivers/staging/olpc_dcon/olpc_dcon.c:787:19: warning: symbol 'dcon_driver' was not declared. Should it be static? Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/olpc_dcon/olpc_dcon.c | 2 +- 1 file changed, 1 inser

[PATCH v3 0/2] staging: iio: accel: Multiple coding style fixes

The following patches fix almost all warnings reported by checkpatch.pl. v2 -> v3 changes: - Folded blank line patches into a single patch. - Fit code in 80-chars limit and still be human-readable. Murilo Opsfelder Araujo (2): staging: iio: accel: Add blank lines between declarations and c

[PATCH v3 2/2] staging: iio: accel: sca3000_core.c: Adjust code to fit 80-chars limit

Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/iio/accel/sca3000_core.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/iio/accel/sca3000_core.c b/drivers/staging/iio/accel/sca3000_core.c index bc53fedb..e4e5639 100644 --- a/drivers/staging/iio

[PATCH v3 1/2] staging: iio: accel: Add blank lines between declarations and code

This patch adds missing blank lines between declarations and code and fixes lines starting by spaces, satisfying checkpatch.pl. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/iio/accel/adis16201_core.c | 5 +++-- drivers/staging/iio/accel/adis16203_core.c | 2 ++ drivers/staging/iio

[PATCH v2 8/8] staging: iio: accel: sca3000_core.c: Add blank line between declarations and code

This patch adds missing blank line between declarations and code, and ajust code to fit 80-chars limit. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/iio/accel/sca3000_core.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/staging/iio/accel

[PATCH v2 6/8] staging: iio: accel: lis3l02dq_core.c: Add blank lines between declarations and code

This patch adds missing blank line between declarations and code, satisfying checkpatch.pl. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/iio/accel/lis3l02dq_core.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/staging/iio/accel/lis3l02dq_core.c b/drivers/staging

[PATCH v2 7/8] staging: iio: accel: lis3l02dq_ring.c: Add blank line between declarations and code

This patch adds missing blank line between declarations and code, satisfying checkpatch.pl. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/iio/accel/lis3l02dq_ring.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/iio/accel/lis3l02dq_ring.c b/drivers/staging/iio

[PATCH v2 2/8] staging: iio: accel: adis16201_core.c: Add blank line between declarations and code

This patch adds missing blank line between declarations and code, and also fixes lines starting by space, satisfying checkpatch.pl. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/iio/accel/adis16201_core.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a

[PATCH v2 4/8] staging: iio: accel: adis16209_core.c: Add blank line between declarations and code

This patch adds missing blank line between declarations and code, satisfying checkpatch.pl. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/iio/accel/adis16209_core.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/iio/accel/adis16209_core.c b/drivers/staging/iio

[PATCH v2 5/8] staging: iio: accel: adis16240_core.c: Add blank line between declarations and code

This patch adds missing blank line between declarations and code, satisfying checkpatch.pl. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/iio/accel/adis16240_core.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/iio/accel/adis16240_core.c b/drivers/staging/iio

[PATCH v2 0/8] staging: iio: accel: Multiple coding style fixes

The following patches fix almost all warnings reported by checkpatch.pl. Changes from v1: - Updated commit messages to better reflect changes. Murilo Opsfelder Araujo (8): staging: iio: accel: adis16203_core.c: Add blank lines between declarations and code staging: iio: accel

[PATCH v2 3/8] staging: iio: accel: adis16204_core.c: Add blank line between declarations and code

This patch adds missing blank line between declarations and code, satisfying checkpatch.pl. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/iio/accel/adis16204_core.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/iio/accel/adis16204_core.c b/drivers/staging/iio

[PATCH v2 1/8] staging: iio: accel: adis16203_core.c: Add blank lines between declarations and code

This patch adds missing blank lines between declarations and code, satisfying checkpatch.pl. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/iio/accel/adis16203_core.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/staging/iio/accel/adis16203_core.c b/drivers/staging

[PATCH 5/8] staging: iio: accel: adis16240_core.c: fix coding style

This patch adds missing blank line after declaration. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/iio/accel/adis16240_core.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/iio/accel/adis16240_core.c b/drivers/staging/iio/accel/adis16240_core.c index

[PATCH 8/8] staging: iio: accel: sca3000_core.c: fix coding style

This patch adds missing blank line after declaration and keep line in 80-chars limit. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/iio/accel/sca3000_core.c |4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/staging/iio/accel/sca3000_core.c b/drivers

[PATCH 6/8] staging: iio: accel: lis3l02dq_core.c: fix coding style

This patch adds missing blank line after declarations. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/iio/accel/lis3l02dq_core.c |4 1 file changed, 4 insertions(+) diff --git a/drivers/staging/iio/accel/lis3l02dq_core.c b/drivers/staging/iio/accel/lis3l02dq_core.c index

[PATCH 7/8] staging: iio: accel: lis3l02dq_ring.c: fix coding style

This patch adds missing blank lines after declarations. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/iio/accel/lis3l02dq_ring.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/iio/accel/lis3l02dq_ring.c b/drivers/staging/iio/accel/lis3l02dq_ring.c index

[PATCH 2/8] staging: iio: accel: adis16201_core.c: fix coding style

This patch adds missing blank line after declaration and fixes lines starting by space. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/iio/accel/adis16201_core.c |5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/staging/iio/accel/adis16201_core.c b

[PATCH 4/8] staging: iio: accel: adis16209_core.c: fix coding style

This patch adds missing blank line after declaration. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/iio/accel/adis16209_core.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/iio/accel/adis16209_core.c b/drivers/staging/iio/accel/adis16209_core.c index

[PATCH 0/8] staging: iio: accel: multiple coding style fixes

The following patches fix (almost) all warnings reported by checkpatch.pl. Murilo Opsfelder Araujo (8): staging: iio: accel: adis16203_core.c: fix coding style staging: iio: accel: adis16201_core.c: fix coding style staging: iio: accel: adis16204_core.c: fix coding style staging: iio

[PATCH 3/8] staging: iio: accel: adis16204_core.c: fix coding style

This patch adds missing blank line after declaration. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/iio/accel/adis16204_core.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/iio/accel/adis16204_core.c b/drivers/staging/iio/accel/adis16204_core.c index

[PATCH 1/8] staging: iio: accel: adis16203_core.c: fix coding style

This patch adds missing blank line after declarations. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/iio/accel/adis16203_core.c |2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/staging/iio/accel/adis16203_core.c b/drivers/staging/iio/accel/adis16203_core.c index

Re: [PATCH] Staging: android: sync.c: fix missing blank line after declaration

On 07/30/2014 09:17 PM, Greg KH wrote: Also doesn't apply, are you sure you are using the staging-next branch of staging.git on git.kernel.org? I was using linux-next, not staging-next. sync.c has no checkpatch.pl warnings in staging-next. Sorry for the false alarm, guys. -- Murilo ___

Re: [PATCH] Staging: android: sw_sync.c: fix missing blank line after declaration

On 07/30/2014 09:17 PM, Greg KH wrote: Does not apply to my tree at all :( I was using linux-next. No checkpatch.pl warnings for sw_sync.c in staging-next. -- Murilo ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverp

[PATCH v2] Staging: android: timed_gpio.c: improved logic of gpio_get_time()

This patch improves the logic of gpio_get_time() and, thereafter, makes checkpatch.pl happy. Signed-off-by: Murilo Opsfelder Araujo --- Thanks for reviweing my patch, Dan. I think I got the proper way without using option --scissors in git-am. What about this new commit message? drivers

[PATCH] Staging: android: timed_gpio.c: improved logic of gpio_get_time()

Thanks for reviewing my first patch, Joe. How about this new one, guys? -- >8 -- Consequently, made checkpatch.pl happy. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/android/timed_gpio.c | 15 --- 1 file changed, 8 insertions(+), 7 deletions(-) diff --gi

[PATCH] Staging: bcm: Bcmchar.c: remove else statement after return

This patch makes checkpatch.pl script happier by fixing all warnings related to else statement after break or return. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/bcm/Bcmchar.c | 21 ++--- 1 file changed, 10 insertions(+), 11 deletions(-) diff --git a/drivers

[PATCH] Staging: android: timed_output.c: use kstrtoint() instead of sscanf()

This patch makes checkpatch.pl happy by fixing the following warning: WARNING: Prefer kstrto to single variable sscanf Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/android/timed_output.c |4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/staging

[PATCH] Staging: android: timed_gpio.c: remove else statement after return

This patch makes checkpatch.pl script happy by fixing the following warning: WARNING: else is not generally useful after a break or return Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/android/timed_gpio.c |5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a

[PATCH] Staging: android: sync.c: fix missing blank line after declaration

Fix coding style issue. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/android/sync.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/android/sync.c b/drivers/staging/android/sync.c index e7b2e02..69c16cb 100644 --- a/drivers/staging/android

[PATCH] Staging: android: sw_sync.c: fix missing blank line after declaration

Fix coding style issue. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/android/sw_sync.c |2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/staging/android/sw_sync.c b/drivers/staging/android/sw_sync.c index a76db3f..863d4b1 100644 --- a/drivers/staging/android