[PATCHv6 4/5] Staging: most: hdm-dim2/dim2_hal.c. Fix Using plain integer as NULL pointer warnings

This patch fixes the warning generated by sparse: Using plain integer as NULL pointer by replacing the offending 0 with NULL. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/hdm-dim2/dim2_hal.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCHv6 2/5] Staging: most: mostcore/core.c. Fix Using plain integer as NULL pointer warnings

This patch fixes the warning generated by sparse: Using plain integer as NULL pointer by replacing the offending 0 with NULL. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- v6: styling. Change the NULL test to comply with the Kernel coding style drivers/staging/most/mostcore/core.c

[PATCHv6 1/5] Staging: most: mostcore/core.c. Fix missing static keyword warnings

This is a patch to the mostcore/core.c file. It makes several local functions and structures static to prevent global visibility. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- v6: fixed warning line over 80 characters as pointed out by Sudip Mukherjee drivers/staging/most

[PATCHv6 3/5] Staging: most: hdm-usb/hdm_usb.c. Fix missing static keyword warnings

This is a patch to the most/hdm-usb/hdm_usb.c file. It makes several local functions and structures static to prevent global visibility. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- v6: fixed alignment with parameter list of a function. drivers/staging/most/hdm-usb/hdm_usb.c | 16

[PATCHv6 5/5] Staging: most: aim-cdev/cdev.c. Fix missing static keyword warnings

This is a patch to the most/aim_cdev.c file. It makes several local functions and structures static to prevent global visibility. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/aim-cdev/cdev.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions

[PATCHv6 0/5] Staging: most: several warnings fix reported by sparse

Kroah-Hartman v5: Because of using a non-updated tree previously two patches did not apply v6: Clear warnings and styling issues Adrian Remonda (5): Staging: most: mostcore/core.c. Fix missing static keyword warnings Staging: most: mostcore/core.c. Fix Using plain integer

[PATCHv7 0/5] Staging: most: several warnings fix reported by sparse

Kroah-Hartman v5: Because of using a non-updated tree previously two patches did not apply v6: Clear warnings and styling issues v7: Update patch format Adrian Remonda (5): Staging: most: mostcore/core.c. Fix missing static keyword warnings Staging: most

[PATCHv7 1/5] Staging: most: mostcore/core.c. Fix missing static keyword warnings

This is a patch to the mostcore/core.c file. It makes several local functions and structures static to prevent global visibility. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/mostcore/core.c | 15 --- 1 file changed, 8 insertions(+), 7 deletions

[PATCHv7 3/5] Staging: most: hdm-usb/hdm_usb.c. Fix missing static keyword warnings

This is a patch to the most/hdm-usb/hdm_usb.c file. It makes several local functions and structures static to prevent global visibility. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/hdm-usb/hdm_usb.c | 16 1 file changed, 8 insertions(+), 8

[PATCHv7 2/5] Staging: most: mostcore/core.c. Fix Using plain integer as NULL pointer warnings

This patch fixes the warning generated by sparse: Using plain integer as NULL pointer by replacing the pointer test against 0 with a logical test. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/mostcore/core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[PATCHv7 5/5] Staging: most: aim-cdev/cdev.c. Fix missing static keyword warnings

This is a patch to the most/aim_cdev.c file. It makes several local functions and structures static to prevent global visibility. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/aim-cdev/cdev.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions

[PATCHv7 4/5] Staging: most: hdm-dim2/dim2_hal.c. Fix Using plain integer as NULL pointer warnings

This patch fixes the warning generated by sparse: Using plain integer as NULL pointer by replacing the offending 0 with NULL. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/hdm-dim2/dim2_hal.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCHv6 2/5] Staging: most: mostcore/core.c. Fix Using plain integer as NULL pointer warnings

This patch fixes the warning generated by sparse: Using plain integer as NULL pointer by replacing the offending 0 with NULL. v6: styling. Change the NULL test to comply with the Kernel coding style Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/mostcore/core.c

[PATCHv6 1/5] Staging: most: mostcore/core.c. Fix missing static keyword warnings

This is a patch to the mostcore/core.c file. It makes several local functions and structures static to prevent global visibility. v6: fixed warning line over 80 characters as pointed out by Sudip Mukherjee Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/mostcore

[PATCHv6 3/5] Staging: most: hdm-usb/hdm_usb.c. Fix missing static keyword warnings

