diff options
author | Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> | 2008-07-24 16:53:32 -0400 |
---|---|---|
committer | Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> | 2008-07-24 16:53:32 -0400 |
commit | ced3ec8aa7d0fa3300187ee47c144a22ccfc974e (patch) | |
tree | ea48651fe1b372685020cd67ac133240829c1576 /drivers/ide/pci/sis5513.c | |
parent | 04ba6e739e9c0623c25f94b191fd20dfbd1b26e3 (diff) |
ide: prefix messages from IDE PCI host drivers by driver name
Prefix messages from IDE PCI host drivers by driver name instead of marketed
chipset name (it is still possible to exactly identify the particular chipset
basing on driver messages).
As a bonus this provides nice code savings for some drivers:
text data bss dec hex filename
3826 112 8 3946 f6a drivers/ide/pci/amd74xx.o.before
2786 112 8 2906 b5a drivers/ide/pci/amd74xx.o.after
764 108 0 872 368 drivers/ide/pci/cs5520.o.before
680 108 0 788 314 drivers/ide/pci/cs5520.o.after
1680 112 4 1796 704 drivers/ide/pci/generic.o.before
1155 112 4 1271 4f7 drivers/ide/pci/generic.o.after
7128 792 0 7920 1ef0 drivers/ide/pci/hpt366.o.before
6984 792 0 7776 1e60 drivers/ide/pci/hpt366.o.after
2800 148 0 2948 b84 drivers/ide/pci/pdc202xx_new.o.before
2523 148 0 2671 a6f drivers/ide/pci/pdc202xx_new.o.after
2831 148 0 2979 ba3 drivers/ide/pci/pdc202xx_old.o.before
2683 148 0 2831 b0f drivers/ide/pci/pdc202xx_old.o.after
3776 112 4 3892 f34 drivers/ide/pci/piix.o.before
2804 112 4 2920 b68 drivers/ide/pci/piix.o.after
4693 116 0 4809 12c9 drivers/ide/pci/siimage.o.before
4600 116 0 4716 126c drivers/ide/pci/siimage.o.after
Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Diffstat (limited to 'drivers/ide/pci/sis5513.c')
-rw-r--r-- | drivers/ide/pci/sis5513.c | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/drivers/ide/pci/sis5513.c b/drivers/ide/pci/sis5513.c index 518d8ab413f3..6fcb46c87871 100644 --- a/drivers/ide/pci/sis5513.c +++ b/drivers/ide/pci/sis5513.c | |||
@@ -52,6 +52,8 @@ | |||
52 | #include <linux/init.h> | 52 | #include <linux/init.h> |
53 | #include <linux/ide.h> | 53 | #include <linux/ide.h> |
54 | 54 | ||
55 | #define DRV_NAME "sis5513" | ||
56 | |||
55 | /* registers layout and init values are chipset family dependant */ | 57 | /* registers layout and init values are chipset family dependant */ |
56 | 58 | ||
57 | #define ATA_16 0x01 | 59 | #define ATA_16 0x01 |
@@ -380,7 +382,7 @@ static int __devinit sis_find_family(struct pci_dev *dev) | |||
380 | } | 382 | } |
381 | pci_dev_put(host); | 383 | pci_dev_put(host); |
382 | 384 | ||
383 | printk(KERN_INFO "SIS5513 %s: %s %s controller\n", | 385 | printk(KERN_INFO DRV_NAME " %s: %s %s controller\n", |
384 | pci_name(dev), SiSHostChipInfo[i].name, | 386 | pci_name(dev), SiSHostChipInfo[i].name, |
385 | chipset_capability[chipset_family]); | 387 | chipset_capability[chipset_family]); |
386 | } | 388 | } |
@@ -397,7 +399,7 @@ static int __devinit sis_find_family(struct pci_dev *dev) | |||
397 | pci_write_config_dword(dev, 0x54, idemisc); | 399 | pci_write_config_dword(dev, 0x54, idemisc); |
398 | 400 | ||
399 | if (trueid == 0x5518) { | 401 | if (trueid == 0x5518) { |
400 | printk(KERN_INFO "SIS5513 %s: SiS 962/963 MuTIOL IDE UDMA133 controller\n", | 402 | printk(KERN_INFO DRV_NAME " %s: SiS 962/963 MuTIOL IDE UDMA133 controller\n", |
401 | pci_name(dev)); | 403 | pci_name(dev)); |
402 | chipset_family = ATA_133; | 404 | chipset_family = ATA_133; |
403 | 405 | ||
@@ -407,7 +409,7 @@ static int __devinit sis_find_family(struct pci_dev *dev) | |||
407 | */ | 409 | */ |
408 | if ((idemisc & 0x40000000) == 0) { | 410 | if ((idemisc & 0x40000000) == 0) { |
409 | pci_write_config_dword(dev, 0x54, idemisc | 0x40000000); | 411 | pci_write_config_dword(dev, 0x54, idemisc | 0x40000000); |
410 | printk(KERN_INFO "SIS5513 %s: Switching to 5513 register mapping\n", | 412 | printk(KERN_INFO DRV_NAME " %s: Switching to 5513 register mapping\n", |
411 | pci_name(dev)); | 413 | pci_name(dev)); |
412 | } | 414 | } |
413 | } | 415 | } |
@@ -432,11 +434,11 @@ static int __devinit sis_find_family(struct pci_dev *dev) | |||
432 | pci_dev_put(lpc_bridge); | 434 | pci_dev_put(lpc_bridge); |
433 | 435 | ||
434 | if (lpc_bridge->revision == 0x10 && (prefctl & 0x80)) { | 436 | if (lpc_bridge->revision == 0x10 && (prefctl & 0x80)) { |
435 | printk(KERN_INFO "SIS5513 %s: SiS 961B MuTIOL IDE UDMA133 controller\n", | 437 | printk(KERN_INFO DRV_NAME " %s: SiS 961B MuTIOL IDE UDMA133 controller\n", |
436 | pci_name(dev)); | 438 | pci_name(dev)); |
437 | chipset_family = ATA_133a; | 439 | chipset_family = ATA_133a; |
438 | } else { | 440 | } else { |
439 | printk(KERN_INFO "SIS5513 %s: SiS 961 MuTIOL IDE UDMA100 controller\n", | 441 | printk(KERN_INFO DRV_NAME " %s: SiS 961 MuTIOL IDE UDMA100 controller\n", |
440 | pci_name(dev)); | 442 | pci_name(dev)); |
441 | chipset_family = ATA_100; | 443 | chipset_family = ATA_100; |
442 | } | 444 | } |
@@ -560,7 +562,7 @@ static const struct ide_port_ops sis_ata133_port_ops = { | |||
560 | }; | 562 | }; |
561 | 563 | ||
562 | static const struct ide_port_info sis5513_chipset __devinitdata = { | 564 | static const struct ide_port_info sis5513_chipset __devinitdata = { |
563 | .name = "SIS5513", | 565 | .name = DRV_NAME, |
564 | .init_chipset = init_chipset_sis5513, | 566 | .init_chipset = init_chipset_sis5513, |
565 | .enablebits = { {0x4a, 0x02, 0x02}, {0x4a, 0x04, 0x04} }, | 567 | .enablebits = { {0x4a, 0x02, 0x02}, {0x4a, 0x04, 0x04} }, |
566 | .host_flags = IDE_HFLAG_LEGACY_IRQS | IDE_HFLAG_NO_AUTODMA, | 568 | .host_flags = IDE_HFLAG_LEGACY_IRQS | IDE_HFLAG_NO_AUTODMA, |