Staging: bcm: Change the style of comparing structures to null in InterfaceMisc.c. (de3615f4) · Commits · 戴 / test

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

+3 −3

Original line number	Diff line number	Diff line
		@@ -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);
		}

Admin message