This is a patch to the most/hdm-usb/hdm_usb.c file. It makes several local functions and structures static to prevent global visibility. v6: fixed alignment with parameter list of a function. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/hdm-usb/hdm_usb.c | 16

[PATCHv6 0/5] Staging: most: several warnings fix reported by sparse

Kroah-Hartman v5: Because of using a non-updated tree previously two patches did not apply v6: Clear warnings and styling issues Adrian Remonda (5): Staging: most: mostcore/core.c. Fix missing static keyword warnings Staging: most: mostcore/core.c. Fix Using plain integer

[PATCHv6 4/5] Staging: most: hdm-dim2/dim2_hal.c. Fix Using plain integer as NULL pointer warnings

This patch fixes the warning generated by sparse: Using plain integer as NULL pointer by replacing the offending 0 with NULL. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/hdm-dim2/dim2_hal.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCHv6 5/5] Staging: most: aim-cdev/cdev.c. Fix missing static keyword warnings

This is a patch to the most/aim_cdev.c file. It makes several local functions and structures static to prevent global visibility. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/aim-cdev/cdev.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions

[PATCHv5 2/5] Staging: most: mostcore/core.c. Fix Using plain integer as NULL pointer warnings

This patch fixes the warning generated by sparse: Using plain integer as NULL pointer by replacing the offending 0 with NULL. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/mostcore/core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers

[PATCHv5 5/5] Staging: most: aim-cdev/cdev.c. Fix missing static keyword warnings

This is a patch to the most/aim_cdev.c file. It makes several local functions and structures static to prevent global visibility. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/aim-cdev/cdev.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions

[PATCHv5 4/5] Staging: most: hdm-dim2/dim2_hal.c. Fix Using plain integer as NULL pointer warnings

This patch fixes the warning generated by sparse: Using plain integer as NULL pointer by replacing the offending 0 with NULL. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/hdm-dim2/dim2_hal.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCHv5 1/5] Staging: most: mostcore/core.c. Fix missing static keyword warnings

This is a patch to the mostcore/core.c file. It makes several local functions and structures static to prevent global visibility. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/mostcore/core.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions

[PATCHv5 0/5] Staging: most: several warnings fix reported by sparse

Kroah-Hartman v5: Because of using a non-updated tree previously two patches did not apply Adrian Remonda (5): Staging: most: mostcore/core.c. Fix missing static keyword warnings Staging: most: mostcore/core.c. Fix Using plain integer as NULL pointer warnings Staging: most

[PATCHv5 3/5] Staging: most: hdm-usb/hdm_usb.c. Fix missing static keyword warnings

This is a patch to the most/hdm-usb/hdm_usb.c file. It makes several local functions and structures static to prevent global visibility. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/hdm-usb/hdm_usb.c | 16 1 file changed, 8 insertions(+), 8

[PATCHv4 0/6] Staging: most: several warnings fix reported by sparse

Kroah-Hartman Adrian Remonda (6): Staging: most: mostcore/core.c. Fix missing static keyword warnings Staging: most: mostcore/core.c. Fix Using plain integer as NULL pointer warnings Staging: most: hdm-usb/hdm_usb.c. Fix missing static keyword warnings Staging: most: hdm-dim2

[PATCHv4 5/6] Staging: most: aim-cdev/cdev.c. Fix missing static keyword warnings

This is a patch to the most/aim_cdev.c file. It makes several local functions and structures static to prevent global visibility. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/aim-cdev/cdev.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions

[PATCHv4 3/6] Staging: most: hdm-usb/hdm_usb.c. Fix missing static keyword warnings

This is a patch to the most/hdm-usb/hdm_usb.c file. It makes several local functions and structures static to prevent global visibility. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/hdm-usb/hdm_usb.c | 16 1 file changed, 8 insertions(+), 8

[PATCHv4 2/6] Staging: most: mostcore/core.c. Fix Using plain integer as NULL pointer warnings

This patch fixes the warning generated by sparse: Using plain integer as NULL pointer by replacing the offending 0 with NULL. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/mostcore/core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers

[PATCHv4 6/6] Staging: most: aim-network/networking.c. Fix Using plain integer as NULL pointer warnings

This patch fixes the warning generated by sparse: Using plain integer as NULL pointer by replacing the offending 0 with NULL. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/aim-network/networking.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff

