[PATCH 13/20] staging: rtlwifi: halmac: mark expected switch fall-through

2017-10-12 Thread Gustavo A. R. Silva
In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Signed-off-by: Gustavo A. R. Silva <garsi...@embeddedor.com> --- drivers/staging/rtlwifi/halmac/rtl_halmac.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/d

[PATCH] staging: vt6655: mark expected switch fall-throughs in vnt_set_keymode

2017-10-10 Thread Gustavo A. R. Silva
In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Signed-off-by: Gustavo A. R. Silva <garsi...@embeddedor.com> --- drivers/staging/vt6655/key.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/staging/

[PATCH] staging: typec: tcpci: mark expected switch fall-through in tcpci_to_typec_cc

2017-10-10 Thread Gustavo A. R. Silva
In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Signed-off-by: Gustavo A. R. Silva <garsi...@embeddedor.com> --- drivers/staging/typec/tcpci.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/typec/tcpci.c b/d

[PATCH] staging: rtlwifi: mark expected switch fall-through in rtl_make_smps_action

2017-10-10 Thread Gustavo A. R. Silva
In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Signed-off-by: Gustavo A. R. Silva <gust...@embeddedor.com> --- drivers/staging/rtlwifi/base.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/s

[PATCH] staging: iio: adc: fix error return code in ad7606_par_probe()

2017-08-09 Thread Gustavo A. R. Silva
of Coccinelle. Signed-off-by: Gustavo A. R. Silva <gust...@embeddedor.com> --- drivers/staging/iio/adc/ad7606_par.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/iio/adc/ad7606_par.c b/drivers/staging/iio/adc/ad7606_par.c index cd6c410c..3eb6f8f

[PATCH] staging: bcm2835-audio: constify snd_pcm_ops structures

2017-07-20 Thread Gustavo A. R. Silva
, e2; position p; @@ snd_pcm_set_ops(e1, e2, @p) @bad@ position p != {r.p,ok1.p,ok2.p}; identifier r.i; struct snd_pcm_ops e; @@ e@i@p @depends on !bad disable optional_qualifier@ identifier r.i; @@ static +const struct snd_pcm_ops i = { ... }; Signed-off-by: Gustavo A. R. Silva <g

[PATCH 2/2] staging: ccree: use sizeof(*var) in kmalloc

2017-07-09 Thread Gustavo A. R. Silva
Fix the following checkpatch warning: CHECK: Prefer kmalloc(sizeof(*buff_mgr_handle)...) over kmalloc(sizeof(struct buff_mgr_handle)...) Signed-off-by: Gustavo A. R. Silva <garsi...@embeddedor.com> --- drivers/staging/ccree/ssi_buffer_mgr.c | 2 +- 1 file changed, 1 insertion(+), 1 de

[PATCH 1/2] staging: ccree: remove unnecessary cast on kmalloc

2017-07-09 Thread Gustavo A. R. Silva
A. R. Silva <garsi...@embeddedor.com> --- drivers/staging/ccree/ssi_buffer_mgr.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/ccree/ssi_buffer_mgr.c b/drivers/staging/ccree/ssi_buffer_mgr.c index b35871e..18a8694 100644 --- a/drivers/staging

[PATCH] staging: lustre: ptlrpc: remove unnecessary code

2017-05-10 Thread Gustavo A. R. Silva
offset is an unsigned variable and, greater-than-or-equal-to-zero comparison of an unsigned variable is always true. Addresses-Coverity-ID: 1373919 Signed-off-by: Gustavo A. R. Silva <garsi...@embeddedor.com> --- drivers/staging/lustre/lustre/ptlrpc/layout.c | 2 +- 1 file changed, 1 ins

[PATCH] staging: speakup: fix warning for static declaration

2017-03-27 Thread Gustavo A. R. Silva
Fix the following sparse warning: symbol 'spk_serial_out' was not declared. Should it be static? Signed-off-by: Gustavo A. R. Silva <garsi...@embeddedor.com> --- drivers/staging/speakup/serialio.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/s

[PATCH] Staging: lustre: lustre: lmv: Compress return logic into one line.

2017-01-04 Thread Gustavo A. R. Silva
Simplify return logic to avoid unnecessary variable assignments. These issues were detected using Coccinelle and the following semantic patch: @@ local idexpression ret; expression e; @@ -ret = +return e; -return ret; Signed-off-by: Gustavo A. R. Silva <garsi...@embeddedor.

[PATCH] Staging: fbtft: Fix too many leading tabs warning

