From: Dan Carpenter Date: Sun, 28 Jul 2013 20:27:38 +0000 (+0300) Subject: ALSA: usx2y: remove an unneeded check X-Git-Url: https://git.stricted.de/?a=commitdiff_plain;h=85054b2153f18eac16df9ff88913c98adea6a23e;p=GitHub%2Fexynos8895%2Fandroid_kernel_samsung_universal8895.git ALSA: usx2y: remove an unneeded check The test here is always true because S[i].urb is an array not a pointer. Also it's bogus because the intent was to test: if (S->urb[i]) { instead of: if (S[i].urb) { Anyway, usb_kill_urb() and usb_free_urb() accept NULL pointers so we can just remove this. Signed-off-by: Dan Carpenter Signed-off-by: Takashi Iwai --- diff --git a/sound/usb/usx2y/usbusx2y.c b/sound/usb/usx2y/usbusx2y.c index 1f9bbd55553f..5a51b18c50fe 100644 --- a/sound/usb/usx2y/usbusx2y.c +++ b/sound/usb/usx2y/usbusx2y.c @@ -305,11 +305,9 @@ static void usX2Y_unlinkSeq(struct snd_usX2Y_AsyncSeq *S) { int i; for (i = 0; i < URBS_AsyncSeq; ++i) { - if (S[i].urb) { - usb_kill_urb(S->urb[i]); - usb_free_urb(S->urb[i]); - S->urb[i] = NULL; - } + usb_kill_urb(S->urb[i]); + usb_free_urb(S->urb[i]); + S->urb[i] = NULL; } kfree(S->buffer); }