USB: cypress_m8: Get rid of pointless NULL check
authorMike Isely <isely@pobox.com>
Mon, 11 Feb 2008 02:23:28 +0000 (20:23 -0600)
committerGreg Kroah-Hartman <gregkh@suse.de>
Fri, 25 Apr 2008 04:16:37 +0000 (21:16 -0700)
Remove a NULL check in cypress_m8; the check is useless in this
context because it is referenced earlier in the same code path thus
the kernel would be oops'ed before reaching this point anyway.  (And
it's really pointless here anyway; if this pointer somehow is NULL the
driver is going to have serious problems in many other places.)

Signed-off-by: Mike Isely <isely@pobox.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/usb/serial/cypress_m8.c

index f0c5d2a7ab94da9f25cd7ed6b311974106796219..bdeda09369511247c19fca11daddeb7bc82ab7af 100644 (file)
@@ -1399,13 +1399,11 @@ static void cypress_read_int_callback(struct urb *urb)
 
        spin_lock_irqsave(&priv->lock, flags);
        /* check to see if status has changed */
-       if (priv != NULL) {
-               if (priv->current_status != priv->prev_status) {
-                       priv->diff_status |= priv->current_status ^
-                               priv->prev_status;
-                       wake_up_interruptible(&priv->delta_msr_wait);
-                       priv->prev_status = priv->current_status;
-               }
+       if (priv->current_status != priv->prev_status) {
+               priv->diff_status |= priv->current_status ^
+                       priv->prev_status;
+               wake_up_interruptible(&priv->delta_msr_wait);
+               priv->prev_status = priv->current_status;
        }
        spin_unlock_irqrestore(&priv->lock, flags);