2016-10-11 Thread Gustavo A. R. Silva
This is a patch to the fb_agm1264k-fl.c file that fixes up the following issue: WARNING: Too many leading tabs - consider code refactoring Add new function iterate_diffusion_matrix() to refactor a piece of code inside write_vmem(). Signed-off-by: Gustavo A. R. Silva <garsi...@embeddedor.

[PATCH 2/2] Staging: comedi: kcomedilib: Fixed a typo

2015-01-11 Thread Gustavo A. R. Silva
Fixed a typo in kcomedilib_main.c file. Signed-off-by: Gustavo A. R. Silva garsi...@embeddedor.com --- drivers/staging/comedi/kcomedilib/kcomedilib_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/comedi/kcomedilib/kcomedilib_main.c b/drivers/staging

[PATCH 3/3] Staging: nvec: Fixed a typo

2015-01-11 Thread Gustavo A. R. Silva
Fixed a typo in nvec.c file Signed-off-by: Gustavo A. R. Silva garsi...@embeddedor.com --- drivers/staging/nvec/nvec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/nvec/nvec.c b/drivers/staging/nvec/nvec.c index 093535c..2637059 100644 --- a/drivers/staging

[PATCH 4/4] Staging: octeon-usb: Fixed a typo

2015-01-11 Thread Gustavo A. R. Silva
Fixed a typo in octeon-hcd.c file Signed-off-by: Gustavo A. R. Silva garsi...@embeddedor.com --- 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

[PATCH] Staging: vt6656: Fixed typo in a variable name

2015-01-11 Thread Gustavo A. R. Silva
Fixed typo in a variable name in dpc.h file. Signed-off-by: Gustavo A. R. Silva garsi...@embeddedor.com --- drivers/staging/vt6656/dpc.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/vt6656/dpc.h b/drivers/staging/vt6656/dpc.h index fab195f..95e0e83 100644

[PATCH] Staging: octeon-usb: Fixed a typo

2015-01-11 Thread Gustavo A. R. Silva
Fixed a typo in octeon-hcd.c file Signed-off-by: Gustavo A. R. Silva garsi...@embeddedor.com --- 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

[PATCH] Staging: nvec: Fixed a typo

2015-01-11 Thread Gustavo A. R. Silva
Fixed a typo in nvec.c file Signed-off-by: Gustavo A. R. Silva garsi...@embeddedor.com --- drivers/staging/nvec/nvec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/nvec/nvec.c b/drivers/staging/nvec/nvec.c index 093535c..2637059 100644 --- a/drivers/staging

[PATCH] Staging: vt6656: Fixed typo in a macro name

2015-01-11 Thread Gustavo A. R. Silva
Fixed typo in a macro name in card.c and device.h files. Signed-off-by: Gustavo A. R. Silva garsi...@embeddedor.com --- drivers/staging/vt6656/card.c | 2 +- drivers/staging/vt6656/device.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/vt6656/card.c b

[PATCH] Staging: comedi: kcomedilib: Fixed a typo

2015-01-11 Thread Gustavo A. R. Silva
Fixed a typo in kcomedilib_main.c file. Signed-off-by: Gustavo A. R. Silva garsi...@embeddedor.com --- drivers/staging/comedi/kcomedilib/kcomedilib_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/comedi/kcomedilib/kcomedilib_main.c b/drivers/staging

[PATCH] Staging: octeon: Fixed a typo

2015-01-11 Thread Gustavo A. R. Silva
Fixed a typo in ethernet.c file. Signed-off-by: Gustavo A. R. Silva garsi...@embeddedor.com --- drivers/staging/octeon/ethernet.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/octeon/ethernet.c b/drivers/staging/octeon/ethernet.c index ee32149..460e854

[PATCH] Staging: olpc_dcon: Fixed a typo

2015-01-11 Thread Gustavo A. R. Silva
Fixed a typo in olpc_dcon_xo_1.c file. Signed-off-by: Gustavo A. R. Silva garsi...@embeddedor.com --- drivers/staging/olpc_dcon/olpc_dcon_xo_1.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/olpc_dcon/olpc_dcon_xo_1.c b/drivers/staging/olpc_dcon

[PATCH] Staging: vt6655: Fixed a typo

2015-01-11 Thread Gustavo A. R. Silva
Fixed a typo in mac.c file. Signed-off-by: Gustavo A. R. Silva garsi...@embeddedor.com --- drivers/staging/vt6655/mac.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/vt6655/mac.c b/drivers/staging/vt6655/mac.c index 8f0d652..1975c91 100644 --- a/drivers

<    1   2