Re: [PATCH v2] staging: wilc1000: Fix sparse warnings incorrect type assignment

2017-03-08 Thread Andrea Ghittino
Thank you for your time regard, andrea On Wed, Mar 8, 2017 at 7:22 AM, Dan Carpenter wrote: > I think this change is buggy. > > On Tue, Mar 07, 2017 at 10:36:53PM +0100, Andrea Ghittino wrote: >> Fixed sparse warnings related to the conversion of le16 and le32 to u16 and >

Re: [PATCH] staging: unisys: fix sparse warnings

2017-03-07 Thread Andrea Ghittino
On Tue, Mar 07, 2017 at 11:17:36PM +0300, Dan Carpenter wrote: > On Sat, Mar 04, 2017 at 06:20:50PM +0100, Andrea Ghittino wrote: > > Sparse generates two warnings related to incorrect type in assignment. > > This patch changes the types in the struct defined in unisys > > Ca

[PATCH v2] staging: wilc1000: Fix sparse warnings incorrect type assignment

2017-03-07 Thread Andrea Ghittino
//wilc_wfi_cfgoperations.c:2084:51:expected unsigned int [unsigned] [assigned] [usertype] ht_tx_bf_cap drivers/staging/wilc1000//wilc_wfi_cfgoperations.c:2084:51:got restricted __le32 const [usertype] tx_BF_cap_info Signed-off-by: Andrea Ghittino --- changelog: v2) Changed patch

[PATCH] staging: wilc1000: Fix sparse warnings incorrect type assignment

2017-03-06 Thread Andrea Ghittino
/wilc1000//wilc_wfi_cfgoperations.c:2084:51:got restricted __le32 const [usertype] tx_BF_cap_info Signed-off-by: Andrea Ghittino --- Compile tested only wilc_wfi_cfgoperations.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/wilc1000

[PATCH] staging: unisys: fix sparse warnings

2017-03-04 Thread Andrea Ghittino
Sparse generates two warnings related to incorrect type in assignment. This patch changes the types in the struct defined in unisys Signed-off-by: Andrea Ghittino --- Compile tested only iochannel.h |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging

[PATCH v4] staging: greybus: arche-platform: fix line over 80 characters style warnings

2016-12-01 Thread Andrea Ghittino
Fixes greybus "line over 80 characters" style warnings found by checkpatch.pl tool Signed-off-by: Andrea Ghittino --- changelog: v2) Review patch based on Vaibhav Hiremath review v3) Fixed email text v4) Review the patch based on received comments drivers/staging/greybus/arche-apb-ctr

Re: [PATCH] staging: greybuis: fix permission style warnings

2016-11-30 Thread Andrea Ghittino
On Wed, Nov 30, 2016 at 12:21 AM, Dan Carpenter wrote: > On Tue, Nov 29, 2016 at 03:59:46PM -0600, Alex Elder wrote: >> On 11/26/2016 03:50 PM, Andrea Ghittino wrote: >> > Fixes greybus user/groups permission style warnings >> > found by checkpatch.pl tool >> >

[PATCH v3] staging: sm750fb: fix tabstop style warning

2016-11-29 Thread Andrea Ghittino
Fixes sm750fb tabstop style warning found by checkpatch.pl tool Signed-off-by: Andrea Ghittino --- changelog: v2) Fixed email subject v3) Fixed email format drivers/staging/sm750fb/sm750_accel.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/sm750fb

[PATCH v3] staging: greybus: arche-platform: fix line over 80 characters style warnings

2016-11-29 Thread Andrea Ghittino
Fixes greybus "line over 80 characters" style warnings found by checkpatch.pl tool Signed-off-by: Andrea Ghittino --- changelog: v2) Review patch based on Vaibhav Hiremath review v3) Fixed email text drivers/staging/greybus/arche-apb-ctrl.c |5 +++ drivers/staging/greybus/arche-

[PATCH v2] staging: sm750fb: fix tabstop style warning

2016-11-29 Thread Andrea Ghittino
Fixes sm750fb tabstop style warning found by checkpatch.pl tool Signed-off-by: Andrea Ghittino --- diff --git a/drivers/staging/sm750fb/sm750_accel.c b/drivers/staging/sm750fb/sm750_accel.c index 6fd18d2..af0db57 100644 --- a/drivers/staging/sm750fb/sm750_accel.c +++ b/drivers/staging/sm750fb

[PATCH v2] staging: greybus: arche-platform: fix line over 80 characters style warnings

2016-11-29 Thread Andrea Ghittino
Fixes greybus "line over 80 characters" style warnings found by checkpatch.pl tool Signed-off-by: Andrea Ghittino --- diff --git a/drivers/staging/greybus/arche-apb-ctrl.c b/drivers/staging/greybus/arche-apb-ctrl.c index 3fda0cd..755120a 100644 --- a/drivers/staging/greybus/arche-

[PATCH] staging: sm750fb: fix function definition argument style warning

2016-11-28 Thread Andrea Ghittino
Fixes sm750fb function definition argument style warning found by checkpatch.pl tool Signed-off-by: Andrea Ghittino --- diff --git a/drivers/staging/sm750fb/ddk750_chip.h b/drivers/staging/sm750fb/ddk750_chip.h index e97e859..30653dd 100644 --- a/drivers/staging/sm750fb/ddk750_chip.h +++ b

Re: [PATCH] staging: sm750fb: fix function definition argument style warning

2016-11-28 Thread Andrea Ghittino
On Mon, Nov 28, 2016 at 12:48 AM, Randy Dunlap wrote: > On 11/27/16 13:11, Andrea Ghittino wrote: >> Fixes sm750fb function definition argument style warning >> found by checkpatch.pl tool >> >> Signed-off-by: Andrea Ghittino >> --- > > Does checkpatch no

