powerpc: Use pr_warn instead of pr_warning
At some point, pr_warning will be removed so all logging messages use a consistent <prefix>_warn style. Update arch/powerpc/ Miscellanea: o Coalesce formats o Realign arguments o Use %s, __func__ instead of embedded function names o Remove unnecessary line continuations Signed-off-by: Joe Perches <joe@perches.com> Acked-by: Geoff Levand <geoff@infradead.org> [mpe: Rebase due to some %pOF changes.] Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
This commit is contained in:
parent
ae64f9bd1d
commit
f2c2cbcc35
|
@ -1276,8 +1276,8 @@ static void pcibios_allocate_bus_resources(struct pci_bus *bus)
|
|||
i + PCI_BRIDGE_RESOURCES) == 0)
|
||||
continue;
|
||||
}
|
||||
pr_warning("PCI: Cannot allocate resource region "
|
||||
"%d of PCI bridge %d, will remap\n", i, bus->number);
|
||||
pr_warn("PCI: Cannot allocate resource region %d of PCI bridge %d, will remap\n",
|
||||
i, bus->number);
|
||||
clear_resource:
|
||||
/* The resource might be figured out when doing
|
||||
* reassignment based on the resources required
|
||||
|
|
|
@ -214,9 +214,8 @@ int __meminit vmemmap_populate(unsigned long start, unsigned long end, int node)
|
|||
|
||||
rc = vmemmap_create_mapping(start, page_size, __pa(p));
|
||||
if (rc < 0) {
|
||||
pr_warning(
|
||||
"vmemmap_populate: Unable to create vmemmap mapping: %d\n",
|
||||
rc);
|
||||
pr_warn("%s: Unable to create vmemmap mapping: %d\n",
|
||||
__func__, rc);
|
||||
return -EFAULT;
|
||||
}
|
||||
}
|
||||
|
|
|
@ -138,8 +138,7 @@ int arch_add_memory(int nid, u64 start, u64 size, bool want_memblock)
|
|||
start = (unsigned long)__va(start);
|
||||
rc = create_section_mapping(start, start + size);
|
||||
if (rc) {
|
||||
pr_warning(
|
||||
"Unable to create mapping for hot added memory 0x%llx..0x%llx: %d\n",
|
||||
pr_warn("Unable to create mapping for hot added memory 0x%llx..0x%llx: %d\n",
|
||||
start, start + size, rc);
|
||||
return -EFAULT;
|
||||
}
|
||||
|
|
|
@ -387,8 +387,8 @@ static unsigned int __init get_fifo_size(struct device_node *np,
|
|||
if (fp)
|
||||
return *fp;
|
||||
|
||||
pr_warning("no %s property in %pOF node, defaulting to %d\n",
|
||||
prop_name, np, DEFAULT_FIFO_SIZE);
|
||||
pr_warn("no %s property in %pOF node, defaulting to %d\n",
|
||||
prop_name, np, DEFAULT_FIFO_SIZE);
|
||||
|
||||
return DEFAULT_FIFO_SIZE;
|
||||
}
|
||||
|
|
|
@ -252,8 +252,7 @@ static int socrates_fpga_pic_host_xlate(struct irq_domain *h,
|
|||
/* type is configurable */
|
||||
if (intspec[1] != IRQ_TYPE_LEVEL_LOW &&
|
||||
intspec[1] != IRQ_TYPE_LEVEL_HIGH) {
|
||||
pr_warning("FPGA PIC: invalid irq type, "
|
||||
"setting default active low\n");
|
||||
pr_warn("FPGA PIC: invalid irq type, setting default active low\n");
|
||||
*out_flags = IRQ_TYPE_LEVEL_LOW;
|
||||
} else {
|
||||
*out_flags = intspec[1];
|
||||
|
@ -267,7 +266,7 @@ static int socrates_fpga_pic_host_xlate(struct irq_domain *h,
|
|||
if (intspec[2] <= 2)
|
||||
fpga_irq->irq_line = intspec[2];
|
||||
else
|
||||
pr_warning("FPGA PIC: invalid irq routing\n");
|
||||
pr_warn("FPGA PIC: invalid irq routing\n");
|
||||
|
||||
return 0;
|
||||
}
|
||||
|
@ -293,7 +292,7 @@ void socrates_fpga_pic_init(struct device_node *pic)
|
|||
for (i = 0; i < 3; i++) {
|
||||
socrates_fpga_irqs[i] = irq_of_parse_and_map(pic, i);
|
||||
if (!socrates_fpga_irqs[i]) {
|
||||
pr_warning("FPGA PIC: can't get irq%d.\n", i);
|
||||
pr_warn("FPGA PIC: can't get irq%d\n", i);
|
||||
continue;
|
||||
}
|
||||
irq_set_chained_handler(socrates_fpga_irqs[i],
|
||||
|
|
|
@ -101,7 +101,7 @@ static int __init mpc86xx_hpcn_probe(void)
|
|||
|
||||
/* Be nice and don't give silent boot death. Delete this in 2.6.27 */
|
||||
if (of_machine_is_compatible("mpc86xx")) {
|
||||
pr_warning("WARNING: your dts/dtb is old. You must update before the next kernel release\n");
|
||||
pr_warn("WARNING: your dts/dtb is old. You must update before the next kernel release.\n");
|
||||
return 1;
|
||||
}
|
||||
|
||||
|
|
|
@ -589,7 +589,7 @@ int pasemi_dma_init(void)
|
|||
pasemi_write_dma_reg(PAS_DMA_COM_RXCMD, 0);
|
||||
while (pasemi_read_dma_reg(PAS_DMA_COM_RXSTA) & 1) {
|
||||
if (time_after(jiffies, timeout)) {
|
||||
pr_warning("Warning: Could not disable RX section\n");
|
||||
pr_warn("Warning: Could not disable RX section\n");
|
||||
break;
|
||||
}
|
||||
}
|
||||
|
@ -598,7 +598,7 @@ int pasemi_dma_init(void)
|
|||
pasemi_write_dma_reg(PAS_DMA_COM_TXCMD, 0);
|
||||
while (pasemi_read_dma_reg(PAS_DMA_COM_TXSTA) & 1) {
|
||||
if (time_after(jiffies, timeout)) {
|
||||
pr_warning("Warning: Could not disable TX section\n");
|
||||
pr_warn("Warning: Could not disable TX section\n");
|
||||
break;
|
||||
}
|
||||
}
|
||||
|
|
|
@ -239,8 +239,8 @@ int opal_message_notifier_register(enum opal_msg_type msg_type,
|
|||
struct notifier_block *nb)
|
||||
{
|
||||
if (!nb || msg_type >= OPAL_MSG_TYPE_MAX) {
|
||||
pr_warning("%s: Invalid arguments, msg_type:%d\n",
|
||||
__func__, msg_type);
|
||||
pr_warn("%s: Invalid arguments, msg_type:%d\n",
|
||||
__func__, msg_type);
|
||||
return -EINVAL;
|
||||
}
|
||||
|
||||
|
@ -281,8 +281,8 @@ static void opal_handle_message(void)
|
|||
|
||||
/* check for errors. */
|
||||
if (ret) {
|
||||
pr_warning("%s: Failed to retrieve opal message, err=%lld\n",
|
||||
__func__, ret);
|
||||
pr_warn("%s: Failed to retrieve opal message, err=%lld\n",
|
||||
__func__, ret);
|
||||
return;
|
||||
}
|
||||
|
||||
|
|
|
@ -1059,8 +1059,8 @@ static struct pnv_ioda_pe *pnv_ioda_setup_dev_PE(struct pci_dev *dev)
|
|||
|
||||
pe = pnv_ioda_alloc_pe(phb);
|
||||
if (!pe) {
|
||||
pr_warning("%s: Not enough PE# available, disabling device\n",
|
||||
pci_name(dev));
|
||||
pr_warn("%s: Not enough PE# available, disabling device\n",
|
||||
pci_name(dev));
|
||||
return NULL;
|
||||
}
|
||||
|
||||
|
@ -1164,7 +1164,7 @@ static struct pnv_ioda_pe *pnv_ioda_setup_bus_PE(struct pci_bus *bus, bool all)
|
|||
pe = pnv_ioda_alloc_pe(phb);
|
||||
|
||||
if (!pe) {
|
||||
pr_warning("%s: Not enough PE# available for PCI bus %04x:%02x\n",
|
||||
pr_warn("%s: Not enough PE# available for PCI bus %04x:%02x\n",
|
||||
__func__, pci_domain_nr(bus), bus->number);
|
||||
return NULL;
|
||||
}
|
||||
|
@ -3293,7 +3293,7 @@ static void pnv_pci_ioda_create_dbgfs(void)
|
|||
sprintf(name, "PCI%04x", hose->global_number);
|
||||
phb->dbgfs = debugfs_create_dir(name, powerpc_debugfs_root);
|
||||
if (!phb->dbgfs) {
|
||||
pr_warning("%s: Error on creating debugfs on PHB#%x\n",
|
||||
pr_warn("%s: Error on creating debugfs on PHB#%x\n",
|
||||
__func__, hose->global_number);
|
||||
continue;
|
||||
}
|
||||
|
@ -4026,7 +4026,7 @@ static void __init pnv_pci_init_ioda_phb(struct device_node *np,
|
|||
/* Reset IODA tables to a clean state */
|
||||
rc = opal_pci_reset(phb_id, OPAL_RESET_PCI_IODA_TABLE, OPAL_ASSERT_RESET);
|
||||
if (rc)
|
||||
pr_warning(" OPAL Error %ld performing IODA table reset !\n", rc);
|
||||
pr_warn(" OPAL Error %ld performing IODA table reset !\n", rc);
|
||||
|
||||
/*
|
||||
* If we're running in kdump kernel, the previous kernel never
|
||||
|
|
|
@ -663,8 +663,8 @@ static void ps3_find_and_add_device(u64 bus_id, u64 dev_id)
|
|||
if (rem)
|
||||
break;
|
||||
}
|
||||
pr_warning("%s:%u: device %llu:%llu not found\n", __func__, __LINE__,
|
||||
bus_id, dev_id);
|
||||
pr_warn("%s:%u: device %llu:%llu not found\n",
|
||||
__func__, __LINE__, bus_id, dev_id);
|
||||
return;
|
||||
|
||||
found:
|
||||
|
@ -859,11 +859,9 @@ static int ps3_probe_thread(void *data)
|
|||
|
||||
if (notify_event->event_type != notify_region_probe ||
|
||||
notify_event->bus_id != dev.sbd.bus_id) {
|
||||
pr_warning("%s:%u: bad notify_event: event %llu, "
|
||||
"dev_id %llu, dev_type %llu\n",
|
||||
__func__, __LINE__, notify_event->event_type,
|
||||
notify_event->dev_id,
|
||||
notify_event->dev_type);
|
||||
pr_warn("%s:%u: bad notify_event: event %llu, dev_id %llu, dev_type %llu\n",
|
||||
__func__, __LINE__, notify_event->event_type,
|
||||
notify_event->dev_id, notify_event->dev_type);
|
||||
continue;
|
||||
}
|
||||
|
||||
|
|
|
@ -607,8 +607,8 @@ static int dma_ioc0_map_pages(struct ps3_dma_region *r, unsigned long phys_addr,
|
|||
r->ioid,
|
||||
iopte_flag);
|
||||
if (result) {
|
||||
pr_warning("%s:%d: lv1_put_iopte failed: %s\n",
|
||||
__func__, __LINE__, ps3_result(result));
|
||||
pr_warn("%s:%d: lv1_put_iopte failed: %s\n",
|
||||
__func__, __LINE__, ps3_result(result));
|
||||
goto fail_map;
|
||||
}
|
||||
DBG("%s: pg=%d bus=%#lx, lpar=%#lx, ioid=%#x\n", __func__,
|
||||
|
|
|
@ -699,7 +699,7 @@ static void os_area_queue_work_handler(struct work_struct *work)
|
|||
|
||||
error = update_flash_db();
|
||||
if (error)
|
||||
pr_warning("%s: Could not update FLASH ROM\n", __func__);
|
||||
pr_warn("%s: Could not update FLASH ROM\n", __func__);
|
||||
|
||||
pr_debug(" <- %s:%d\n", __func__, __LINE__);
|
||||
}
|
||||
|
|
|
@ -816,15 +816,15 @@ static void remove_ddw(struct device_node *np, bool remove_prop)
|
|||
ret = tce_clearrange_multi_pSeriesLP(0,
|
||||
1ULL << (be32_to_cpu(dwp->window_shift) - PAGE_SHIFT), dwp);
|
||||
if (ret)
|
||||
pr_warning("%pOF failed to clear tces in window.\n",
|
||||
np);
|
||||
pr_warn("%pOF failed to clear tces in window.\n",
|
||||
np);
|
||||
else
|
||||
pr_debug("%pOF successfully cleared tces in window.\n",
|
||||
np);
|
||||
|
||||
ret = rtas_call(ddw_avail[2], 1, 1, NULL, liobn);
|
||||
if (ret)
|
||||
pr_warning("%pOF: failed to remove direct window: rtas returned "
|
||||
pr_warn("%pOF: failed to remove direct window: rtas returned "
|
||||
"%d to ibm,remove-pe-dma-window(%x) %llx\n",
|
||||
np, ret, ddw_avail[2], liobn);
|
||||
else
|
||||
|
@ -836,7 +836,7 @@ static void remove_ddw(struct device_node *np, bool remove_prop)
|
|||
if (remove_prop)
|
||||
ret = of_remove_property(np, win64);
|
||||
if (ret)
|
||||
pr_warning("%pOF: failed to remove direct window property: %d\n",
|
||||
pr_warn("%pOF: failed to remove direct window property: %d\n",
|
||||
np, ret);
|
||||
}
|
||||
|
||||
|
|
|
@ -371,8 +371,8 @@ void pseries_disable_reloc_on_exc(void)
|
|||
mdelay(get_longbusy_msecs(rc));
|
||||
}
|
||||
if (rc != H_SUCCESS)
|
||||
pr_warning("Warning: Failed to disable relocation on "
|
||||
"exceptions: %ld\n", rc);
|
||||
pr_warn("Warning: Failed to disable relocation on exceptions: %ld\n",
|
||||
rc);
|
||||
}
|
||||
EXPORT_SYMBOL(pseries_disable_reloc_on_exc);
|
||||
|
||||
|
|
|
@ -448,7 +448,7 @@ static void setup_pci_atmu(struct pci_controller *hose)
|
|||
#endif
|
||||
/* adjusting outbound windows could reclaim space in mem map */
|
||||
if (paddr_hi < 0xffffffffull)
|
||||
pr_warning("%pOF: WARNING: Outbound window cfg leaves "
|
||||
pr_warn("%pOF: WARNING: Outbound window cfg leaves "
|
||||
"gaps in memory map. Adjusting the memory map "
|
||||
"could reduce unnecessary bounce buffering.\n",
|
||||
hose->dn);
|
||||
|
@ -531,7 +531,7 @@ int fsl_add_bridge(struct platform_device *pdev, int is_primary)
|
|||
dev = pdev->dev.of_node;
|
||||
|
||||
if (!of_device_is_available(dev)) {
|
||||
pr_warning("%pOF: disabled\n", dev);
|
||||
pr_warn("%pOF: disabled\n", dev);
|
||||
return -ENODEV;
|
||||
}
|
||||
|
||||
|
@ -808,8 +808,8 @@ int __init mpc83xx_add_bridge(struct device_node *dev)
|
|||
is_mpc83xx_pci = 1;
|
||||
|
||||
if (!of_device_is_available(dev)) {
|
||||
pr_warning("%pOF: disabled by the firmware.\n",
|
||||
dev);
|
||||
pr_warn("%pOF: disabled by the firmware.\n",
|
||||
dev);
|
||||
return -ENODEV;
|
||||
}
|
||||
pr_debug("Adding PCI host bridge %pOF\n", dev);
|
||||
|
|
|
@ -1008,9 +1008,8 @@ static int mpic_host_map(struct irq_domain *h, unsigned int virq,
|
|||
if (hw == mpic->spurious_vec)
|
||||
return -EINVAL;
|
||||
if (mpic->protected && test_bit(hw, mpic->protected)) {
|
||||
pr_warning("mpic: Mapping of source 0x%x failed, "
|
||||
"source protected by firmware !\n",\
|
||||
(unsigned int)hw);
|
||||
pr_warn("mpic: Mapping of source 0x%x failed, source protected by firmware !\n",
|
||||
(unsigned int)hw);
|
||||
return -EPERM;
|
||||
}
|
||||
|
||||
|
@ -1040,9 +1039,8 @@ static int mpic_host_map(struct irq_domain *h, unsigned int virq,
|
|||
return 0;
|
||||
|
||||
if (hw >= mpic->num_sources) {
|
||||
pr_warning("mpic: Mapping of source 0x%x failed, "
|
||||
"source out of range !\n",\
|
||||
(unsigned int)hw);
|
||||
pr_warn("mpic: Mapping of source 0x%x failed, source out of range !\n",
|
||||
(unsigned int)hw);
|
||||
return -EINVAL;
|
||||
}
|
||||
|
||||
|
|
|
@ -241,18 +241,16 @@ static int __init icp_native_map_one_cpu(int hw_id, unsigned long addr,
|
|||
cpu, hw_id);
|
||||
|
||||
if (!request_mem_region(addr, size, rname)) {
|
||||
pr_warning("icp_native: Could not reserve ICP MMIO"
|
||||
" for CPU %d, interrupt server #0x%x\n",
|
||||
cpu, hw_id);
|
||||
pr_warn("icp_native: Could not reserve ICP MMIO for CPU %d, interrupt server #0x%x\n",
|
||||
cpu, hw_id);
|
||||
return -EBUSY;
|
||||
}
|
||||
|
||||
icp_native_regs[cpu] = ioremap(addr, size);
|
||||
kvmppc_set_xics_phys(cpu, addr);
|
||||
if (!icp_native_regs[cpu]) {
|
||||
pr_warning("icp_native: Failed ioremap for CPU %d, "
|
||||
"interrupt server #0x%x, addr %#lx\n",
|
||||
cpu, hw_id, addr);
|
||||
pr_warn("icp_native: Failed ioremap for CPU %d, interrupt server #0x%x, addr %#lx\n",
|
||||
cpu, hw_id, addr);
|
||||
release_mem_region(addr, size);
|
||||
return -ENOMEM;
|
||||
}
|
||||
|
|
|
@ -131,8 +131,8 @@ static int ics_opal_set_affinity(struct irq_data *d,
|
|||
|
||||
wanted_server = xics_get_irq_server(d->irq, cpumask, 1);
|
||||
if (wanted_server < 0) {
|
||||
pr_warning("%s: No online cpus in the mask %*pb for irq %d\n",
|
||||
__func__, cpumask_pr_args(cpumask), d->irq);
|
||||
pr_warn("%s: No online cpus in the mask %*pb for irq %d\n",
|
||||
__func__, cpumask_pr_args(cpumask), d->irq);
|
||||
return -1;
|
||||
}
|
||||
server = ics_opal_mangle_server(wanted_server);
|
||||
|
|
|
@ -141,8 +141,8 @@ static int ics_rtas_set_affinity(struct irq_data *d,
|
|||
|
||||
irq_server = xics_get_irq_server(d->irq, cpumask, 1);
|
||||
if (irq_server == -1) {
|
||||
pr_warning("%s: No online cpus in the mask %*pb for irq %d\n",
|
||||
__func__, cpumask_pr_args(cpumask), d->irq);
|
||||
pr_warn("%s: No online cpus in the mask %*pb for irq %d\n",
|
||||
__func__, cpumask_pr_args(cpumask), d->irq);
|
||||
return -1;
|
||||
}
|
||||
|
||||
|
|
|
@ -243,8 +243,8 @@ void xics_migrate_irqs_away(void)
|
|||
|
||||
/* This is expected during cpu offline. */
|
||||
if (cpu_online(cpu))
|
||||
pr_warning("IRQ %u affinity broken off cpu %u\n",
|
||||
virq, cpu);
|
||||
pr_warn("IRQ %u affinity broken off cpu %u\n",
|
||||
virq, cpu);
|
||||
|
||||
/* Reset affinity to all cpus */
|
||||
raw_spin_unlock_irqrestore(&desc->lock, flags);
|
||||
|
@ -466,7 +466,7 @@ void __init xics_init(void)
|
|||
rc = icp_opal_init();
|
||||
}
|
||||
if (rc < 0) {
|
||||
pr_warning("XICS: Cannot find a Presentation Controller !\n");
|
||||
pr_warn("XICS: Cannot find a Presentation Controller !\n");
|
||||
return;
|
||||
}
|
||||
|
||||
|
@ -481,7 +481,7 @@ void __init xics_init(void)
|
|||
if (rc < 0)
|
||||
rc = ics_opal_init();
|
||||
if (rc < 0)
|
||||
pr_warning("XICS: Cannot find a Source Controller !\n");
|
||||
pr_warn("XICS: Cannot find a Source Controller !\n");
|
||||
|
||||
/* Initialize common bits */
|
||||
xics_get_server_size();
|
||||
|
|
Loading…
Reference in New Issue