What is the purpose of setting this to NULL, other drivers don't do that too?+unprobe:upon failure, you fail to set drvdata back to NULL
+ for (j = i; j > 0; j--) {
+ struct gt96100_if_t *gtif = >ifs[j - 1];
+ gt96100_remove1(gtif);
+ }
+ kfree(gtifs);
There are GT96100_ETH{0,1}_BASEs instead of bars,+ struct gt96100_private *gp = netdev_priv(gtif->dev);shouldn't this be using pci_request_regions() / pci_release_regions() ?
+ unregister_netdev(gtif->dev);
+ dmafree(RX_HASH_TABLE_SIZE, gp->hash_table_dma);
+ dmafree(PKT_BUF_SZ*RX_RING_SIZE, gp->rx_buff);
+ dmafree(sizeof(gt96100_rd_t) * RX_RING_SIZE
+ + sizeof(gt96100_td_t) * TX_RING_SIZE,
+ gp->rx_ring);
+ free_netdev(gtif->dev);
+ release_region(gtif->iobase, gp->io_size);