[PATCH] staging: greybus: fix line over 80 characters style warnings

2016-11-28 Thread Andrea Ghittino
Fixes greybus "line over 80 characters" style warnings found by checkpatch.pl tool Signed-off-by: Andrea Ghittino --- diff --git a/drivers/staging/greybus/arche-apb-ctrl.c b/drivers/staging/greybus/arche-apb-ctrl.c index 3fda0cd..755120a 100644 --- a/drivers/staging/greybus/arche-

[PATCH] staging: greybus: fix line over 80 characters style warnings

2016-11-28 Thread Andrea Ghittino
Fixes greybus "line over 80 characters" style warnings found by checkpatch.pl tool Signed-off-by: Andrea Ghittino --- diff --git a/drivers/staging/greybus/arche-apb-ctrl.c b/drivers/staging/greybus/arche-apb-ctrl.c index 3fda0cd..755120a 100644 --- a/drivers/staging/greybus/arche-

Re: [PATCH] staging: greybuis: fix line over 80 characters style warnings

2016-11-28 Thread Andrea Ghittino
On Mon, Nov 28, 2016 at 10:14:03AM +0530, Vaibhav Hiremath wrote: > On Sun, Nov 27, 2016 at 3:43 AM, Andrea Ghittino wrote: > > Fixes greybus "line over 80 characters" style warnings > > found by checkpatch.pl tool > > > > It is always trade-off between re

[PATCH] staging: sm750fb: fix function definition argument style warning

2016-11-27 Thread Andrea Ghittino
Fixes sm750fb function definition argument style warning found by checkpatch.pl tool Signed-off-by: Andrea Ghittino --- diff --git a/drivers/staging/sm750fb/ddk750_chip.h b/drivers/staging/sm750fb/ddk750_chip.h index e97e859..30653dd 100644 --- a/drivers/staging/sm750fb/ddk750_chip.h +++ b

[PATCH] staging: sm750fb: fix tabstop style warning

2016-11-27 Thread Andrea Ghittino
Fixes sm750fb tabstop style warning found by checkpatch.pl tool Signed-off-by: Andrea Ghittino --- diff --git a/drivers/staging/sm750fb/sm750_accel.c b/drivers/staging/sm750fb/sm750_accel.c index 6fd18d2..af0db57 100644 --- a/drivers/staging/sm750fb/sm750_accel.c +++ b/drivers/staging/sm750fb

[PATCH] staging: sm750fb: fix function definition argument style warning

2016-11-27 Thread Andrea Ghittino
Fixes sm750fb function definition argument style warning found by checkpatch.pl tool Signed-off-by: Andrea Ghittino --- diff --git a/drivers/staging/sm750fb/ddk750_chip.h b/drivers/staging/sm750fb/ddk750_chip.h index e97e859..30653dd 100644 --- a/drivers/staging/sm750fb/ddk750_chip.h +++ b

[PATCH] staging: sm750fb: fix permission style warning

2016-11-27 Thread Andrea Ghittino
Fixes sm750fb user/groups permission style warning found by checkpatch.pl tool Signed-off-by: Andrea Ghittino --- diff --git a/drivers/staging/sm750fb/sm750.c b/drivers/staging/sm750fb/sm750.c index 2d22c51..2d33b80 100644 --- a/drivers/staging/sm750fb/sm750.c +++ b/drivers/staging/sm750fb/sm750

[PATCH] staging: greybuis: fix line over 80 characters style warnings

2016-11-26 Thread Andrea Ghittino
Fixes greybus "line over 80 characters" style warnings found by checkpatch.pl tool Signed-off-by: Andrea Ghittino --- diff --git a/drivers/staging/greybus/arche-apb-ctrl.c b/drivers/staging/greybus/arche-apb-ctrl.c index 3fda0cd..755120a 100644 --- a/drivers/staging/greybus/arche-

[PATCH] staging: greybuis: fix permission style warnings

2016-11-26 Thread Andrea Ghittino
Fixes greybus user/groups permission style warnings found by checkpatch.pl tool Signed-off-by: Andrea Ghittino --- diff --git a/drivers/staging/greybus/camera.c b/drivers/staging/greybus/camera.c index 1c5b41a..4424f63 100644 --- a/drivers/staging/greybus/camera.c +++ b/drivers/staging/greybus

[PATCH] staging: octeon-usb: fix code style warnings

2016-11-23 Thread Andrea Ghittino
This patch fix two code style warnings related to "line over 80 characters" in drivers/staging/octeon-usb/octeon-hcd.c file. Signed-off-by: Andrea Ghittino --- diff --git a/drivers/staging/octeon-usb/octeon-hcd.c b/drivers/staging/octeon-usb/octeon-hcd.c index 9a7858a..92f3a7e 10

[PATCH] staging: octeon-usb: fix code style warnings

2016-11-22 Thread Andrea Ghittino
This patch fix two code style warnings related to "line over 80 characters" in drivers/staging/octeon-usb/octeon-hcd.c file. Signed-off-by: Andrea Ghittino --- diff --git a/drivers/staging/octeon-usb/octeon-hcd.c b/drivers/staging/octeon-usb/octeon-hcd.c index 9a7858a..92f3a7e 10

[PATCH] staging: octeon-usb: fix code style warnings

2016-11-22 Thread Andrea Ghittino
This patch fix two code style warnings related to "line over 80 characters" in drivers/staging/octeon-usb/octeon-hcd.c file. index 9a7858a..92f3a7e 100644 --- a/drivers/staging/octeon-usb/octeon-hcd.c +++ b/drivers/staging/octeon-usb/octeon-hcd.c @@ -2636,12 +2636,12 @@ static int cvmx_usb_poll_c