Commit bb77bd31 authored by Zheyu Ma's avatar Zheyu Ma Committed by David S. Miller
Browse files

ethernet: sun: Free the coherent when failing in probing 



When the driver fails to register net device, it should free the DMA
region first, and then do other cleanup.

Signed-off-by: default avatarZheyu Ma <zheyuma97@gmail.com>
Reviewed-by: default avatarAndrew Lunn <andrew@lunn.ch>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent dd830aed

drivers/net/ethernet/sun/sunhme.c

+5 −1
Original line number Diff line number Diff line
@@ -3146,7 +3146,7 @@ static int happy_meal_pci_probe(struct pci_dev *pdev, 
	if (err) {

		printk(KERN_ERR "happymeal(PCI): Cannot register net device, "

		       "aborting.\n");

		goto err_out_iounmap;

		goto err_out_free_coherent;

	}



	pci_set_drvdata(pdev, hp);
@@ -3179,6 +3179,10 @@ static int happy_meal_pci_probe(struct pci_dev *pdev, 


	return 0;



err_out_free_coherent:

	dma_free_coherent(hp->dma_dev, PAGE_SIZE,

			  hp->happy_block, hp->hblock_dvma);



err_out_iounmap:

	iounmap(hp->gregs);