5 files changed, 23 insertions, 25 deletions

diff --git a/drivers/net/wan/c101.c b/drivers/net/wan/c101.c
index 43d854ace233..b60ef02db7b0 100644
--- a/drivers/net/wan/c101.c
+++ b/drivers/net/wan/c101.c
@@ -326,21 +326,21 @@ static int __init c101_run(unsigned long irq, unsigned long winbase)
         if (request_irq(irq, sca_intr, 0, devname, card)) {
                 printk(KERN_ERR "c101: could not allocate IRQ\n");
                 c101_destroy_card(card);
-                return(-EBUSY);
+                return -EBUSY;
         }
         card->irq = irq;
 
         if (!request_mem_region(winbase, C101_MAPPED_RAM_SIZE, devname)) {
                 printk(KERN_ERR "c101: could not request RAM window\n");
                 c101_destroy_card(card);
-                return(-EBUSY);
+                return -EBUSY;
         }
         card->phy_winbase = winbase;
         card->win0base = ioremap(winbase, C101_MAPPED_RAM_SIZE);
         if (!card->win0base) {
                 printk(KERN_ERR "c101: could not map I/O address\n");
                 c101_destroy_card(card);
-                return -EBUSY;
+                return -EFAULT;
         }
 
         card->tx_ring_buffers = TX_RING_BUFFERS;
diff --git a/drivers/net/wan/hdlc_generic.c b/drivers/net/wan/hdlc_generic.c
index 46cef8f92133..57f9538b8fb5 100644
--- a/drivers/net/wan/hdlc_generic.c
+++ b/drivers/net/wan/hdlc_generic.c
@@ -259,7 +259,7 @@ int hdlc_ioctl(struct net_device *dev, struct ifreq *ifr, int cmd)
         }
 }
 
-static void hdlc_setup(struct net_device *dev)
+void hdlc_setup(struct net_device *dev)
 {
         hdlc_device *hdlc = dev_to_hdlc(dev);
 
@@ -288,26 +288,6 @@ struct net_device *alloc_hdlcdev(void *priv)
         return dev;
 }
 
-int register_hdlc_device(struct net_device *dev)
-{
-        int result = dev_alloc_name(dev, "hdlc%d");
-        if (result < 0)
-                return result;
-
-        result = register_netdev(dev);
-        if (result != 0)
-                return -EIO;
-
-#if 0
-        if (netif_carrier_ok(dev))
-                netif_carrier_off(dev); /* no carrier until DCD goes up */
-#endif
-
-        return 0;
-}
-
-
-
 void unregister_hdlc_device(struct net_device *dev)
 {
         rtnl_lock();
@@ -326,8 +306,8 @@ EXPORT_SYMBOL(hdlc_open);
 EXPORT_SYMBOL(hdlc_close);
 EXPORT_SYMBOL(hdlc_set_carrier);
 EXPORT_SYMBOL(hdlc_ioctl);
+EXPORT_SYMBOL(hdlc_setup);
 EXPORT_SYMBOL(alloc_hdlcdev);
-EXPORT_SYMBOL(register_hdlc_device);
 EXPORT_SYMBOL(unregister_hdlc_device);
 
 static struct packet_type hdlc_packet_type = {
diff --git a/drivers/net/wan/n2.c b/drivers/net/wan/n2.c
index cd32751b64eb..b7d88db89a5c 100644
--- a/drivers/net/wan/n2.c
+++ b/drivers/net/wan/n2.c
@@ -387,6 +387,11 @@ static int __init n2_run(unsigned long io, unsigned long irq,
         }
         card->phy_winbase = winbase;
         card->winbase = ioremap(winbase, USE_WINDOWSIZE);
+        if (!card->winbase) {
+                printk(KERN_ERR "n2: ioremap() failed\n");
+                n2_destroy_card(card);
+                return -EFAULT;
+        }
 
         outb(0, io + N2_PCR);
         outb(winbase >> 12, io + N2_BAR);
diff --git a/drivers/net/wan/pci200syn.c b/drivers/net/wan/pci200syn.c
index f485a97844cc..670e8bd2245c 100644
--- a/drivers/net/wan/pci200syn.c
+++ b/drivers/net/wan/pci200syn.c
@@ -354,6 +354,7 @@ static int __devinit pci200_pci_init_one(struct pci_dev *pdev,
             card->rambase == NULL) {
                 printk(KERN_ERR "pci200syn: ioremap() failed\n");
                 pci200_pci_remove_one(pdev);
+                return -EFAULT;
         }
 
         /* Reset PLX */
diff --git a/drivers/net/wan/wanxl.c b/drivers/net/wan/wanxl.c
index 29a756dd979b..437e0e938e38 100644
--- a/drivers/net/wan/wanxl.c
+++ b/drivers/net/wan/wanxl.c
@@ -634,7 +634,13 @@ static int __devinit wanxl_pci_init_one(struct pci_dev *pdev,
 
         /* set up PLX mapping */
         plx_phy = pci_resource_start(pdev, 0);
+
         card->plx = ioremap_nocache(plx_phy, 0x70);
+        if (!card->plx) {
+                printk(KERN_ERR "wanxl: ioremap() failed\n");
+                wanxl_pci_remove_one(pdev);
+                return -EFAULT;
+        }
 
 #if RESET_WHILE_LOADING
         wanxl_reset(card);
@@ -700,6 +706,12 @@ static int __devinit wanxl_pci_init_one(struct pci_dev *pdev,
         }
 
         mem = ioremap_nocache(mem_phy, PDM_OFFSET + sizeof(firmware));
+        if (!mem) {
+                printk(KERN_ERR "wanxl: ioremap() failed\n");
+                wanxl_pci_remove_one(pdev);
+                return -EFAULT;
+        }
+
         for (i = 0; i < sizeof(firmware); i += 4)
                 writel(htonl(*(u32*)(firmware + i)), mem + PDM_OFFSET + i);