[PATCH] drivers/staging/comedi/drivers/pcl816.c: style fix

2015-06-16 Thread Guillermo O. Freschi
Removed some spaces before a tab character. --- drivers/staging/comedi/drivers/pcl816.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/comedi/drivers/pcl816.c b/drivers/staging/comedi/drivers/pcl816.c index 1ccb2f1..781b321 100644 --- a/drivers/staging/comedi/

[PATCH v2] drivers/staging/comedi/drivers/pcl816.c: style fix

2015-06-16 Thread Guillermo O. Freschi
Removed some spaces before a tab character. Signed-off-by: Guillermo O. Freschi --- drivers/staging/comedi/drivers/pcl816.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/comedi/drivers/pcl816.c b/drivers/staging/comedi/drivers/pcl816.c index 1ccb2f1

[PATCH] ion_chunk_heap.c: Fixed line over 80 characters

2015-06-21 Thread Guillermo O. Freschi
Simple style fix. Signed-off-by: Guillermo O. Freschi --- drivers/staging/android/ion/ion_chunk_heap.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/android/ion/ion_chunk_heap.c b/drivers/staging/android/ion/ion_chunk_heap.c index 5474615..0813163

[PATCH v2] Style fixes

2017-01-13 Thread Guillermo O. Freschi
Missing braces on `if` statement. Signed-off-by: Guillermo O. Freschi Reviewed-by: Andreas Dilger --- drivers/staging/lustre/lustre/ldlm/ldlm_lock.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/drivers/staging/lustre/lustre/ldlm/ldlm_lock.c b/drivers/staging

[PATCH 2/2] Style fixes

2016-12-26 Thread Guillermo O. Freschi
Missing braces on `if` statement; misaligned parameter. Signed-off-by: Guillermo O. Freschi --- drivers/staging/lustre/lustre/ldlm/ldlm_lock.c | 12 +++- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/drivers/staging/lustre/lustre/ldlm/ldlm_lock.c b/drivers/staging

[PATCH 1/2] Fixed signedness check

2016-12-26 Thread Guillermo O. Freschi
Was `unsigned int`, but `enum`s are signed. Signed-off-by: Guillermo O. Freschi --- drivers/staging/lustre/lustre/ldlm/ldlm_lock.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/ldlm/ldlm_lock.c b/drivers/staging/lustre/lustre/ldlm/ldlm_lock.c

[PATCH 2/2] drivers: staging/lustre: conrpc.c - fix sparse warning: warning: incorrect type in argument * (different address spaces)

2017-06-07 Thread Guillermo O. Freschi
The calls to `copy_to_user` suggest that `ent` is pointing to userspace memory, but `ent` is missing the `__user` qualifier. Signed-off-by: Guillermo O. Freschi --- drivers/staging/lustre/lnet/selftest/conrpc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging

[PATCH 1/2] drivers: staging/lustre: conrpc.c - fix sparse error: incompatible types in comparison expression (different address spaces)

2017-06-07 Thread Guillermo O. Freschi
This patch reveals hidden warnings after this line. (This is a stepping-stone patch for the following one, it should be ignored as per Andreas Dilger.) Signed-off-by: Guillermo O. Freschi --- drivers/staging/lustre/lnet/selftest/conrpc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[PATCH] drivers: staging/lustre: conrpc.c - fix sparse error: incompatible types in comparison expression (different address spaces)

2017-05-05 Thread Guillermo O. Freschi
Signed-off-by: Guillermo O. Freschi --- drivers/staging/lustre/lnet/selftest/conrpc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lnet/selftest/conrpc.c b/drivers/staging/lustre/lnet/selftest/conrpc.c index da36c55b86d3..f4578200994f 100644 --- a

[PATCH] staging/lustre: echo_client.c - fix sparse warning: cast removes address space of expression

2017-06-07 Thread Guillermo O. Freschi
The function only cares about the value of the pointer, not the contents of the pointed-to data. Adding a `__force` annotation to the cast removes this warning. Signed-off-by: Guillermo O. Freschi --- drivers/staging/lustre/lustre/obdecho/echo_client.c | 2 +- 1 file changed, 1 insertion(+), 1