Staging: bcm: Change the style of comparing structures to null in InterfaceMisc.c.
authorKevin McKinney <klmckinney1@gmail.com>
Sat, 13 Oct 2012 03:49:33 +0000 (23:49 -0400)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 19 Oct 2012 20:42:23 +0000 (13:42 -0700)
This patch changes the style of comparing
structures to null. Instead of this:
"if (foo == NULL) {" or "if (foo != NULL) {",
the new logic uses: "if (!foo) {" or
"if (foo) {".

Signed-off-by: Kevin McKinney <klmckinney1@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/bcm/InterfaceMisc.c

index 5a4768ddd1ca5ffb1cca8447f7af3b2cbe58465f..f7166f1be5fcbf625ce53216ce5581c66c284b9e 100644 (file)
@@ -8,7 +8,7 @@ int InterfaceRDM(PS_INTERFACE_ADAPTER psIntfAdapter,
        int bytes;
        unsigned short usRetries = 0;
 
-       if (psIntfAdapter == NULL) {
+       if (!psIntfAdapter) {
                BCM_DEBUG_PRINT(psIntfAdapter->psAdapter, DBG_TYPE_PRINTK, 0, 0, "Interface Adapter is NULL");
                return -EINVAL;
        }
@@ -65,7 +65,7 @@ int InterfaceWRM(PS_INTERFACE_ADAPTER psIntfAdapter,
        int retval = 0;
        unsigned short usRetries = 0;
 
-       if (psIntfAdapter == NULL) {
+       if (!psIntfAdapter) {
                BCM_DEBUG_PRINT(psIntfAdapter->psAdapter, DBG_TYPE_PRINTK, 0, 0, "Interface Adapter  is NULL");
                return -EINVAL;
        }
@@ -193,7 +193,7 @@ VOID Bcm_kill_all_URBs(PS_INTERFACE_ADAPTER psIntfAdapter)
         */
 
        /* Cancel submitted Interrupt-URB's */
-       if (psIntfAdapter->psInterruptUrb != NULL) {
+       if (psIntfAdapter->psInterruptUrb) {
                if (psIntfAdapter->psInterruptUrb->status == -EINPROGRESS)
                        usb_kill_urb(psIntfAdapter->psInterruptUrb);
        }