[PATCHv4 4/6] Staging: most: hdm-dim2/dim2_hal.c. Fix Using plain integer as NULL pointer warnings

This patch fixes the warning generated by sparse: Using plain integer as NULL pointer by replacing the offending 0 with NULL. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/hdm-dim2/dim2_hal.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCHv4 1/6] Staging: most: mostcore/core.c. Fix missing static keyword warnings

This is a patch to the mostcore/core.c file. It makes several local functions and structures static to prevent global visibility. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/mostcore/core.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions

[PATCHv3 0/6] Staging: most: several warnings fix reported by sparse

This patch series fix several warnings reported by the Sparse tool v2: Fixed patch format and comments as noted by Greg Kroah-Hartman and clear a few more warnings v3: Fixed patch format as noted by Greg Kroah-Hartman Adrian Remonda (6): Staging: most: Fix

[PATCHv3 5/6] Staging: most: Fix missing static keyword warnings

This is a patch to the most/aim_cdev.c file. It makes several local functions and structures static to prevent global visibility. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/aim-cdev/cdev.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions

[PATCHv3 6/6] Staging: most: Fix Using plain integer as NULL pointer warnings

This patch fixes the warning generated by sparse: Using plain integer as NULL pointer by replacing the offending 0 with NULL. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/aim-network/networking.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff

[PATCHv3 4/6] Staging: most: Fix Using plain integer as NULL pointer warnings

This patch fixes the warning generated by sparse: Using plain integer as NULL pointer by replacing the offending 0 with NULL. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/hdm-dim2/dim2_hal.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCHv3 2/6] Staging: most: Fix Using plain integer as NULL pointer warnings

This patch fixes the warning generated by sparse: Using plain integer as NULL pointer by replacing the offending 0 with NULL. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/mostcore/core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCHv3 3/6] Staging: most: Fix missing static keyword warnings

This is a patch to the most/hdm-usb/hdm_usb.c file. It makes several local functions and structures static to prevent global visibility. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/hdm-usb/hdm_usb.c | 14 +++--- 1 file changed, 7 insertions(+), 7

[PATCH 4/4] Plain integer instead of NULL pointer fix

This patch fix the using plain integer as NULL pointer Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/hdm-dim2/dim2_hal.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/most/hdm-dim2/dim2_hal.c b/drivers/staging/most/hdm-dim2

[PATCH 3/4] Staging: most: static keyword missing

This is a patch to the most/hdm-usb/hdm_usb.c file. It makes several local functions static to prevent global visibility. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/hdm-usb/hdm_usb.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff

[PATCH 0/4] Staging: most: several warnings fix reported by sparse

This patch series fix several warnings reported by the Sparse tool Adrian Remonda (4): This is a patch to the mostcore/core.c file. It makes several local functions static to prevent global visibility. This patch fix the using plain integer as NULL pointer This is a patch

[PATCH 2/4] Plain integer instead of NULL pointer fix

This patch fix the using plain integer as NULL pointer Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/mostcore/core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/most/mostcore/core.c b/drivers/staging/most/mostcore/core.c

[PATCH 1/4] Staging: most: static keyword missing

This is a patch to the mostcore/core.c file. It makes several local functions static to prevent global visibility. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/mostcore/core.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git

[PATCHv2 5/6] Staging: most: static keyword missing

This is a patch to the most/aim_cdev.c file. It makes several local functions and structures static to prevent global visibility. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/aim-cdev/cdev.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions

[PATCHv2 6/6] Staging: most: Fix sparse warning Using plain integer as NULL pointer

This patch fixes the warning generated by sparse: Using plain integer as NULL pointer by replacing the offending 0 with NULL. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/aim-network/networking.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff

[PATCHv2 1/6] Staging: most: static keyword missing

This is a patch to the mostcore/core.c file. It makes several local functions and structures static to prevent global visibility. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/mostcore/core.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions

[PATCHv2 2/6] Staging: most: Fix sparse warning Using plain integer as NULL pointer

This patch fixes the warning generated by sparse: Using plain integer as NULL pointer by replacing the offending 0 with NULL. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/mostcore/core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCHv2 3/6] Staging: most: static keyword missing

This is a patch to the most/hdm-usb/hdm_usb.c file. It makes several local functions and structures static to prevent global visibility. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/hdm-usb/hdm_usb.c | 14 +++--- 1 file changed, 7 insertions(+), 7

