irda: Remove IRDA_<TYPE> logging macros
[pandora-kernel.git] / drivers / net / irda / vlsi_ir.c
index a2e5561..fd4dede 100644 (file)
@@ -429,8 +429,8 @@ static struct vlsi_ring *vlsi_alloc_ring(struct pci_dev *pdev, struct ring_descr
                if (rd->buf == NULL ||
                    !(busaddr = pci_map_single(pdev, rd->buf, len, dir))) {
                        if (rd->buf) {
-                               IRDA_ERROR("%s: failed to create PCI-MAP for %p",
-                                          __func__, rd->buf);
+                               net_err_ratelimited("%s: failed to create PCI-MAP for %p\n",
+                                                   __func__, rd->buf);
                                kfree(rd->buf);
                                rd->buf = NULL;
                        }
@@ -483,11 +483,8 @@ static int vlsi_create_hwif(vlsi_irda_dev_t *idev)
 
        ringarea = pci_zalloc_consistent(idev->pdev, HW_RING_AREA_SIZE,
                                         &idev->busaddr);
-       if (!ringarea) {
-               IRDA_ERROR("%s: insufficient memory for descriptor rings\n",
-                          __func__);
+       if (!ringarea)
                goto out;
-       }
 
        hwmap = (struct ring_descr_hw *)ringarea;
        idev->rx_ring = vlsi_alloc_ring(idev->pdev, hwmap, ringsize[1],
@@ -581,7 +578,7 @@ static int vlsi_process_rx(struct vlsi_ring *r, struct ring_descr *rd)
        }
 
        if (!rd->skb) {
-               IRDA_WARNING("%s: rx packet lost\n", __func__);
+               net_warn_ratelimited("%s: rx packet lost\n", __func__);
                ret |= VLSI_RX_DROP;
                goto done;
        }
@@ -610,8 +607,8 @@ static void vlsi_fill_rx(struct vlsi_ring *r)
 
        for (rd = ring_last(r); rd != NULL; rd = ring_put(r)) {
                if (rd_is_active(rd)) {
-                       IRDA_WARNING("%s: driver bug: rx descr race with hw\n",
-                                    __func__);
+                       net_warn_ratelimited("%s: driver bug: rx descr race with hw\n",
+                                            __func__);
                        vlsi_ring_debug(r);
                        break;
                }
@@ -670,7 +667,7 @@ static void vlsi_rx_interrupt(struct net_device *ndev)
 
        if (ring_first(r) == NULL) {
                /* we are in big trouble, if this should ever happen */
-               IRDA_ERROR("%s: rx ring exhausted!\n", __func__);
+               net_err_ratelimited("%s: rx ring exhausted!\n", __func__);
                vlsi_ring_debug(r);
        }
        else
@@ -783,8 +780,8 @@ static int vlsi_set_baud(vlsi_irda_dev_t *idev, unsigned iobase)
                config = IRCFG_SIR | IRCFG_SIRFILT  | IRCFG_RXANY;
                switch(baudrate) {
                        default:
-                               IRDA_WARNING("%s: undefined baudrate %d - fallback to 9600!\n",
-                                            __func__, baudrate);
+                               net_warn_ratelimited("%s: undefined baudrate %d - fallback to 9600!\n",
+                                                    __func__, baudrate);
                                baudrate = 9600;
                                /* fallthru */
                        case 2400:
@@ -825,14 +822,16 @@ static int vlsi_set_baud(vlsi_irda_dev_t *idev, unsigned iobase)
                config ^= IRENABLE_SIR_ON;
 
        if (config != (IRENABLE_PHYANDCLOCK|IRENABLE_ENRXST)) {
-               IRDA_WARNING("%s: failed to set %s mode!\n", __func__,
-                       (mode==IFF_SIR)?"SIR":((mode==IFF_MIR)?"MIR":"FIR"));
+               net_warn_ratelimited("%s: failed to set %s mode!\n",
+                                    __func__,
+                                    mode == IFF_SIR ? "SIR" :
+                                    mode == IFF_MIR ? "MIR" : "FIR");
                ret = -1;
        }
        else {
                if (inw(iobase+VLSI_PIO_PHYCTL) != nphyctl) {
-                       IRDA_WARNING("%s: failed to apply baudrate %d\n",
-                                    __func__, baudrate);
+                       net_warn_ratelimited("%s: failed to apply baudrate %d\n",
+                                            __func__, baudrate);
                        ret = -1;
                }
                else {
@@ -977,8 +976,8 @@ static netdev_tx_t vlsi_hard_start_xmit(struct sk_buff *skb,
                 */
 
                if (len >= r->len-5)
-                        IRDA_WARNING("%s: possible buffer overflow with SIR wrapping!\n",
-                                     __func__);
+                       net_warn_ratelimited("%s: possible buffer overflow with SIR wrapping!\n",
+                                            __func__);
        }
        else {
                /* hw deals with MIR/FIR mode wrapping */
@@ -1044,7 +1043,7 @@ static netdev_tx_t vlsi_hard_start_xmit(struct sk_buff *skb,
 drop_unlock:
        spin_unlock_irqrestore(&idev->lock, flags);
 drop:
-       IRDA_WARNING("%s: dropping packet - %s\n", __func__, msg);
+       net_warn_ratelimited("%s: dropping packet - %s\n", __func__, msg);
        dev_kfree_skb_any(skb);
        ndev->stats.tx_errors++;
        ndev->stats.tx_dropped++;
@@ -1183,8 +1182,8 @@ static int vlsi_start_clock(struct pci_dev *pdev)
                }
                if (count < 3) {
                        if (clksrc == 1) { /* explicitly asked for PLL hence bail out */
-                               IRDA_ERROR("%s: no PLL or failed to lock!\n",
-                                          __func__);
+                               net_err_ratelimited("%s: no PLL or failed to lock!\n",
+                                                   __func__);
                                clkctl = CLKCTL_CLKSTP;
                                pci_write_config_byte(pdev, VLSI_PCI_CLKCTL, clkctl);
                                return -1;
@@ -1265,7 +1264,7 @@ static int vlsi_init_chip(struct pci_dev *pdev)
        /* start the clock and clean the registers */
 
        if (vlsi_start_clock(pdev)) {
-               IRDA_ERROR("%s: no valid clock source\n", __func__);
+               net_err_ratelimited("%s: no valid clock source\n", __func__);
                return -1;
        }
        iobase = ndev->base_addr;
@@ -1389,8 +1388,8 @@ static void vlsi_tx_timeout(struct net_device *ndev)
                idev->new_baud = idev->baud;            /* keep current baudrate */
 
        if (vlsi_start_hw(idev))
-               IRDA_ERROR("%s: failed to restart hw - %s(%s) unusable!\n",
-                          __func__, pci_name(idev->pdev), ndev->name);
+               net_err_ratelimited("%s: failed to restart hw - %s(%s) unusable!\n",
+                                   __func__, pci_name(idev->pdev), ndev->name);
        else
                netif_start_queue(ndev);
 }
@@ -1434,8 +1433,8 @@ static int vlsi_ioctl(struct net_device *ndev, struct ifreq *rq, int cmd)
                        irq->ifr_receiving = (fifocnt!=0) ? 1 : 0;
                        break;
                default:
-                       IRDA_WARNING("%s: notsupp - cmd=%04x\n",
-                                    __func__, cmd);
+                       net_warn_ratelimited("%s: notsupp - cmd=%04x\n",
+                                            __func__, cmd);
                        ret = -EOPNOTSUPP;
        }       
        
@@ -1479,8 +1478,8 @@ static irqreturn_t vlsi_interrupt(int irq, void *dev_instance)
        spin_unlock_irqrestore(&idev->lock,flags);
 
        if (boguscount <= 0)
-               IRDA_MESSAGE("%s: too much work in interrupt!\n",
-                            __func__);
+               net_info_ratelimited("%s: too much work in interrupt!\n",
+                                    __func__);
        return IRQ_RETVAL(handled);
 }
 
@@ -1493,7 +1492,7 @@ static int vlsi_open(struct net_device *ndev)
        char    hwname[32];
 
        if (pci_request_regions(idev->pdev, drivername)) {
-               IRDA_WARNING("%s: io resource busy\n", __func__);
+               net_warn_ratelimited("%s: io resource busy\n", __func__);
                goto errout;
        }
        ndev->base_addr = pci_resource_start(idev->pdev,0);
@@ -1507,8 +1506,8 @@ static int vlsi_open(struct net_device *ndev)
 
        if (request_irq(ndev->irq, vlsi_interrupt, IRQF_SHARED,
                        drivername, ndev)) {
-               IRDA_WARNING("%s: couldn't get IRQ: %d\n",
-                            __func__, ndev->irq);
+               net_warn_ratelimited("%s: couldn't get IRQ: %d\n",
+                                    __func__, ndev->irq);
                goto errout_io;
        }
 
@@ -1529,7 +1528,8 @@ static int vlsi_open(struct net_device *ndev)
 
        netif_start_queue(ndev);
 
-       IRDA_MESSAGE("%s: device %s operational\n", __func__, ndev->name);
+       net_info_ratelimited("%s: device %s operational\n",
+                            __func__, ndev->name);
 
        return 0;
 
@@ -1563,7 +1563,7 @@ static int vlsi_close(struct net_device *ndev)
 
        pci_release_regions(idev->pdev);
 
-       IRDA_MESSAGE("%s: device %s stopped\n", __func__, ndev->name);
+       net_info_ratelimited("%s: device %s stopped\n", __func__, ndev->name);
 
        return 0;
 }
@@ -1590,7 +1590,8 @@ static int vlsi_irda_init(struct net_device *ndev)
 
        if (pci_set_dma_mask(pdev,DMA_MASK_USED_BY_HW) ||
            pci_set_dma_mask(pdev,DMA_MASK_MSTRPAGE)) {
-               IRDA_ERROR("%s: aborting due to PCI BM-DMA address limitations\n", __func__);
+               net_err_ratelimited("%s: aborting due to PCI BM-DMA address limitations\n",
+                                   __func__);
                return -1;
        }
 
@@ -1632,19 +1633,19 @@ vlsi_irda_probe(struct pci_dev *pdev, const struct pci_device_id *id)
        else
                pdev->current_state = 0; /* hw must be running now */
 
-       IRDA_MESSAGE("%s: IrDA PCI controller %s detected\n",
-                    drivername, pci_name(pdev));
+       net_info_ratelimited("%s: IrDA PCI controller %s detected\n",
+                            drivername, pci_name(pdev));
 
        if ( !pci_resource_start(pdev,0) ||
             !(pci_resource_flags(pdev,0) & IORESOURCE_IO) ) {
-               IRDA_ERROR("%s: bar 0 invalid", __func__);
+               net_err_ratelimited("%s: bar 0 invalid", __func__);
                goto out_disable;
        }
 
        ndev = alloc_irdadev(sizeof(*idev));
        if (ndev==NULL) {
-               IRDA_ERROR("%s: Unable to allocate device memory.\n",
-                          __func__);
+               net_err_ratelimited("%s: Unable to allocate device memory.\n",
+                                   __func__);
                goto out_disable;
        }
 
@@ -1659,7 +1660,7 @@ vlsi_irda_probe(struct pci_dev *pdev, const struct pci_device_id *id)
                goto out_freedev;
 
        if (register_netdev(ndev) < 0) {
-               IRDA_ERROR("%s: register_netdev failed\n", __func__);
+               net_err_ratelimited("%s: register_netdev failed\n", __func__);
                goto out_freedev;
        }
 
@@ -1669,14 +1670,15 @@ vlsi_irda_probe(struct pci_dev *pdev, const struct pci_device_id *id)
                ent = proc_create_data(ndev->name, S_IFREG|S_IRUGO,
                                       vlsi_proc_root, VLSI_PROC_FOPS, ndev);
                if (!ent) {
-                       IRDA_WARNING("%s: failed to create proc entry\n",
-                                    __func__);
+                       net_warn_ratelimited("%s: failed to create proc entry\n",
+                                            __func__);
                } else {
                        proc_set_size(ent, 0);
                }
                idev->proc_entry = ent;
        }
-       IRDA_MESSAGE("%s: registered device %s\n", drivername, ndev->name);
+       net_info_ratelimited("%s: registered device %s\n",
+                            drivername, ndev->name);
 
        pci_set_drvdata(pdev, ndev);
        mutex_unlock(&idev->mtx);
@@ -1698,7 +1700,7 @@ static void vlsi_irda_remove(struct pci_dev *pdev)
        vlsi_irda_dev_t *idev;
 
        if (!ndev) {
-               IRDA_ERROR("%s: lost netdevice?\n", drivername);
+               net_err_ratelimited("%s: lost netdevice?\n", drivername);
                return;
        }
 
@@ -1714,7 +1716,7 @@ static void vlsi_irda_remove(struct pci_dev *pdev)
 
        free_netdev(ndev);
 
-       IRDA_MESSAGE("%s: %s removed\n", drivername, pci_name(pdev));
+       net_info_ratelimited("%s: %s removed\n", drivername, pci_name(pdev));
 }
 
 #ifdef CONFIG_PM
@@ -1733,8 +1735,8 @@ static int vlsi_irda_suspend(struct pci_dev *pdev, pm_message_t state)
        vlsi_irda_dev_t *idev;
 
        if (!ndev) {
-               IRDA_ERROR("%s - %s: no netdevice\n",
-                          __func__, pci_name(pdev));
+               net_err_ratelimited("%s - %s: no netdevice\n",
+                                   __func__, pci_name(pdev));
                return 0;
        }
        idev = netdev_priv(ndev);
@@ -1745,7 +1747,9 @@ static int vlsi_irda_suspend(struct pci_dev *pdev, pm_message_t state)
                        pdev->current_state = state.event;
                }
                else
-                       IRDA_ERROR("%s - %s: invalid suspend request %u -> %u\n", __func__, pci_name(pdev), pdev->current_state, state.event);
+                       net_err_ratelimited("%s - %s: invalid suspend request %u -> %u\n",
+                                           __func__, pci_name(pdev),
+                                           pdev->current_state, state.event);
                mutex_unlock(&idev->mtx);
                return 0;
        }
@@ -1772,16 +1776,16 @@ static int vlsi_irda_resume(struct pci_dev *pdev)
        vlsi_irda_dev_t *idev;
 
        if (!ndev) {
-               IRDA_ERROR("%s - %s: no netdevice\n",
-                          __func__, pci_name(pdev));
+               net_err_ratelimited("%s - %s: no netdevice\n",
+                                   __func__, pci_name(pdev));
                return 0;
        }
        idev = netdev_priv(ndev);
        mutex_lock(&idev->mtx);
        if (pdev->current_state == 0) {
                mutex_unlock(&idev->mtx);
-               IRDA_WARNING("%s - %s: already resumed\n",
-                            __func__, pci_name(pdev));
+               net_warn_ratelimited("%s - %s: already resumed\n",
+                                    __func__, pci_name(pdev));
                return 0;
        }
        
@@ -1800,7 +1804,7 @@ static int vlsi_irda_resume(struct pci_dev *pdev)
                 * now we explicitly set pdev->current_state = 0 after enabling the
                 * device and independently resume_ok should catch any garbage config.
                 */
-               IRDA_WARNING("%s - hm, nothing to resume?\n", __func__);
+               net_warn_ratelimited("%s - hm, nothing to resume?\n", __func__);
                mutex_unlock(&idev->mtx);
                return 0;
        }
@@ -1837,7 +1841,8 @@ static int __init vlsi_mod_init(void)
        int     i, ret;
 
        if (clksrc < 0  ||  clksrc > 3) {
-               IRDA_ERROR("%s: invalid clksrc=%d\n", drivername, clksrc);
+               net_err_ratelimited("%s: invalid clksrc=%d\n",
+                                   drivername, clksrc);
                return -1;
        }
 
@@ -1850,7 +1855,10 @@ static int __init vlsi_mod_init(void)
                        case 64:
                                break;
                        default:
-                               IRDA_WARNING("%s: invalid %s ringsize %d, using default=8", drivername, (i)?"rx":"tx", ringsize[i]);
+                               net_warn_ratelimited("%s: invalid %s ringsize %d, using default=8\n",
+                                                    drivername,
+                                                    i ? "rx" : "tx",
+                                                    ringsize[i]);
                                ringsize[i] = 8;
                                break;
                }