From: Kevin McKinney Date: Sat, 13 Oct 2012 03:49:33 +0000 (-0400) Subject: Staging: bcm: Change the style of comparing structures to null in InterfaceMisc.c. X-Git-Tag: firefly_0821_release~3680^2~1519^2~1190 X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=de3615f46cc5892a82f18324e62ec2c5c99e959f;p=firefly-linux-kernel-4.4.55.git Staging: bcm: Change the style of comparing structures to null in InterfaceMisc.c. 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 Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/staging/bcm/InterfaceMisc.c b/drivers/staging/bcm/InterfaceMisc.c index 5a4768ddd1ca..f7166f1be5fc 100644 --- a/drivers/staging/bcm/InterfaceMisc.c +++ b/drivers/staging/bcm/InterfaceMisc.c @@ -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); }