Commit e3c5530b authored Apr 09, 2012 by Jesper Juhl Committed by Jiri Kosina Apr 30, 2012

tg3: remove redundant NULL test before release_firmware() call



There is no need to test for a NULL pointer before calling
release_firmware - the function does that on its own.

Signed-off-by: Jesper Juhl <jj@chaosbits.net>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>

parent b2cbf2e3

drivers/net/ethernet/broadcom/tg3.c

+1 −2

Original line number	Original line	Diff line number	Diff line
	@@ -15842,7 +15842,6 @@ static void __devexit tg3_remove_one(struct pci_dev *pdev)
	if (dev) {		if (dev) {
	struct tg3 *tp = netdev_priv(dev);		struct tg3 *tp = netdev_priv(dev);

	if (tp->fw)
	release_firmware(tp->fw);		release_firmware(tp->fw);

	tg3_reset_task_cancel(tp);		tg3_reset_task_cancel(tp);