irda: Convert IRDA_DEBUG to pr_debug

Use the normal kernel debugging mechanism which also
enables dynamic_debug at the same time.

Other miscellanea:

o Remove sysctl for irda_debug
o Remove function tracing like uses (use ftrace instead)
o Coalesce formats
o Realign arguments
o Remove unnecessary OOM messages

Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

1 files changed, 66 insertions, 101 deletions

diff --git a/net/irda/irlmp.c b/net/irda/irlmp.c
index 6178e71f3a51..a26c401ef4a4 100644
--- a/net/irda/irlmp.c
+++ b/net/irda/irlmp.c
@@ -83,7 +83,6 @@ const char *irlmp_reason_str(LM_REASON reason)
  */
 int __init irlmp_init(void)
 {
-        IRDA_DEBUG(1, "%s()\n", __func__);
         /* Initialize the irlmp structure. */
         irlmp = kzalloc( sizeof(struct irlmp_cb), GFP_KERNEL);
         if (irlmp == NULL)
@@ -207,8 +206,6 @@ EXPORT_SYMBOL(irlmp_open_lsap);
  */
 static void __irlmp_close_lsap(struct lsap_cb *self)
 {
-        IRDA_DEBUG(4, "%s()\n", __func__);
-
         IRDA_ASSERT(self != NULL, return;);
         IRDA_ASSERT(self->magic == LMP_LSAP_MAGIC, return;);
 
@@ -267,9 +264,8 @@ void irlmp_close_lsap(struct lsap_cb *self)
                                       NULL);
         }
         if (!lsap) {
-                IRDA_DEBUG(0,
-                     "%s(), Looks like somebody has removed me already!\n",
-                           __func__);
+                pr_debug("%s(), Looks like somebody has removed me already!\n",
+                         __func__);
                 return;
         }
         __irlmp_close_lsap(self);
