Category: Linux

ARM: msm: dma: use list_move_tail instead of list_del/list_add_tail

Author: Wei Yongjun <yongjun_wei@trendmicro.com.cn> Using list_move_tail() instead of list_del() + list_add_tail(). spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun Signed-off-by: David Brown — arch/arm/mach-msm/dma.c | 3 +– 1 file changed, 1 insertion(+), 2 deletions(-)   diff –git a/arch/arm/mach-msm/dma.c b/arch/arm/mach-msm/dma.c index 02cae5e..354b91d 100644 — a/arch/arm/mach-msm/dma.c +++ b/arch/arm/mach-msm/dma.c @@ -223,8 …

Continue reading

staging: gdm72xx: use list_move_tail instead of list_del/list_add_tail

Author: Wei Yongjun <yongjun_wei@trendmicro.com.cn> Using list_move_tail() instead of list_del() + list_add_tail(). spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun Signed-off-by: Greg Kroah-Hartman — drivers/staging/gdm72xx/gdm_usb.c | 3 +– 1 file changed, 1 insertion(+), 2 deletions(-)   diff –git a/drivers/staging/gdm72xx/gdm_usb.c b/drivers/staging/gdm72xx/gdm_usb.c index c7a22fa..932a797 100644 — a/drivers/staging/gdm72xx/gdm_usb.c +++ b/drivers/staging/gdm72xx/gdm_usb.c @@ -178,8 …

Continue reading

staging: ozwpan: use list_move_tail instead of list_del/list_add_tail

Author: Wei Yongjun <yongjun_wei@trendmicro.com.cn> Using list_move_tail() instead of list_del() + list_add_tail(). spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun Signed-off-by: Greg Kroah-Hartman — drivers/staging/ozwpan/ozhcd.c | 15 +++++———- 1 file changed, 5 insertions(+), 10 deletions(-)   diff –git a/drivers/staging/ozwpan/ozhcd.c b/drivers/staging/ozwpan/ozhcd.c index 76821cb..2e087ac 100644 — a/drivers/staging/ozwpan/ozhcd.c +++ b/drivers/staging/ozwpan/ozhcd.c @@ -1068,8 …

Continue reading

target: use list_move_tail instead of list_del/list_add_tail

Author: Wei Yongjun <yongjun_wei@trendmicro.com.cn> Using list_move_tail() instead of list_del() + list_add_tail(). spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun Signed-off-by: Nicholas Bellinger — drivers/target/sbp/sbp_target.c | 3 +– 1 file changed, 1 insertion(+), 2 deletions(-)   diff –git a/drivers/target/sbp/sbp_target.c b/drivers/target/sbp/sbp_target.c index 4d97977..0ce44cd 100644 — a/drivers/target/sbp/sbp_target.c +++ b/drivers/target/sbp/sbp_target.c @@ -660,8 …

Continue reading

USB: FHCI: use list_move_tail instead of list_del/list_add_tail

Author: Wei Yongjun <yongjun_wei@trendmicro.com.cn> Using list_move_tail() instead of list_del() + list_add_tail(). spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun Signed-off-by: Greg Kroah-Hartman — drivers/usb/host/fhci-sched.c | 3 +– 1 file changed, 1 insertion(+), 2 deletions(-)   diff –git a/drivers/usb/host/fhci-sched.c b/drivers/usb/host/fhci-sched.c index 2dc8a40..8f18538 100644 — a/drivers/usb/host/fhci-sched.c +++ b/drivers/usb/host/fhci-sched.c @@ -261,8 …

Continue reading

vmlfb: use list_move_tail instead of list_del/list_add_tail

Author: Wei Yongjun <yongjun_wei@trendmicro.com.cn> Using list_move_tail() instead of list_del() + list_add_tail(). spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun Signed-off-by: Florian Tobias Schandinat — drivers/video/vermilion/vermilion.c | 3 +– 1 file changed, 1 insertion(+), 2 deletions(-)   diff –git a/drivers/video/vermilion/vermilion.c b/drivers/video/vermilion/vermilion.c index 970e43d..083f0c8 100644 — a/drivers/video/vermilion/vermilion.c +++ b/drivers/video/vermilion/vermilion.c @@ …

Continue reading

pnfsblock: use list_move_tail instead of list_del/list_add_tail

Author: Wei Yongjun <yongjun_wei@trendmicro.com.cn> Using list_move_tail() instead of list_del() + list_add_tail(). spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun Signed-off-by: Trond Myklebust — fs/nfs/blocklayout/extents.c | 3 +– 1 file changed, 1 insertion(+), 2 deletions(-)   diff –git a/fs/nfs/blocklayout/extents.c b/fs/nfs/blocklayout/extents.c index 1f9a603..9c3e117 100644 — a/fs/nfs/blocklayout/extents.c +++ b/fs/nfs/blocklayout/extents.c @@ -683,8 …

Continue reading

bnx2x: use list_move_tail instead of list_del/list_add_tail

Author: Wei Yongjun <yongjun_wei@trendmicro.com.cn> Using list_move_tail() instead of list_del() + list_add_tail(). spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun Signed-off-by: David S. Miller — drivers/net/ethernet/broadcom/bnx2x/bnx2x_sp.c | 3 +– 1 file changed, 1 insertion(+), 2 deletions(-)   diff –git a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_sp.c b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_sp.c index 62f754b..5a5fbf5 100644 — a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_sp.c +++ b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_sp.c @@ …

Continue reading

[media] omap3isp: Fix error return code in probe function

Author: Peter Senna Tschudin <peter.senna@gmail.com> Convert a nonnegative error return code to a negative one, as returned elsewhere in the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // ( if@p1 (\(ret < 0\|ret != 0\)) { ... return ret; } | ret@p1 = 0 ) ... …

Continue reading

[SCSI] zfcp: remove invalid reference to list iterator variable

Author: Julia Lawall <Julia.Lawall@lip6.fr> If list_for_each_entry, etc complete a traversal of the list, the iterator variable ends up pointing to an address at an offset from the list head, and not a meaningful structure. Thus this value should not be used after the end of the iterator. Replace port->adapter->scsi_host by adapter->scsi_host. This problem was found …

Continue reading