UPSTREAM: staging: gdm72xx: use list_move instead of list_del/list_add
authorWei Yongjun <yongjun_wei@trendmicro.com.cn>
Thu, 6 Sep 2012 04:36:49 +0000 (12:36 +0800)
committerGerrit <chrome-bot@google.com>
Fri, 7 Sep 2012 04:56:32 +0000 (21:56 -0700)
Using list_move() instead of list_del() + list_add().

spatch with a semantic match is used to found this problem.
(http://coccinelle.lip6.fr/)

Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
(cherry picked from commit 73295fe185e58b8c0b288fee61a8a994015ab4b2)

Change-Id: If2acdc5f7c17fdbb707b80e46e5e433733932454
Reviewed-on: https://gerrit.chromium.org/gerrit/32492
Reviewed-by: Olof Johansson <olofj@chromium.org>
Commit-Ready: Ben Chan <benchan@chromium.org>
Tested-by: Ben Chan <benchan@chromium.org>
drivers/staging/gdm72xx/gdm_usb.c

index 932a797..41e08a7 100644 (file)
@@ -186,8 +186,7 @@ static struct usb_rx *get_rx_struct(struct rx_cxt *rx)
 /* Before this function is called, spin lock should be locked. */
 static void put_rx_struct(struct rx_cxt *rx, struct usb_rx *r)
 {
-       list_del(&r->list);
-       list_add(&r->list, &rx->free_list);
+       list_move(&r->list, &rx->free_list);
 }
 
 static int init_usb(struct usbwm_dev *udev)