Commit c83642d5 authored by Bjorn Helgaas's avatar Bjorn Helgaas Committed by Andi Kleen

ACPI: use dev_printk when possible

Convert printks to use dev_printk().  The most obvious change will
be messages like this:

   -ACPI: PCI Interrupt 0000:00:04.0[A] -> GSI 31 (level, low) -> IRQ 31
   +cciss 0000:00:04.0: PCI INT A -> GSI 31 (level, low) -> IRQ 31
Signed-off-by: default avatarBjorn Helgaas <bjorn.helgaas@hp.com>
Signed-off-by: default avatarLen Brown <len.brown@intel.com>
Signed-off-by: default avatarAndi Kleen <ak@linux.intel.com>
parent 40ab4f4c
...@@ -162,7 +162,7 @@ do_prt_fixups(struct acpi_prt_entry *entry, struct acpi_pci_routing_table *prt) ...@@ -162,7 +162,7 @@ do_prt_fixups(struct acpi_prt_entry *entry, struct acpi_pci_routing_table *prt)
!strcmp(prt->source, quirk->source) && !strcmp(prt->source, quirk->source) &&
strlen(prt->source) >= strlen(quirk->actual_source)) { strlen(prt->source) >= strlen(quirk->actual_source)) {
printk(KERN_WARNING PREFIX "firmware reports " printk(KERN_WARNING PREFIX "firmware reports "
"%04x:%02x:%02x[%c] connected to %s; " "%04x:%02x:%02x PCI INT %c connected to %s; "
"changing to %s\n", "changing to %s\n",
entry->id.segment, entry->id.bus, entry->id.segment, entry->id.bus,
entry->id.device, 'A' + entry->pin, entry->id.device, 'A' + entry->pin,
...@@ -429,7 +429,7 @@ acpi_pci_irq_derive(struct pci_dev *dev, ...@@ -429,7 +429,7 @@ acpi_pci_irq_derive(struct pci_dev *dev,
{ {
struct pci_dev *bridge = dev; struct pci_dev *bridge = dev;
int irq = -1; int irq = -1;
u8 bridge_pin = 0; u8 bridge_pin = 0, orig_pin = pin;
if (!dev) if (!dev)
...@@ -463,8 +463,8 @@ acpi_pci_irq_derive(struct pci_dev *dev, ...@@ -463,8 +463,8 @@ acpi_pci_irq_derive(struct pci_dev *dev,
} }
if (irq < 0) { if (irq < 0) {
printk(KERN_WARNING PREFIX "Unable to derive IRQ for device %s\n", dev_warn(&dev->dev, "can't derive routing for PCI INT %c\n",
pci_name(dev)); 'A' + orig_pin);
return -1; return -1;
} }
...@@ -487,6 +487,7 @@ int acpi_pci_irq_enable(struct pci_dev *dev) ...@@ -487,6 +487,7 @@ int acpi_pci_irq_enable(struct pci_dev *dev)
int triggering = ACPI_LEVEL_SENSITIVE; int triggering = ACPI_LEVEL_SENSITIVE;
int polarity = ACPI_ACTIVE_LOW; int polarity = ACPI_ACTIVE_LOW;
char *link = NULL; char *link = NULL;
char link_desc[16];
int rc; int rc;
...@@ -503,7 +504,7 @@ int acpi_pci_irq_enable(struct pci_dev *dev) ...@@ -503,7 +504,7 @@ int acpi_pci_irq_enable(struct pci_dev *dev)
pin--; pin--;
if (!dev->bus) { if (!dev->bus) {
printk(KERN_ERR PREFIX "Invalid (NULL) 'bus' field\n"); dev_err(&dev->dev, "invalid (NULL) 'bus' field\n");
return -ENODEV; return -ENODEV;
} }
...@@ -538,8 +539,7 @@ int acpi_pci_irq_enable(struct pci_dev *dev) ...@@ -538,8 +539,7 @@ int acpi_pci_irq_enable(struct pci_dev *dev)
* driver reported one, then use it. Exit in any case. * driver reported one, then use it. Exit in any case.
*/ */
if (irq < 0) { if (irq < 0) {
printk(KERN_WARNING PREFIX "PCI Interrupt %s[%c]: no GSI", dev_warn(&dev->dev, "PCI INT %c: no GSI", 'A' + pin);
pci_name(dev), ('A' + pin));
/* Interrupt Line values above 0xF are forbidden */ /* Interrupt Line values above 0xF are forbidden */
if (dev->irq > 0 && (dev->irq <= 0xF)) { if (dev->irq > 0 && (dev->irq <= 0xF)) {
printk(" - using IRQ %d\n", dev->irq); printk(" - using IRQ %d\n", dev->irq);
...@@ -554,21 +554,21 @@ int acpi_pci_irq_enable(struct pci_dev *dev) ...@@ -554,21 +554,21 @@ int acpi_pci_irq_enable(struct pci_dev *dev)
rc = acpi_register_gsi(irq, triggering, polarity); rc = acpi_register_gsi(irq, triggering, polarity);
if (rc < 0) { if (rc < 0) {
printk(KERN_WARNING PREFIX "PCI Interrupt %s[%c]: failed " dev_warn(&dev->dev, "PCI INT %c: failed to register GSI\n",
"to register GSI\n", pci_name(dev), ('A' + pin)); 'A' + pin);
return rc; return rc;
} }
dev->irq = rc; dev->irq = rc;
printk(KERN_INFO PREFIX "PCI Interrupt %s[%c] -> ",
pci_name(dev), 'A' + pin);
if (link) if (link)
printk("Link [%s] -> ", link); snprintf(link_desc, sizeof(link_desc), " -> Link[%s]", link);
else
link_desc[0] = '\0';
printk("GSI %u (%s, %s) -> IRQ %d\n", irq, dev_info(&dev->dev, "PCI INT %c%s -> GSI %u (%s, %s) -> IRQ %d\n",
(triggering == ACPI_LEVEL_SENSITIVE) ? "level" : "edge", 'A' + pin, link_desc, irq,
(polarity == ACPI_ACTIVE_LOW) ? "low" : "high", dev->irq); (triggering == ACPI_LEVEL_SENSITIVE) ? "level" : "edge",
(polarity == ACPI_ACTIVE_LOW) ? "low" : "high", dev->irq);
return 0; return 0;
} }
...@@ -616,10 +616,6 @@ void acpi_pci_irq_disable(struct pci_dev *dev) ...@@ -616,10 +616,6 @@ void acpi_pci_irq_disable(struct pci_dev *dev)
* (e.g. PCI_UNDEFINED_IRQ). * (e.g. PCI_UNDEFINED_IRQ).
*/ */
printk(KERN_INFO PREFIX "PCI interrupt for device %s disabled\n", dev_info(&dev->dev, "PCI INT %c disabled\n", 'A' + pin);
pci_name(dev));
acpi_unregister_gsi(gsi); acpi_unregister_gsi(gsi);
return;
} }
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment