[PATCH 20/49] drivers/staging/octeon: Convert printks to pr_<level>
Joe Perches
joe at perches.com
Wed Oct 14 18:54:04 PDT 2009
Signed-off-by: Joe Perches <joe at perches.com>
---
drivers/staging/octeon/cvmx-pow.h | 55 ++++++---------
drivers/staging/octeon/ethernet-mem.c | 18 ++---
drivers/staging/octeon/ethernet-spi.c | 115 ++++++++++++--------------------
drivers/staging/octeon/ethernet.c | 21 ++----
4 files changed, 80 insertions(+), 129 deletions(-)
diff --git a/drivers/staging/octeon/cvmx-pow.h b/drivers/staging/octeon/cvmx-pow.h
index c5d66f2..d87e149 100644
--- a/drivers/staging/octeon/cvmx-pow.h
+++ b/drivers/staging/octeon/cvmx-pow.h
@@ -1084,8 +1084,7 @@ static inline void cvmx_pow_tag_sw_wait(void)
if (unlikely(switch_complete))
break;
if (unlikely(cvmx_get_cycle() > start_cycle + MAX_CYCLES)) {
- pr_warning("Tag switch is taking a long time, "
- "possible deadlock\n");
+ pr_warning("Tag switch is taking a long time, possible deadlock\n");
start_cycle = -MAX_CYCLES - 1;
}
}
@@ -1302,13 +1301,11 @@ static inline void cvmx_pow_tag_sw_nocheck(uint32_t tag,
pr_warning("%s called with NULL tag\n", __func__);
if ((current_tag.s.type == tag_type)
&& (current_tag.s.tag == tag))
- pr_warning("%s called to perform a tag switch to the "
- "same tag\n",
- __func__);
+ pr_warning("%s called to perform a tag switch to the same tag\n",
+ __func__);
if (tag_type == CVMX_POW_TAG_TYPE_NULL)
- pr_warning("%s called to perform a tag switch to "
- "NULL. Use cvmx_pow_tag_sw_null() instead\n",
- __func__);
+ pr_warning("%s called to perform a tag switch to NULL. Use cvmx_pow_tag_sw_null() instead\n",
+ __func__);
}
/*
@@ -1411,19 +1408,16 @@ static inline void cvmx_pow_tag_sw_full_nocheck(cvmx_wqe_t *wqp, uint32_t tag,
__func__);
if ((current_tag.s.type == tag_type)
&& (current_tag.s.tag == tag))
- pr_warning("%s called to perform a tag switch to "
- "the same tag\n",
- __func__);
+ pr_warning("%s called to perform a tag switch to the same tag\n",
+ __func__);
if (tag_type == CVMX_POW_TAG_TYPE_NULL)
- pr_warning("%s called to perform a tag switch to "
- "NULL. Use cvmx_pow_tag_sw_null() instead\n",
- __func__);
+ pr_warning("%s called to perform a tag switch to NULL. Use cvmx_pow_tag_sw_null() instead\n",
+ __func__);
if (wqp != cvmx_phys_to_ptr(0x80))
if (wqp != cvmx_pow_get_current_wqp())
- pr_warning("%s passed WQE(%p) doesn't match "
- "the address in the POW(%p)\n",
- __func__, wqp,
- cvmx_pow_get_current_wqp());
+ pr_warning("%s passed WQE(%p) doesn't match the address in the POW(%p)\n",
+ __func__, wqp,
+ cvmx_pow_get_current_wqp());
}
/*
@@ -1510,9 +1504,8 @@ static inline void cvmx_pow_tag_sw_null_nocheck(void)
pr_warning("%s called with NULL_NULL tag\n",
__func__);
if (current_tag.s.type == CVMX_POW_TAG_TYPE_NULL)
- pr_warning("%s called when we already have a "
- "NULL tag\n",
- __func__);
+ pr_warning("%s called when we already have a NULL tag\n",
+ __func__);
}
tag_req.u64 = 0;
@@ -1659,9 +1652,8 @@ static inline void cvmx_pow_set_priority(uint64_t core_num,
prio_mask |= 1 << priority[i];
if (prio_mask ^ ((1 << cvmx_pop(prio_mask)) - 1)) {
- pr_err("POW static priorities should be "
- "contiguous (0x%llx)\n",
- (unsigned long long)prio_mask);
+ pr_err("POW static priorities should be contiguous (0x%llx)\n",
+ (unsigned long long)prio_mask);
return;
}
}
@@ -1728,14 +1720,12 @@ static inline void cvmx_pow_tag_sw_desched_nocheck(
pr_warning("%s called with NULL_NULL tag\n",
__func__);
if (current_tag.s.type == CVMX_POW_TAG_TYPE_NULL)
- pr_warning("%s called with NULL tag. Deschedule not "
- "allowed from NULL state\n",
- __func__);
+ pr_warning("%s called with NULL tag. Deschedule not allowed from NULL state\n",
+ __func__);
if ((current_tag.s.type != CVMX_POW_TAG_TYPE_ATOMIC)
&& (tag_type != CVMX_POW_TAG_TYPE_ATOMIC))
- pr_warning("%s called where neither the before or "
- "after tag is ATOMIC\n",
- __func__);
+ pr_warning("%s called where neither the before or after tag is ATOMIC\n",
+ __func__);
}
tag_req.u64 = 0;
@@ -1835,9 +1825,8 @@ static inline void cvmx_pow_desched(uint64_t no_sched)
pr_warning("%s called with NULL_NULL tag\n",
__func__);
if (current_tag.s.type == CVMX_POW_TAG_TYPE_NULL)
- pr_warning("%s called with NULL tag. Deschedule not "
- "expected from NULL state\n",
- __func__);
+ pr_warning("%s called with NULL tag. Deschedule not expected from NULL state\n",
+ __func__);
}
/* Need to make sure any writes to the work queue entry are complete */
diff --git a/drivers/staging/octeon/ethernet-mem.c b/drivers/staging/octeon/ethernet-mem.c
index b595903..ea5b444 100644
--- a/drivers/staging/octeon/ethernet-mem.c
+++ b/drivers/staging/octeon/ethernet-mem.c
@@ -120,8 +120,7 @@ static int cvm_oct_fill_hw_memory(int pool, int size, int elements)
panic("Unable to allocate %u bytes for FPA pool %d\n",
elements * size, pool);
- pr_notice("Memory range %p - %p reserved for "
- "hardware\n", memory,
+ pr_notice("Memory range %p - %p reserved for hardware\n", memory,
memory + elements * size - 1);
while (freed) {
@@ -134,9 +133,8 @@ static int cvm_oct_fill_hw_memory(int pool, int size, int elements)
/* We need to force alignment to 128 bytes here */
memory = kmalloc(size + 127, GFP_ATOMIC);
if (unlikely(memory == NULL)) {
- pr_warning("Unable to allocate %u bytes for "
- "FPA pool %d\n",
- elements * size, pool);
+ pr_warning("Unable to allocate %u bytes for FPA pool %d\n",
+ elements * size, pool);
break;
}
memory = (char *)(((unsigned long)memory + 127) & -128);
@@ -169,13 +167,11 @@ static void cvm_oct_free_hw_memory(int pool, int size, int elements)
} while (memory);
if (elements < 0)
- pr_warning("Freeing of pool %u had too many "
- "buffers (%d)\n",
- pool, elements);
+ pr_warning("Freeing of pool %u had too many buffers (%d)\n",
+ pool, elements);
else if (elements > 0)
- pr_warning("Warning: Freeing of pool %u is "
- "missing %d buffers\n",
- pool, elements);
+ pr_warning("Warning: Freeing of pool %u is missing %d buffers\n",
+ pool, elements);
}
}
diff --git a/drivers/staging/octeon/ethernet-spi.c b/drivers/staging/octeon/ethernet-spi.c
index 66190b0..e8db8f0 100644
--- a/drivers/staging/octeon/ethernet-spi.c
+++ b/drivers/staging/octeon/ethernet-spi.c
@@ -24,6 +24,9 @@
* This file may also be available under a different license from Cavium.
* Contact Cavium Networks for more information
**********************************************************************/
+
+#define pr_fmt(fmt) "SPI: " fmt
+
#include <linux/kernel.h>
#include <linux/netdevice.h>
#include <linux/mii.h>
@@ -62,35 +65,27 @@ static irqreturn_t cvm_oct_spi_rml_interrupt(int cpl, void *dev_id)
spx_int_reg.u64 &= cvmx_read_csr(CVMX_SPXX_INT_MSK(1));
if (spx_int_reg.s.spf)
- pr_err("SPI1: SRX Spi4 interface down\n");
+ pr_err("%d: SRX Spi4 interface down\n", 1);
if (spx_int_reg.s.calerr)
- pr_err("SPI1: SRX Spi4 Calendar table "
- "parity error\n");
+ pr_err("%d: SRX Spi4 Calendar table parity error\n", 1);
if (spx_int_reg.s.syncerr)
- pr_err("SPI1: SRX Consecutive Spi4 DIP4 "
- "errors have exceeded "
- "SPX_ERR_CTL[ERRCNT]\n");
+ pr_err("%d: SRX Consecutive Spi4 DIP4 errors have exceeded SPX_ERR_CTL[ERRCNT]\n", 1);
if (spx_int_reg.s.diperr)
- pr_err("SPI1: SRX Spi4 DIP4 error\n");
+ pr_err("%d: SRX Spi4 DIP4 error\n", 1);
if (spx_int_reg.s.tpaovr)
- pr_err("SPI1: SRX Selected port has hit "
- "TPA overflow\n");
+ pr_err("%d: SRX Selected port has hit TPA overflow\n", 1);
if (spx_int_reg.s.rsverr)
- pr_err("SPI1: SRX Spi4 reserved control "
- "word detected\n");
+ pr_err("%d: SRX Spi4 reserved control word detected\n", 1);
if (spx_int_reg.s.drwnng)
- pr_err("SPI1: SRX Spi4 receive FIFO "
- "drowning/overflow\n");
+ pr_err("%d: SRX Spi4 receive FIFO drowning/overflow\n", 1);
if (spx_int_reg.s.clserr)
- pr_err("SPI1: SRX Spi4 packet closed on "
- "non-16B alignment without EOP\n");
+ pr_err("%d: SRX Spi4 packet closed on non-16B alignment without EOP\n", 1);
if (spx_int_reg.s.spiovr)
- pr_err("SPI1: SRX Spi4 async FIFO overflow\n");
+ pr_err("%d: SRX Spi4 async FIFO overflow\n", 1);
if (spx_int_reg.s.abnorm)
- pr_err("SPI1: SRX Abnormal packet "
- "termination (ERR bit)\n");
+ pr_err("%d: SRX Abnormal packet termination (ERR bit)\n", 1);
if (spx_int_reg.s.prtnxa)
- pr_err("SPI1: SRX Port out of range\n");
+ pr_err("%d: SRX Port out of range\n", 1);
}
stx_int_reg.u64 = cvmx_read_csr(CVMX_STXX_INT_REG(1));
@@ -99,31 +94,23 @@ static irqreturn_t cvm_oct_spi_rml_interrupt(int cpl, void *dev_id)
stx_int_reg.u64 &= cvmx_read_csr(CVMX_STXX_INT_MSK(1));
if (stx_int_reg.s.syncerr)
- pr_err("SPI1: STX Interface encountered a "
- "fatal error\n");
+ pr_err("%d: STX Interface encountered a fatal error\n", 1);
if (stx_int_reg.s.frmerr)
- pr_err("SPI1: STX FRMCNT has exceeded "
- "STX_DIP_CNT[MAXFRM]\n");
+ pr_err("%d: STX FRMCNT has exceeded STX_DIP_CNT[MAXFRM]\n", 1);
if (stx_int_reg.s.unxfrm)
- pr_err("SPI1: STX Unexpected framing "
- "sequence\n");
+ pr_err("%d: STX Unexpected framing sequence\n", 1);
if (stx_int_reg.s.nosync)
- pr_err("SPI1: STX ERRCNT has exceeded "
- "STX_DIP_CNT[MAXDIP]\n");
+ pr_err("%d: STX ERRCNT has exceeded STX_DIP_CNT[MAXDIP]\n", 1);
if (stx_int_reg.s.diperr)
- pr_err("SPI1: STX DIP2 error on the Spi4 "
- "Status channel\n");
+ pr_err("%d: STX DIP2 error on the Spi4 Status channel\n", 1);
if (stx_int_reg.s.datovr)
- pr_err("SPI1: STX Spi4 FIFO overflow error\n");
+ pr_err("%d: STX Spi4 FIFO overflow error\n", 1);
if (stx_int_reg.s.ovrbst)
- pr_err("SPI1: STX Transmit packet burst "
- "too big\n");
+ pr_err("%d: STX Transmit packet burst too big\n", 1);
if (stx_int_reg.s.calpar1)
- pr_err("SPI1: STX Calendar Table Parity "
- "Error Bank1\n");
+ pr_err("%d: STX Calendar Table Parity Error Bank1\n", 1);
if (stx_int_reg.s.calpar0)
- pr_err("SPI1: STX Calendar Table Parity "
- "Error Bank0\n");
+ pr_err("%d: STX Calendar Table Parity Error Bank0\n", 1);
}
cvmx_write_csr(CVMX_SPXX_INT_MSK(1), 0);
@@ -142,35 +129,27 @@ static irqreturn_t cvm_oct_spi_rml_interrupt(int cpl, void *dev_id)
spx_int_reg.u64 &= cvmx_read_csr(CVMX_SPXX_INT_MSK(0));
if (spx_int_reg.s.spf)
- pr_err("SPI0: SRX Spi4 interface down\n");
+ pr_err("%d: SRX Spi4 interface down\n", 0);
if (spx_int_reg.s.calerr)
- pr_err("SPI0: SRX Spi4 Calendar table "
- "parity error\n");
+ pr_err("%d: SRX Spi4 Calendar table parity error\n", 0);
if (spx_int_reg.s.syncerr)
- pr_err("SPI0: SRX Consecutive Spi4 DIP4 "
- "errors have exceeded "
- "SPX_ERR_CTL[ERRCNT]\n");
+ pr_err("%d: SRX Consecutive Spi4 DIP4 errors have exceeded SPX_ERR_CTL[ERRCNT]\n", 0);
if (spx_int_reg.s.diperr)
- pr_err("SPI0: SRX Spi4 DIP4 error\n");
+ pr_err("%d: SRX Spi4 DIP4 error\n", 0);
if (spx_int_reg.s.tpaovr)
- pr_err("SPI0: SRX Selected port has hit "
- "TPA overflow\n");
+ pr_err("%d: SRX Selected port has hit TPA overflow\n", 0);
if (spx_int_reg.s.rsverr)
- pr_err("SPI0: SRX Spi4 reserved control "
- "word detected\n");
+ pr_err("%d: SRX Spi4 reserved control word detected\n", 0);
if (spx_int_reg.s.drwnng)
- pr_err("SPI0: SRX Spi4 receive FIFO "
- "drowning/overflow\n");
+ pr_err("%d: SRX Spi4 receive FIFO drowning/overflow\n", 0);
if (spx_int_reg.s.clserr)
- pr_err("SPI0: SRX Spi4 packet closed on "
- "non-16B alignment without EOP\n");
+ pr_err("%d: SRX Spi4 packet closed on non-16B alignment without EOP\n", 0);
if (spx_int_reg.s.spiovr)
- pr_err("SPI0: SRX Spi4 async FIFO overflow\n");
+ pr_err("%d: SRX Spi4 async FIFO overflow\n", 0);
if (spx_int_reg.s.abnorm)
- pr_err("SPI0: SRX Abnormal packet "
- "termination (ERR bit)\n");
+ pr_err("%d: SRX Abnormal packet termination (ERR bit)\n", 0);
if (spx_int_reg.s.prtnxa)
- pr_err("SPI0: SRX Port out of range\n");
+ pr_err("%d: SRX Port out of range\n", 0);
}
stx_int_reg.u64 = cvmx_read_csr(CVMX_STXX_INT_REG(0));
@@ -179,31 +158,23 @@ static irqreturn_t cvm_oct_spi_rml_interrupt(int cpl, void *dev_id)
stx_int_reg.u64 &= cvmx_read_csr(CVMX_STXX_INT_MSK(0));
if (stx_int_reg.s.syncerr)
- pr_err("SPI0: STX Interface encountered a "
- "fatal error\n");
+ pr_err("%d: STX Interface encountered a fatal error\n", 0);
if (stx_int_reg.s.frmerr)
- pr_err("SPI0: STX FRMCNT has exceeded "
- "STX_DIP_CNT[MAXFRM]\n");
+ pr_err("%d: STX FRMCNT has exceeded STX_DIP_CNT[MAXFRM]\n", 0);
if (stx_int_reg.s.unxfrm)
- pr_err("SPI0: STX Unexpected framing "
- "sequence\n");
+ pr_err("%d: STX Unexpected framing sequence\n", 0);
if (stx_int_reg.s.nosync)
- pr_err("SPI0: STX ERRCNT has exceeded "
- "STX_DIP_CNT[MAXDIP]\n");
+ pr_err("%d: STX ERRCNT has exceeded STX_DIP_CNT[MAXDIP]\n", 0);
if (stx_int_reg.s.diperr)
- pr_err("SPI0: STX DIP2 error on the Spi4 "
- "Status channel\n");
+ pr_err("%d: STX DIP2 error on the Spi4 Status channel\n", 0);
if (stx_int_reg.s.datovr)
- pr_err("SPI0: STX Spi4 FIFO overflow error\n");
+ pr_err("%d: STX Spi4 FIFO overflow error\n", 0);
if (stx_int_reg.s.ovrbst)
- pr_err("SPI0: STX Transmit packet burst "
- "too big\n");
+ pr_err("%d: STX Transmit packet burst too big\n", 0);
if (stx_int_reg.s.calpar1)
- pr_err("SPI0: STX Calendar Table Parity "
- "Error Bank1\n");
+ pr_err("%d: STX Calendar Table Parity Error Bank1\n", 0);
if (stx_int_reg.s.calpar0)
- pr_err("SPI0: STX Calendar Table Parity "
- "Error Bank0\n");
+ pr_err("%d: STX Calendar Table Parity Error Bank0\n", 0);
}
cvmx_write_csr(CVMX_SPXX_INT_MSK(0), 0);
diff --git a/drivers/staging/octeon/ethernet.c b/drivers/staging/octeon/ethernet.c
index b847951..228caa6 100644
--- a/drivers/staging/octeon/ethernet.c
+++ b/drivers/staging/octeon/ethernet.c
@@ -672,19 +672,16 @@ static int __init cvm_oct_init_module(void)
skb_queue_head_init(&priv->tx_free_list[qos]);
if (register_netdev(dev) < 0) {
- pr_err("Failed to register ethernet "
- "device for POW\n");
+ pr_err("Failed to register ethernet device for POW\n");
kfree(dev);
} else {
cvm_oct_device[CVMX_PIP_NUM_INPUT_PORTS] = dev;
- pr_info("%s: POW send group %d, receive "
- "group %d\n",
- dev->name, pow_send_group,
- pow_receive_group);
+ pr_info("%s: POW send group %d, receive group %d\n",
+ dev->name, pow_send_group,
+ pow_receive_group);
}
} else {
- pr_err("Failed to allocate ethernet device "
- "for POW\n");
+ pr_err("Failed to allocate ethernet device for POW\n");
}
}
@@ -702,8 +699,7 @@ static int __init cvm_oct_init_module(void)
struct net_device *dev =
alloc_etherdev(sizeof(struct octeon_ethernet));
if (!dev) {
- pr_err("Failed to allocate ethernet device "
- "for port %d\n", port);
+ pr_err("Failed to allocate ethernet device for port %d\n", port);
continue;
}
if (disable_core_queueing)
@@ -766,9 +762,8 @@ static int __init cvm_oct_init_module(void)
if (!dev->netdev_ops) {
kfree(dev);
} else if (register_netdev(dev) < 0) {
- pr_err("Failed to register ethernet device "
- "for interface %d, port %d\n",
- interface, priv->port);
+ pr_err("Failed to register ethernet device for interface %d, port %d\n",
+ interface, priv->port);
kfree(dev);
} else {
cvm_oct_device[priv->port] = dev;
--
1.6.5.rc3.dirty
More information about the devel
mailing list