[PATCHv2 0/6] Staging: most: several warnings fix reported by sparse

This patch series fix several warnings reported by the Sparse tool Adrian Remonda (6): Staging: most: static keyword missing Staging: most: Fix sparse warning Using plain integer as NULL pointer Staging: most: static keyword missing Staging: most: Fix sparse warning Using plain

[PATCHv2 4/6] Staging: most: Fix sparse warning Using plain integer as NULL pointer

This patch fixes the warning generated by sparse: Using plain integer as NULL pointer by replacing the offending 0 with NULL. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/hdm-dim2/dim2_hal.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 2/4] Staging: lustre: sparse static warning fix

/lustre/ptlrpc/ptlrpc_internal.h Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/lustre/lustre/ptlrpc/lproc_ptlrpc.c| 6 -- drivers/staging/lustre/lustre/ptlrpc/ptlrpc_internal.h | 7 +++ 2 files changed, 7 insertions(+), 6 deletions(-) diff --git a/drivers

[PATCH 1/4] Staging: lustre: sparse static warning fix

/ptlrpc/ptlrpc_internal.h Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/lustre/lustre/ptlrpc/nrs.c | 4 drivers/staging/lustre/lustre/ptlrpc/ptlrpc_internal.h | 4 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/lustre

[PATCH 4/4] Staging: lustre: sparse lock warning fix

Fixed sparse warning: context imbalance in 'nrs_resource_put_safe' - 'different lock contexts for basic block' by releasing the lock on each iteration of the for loop. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/lustre/lustre/ptlrpc/nrs.c | 6 ++ 1 file changed

[PATCH 3/4] Staging: lustre: Fixed typo

In the explanation of the function the name of the function was incorrect Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/lustre/lustre/ptlrpc/nrs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/ptlrpc/nrs.c b/drivers

[PATCH] Staging: comedi: Removed expression that has no effect

This patch remove a register from an '|' expression. It is wrong since after in the code, the variable ai_trig gets written back to that register. modified: drivers/staging/comedi/drivers/ni_mio_common.c Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging

[PATCH] Staging: lustre: Added missing __user keyword to several struct fields

modified: drivers/staging/lustre/lnet/selftest/console.c modified: drivers/staging/lustre/lnet/selftest/console.h modified: drivers/staging/lustre/include/linux/libcfs/libcfs_ioctl.h modified: drivers/staging/lustre/lnet/selftest/console.h Signed-off-by: Adrian

Question regarding sparse warning in staging/lustre

Hello, I'm cleaning the drivers/staging/lustre driver. I have got the next warning from sparse: drivers/staging/lustre/lnet/selftest//conctl.c:918:30: warning: incorrect type in argument 1 (different address spaces) drivers/staging/lustre/lnet/selftest//conctl.c:918:30:expected void

[PATCHv3 1/2] Staging: rtl8188eu: Lines over 80 characters fixed.

This is a patch to the hal/rtl8188eu_recv.c file that fixes up a line over 80 characters warning found by the checkpatch.pl tool. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/rtl8188eu/hal/rtl8188eu_recv.c | 28 +- 1 file changed, 18

[PATCHv3 2/2] Staging: rtl8188eu: Removed unneeded code.

the allocation. Suggested-by: Larry Finger larry.fin...@lwfinger.net Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/rtl8188eu/hal/rtl8188eu_recv.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/drivers/staging/rtl8188eu/hal/rtl8188eu_recv.c b/drivers

[PATCHv2 1/2] Staging: rtl8188eu: Lines over 80 characters fixed.

This is a patch to the hal/rtl8188eu_recv.c file that fixes up a line over 80 characters warning found by the checkpatch.pl tool. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/rtl8188eu/hal/rtl8188eu_recv.c | 28 +- 1 file changed, 18

[PATCH] Staging: rtl8188eu: Missing a blank line after declarations.

This is a patch to the rtl8188e_xmit.c file that fixes up a missing blank line warning found by the checkpatch.pl tool. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/rtl8188eu/hal/rtl8188e_xmit.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging

[PATCH] Staging: rtl8188eu: Lines over 80 characters fixed.

This is a patch to the hal/rtl8188eu_recv.c file that fixes up a line over 80 characters warning found by the checkpatch.pl tool. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/rtl8188eu/hal/rtl8188eu_recv.c | 28 +- 1 file changed, 18