diff options
Diffstat (limited to 'drivers/net/irda/mcs7780.c')
-rw-r--r-- | drivers/net/irda/mcs7780.c | 53 |
1 files changed, 27 insertions, 26 deletions
diff --git a/drivers/net/irda/mcs7780.c b/drivers/net/irda/mcs7780.c index 16f8ffb50e04..e4d678fbeb2f 100644 --- a/drivers/net/irda/mcs7780.c +++ b/drivers/net/irda/mcs7780.c | |||
@@ -197,14 +197,14 @@ error: | |||
197 | /* Setup a communication between mcs7780 and agilent chip. */ | 197 | /* Setup a communication between mcs7780 and agilent chip. */ |
198 | static inline int mcs_setup_transceiver_agilent(struct mcs_cb *mcs) | 198 | static inline int mcs_setup_transceiver_agilent(struct mcs_cb *mcs) |
199 | { | 199 | { |
200 | IRDA_WARNING("This transceiver type is not supported yet.\n"); | 200 | net_warn_ratelimited("This transceiver type is not supported yet\n"); |
201 | return 1; | 201 | return 1; |
202 | } | 202 | } |
203 | 203 | ||
204 | /* Setup a communication between mcs7780 and sharp chip. */ | 204 | /* Setup a communication between mcs7780 and sharp chip. */ |
205 | static inline int mcs_setup_transceiver_sharp(struct mcs_cb *mcs) | 205 | static inline int mcs_setup_transceiver_sharp(struct mcs_cb *mcs) |
206 | { | 206 | { |
207 | IRDA_WARNING("This transceiver type is not supported yet.\n"); | 207 | net_warn_ratelimited("This transceiver type is not supported yet\n"); |
208 | return 1; | 208 | return 1; |
209 | } | 209 | } |
210 | 210 | ||
@@ -213,9 +213,9 @@ static inline int mcs_setup_transceiver(struct mcs_cb *mcs) | |||
213 | { | 213 | { |
214 | int ret = 0; | 214 | int ret = 0; |
215 | __u16 rval; | 215 | __u16 rval; |
216 | char *msg; | 216 | const char *msg; |
217 | 217 | ||
218 | msg = "Basic transceiver setup error."; | 218 | msg = "Basic transceiver setup error"; |
219 | 219 | ||
220 | /* read value of MODE Register, set the DRIVER and RESET bits | 220 | /* read value of MODE Register, set the DRIVER and RESET bits |
221 | * and write value back out to MODE Register | 221 | * and write value back out to MODE Register |
@@ -261,7 +261,7 @@ static inline int mcs_setup_transceiver(struct mcs_cb *mcs) | |||
261 | if(unlikely(ret)) | 261 | if(unlikely(ret)) |
262 | goto error; | 262 | goto error; |
263 | 263 | ||
264 | msg = "transceiver model specific setup error."; | 264 | msg = "transceiver model specific setup error"; |
265 | switch (mcs->transceiver_type) { | 265 | switch (mcs->transceiver_type) { |
266 | case MCS_TSC_VISHAY: | 266 | case MCS_TSC_VISHAY: |
267 | ret = mcs_setup_transceiver_vishay(mcs); | 267 | ret = mcs_setup_transceiver_vishay(mcs); |
@@ -276,8 +276,8 @@ static inline int mcs_setup_transceiver(struct mcs_cb *mcs) | |||
276 | break; | 276 | break; |
277 | 277 | ||
278 | default: | 278 | default: |
279 | IRDA_WARNING("Unknown transceiver type: %d\n", | 279 | net_warn_ratelimited("Unknown transceiver type: %d\n", |
280 | mcs->transceiver_type); | 280 | mcs->transceiver_type); |
281 | ret = 1; | 281 | ret = 1; |
282 | } | 282 | } |
283 | if (unlikely(ret)) | 283 | if (unlikely(ret)) |
@@ -300,7 +300,7 @@ static inline int mcs_setup_transceiver(struct mcs_cb *mcs) | |||
300 | goto error; | 300 | goto error; |
301 | } | 301 | } |
302 | 302 | ||
303 | msg = "transceiver reset."; | 303 | msg = "transceiver reset"; |
304 | 304 | ||
305 | ret = mcs_get_reg(mcs, MCS_MODE_REG, &rval); | 305 | ret = mcs_get_reg(mcs, MCS_MODE_REG, &rval); |
306 | if (unlikely(ret != 2)) | 306 | if (unlikely(ret != 2)) |
@@ -315,7 +315,7 @@ static inline int mcs_setup_transceiver(struct mcs_cb *mcs) | |||
315 | return ret; | 315 | return ret; |
316 | 316 | ||
317 | error: | 317 | error: |
318 | IRDA_ERROR("%s\n", msg); | 318 | net_err_ratelimited("%s\n", msg); |
319 | return ret; | 319 | return ret; |
320 | } | 320 | } |
321 | 321 | ||
@@ -399,8 +399,8 @@ static void mcs_unwrap_mir(struct mcs_cb *mcs, __u8 *buf, int len) | |||
399 | 399 | ||
400 | new_len = len - 2; | 400 | new_len = len - 2; |
401 | if(unlikely(new_len <= 0)) { | 401 | if(unlikely(new_len <= 0)) { |
402 | IRDA_ERROR("%s short frame length %d\n", | 402 | net_err_ratelimited("%s short frame length %d\n", |
403 | mcs->netdev->name, new_len); | 403 | mcs->netdev->name, new_len); |
404 | ++mcs->netdev->stats.rx_errors; | 404 | ++mcs->netdev->stats.rx_errors; |
405 | ++mcs->netdev->stats.rx_length_errors; | 405 | ++mcs->netdev->stats.rx_length_errors; |
406 | return; | 406 | return; |
@@ -409,8 +409,8 @@ static void mcs_unwrap_mir(struct mcs_cb *mcs, __u8 *buf, int len) | |||
409 | fcs = irda_calc_crc16(~fcs, buf, len); | 409 | fcs = irda_calc_crc16(~fcs, buf, len); |
410 | 410 | ||
411 | if(fcs != GOOD_FCS) { | 411 | if(fcs != GOOD_FCS) { |
412 | IRDA_ERROR("crc error calc 0x%x len %d\n", | 412 | net_err_ratelimited("crc error calc 0x%x len %d\n", |
413 | fcs, new_len); | 413 | fcs, new_len); |
414 | mcs->netdev->stats.rx_errors++; | 414 | mcs->netdev->stats.rx_errors++; |
415 | mcs->netdev->stats.rx_crc_errors++; | 415 | mcs->netdev->stats.rx_crc_errors++; |
416 | return; | 416 | return; |
@@ -452,8 +452,8 @@ static void mcs_unwrap_fir(struct mcs_cb *mcs, __u8 *buf, int len) | |||
452 | 452 | ||
453 | new_len = len - 4; | 453 | new_len = len - 4; |
454 | if(unlikely(new_len <= 0)) { | 454 | if(unlikely(new_len <= 0)) { |
455 | IRDA_ERROR("%s short frame length %d\n", | 455 | net_err_ratelimited("%s short frame length %d\n", |
456 | mcs->netdev->name, new_len); | 456 | mcs->netdev->name, new_len); |
457 | ++mcs->netdev->stats.rx_errors; | 457 | ++mcs->netdev->stats.rx_errors; |
458 | ++mcs->netdev->stats.rx_length_errors; | 458 | ++mcs->netdev->stats.rx_length_errors; |
459 | return; | 459 | return; |
@@ -461,7 +461,8 @@ static void mcs_unwrap_fir(struct mcs_cb *mcs, __u8 *buf, int len) | |||
461 | 461 | ||
462 | fcs = ~(crc32_le(~0, buf, new_len)); | 462 | fcs = ~(crc32_le(~0, buf, new_len)); |
463 | if(fcs != get_unaligned_le32(buf + new_len)) { | 463 | if(fcs != get_unaligned_le32(buf + new_len)) { |
464 | IRDA_ERROR("crc error calc 0x%x len %d\n", fcs, new_len); | 464 | net_err_ratelimited("crc error calc 0x%x len %d\n", |
465 | fcs, new_len); | ||
465 | mcs->netdev->stats.rx_errors++; | 466 | mcs->netdev->stats.rx_errors++; |
466 | mcs->netdev->stats.rx_crc_errors++; | 467 | mcs->netdev->stats.rx_crc_errors++; |
467 | return; | 468 | return; |
@@ -583,7 +584,7 @@ static int mcs_speed_change(struct mcs_cb *mcs) | |||
583 | } while(cnt++ < 100 && (rval & MCS_IRINTX)); | 584 | } while(cnt++ < 100 && (rval & MCS_IRINTX)); |
584 | 585 | ||
585 | if (cnt > 100) { | 586 | if (cnt > 100) { |
586 | IRDA_ERROR("unable to change speed\n"); | 587 | net_err_ratelimited("unable to change speed\n"); |
587 | ret = -EIO; | 588 | ret = -EIO; |
588 | goto error; | 589 | goto error; |
589 | } | 590 | } |
@@ -634,8 +635,8 @@ static int mcs_speed_change(struct mcs_cb *mcs) | |||
634 | 635 | ||
635 | default: | 636 | default: |
636 | ret = 1; | 637 | ret = 1; |
637 | IRDA_WARNING("Unknown transceiver type: %d\n", | 638 | net_warn_ratelimited("Unknown transceiver type: %d\n", |
638 | mcs->transceiver_type); | 639 | mcs->transceiver_type); |
639 | } | 640 | } |
640 | if (unlikely(ret)) | 641 | if (unlikely(ret)) |
641 | goto error; | 642 | goto error; |
@@ -731,7 +732,7 @@ static int mcs_net_open(struct net_device *netdev) | |||
731 | sprintf(hwname, "usb#%d", mcs->usbdev->devnum); | 732 | sprintf(hwname, "usb#%d", mcs->usbdev->devnum); |
732 | mcs->irlap = irlap_open(netdev, &mcs->qos, hwname); | 733 | mcs->irlap = irlap_open(netdev, &mcs->qos, hwname); |
733 | if (!mcs->irlap) { | 734 | if (!mcs->irlap) { |
734 | IRDA_ERROR("mcs7780: irlap_open failed\n"); | 735 | net_err_ratelimited("mcs7780: irlap_open failed\n"); |
735 | goto error2; | 736 | goto error2; |
736 | } | 737 | } |
737 | 738 | ||
@@ -851,7 +852,7 @@ static netdev_tx_t mcs_hard_xmit(struct sk_buff *skb, | |||
851 | mcs->out_buf, wraplen, mcs_send_irq, mcs); | 852 | mcs->out_buf, wraplen, mcs_send_irq, mcs); |
852 | 853 | ||
853 | if ((ret = usb_submit_urb(mcs->tx_urb, GFP_ATOMIC))) { | 854 | if ((ret = usb_submit_urb(mcs->tx_urb, GFP_ATOMIC))) { |
854 | IRDA_ERROR("failed tx_urb: %d\n", ret); | 855 | net_err_ratelimited("failed tx_urb: %d\n", ret); |
855 | switch (ret) { | 856 | switch (ret) { |
856 | case -ENODEV: | 857 | case -ENODEV: |
857 | case -EPIPE: | 858 | case -EPIPE: |
@@ -893,13 +894,13 @@ static int mcs_probe(struct usb_interface *intf, | |||
893 | if (!ndev) | 894 | if (!ndev) |
894 | goto error1; | 895 | goto error1; |
895 | 896 | ||
896 | IRDA_DEBUG(1, "MCS7780 USB-IrDA bridge found at %d.\n", udev->devnum); | 897 | pr_debug("MCS7780 USB-IrDA bridge found at %d.\n", udev->devnum); |
897 | 898 | ||
898 | SET_NETDEV_DEV(ndev, &intf->dev); | 899 | SET_NETDEV_DEV(ndev, &intf->dev); |
899 | 900 | ||
900 | ret = usb_reset_configuration(udev); | 901 | ret = usb_reset_configuration(udev); |
901 | if (ret != 0) { | 902 | if (ret != 0) { |
902 | IRDA_ERROR("mcs7780: usb reset configuration failed\n"); | 903 | net_err_ratelimited("mcs7780: usb reset configuration failed\n"); |
903 | goto error2; | 904 | goto error2; |
904 | } | 905 | } |
905 | 906 | ||
@@ -941,8 +942,8 @@ static int mcs_probe(struct usb_interface *intf, | |||
941 | if (ret != 0) | 942 | if (ret != 0) |
942 | goto error2; | 943 | goto error2; |
943 | 944 | ||
944 | IRDA_DEBUG(1, "IrDA: Registered MosChip MCS7780 device as %s\n", | 945 | pr_debug("IrDA: Registered MosChip MCS7780 device as %s\n", |
945 | ndev->name); | 946 | ndev->name); |
946 | 947 | ||
947 | mcs->transceiver_type = transceiver_type; | 948 | mcs->transceiver_type = transceiver_type; |
948 | mcs->sir_tweak = sir_tweak; | 949 | mcs->sir_tweak = sir_tweak; |
@@ -972,7 +973,7 @@ static void mcs_disconnect(struct usb_interface *intf) | |||
972 | free_netdev(mcs->netdev); | 973 | free_netdev(mcs->netdev); |
973 | 974 | ||
974 | usb_set_intfdata(intf, NULL); | 975 | usb_set_intfdata(intf, NULL); |
975 | IRDA_DEBUG(0, "MCS7780 now disconnected.\n"); | 976 | pr_debug("MCS7780 now disconnected.\n"); |
976 | } | 977 | } |
977 | 978 | ||
978 | module_usb_driver(mcs_driver); | 979 | module_usb_driver(mcs_driver); |