diff options
-rw-r--r-- | drivers/firewire/nosy.c | 33 |
1 files changed, 16 insertions, 17 deletions
diff --git a/drivers/firewire/nosy.c b/drivers/firewire/nosy.c index 225e64956823..8528b10763ed 100644 --- a/drivers/firewire/nosy.c +++ b/drivers/firewire/nosy.c | |||
@@ -17,6 +17,7 @@ | |||
17 | * Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA. | 17 | * Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA. |
18 | */ | 18 | */ |
19 | 19 | ||
20 | #include <linux/device.h> | ||
20 | #include <linux/errno.h> | 21 | #include <linux/errno.h> |
21 | #include <linux/fs.h> | 22 | #include <linux/fs.h> |
22 | #include <linux/init.h> | 23 | #include <linux/init.h> |
@@ -45,10 +46,6 @@ | |||
45 | #define TCODE_PHY_PACKET 0x10 | 46 | #define TCODE_PHY_PACKET 0x10 |
46 | #define PCI_DEVICE_ID_TI_PCILYNX 0x8000 | 47 | #define PCI_DEVICE_ID_TI_PCILYNX 0x8000 |
47 | 48 | ||
48 | #define notify(s, args...) printk(KERN_NOTICE s, ## args) | ||
49 | #define error(s, args...) printk(KERN_ERR s, ## args) | ||
50 | #define debug(s, args...) printk(KERN_DEBUG s, ## args) | ||
51 | |||
52 | static char driver_name[] = KBUILD_MODNAME; | 49 | static char driver_name[] = KBUILD_MODNAME; |
53 | 50 | ||
54 | /* this is the physical layout of a PCL, its size is 128 bytes */ | 51 | /* this is the physical layout of a PCL, its size is 128 bytes */ |
@@ -260,15 +257,15 @@ static int | |||
260 | set_phy_reg(struct pcilynx *lynx, int addr, int val) | 257 | set_phy_reg(struct pcilynx *lynx, int addr, int val) |
261 | { | 258 | { |
262 | if (addr > 15) { | 259 | if (addr > 15) { |
263 | debug("PHY register address %d out of range\n", addr); | 260 | dev_err(&lynx->pci_device->dev, |
261 | "PHY register address %d out of range\n", addr); | ||
264 | return -1; | 262 | return -1; |
265 | } | 263 | } |
266 | |||
267 | if (val > 0xff) { | 264 | if (val > 0xff) { |
268 | debug("PHY register value %d out of range\n", val); | 265 | dev_err(&lynx->pci_device->dev, |
266 | "PHY register value %d out of range\n", val); | ||
269 | return -1; | 267 | return -1; |
270 | } | 268 | } |
271 | |||
272 | reg_write(lynx, LINK_PHY, LINK_PHY_WRITE | | 269 | reg_write(lynx, LINK_PHY, LINK_PHY_WRITE | |
273 | LINK_PHY_ADDR(addr) | LINK_PHY_WDATA(val)); | 270 | LINK_PHY_ADDR(addr) | LINK_PHY_WDATA(val)); |
274 | 271 | ||
@@ -540,19 +537,19 @@ add_card(struct pci_dev *dev, const struct pci_device_id *unused) | |||
540 | int ret, i; | 537 | int ret, i; |
541 | 538 | ||
542 | if (pci_set_dma_mask(dev, 0xffffffff)) { | 539 | if (pci_set_dma_mask(dev, 0xffffffff)) { |
543 | error("DMA address limits not supported " | 540 | dev_err(&dev->dev, |
544 | "for PCILynx hardware\n"); | 541 | "DMA address limits not supported for PCILynx hardware\n"); |
545 | return -ENXIO; | 542 | return -ENXIO; |
546 | } | 543 | } |
547 | if (pci_enable_device(dev)) { | 544 | if (pci_enable_device(dev)) { |
548 | error("Failed to enable PCILynx hardware\n"); | 545 | dev_err(&dev->dev, "Failed to enable PCILynx hardware\n"); |
549 | return -ENXIO; | 546 | return -ENXIO; |
550 | } | 547 | } |
551 | pci_set_master(dev); | 548 | pci_set_master(dev); |
552 | 549 | ||
553 | lynx = kzalloc(sizeof *lynx, GFP_KERNEL); | 550 | lynx = kzalloc(sizeof *lynx, GFP_KERNEL); |
554 | if (lynx == NULL) { | 551 | if (lynx == NULL) { |
555 | error("Failed to allocate control structure memory\n"); | 552 | dev_err(&dev->dev, "Failed to allocate control structure\n"); |
556 | ret = -ENOMEM; | 553 | ret = -ENOMEM; |
557 | goto fail_disable; | 554 | goto fail_disable; |
558 | } | 555 | } |
@@ -575,7 +572,7 @@ add_card(struct pci_dev *dev, const struct pci_device_id *unused) | |||
575 | if (lynx->rcv_start_pcl == NULL || | 572 | if (lynx->rcv_start_pcl == NULL || |
576 | lynx->rcv_pcl == NULL || | 573 | lynx->rcv_pcl == NULL || |
577 | lynx->rcv_buffer == NULL) { | 574 | lynx->rcv_buffer == NULL) { |
578 | error("Failed to allocate receive buffer\n"); | 575 | dev_err(&dev->dev, "Failed to allocate receive buffer\n"); |
579 | ret = -ENOMEM; | 576 | ret = -ENOMEM; |
580 | goto fail_deallocate; | 577 | goto fail_deallocate; |
581 | } | 578 | } |
@@ -637,7 +634,8 @@ add_card(struct pci_dev *dev, const struct pci_device_id *unused) | |||
637 | 634 | ||
638 | if (request_irq(dev->irq, irq_handler, IRQF_SHARED, | 635 | if (request_irq(dev->irq, irq_handler, IRQF_SHARED, |
639 | driver_name, lynx)) { | 636 | driver_name, lynx)) { |
640 | error("Failed to allocate shared interrupt %d\n", dev->irq); | 637 | dev_err(&dev->dev, |
638 | "Failed to allocate shared interrupt %d\n", dev->irq); | ||
641 | ret = -EIO; | 639 | ret = -EIO; |
642 | goto fail_deallocate; | 640 | goto fail_deallocate; |
643 | } | 641 | } |
@@ -650,14 +648,15 @@ add_card(struct pci_dev *dev, const struct pci_device_id *unused) | |||
650 | mutex_lock(&card_mutex); | 648 | mutex_lock(&card_mutex); |
651 | ret = misc_register(&lynx->misc); | 649 | ret = misc_register(&lynx->misc); |
652 | if (ret) { | 650 | if (ret) { |
653 | error("Failed to register misc char device\n"); | 651 | dev_err(&dev->dev, "Failed to register misc char device\n"); |
654 | mutex_unlock(&card_mutex); | 652 | mutex_unlock(&card_mutex); |
655 | goto fail_free_irq; | 653 | goto fail_free_irq; |
656 | } | 654 | } |
657 | list_add_tail(&lynx->link, &card_list); | 655 | list_add_tail(&lynx->link, &card_list); |
658 | mutex_unlock(&card_mutex); | 656 | mutex_unlock(&card_mutex); |
659 | 657 | ||
660 | notify("Initialized PCILynx IEEE1394 card, irq=%d\n", dev->irq); | 658 | dev_info(&dev->dev, |
659 | "Initialized PCILynx IEEE1394 card, irq=%d\n", dev->irq); | ||
661 | 660 | ||
662 | return 0; | 661 | return 0; |
663 | 662 | ||
@@ -715,7 +714,7 @@ static void __exit nosy_cleanup(void) | |||
715 | { | 714 | { |
716 | pci_unregister_driver(&lynx_pci_driver); | 715 | pci_unregister_driver(&lynx_pci_driver); |
717 | 716 | ||
718 | notify("Unloaded %s.\n", driver_name); | 717 | pr_info("Unloaded %s\n", driver_name); |
719 | } | 718 | } |
720 | 719 | ||
721 | module_init(nosy_init); | 720 | module_init(nosy_init); |