@@ -340,8 +336,6 @@ void irlmp_unregister_link(__u32 saddr)
 {
         struct lap_cb *link;
 
-        IRDA_DEBUG(4, "%s()\n", __func__);
-
         /* We must remove ourselves from the hashbin *first*. This ensure
          * that no more LSAPs will be open on this link and no discovery
          * will be triggered anymore. Jean II */
@@ -383,9 +377,8 @@ int irlmp_connect_request(struct lsap_cb *self, __u8 dlsap_sel,
         IRDA_ASSERT(self != NULL, return -EBADR;);
         IRDA_ASSERT(self->magic == LMP_LSAP_MAGIC, return -EBADR;);
 
-        IRDA_DEBUG(2,
-              "%s(), slsap_sel=%02x, dlsap_sel=%02x, saddr=%08x, daddr=%08x\n",
-              __func__, self->slsap_sel, dlsap_sel, saddr, daddr);
+        pr_debug("%s(), slsap_sel=%02x, dlsap_sel=%02x, saddr=%08x, daddr=%08x\n",
+                 __func__, self->slsap_sel, dlsap_sel, saddr, daddr);
 
         if (test_bit(0, &self->connected)) {
                 ret = -EISCONN;
@@ -429,7 +422,7 @@ int irlmp_connect_request(struct lsap_cb *self, __u8 dlsap_sel,
                 if (daddr != DEV_ADDR_ANY)
                         discovery = hashbin_find(irlmp->cachelog, daddr, NULL);
                 else {
-                        IRDA_DEBUG(2, "%s(), no daddr\n", __func__);
+                        pr_debug("%s(), no daddr\n", __func__);
                         discovery = (discovery_t *)
                                 hashbin_get_first(irlmp->cachelog);
                 }
@@ -442,7 +435,7 @@ int irlmp_connect_request(struct lsap_cb *self, __u8 dlsap_sel,
         }
         lap = hashbin_lock_find(irlmp->links, saddr, NULL);
         if (lap == NULL) {
-                IRDA_DEBUG(1, "%s(), Unable to find a usable link!\n", __func__);
+                pr_debug("%s(), Unable to find a usable link!\n", __func__);
                 ret = -EHOSTUNREACH;
                 goto err;
         }
@@ -457,14 +450,15 @@ int irlmp_connect_request(struct lsap_cb *self, __u8 dlsap_sel,
                          * disconnected yet (waiting for timeout in LAP).
                          * Maybe we could give LAP a bit of help in this case.
                          */
-                        IRDA_DEBUG(0, "%s(), sorry, but I'm waiting for LAP to timeout!\n", __func__);
+                        pr_debug("%s(), sorry, but I'm waiting for LAP to timeout!\n",
+                                 __func__);
                         ret = -EAGAIN;
                         goto err;
                 }
 
                 /* LAP is already connected to a different node, and LAP
                  * can only talk to one node at a time */
-                IRDA_DEBUG(0, "%s(), sorry, but link is busy!\n", __func__);
+                pr_debug("%s(), sorry, but link is busy!\n", __func__);
                 ret = -EBUSY;
                 goto err;
         }
@@ -525,8 +519,8 @@ void irlmp_connect_indication(struct lsap_cb *self, struct sk_buff *skb)
         IRDA_ASSERT(skb != NULL, return;);
         IRDA_ASSERT(self->lap != NULL, return;);
 
-        IRDA_DEBUG(2, "%s(), slsap_sel=%02x, dlsap_sel=%02x\n",
-                   __func__, self->slsap_sel, self->dlsap_sel);
+        pr_debug("%s(), slsap_sel=%02x, dlsap_sel=%02x\n",
+                 __func__, self->slsap_sel, self->dlsap_sel);
 
         /* Note : self->lap is set in irlmp_link_data_indication(),
          * (case CONNECT_CMD:) because we have no way to set it here.
@@ -566,8 +560,8 @@ int irlmp_connect_response(struct lsap_cb *self, struct sk_buff *userdata)
         /* We set the connected bit and move the lsap to the connected list
          * in the state machine itself. Jean II */
 
-        IRDA_DEBUG(2, "%s(), slsap_sel=%02x, dlsap_sel=%02x\n",
-                   __func__, self->slsap_sel, self->dlsap_sel);
+        pr_debug("%s(), slsap_sel=%02x, dlsap_sel=%02x\n",
+                 __func__, self->slsap_sel, self->dlsap_sel);
 
         /* Make room for MUX control header (3 bytes) */
         IRDA_ASSERT(skb_headroom(userdata) >= LMP_CONTROL_HEADER, return -1;);
@@ -593,8 +587,6 @@ void irlmp_connect_confirm(struct lsap_cb *self, struct sk_buff *skb)
         int lap_header_size;
         int max_seg_size;
 
-        IRDA_DEBUG(3, "%s()\n", __func__);
-
         IRDA_ASSERT(skb != NULL, return;);
         IRDA_ASSERT(self != NULL, return;);
         IRDA_ASSERT(self->magic == LMP_LSAP_MAGIC, return;);
@@ -606,8 +598,8 @@ void irlmp_connect_confirm(struct lsap_cb *self, struct sk_buff *skb)
         lap_header_size = IRLAP_GET_HEADER_SIZE(self->lap->irlap);
         max_header_size = LMP_HEADER + lap_header_size;
 
-        IRDA_DEBUG(2, "%s(), max_header_size=%d\n",
-                   __func__, max_header_size);
+        pr_debug("%s(), max_header_size=%d\n",
+                 __func__, max_header_size);
 
         /* Hide LMP_CONTROL_HEADER header from layer above */
         skb_pull(skb, LMP_CONTROL_HEADER);
@@ -633,16 +625,14 @@ struct lsap_cb *irlmp_dup(struct lsap_cb *orig, void *instance)
         struct lsap_cb *new;
         unsigned long flags;
 
-        IRDA_DEBUG(1, "%s()\n", __func__);
-
         spin_lock_irqsave(&irlmp->unconnected_lsaps->hb_spinlock, flags);
 
         /* Only allowed to duplicate unconnected LSAP's, and only LSAPs
          * that have received a connect indication. Jean II */
         if ((!hashbin_find(irlmp->unconnected_lsaps, (long) orig, NULL)) ||
             (orig->lap == NULL)) {
-                IRDA_DEBUG(0, "%s(), invalid LSAP (wrong state)\n",
-                           __func__);
+                pr_debug("%s(), invalid LSAP (wrong state)\n",
+                         __func__);
                 spin_unlock_irqrestore(&irlmp->unconnected_lsaps->hb_spinlock,
                                        flags);
                 return NULL;
@@ -651,7 +641,7 @@ struct lsap_cb *irlmp_dup(struct lsap_cb *orig, void *instance)
         /* Allocate a new instance */
         new = kmemdup(orig, sizeof(*new), GFP_ATOMIC);
         if (!new)  {
-                IRDA_DEBUG(0, "%s(), unable to kmalloc\n", __func__);
+                pr_debug("%s(), unable to kmalloc\n", __func__);
                 spin_unlock_irqrestore(&irlmp->unconnected_lsaps->hb_spinlock,
                                        flags);
                 return NULL;
@@ -697,7 +687,7 @@ int irlmp_disconnect_request(struct lsap_cb *self, struct sk_buff *userdata)
          * and us that might mess up the hashbins below. This fixes it.
          * Jean II */
         if (! test_and_clear_bit(0, &self->connected)) {
-                IRDA_DEBUG(0, "%s(), already disconnected!\n", __func__);
+                pr_debug("%s(), already disconnected!\n", __func__);
                 dev_kfree_skb(userdata);
                 return -1;
         }
@@ -751,20 +741,20 @@ void irlmp_disconnect_indication(struct lsap_cb *self, LM_REASON reason,
 {
         struct lsap_cb *lsap;
 
-        IRDA_DEBUG(1, "%s(), reason=%s [%d]\n", __func__,
-                   irlmp_reason_str(reason), reason);
+        pr_debug("%s(), reason=%s [%d]\n", __func__,
+                 irlmp_reason_str(reason), reason);
         IRDA_ASSERT(self != NULL, return;);
         IRDA_ASSERT(self->magic == LMP_LSAP_MAGIC, return;);
 
-        IRDA_DEBUG(3, "%s(), slsap_sel=%02x, dlsap_sel=%02x\n",
-                   __func__, self->slsap_sel, self->dlsap_sel);
+        pr_debug("%s(), slsap_sel=%02x, dlsap_sel=%02x\n",
+                 __func__, self->slsap_sel, self->dlsap_sel);
 
         /* Already disconnected ?
          * There is a race condition between irlmp_disconnect_request()
          * and us that might mess up the hashbins below. This fixes it.
          * Jean II */
         if (! test_and_clear_bit(0, &self->connected)) {
-                IRDA_DEBUG(0, "%s(), already disconnected!\n", __func__);
+                pr_debug("%s(), already disconnected!\n", __func__);
                 return;
         }
 
@@ -797,7 +787,7 @@ void irlmp_disconnect_indication(struct lsap_cb *self, LM_REASON reason,
                 self->notify.disconnect_indication(self->notify.instance,
                                                    self, reason, skb);
         } else {
-                IRDA_DEBUG(0, "%s(), no handler\n", __func__);
+                pr_debug("%s(), no handler\n", __func__);
         }
 }
 
@@ -968,8 +958,6 @@ irlmp_notify_client(irlmp_client_t *client,
         int     number;                 /* Number of nodes in the log */
         int     i;
 
-        IRDA_DEBUG(3, "%s()\n", __func__);
-
         /* Check if client wants or not partial/selective log (optimisation) */
         if (!client->disco_callback)
                 return;
@@ -1019,8 +1007,6 @@ void irlmp_discovery_confirm(hashbin_t *log, DISCOVERY_MODE mode)
         irlmp_client_t *client;
         irlmp_client_t *client_next;
 
-        IRDA_DEBUG(3, "%s()\n", __func__);
-
         IRDA_ASSERT(log != NULL, return;);
 
         if (!(HASHBIN_GET_SIZE(log)))
@@ -1054,8 +1040,6 @@ void irlmp_discovery_expiry(discinfo_t *expiries, int number)
         irlmp_client_t *client_next;
         int             i;
 
-        IRDA_DEBUG(3, "%s()\n", __func__);
-
         IRDA_ASSERT(expiries != NULL, return;);
 
         /* For each client - notify callback may touch client list */
@@ -1088,8 +1072,6 @@ void irlmp_discovery_expiry(discinfo_t *expiries, int number)
  */
 discovery_t *irlmp_get_discovery_response(void)
 {
-        IRDA_DEBUG(4, "%s()\n", __func__);
-
         IRDA_ASSERT(irlmp != NULL, return NULL;);
 
         put_unaligned(irlmp->hints.word, (__u16 *)irlmp->discovery_rsp.data.hints);
@@ -1166,8 +1148,6 @@ int irlmp_udata_request(struct lsap_cb *self, struct sk_buff *userdata)
 {
         int     ret;
 
-        IRDA_DEBUG(4, "%s()\n", __func__);
-
         IRDA_ASSERT(userdata != NULL, return -1;);
 
         /* Make room for MUX header */
@@ -1190,8 +1170,6 @@ int irlmp_udata_request(struct lsap_cb *self, struct sk_buff *userdata)
  */
 void irlmp_udata_indication(struct lsap_cb *self, struct sk_buff *skb)
 {
-        IRDA_DEBUG(4, "%s()\n", __func__);
-
         IRDA_ASSERT(self != NULL, return;);
         IRDA_ASSERT(self->magic == LMP_LSAP_MAGIC, return;);
         IRDA_ASSERT(skb != NULL, return;);
@@ -1217,8 +1195,6 @@ int irlmp_connless_data_request(struct lsap_cb *self, struct sk_buff *userdata,
         struct sk_buff *clone_skb;
         struct lap_cb *lap;
 
-        IRDA_DEBUG(4, "%s()\n", __func__);
-
         IRDA_ASSERT(userdata != NULL, return -1;);
 
         /* Make room for MUX and PID header */
@@ -1268,8 +1244,6 @@ int irlmp_connless_data_request(struct lsap_cb *self, struct sk_buff *userdata,
 #ifdef CONFIG_IRDA_ULTRA
 void irlmp_connless_data_indication(struct lsap_cb *self, struct sk_buff *skb)
 {
-        IRDA_DEBUG(4, "%s()\n", __func__);
-
         IRDA_ASSERT(self != NULL, return;);
         IRDA_ASSERT(self->magic == LMP_LSAP_MAGIC, return;);
         IRDA_ASSERT(skb != NULL, return;);
@@ -1311,7 +1285,7 @@ void irlmp_status_indication(struct lap_cb *self,
                         curr->notify.status_indication(curr->notify.instance,
                                                        link, lock);
                 else
-                        IRDA_DEBUG(2, "%s(), no handler\n", __func__);
+                        pr_debug("%s(), no handler\n", __func__);
 
                 curr = next;
         }
@@ -1339,7 +1313,7 @@ void irlmp_flow_indication(struct lap_cb *self, LOCAL_FLOW flow)
         /* Get the number of lsap. That's the only safe way to know
          * that we have looped around... - Jean II */
         lsap_todo = HASHBIN_GET_SIZE(self->lsaps);
-        IRDA_DEBUG(4, "%s() : %d lsaps to scan\n", __func__, lsap_todo);
+        pr_debug("%s() : %d lsaps to scan\n", __func__, lsap_todo);
 
         /* Poll lsap in order until the queue is full or until we
          * tried them all.
@@ -1358,14 +1332,16 @@ void irlmp_flow_indication(struct lap_cb *self, LOCAL_FLOW flow)
                 /* Uh-oh... Paranoia */
                 if(curr == NULL)
                         break;
-                IRDA_DEBUG(4, "%s() : curr is %p, next was %p and is now %p, still %d to go - queue len = %d\n", __func__, curr, next, self->flow_next, lsap_todo, IRLAP_GET_TX_QUEUE_LEN(self->irlap));
+                pr_debug("%s() : curr is %p, next was %p and is now %p, still %d to go - queue len = %d\n",
+                         __func__, curr, next, self->flow_next, lsap_todo,
+                         IRLAP_GET_TX_QUEUE_LEN(self->irlap));
 
                 /* Inform lsap user that it can send one more packet. */
                 if (curr->notify.flow_indication != NULL)
                         curr->notify.flow_indication(curr->notify.instance,
                                                      curr, flow);
                 else
-                        IRDA_DEBUG(1, "%s(), no handler\n", __func__);
+                        pr_debug("%s(), no handler\n", __func__);
         }
 }
 
@@ -1386,32 +1362,30 @@ __u8 *irlmp_hint_to_service(__u8 *hint)
          * since we currently only support 2 hint bytes
          */
         service = kmalloc(16, GFP_ATOMIC);
-        if (!service) {
-                IRDA_DEBUG(1, "%s(), Unable to kmalloc!\n", __func__);
+        if (!service)
                 return NULL;
-        }
 
         if (!hint[0]) {
-                IRDA_DEBUG(1, "<None>\n");
+                pr_debug("<None>\n");
                 kfree(service);
                 return NULL;
         }
         if (hint[0] & HINT_PNP)
-                IRDA_DEBUG(1, "PnP Compatible ");
+                pr_debug("PnP Compatible ");
         if (hint[0] & HINT_PDA)
-                IRDA_DEBUG(1, "PDA/Palmtop ");
+                pr_debug("PDA/Palmtop ");
         if (hint[0] & HINT_COMPUTER)
-                IRDA_DEBUG(1, "Computer ");
+                pr_debug("Computer ");
         if (hint[0] & HINT_PRINTER) {
-                IRDA_DEBUG(1, "Printer ");
+                pr_debug("Printer ");
                 service[i++] = S_PRINTER;
         }
         if (hint[0] & HINT_MODEM)
-                IRDA_DEBUG(1, "Modem ");
+                pr_debug("Modem ");
         if (hint[0] & HINT_FAX)
-                IRDA_DEBUG(1, "Fax ");
+                pr_debug("Fax ");
         if (hint[0] & HINT_LAN) {
-                IRDA_DEBUG(1, "LAN Access ");
+                pr_debug("LAN Access ");
                 service[i++] = S_LAN;
         }
         /*
@@ -1421,22 +1395,22 @@ __u8 *irlmp_hint_to_service(__u8 *hint)
          */
         if (hint[0] & HINT_EXTENSION) {
                 if (hint[1] & HINT_TELEPHONY) {
-                        IRDA_DEBUG(1, "Telephony ");
+                        pr_debug("Telephony ");
                         service[i++] = S_TELEPHONY;
                 }
                 if (hint[1] & HINT_FILE_SERVER)
-                        IRDA_DEBUG(1, "File Server ");
+                        pr_debug("File Server ");
 
                 if (hint[1] & HINT_COMM) {
-                        IRDA_DEBUG(1, "IrCOMM ");
+                        pr_debug("IrCOMM ");
                         service[i++] = S_COMM;
                 }
                 if (hint[1] & HINT_OBEX) {
-                        IRDA_DEBUG(1, "IrOBEX ");
+                        pr_debug("IrOBEX ");
                         service[i++] = S_OBEX;
                 }
         }
-        IRDA_DEBUG(1, "\n");
+        pr_debug("\n");
 
         /* So that client can be notified about any discovery */
         service[i++] = S_ANY;
@@ -1489,14 +1463,13 @@ void *irlmp_register_service(__u16 hints)
 {
         irlmp_service_t *service;
 
-        IRDA_DEBUG(4, "%s(), hints = %04x\n", __func__, hints);
+        pr_debug("%s(), hints = %04x\n", __func__, hints);
 
         /* Make a new registration */
         service = kmalloc(sizeof(irlmp_service_t), GFP_ATOMIC);
-        if (!service) {
-                IRDA_DEBUG(1, "%s(), Unable to kmalloc!\n", __func__);
+        if (!service)
                 return NULL;
-        }
+
         service->hints.word = hints;
         hashbin_insert(irlmp->services, (irda_queue_t *) service,
                        (long) service, NULL);
@@ -1519,15 +1492,13 @@ int irlmp_unregister_service(void *handle)
         irlmp_service_t *service;
         unsigned long flags;
 
-        IRDA_DEBUG(4, "%s()\n", __func__);
-
         if (!handle)
                 return -1;
 
         /* Caller may call with invalid handle (it's legal) - Jean II */
         service = hashbin_lock_find(irlmp->services, (long) handle, NULL);
         if (!service) {
-                IRDA_DEBUG(1, "%s(), Unknown service!\n", __func__);
+                pr_debug("%s(), Unknown service!\n", __func__);
                 return -1;
         }
 
@@ -1564,15 +1535,12 @@ void *irlmp_register_client(__u16 hint_mask, DISCOVERY_CALLBACK1 disco_clb,
 {
         irlmp_client_t *client;
 
-        IRDA_DEBUG(1, "%s()\n", __func__);
         IRDA_ASSERT(irlmp != NULL, return NULL;);
 
         /* Make a new registration */
         client = kmalloc(sizeof(irlmp_client_t), GFP_ATOMIC);
-        if (!client) {
-                IRDA_DEBUG( 1, "%s(), Unable to kmalloc!\n", __func__);
+        if (!client)
                 return NULL;
-        }
 
         /* Register the details */
         client->hint_mask.word = hint_mask;
@@ -1606,7 +1574,7 @@ int irlmp_update_client(void *handle, __u16 hint_mask,
 
         client = hashbin_lock_find(irlmp->clients, (long) handle, NULL);
         if (!client) {
-                IRDA_DEBUG(1, "%s(), Unknown client!\n", __func__);
+                pr_debug("%s(), Unknown client!\n", __func__);
                 return -1;
         }
 
@@ -1629,19 +1597,17 @@ int irlmp_unregister_client(void *handle)
 {
         struct irlmp_client *client;
 
-        IRDA_DEBUG(4, "%s()\n", __func__);
-
         if (!handle)
                 return -1;
 
         /* Caller may call with invalid handle (it's legal) - Jean II */
         client = hashbin_lock_find(irlmp->clients, (long) handle, NULL);
         if (!client) {
-                IRDA_DEBUG(1, "%s(), Unknown client!\n", __func__);
+                pr_debug("%s(), Unknown client!\n", __func__);
                 return -1;
         }
 
-        IRDA_DEBUG(4, "%s(), removing client!\n", __func__);
+        pr_debug("%s(), removing client!\n", __func__);
         hashbin_remove_this(irlmp->clients, (irda_queue_t *) client);
         kfree(client);
 
@@ -1670,8 +1636,6 @@ static int irlmp_slsap_inuse(__u8 slsap_sel)
         IRDA_ASSERT(irlmp->magic == LMP_MAGIC, return TRUE;);
         IRDA_ASSERT(slsap_sel != LSAP_ANY, return TRUE;);
 
-        IRDA_DEBUG(4, "%s()\n", __func__);
-
 #ifdef CONFIG_IRDA_ULTRA
         /* Accept all bindings to the connectionless LSAP */
         if (slsap_sel == LSAP_CONNLESS)
@@ -1705,8 +1669,8 @@ static int irlmp_slsap_inuse(__u8 slsap_sel)
                                     goto errlsap;);
 
                         if ((self->slsap_sel == slsap_sel)) {
-                                IRDA_DEBUG(4, "Source LSAP selector=%02x in use\n",
-                                           self->slsap_sel);
+                                pr_debug("Source LSAP selector=%02x in use\n",
+                                         self->slsap_sel);
                                 goto errlsap;
                         }
                         self = (struct lsap_cb*) hashbin_get_next(lap->lsaps);
@@ -1730,8 +1694,8 @@ static int irlmp_slsap_inuse(__u8 slsap_sel)
         while (self != NULL) {
                 IRDA_ASSERT(self->magic == LMP_LSAP_MAGIC, goto erruncon;);
                 if ((self->slsap_sel == slsap_sel)) {
-                        IRDA_DEBUG(4, "Source LSAP selector=%02x in use (unconnected)\n",
-                                   self->slsap_sel);
+                        pr_debug("Source LSAP selector=%02x in use (unconnected)\n",
+                                 self->slsap_sel);
                         goto erruncon;
                 }
                 self = (struct lsap_cb*) hashbin_get_next(irlmp->unconnected_lsaps);
@@ -1811,8 +1775,8 @@ static __u8 irlmp_find_free_slsap(void)
 
         /* Got it ! */
         lsap_sel = irlmp->last_lsap_sel;
-        IRDA_DEBUG(4, "%s(), found free lsap_sel=%02x\n",
-                   __func__, lsap_sel);
+        pr_debug("%s(), found free lsap_sel=%02x\n",
+                 __func__, lsap_sel);
 
         return lsap_sel;
 }
@@ -1830,26 +1794,27 @@ LM_REASON irlmp_convert_lap_reason( LAP_REASON lap_reason)
 
         switch (lap_reason) {
         case LAP_DISC_INDICATION: /* Received a disconnect request from peer */
-                IRDA_DEBUG( 1, "%s(), LAP_DISC_INDICATION\n", __func__);
+                pr_debug("%s(), LAP_DISC_INDICATION\n", __func__);
                 reason = LM_USER_REQUEST;
                 break;
         case LAP_NO_RESPONSE:    /* To many retransmits without response */
-                IRDA_DEBUG( 1, "%s(), LAP_NO_RESPONSE\n", __func__);
+                pr_debug("%s(), LAP_NO_RESPONSE\n", __func__);
                 reason = LM_LAP_DISCONNECT;
                 break;
         case LAP_RESET_INDICATION:
-                IRDA_DEBUG( 1, "%s(), LAP_RESET_INDICATION\n", __func__);
+                pr_debug("%s(), LAP_RESET_INDICATION\n", __func__);
                 reason = LM_LAP_RESET;
                 break;
         case LAP_FOUND_NONE:
         case LAP_MEDIA_BUSY:
         case LAP_PRIMARY_CONFLICT:
-                IRDA_DEBUG(1, "%s(), LAP_FOUND_NONE, LAP_MEDIA_BUSY or LAP_PRIMARY_CONFLICT\n", __func__);
+                pr_debug("%s(), LAP_FOUND_NONE, LAP_MEDIA_BUSY or LAP_PRIMARY_CONFLICT\n",
+                         __func__);
                 reason = LM_CONNECT_FAILURE;
                 break;
         default:
-                IRDA_DEBUG(1, "%s(), Unknown IrLAP disconnect reason %d!\n",
-                           __func__, lap_reason);
+                pr_debug("%s(), Unknown IrLAP disconnect reason %d!\n",
+                         __func__, lap_reason);
                 reason = LM_LAP_DISCONNECT;
                 break;
         }