Re: [PATCH] staging/iio/accel: checkpatch: fixed long lines by moving comments

2014-12-26 Thread Jonathan Cameron
On 22/12/14 13:48, Sudip Mukherjee wrote: > On Mon, Dec 22, 2014 at 02:23:42AM +0100, Helene Gsaenger wrote: >> Signed-off-by: Helene Gsaenger >> Signed-off-by: Simone Weiss >> --- >> drivers/staging/iio/accel/adis16203.h | 41 >> ++- >> 1 file changed, 31 insert

Re: [PATCH] staging : iio: meter: ade7759: fix space before , coding style issue

2014-12-26 Thread Jonathan Cameron
On 21/12/14 12:42, Mohammad Jamal wrote: > This patch solves the space before , coding style issue found by > checkpatch in ade7759.c > > Signed-off-by: Mohammad Jamal Fixed by Zachery Warren back in November. I'm a bit behind with sending stuff upstream so won't have hit anything other than the

Re: [PATCH] staging: iio: adc: ad7192: fix space before , coding style issue

2014-12-26 Thread Jonathan Cameron
On 21/12/14 12:35, Mohammad Jamal wrote: > This patch solves the space before , error of the checkpatch.pl > > Signed-off-by: Mohammad Jamal Also fixed in that earlier patch from Zachery. > --- > drivers/staging/iio/adc/ad7192.c |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > dif

[PATCH] Staging: line6: pcm: Corrected checkpatch notices in pcm.h

2014-12-26 Thread Damon Swayn
Fixed a coding style issue Signed-off-by: Damon Swayn --- drivers/staging/line6/pcm.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/line6/pcm.h b/drivers/staging/line6/pcm.h index 6aa0d46..5d87934 100644 --- a/drivers/staging/line6/pcm.h +++ b/drivers/

Re: [PATCHv2 5/8] staging: iio: isl29028: deprecate use of isl in compatible string for isil

2014-12-26 Thread Jonathan Cameron
On 16/12/14 21:20, Arnaud Ebalard wrote: > > "isil" and "isl" prefixes are used at various locations inside the kernel > to reference Intersil corporation. This patch is part of a series fixing > those locations were "isl" is used in compatible strings to use the now > expected "isil" prefix inste

Re: [PATCH] staging: iio: accel: sca3000_core.c: Remove unused function

2014-12-26 Thread Jonathan Cameron
On 20/12/14 14:53, Rickard Strandqvist wrote: > Remove the function sca3000_check_status() that is not used anywhere. > > This was partially found by using a static code analysis program called > cppcheck. > > Signed-off-by: Rickard Strandqvist Applied to the togreg branch of iio.git - initiall

Re: [PATCH] Staging: line6: pcm: Corrected checkpatch notices in pcm.h

2014-12-26 Thread Konrad Zapalowicz
On 12/26, Damon Swayn wrote: > Fixed a coding style issue You need to be more specific here, describe what kind of issue this patch is fixing. Keep in mind that this message, when formulated properly, will end up as a comment to the Linux kernel source - it must, therefore, contain valuable inform

[PATCH] staging: lustre: do not ignore try_module_get() fail in obd_class_open()

2014-12-26 Thread Alexey Khoroshilov
obd_class_open() ignores error code of try_module_get(), while it can lead to race with module unload. Found by Linux Driver Verification project (linuxtesting.org). Signed-off-by: Alexey Khoroshilov --- drivers/staging/lustre/lustre/obdclass/linux/linux-module.c | 3 +-- 1 file changed, 1 inse

Prezado usuário

2014-12-26 Thread ADMIN
Prezado usuário Seu e-mail excedeu 2GB criado pelo webmaster, que atualmente está rodando a 2,30 GB, que não é possível enviar ou receber novas mensagens dentro nas próximas 24 horas, até que tenha certeza de que você conta de e-mail. Por favor, digite seus dados abaixo para verificar a sua con

[PATCH] Staging: line6: Updated comma spacing to fit within coding style

2014-12-26 Thread Damon Swayn
Fixed a coding style issue reported by checkpatch surrounding the spacing of comma's Signed-off-by: Damon Swayn --- drivers/staging/line6/pcm.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/line6/pcm.h b/drivers/staging/line6/pcm.h index 6aa0d46..5d879

Re: [PATCH v2] staging: comedi: dmm32at: fix style issues

2014-12-26 Thread Sudip Mukherjee
On Thu, Dec 25, 2014 at 12:28:28PM -0800, David Decotigny wrote: > Before: > 1 ERROR: code indent should use tabs where possible > 1 WARNING: please, no spaces at the start of a line > > After: > (none) > > Signed-off-by: David Decotigny > --- > drivers/staging/comedi/drivers/dmm3

Re: [PATCH] staging/iio/accel: checkpatch: fixed long lines by moving comments

2014-12-26 Thread Sudip Mukherjee
On Fri, Dec 26, 2014 at 10:00:00AM +, Jonathan Cameron wrote: > On 22/12/14 13:48, Sudip Mukherjee wrote: > > On Mon, Dec 22, 2014 at 02:23:42AM +0100, Helene Gsaenger wrote: > >> Signed-off-by: Helene Gsaenger > >> Signed-off-by: Simone Weiss > >> --- > >> drivers/staging/iio/accel/adis1620