diff options
author | Joe Perches <joe@perches.com> | 2014-11-11 17:44:57 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2014-11-12 13:56:41 -0500 |
commit | 955a9d202f470019f42979f0d1caec98843e39ce (patch) | |
tree | fff6124b339bc33e3b5c67668bfcc13098be865c /net/irda/wrapper.c | |
parent | 8d326d818a2a8fc80c7df85dd88cb214804d1499 (diff) |
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>
Diffstat (limited to 'net/irda/wrapper.c')
-rw-r--r-- | net/irda/wrapper.c | 22 |
1 files changed, 11 insertions, 11 deletions
diff --git a/net/irda/wrapper.c b/net/irda/wrapper.c index 9efffeb8d0f1..40a0f993bf13 100644 --- a/net/irda/wrapper.c +++ b/net/irda/wrapper.c | |||
@@ -106,17 +106,17 @@ int async_wrap_skb(struct sk_buff *skb, __u8 *tx_buff, int buffsize) | |||
106 | * Nothing to worry about, but we set the default number of | 106 | * Nothing to worry about, but we set the default number of |
107 | * BOF's | 107 | * BOF's |
108 | */ | 108 | */ |
109 | IRDA_DEBUG(1, "%s(), wrong magic in skb!\n", __func__); | 109 | pr_debug("%s(), wrong magic in skb!\n", __func__); |
110 | xbofs = 10; | 110 | xbofs = 10; |
111 | } else | 111 | } else |
112 | xbofs = cb->xbofs + cb->xbofs_delay; | 112 | xbofs = cb->xbofs + cb->xbofs_delay; |
113 | 113 | ||
114 | IRDA_DEBUG(4, "%s(), xbofs=%d\n", __func__, xbofs); | 114 | pr_debug("%s(), xbofs=%d\n", __func__, xbofs); |
115 | 115 | ||
116 | /* Check that we never use more than 115 + 48 xbofs */ | 116 | /* Check that we never use more than 115 + 48 xbofs */ |
117 | if (xbofs > 163) { | 117 | if (xbofs > 163) { |
118 | IRDA_DEBUG(0, "%s(), too many xbofs (%d)\n", __func__, | 118 | pr_debug("%s(), too many xbofs (%d)\n", __func__, |
119 | xbofs); | 119 | xbofs); |
120 | xbofs = 163; | 120 | xbofs = 163; |
121 | } | 121 | } |
122 | 122 | ||
@@ -286,8 +286,8 @@ async_unwrap_bof(struct net_device *dev, | |||
286 | case INSIDE_FRAME: | 286 | case INSIDE_FRAME: |
287 | /* Not supposed to happen, the previous frame is not | 287 | /* Not supposed to happen, the previous frame is not |
288 | * finished - Jean II */ | 288 | * finished - Jean II */ |
289 | IRDA_DEBUG(1, "%s(), Discarding incomplete frame\n", | 289 | pr_debug("%s(), Discarding incomplete frame\n", |
290 | __func__); | 290 | __func__); |
291 | stats->rx_errors++; | 291 | stats->rx_errors++; |
292 | stats->rx_missed_errors++; | 292 | stats->rx_missed_errors++; |
293 | irda_device_set_media_busy(dev, TRUE); | 293 | irda_device_set_media_busy(dev, TRUE); |
@@ -360,7 +360,7 @@ async_unwrap_eof(struct net_device *dev, | |||
360 | /* Wrong CRC, discard frame! */ | 360 | /* Wrong CRC, discard frame! */ |
361 | irda_device_set_media_busy(dev, TRUE); | 361 | irda_device_set_media_busy(dev, TRUE); |
362 | 362 | ||
363 | IRDA_DEBUG(1, "%s(), crc error\n", __func__); | 363 | pr_debug("%s(), crc error\n", __func__); |
364 | stats->rx_errors++; | 364 | stats->rx_errors++; |
365 | stats->rx_crc_errors++; | 365 | stats->rx_crc_errors++; |
366 | } | 366 | } |
@@ -420,8 +420,8 @@ async_unwrap_other(struct net_device *dev, | |||
420 | rx_buff->fcs = irda_fcs(rx_buff->fcs, byte); | 420 | rx_buff->fcs = irda_fcs(rx_buff->fcs, byte); |
421 | #endif | 421 | #endif |
422 | } else { | 422 | } else { |
423 | IRDA_DEBUG(1, "%s(), Rx buffer overflow, aborting\n", | 423 | pr_debug("%s(), Rx buffer overflow, aborting\n", |
424 | __func__); | 424 | __func__); |
425 | rx_buff->state = OUTSIDE_FRAME; | 425 | rx_buff->state = OUTSIDE_FRAME; |
426 | } | 426 | } |
427 | break; | 427 | break; |
@@ -439,8 +439,8 @@ async_unwrap_other(struct net_device *dev, | |||
439 | #endif | 439 | #endif |
440 | rx_buff->state = INSIDE_FRAME; | 440 | rx_buff->state = INSIDE_FRAME; |
441 | } else { | 441 | } else { |
442 | IRDA_DEBUG(1, "%s(), Rx buffer overflow, aborting\n", | 442 | pr_debug("%s(), Rx buffer overflow, aborting\n", |
443 | __func__); | 443 | __func__); |
444 | rx_buff->state = OUTSIDE_FRAME; | 444 | rx_buff->state = OUTSIDE_FRAME; |
445 | } | 445 | } |
446 | break; | 446 | break; |