aboutsummaryrefslogtreecommitdiffstats
path: root/drivers
diff options
context:
space:
mode:
authorDan Carpenter <error27@gmail.com>2010-10-08 08:57:44 -0400
committerGreg Kroah-Hartman <gregkh@suse.de>2010-10-08 10:29:56 -0400
commit02339374cf3a28c30b5a137ab181eed358589dea (patch)
tree96f7342556c38a47013a3f6ed75dcb48233982d4 /drivers
parentef5d2056456d63528daa762b32afa5c430635f33 (diff)
Staging: bcm: remove unneeded NULL check
The error handling here is wrong. If psIntfAdapter were NULL then we would have a NULL dereference in the debug output on the error path. But this function is only called from usbbcm_device_probe() when psIntfAdapter is non-NULL. Since the check isn't needed and I removed it instead of fixing it. Signed-off-by: Dan Carpenter <error27@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers')
-rw-r--r--drivers/staging/bcm/InterfaceInit.c5
1 files changed, 0 insertions, 5 deletions
diff --git a/drivers/staging/bcm/InterfaceInit.c b/drivers/staging/bcm/InterfaceInit.c
index 81523b1e4f0..824f9a45007 100644
--- a/drivers/staging/bcm/InterfaceInit.c
+++ b/drivers/staging/bcm/InterfaceInit.c
@@ -529,11 +529,6 @@ INT InterfaceAdapterInit(PS_INTERFACE_ADAPTER psIntfAdapter)
529 BOOLEAN bBcm16 = FALSE; 529 BOOLEAN bBcm16 = FALSE;
530 UINT uiData = 0; 530 UINT uiData = 0;
531 531
532 if(psIntfAdapter == NULL)
533 {
534 BCM_DEBUG_PRINT(psIntfAdapter->psAdapter,DBG_TYPE_INITEXIT, DRV_ENTRY, DBG_LVL_ALL, "Interface Adapter is NULL");
535 return -EINVAL;
536 }
537 /* Store the usb dev into interface adapter */ 532 /* Store the usb dev into interface adapter */
538 psIntfAdapter->udev = usb_get_dev(interface_to_usbdev( 533 psIntfAdapter->udev = usb_get_dev(interface_to_usbdev(
539 psIntfAdapter->interface)); 534 psIntfAdapter->interface));