Commit 02b1bae5 authored by David S. Miller's avatar David S. Miller

niu: Refinements to kernel logging.

Fix unconditional empty kerne log message every interrupt.

Kill some informational log messages that are superfluous
and anyways occur before the netdev is registered.
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 228da6c2
...@@ -3204,9 +3204,6 @@ static int fflp_early_init(struct niu *np) ...@@ -3204,9 +3204,6 @@ static int fflp_early_init(struct niu *np)
parent = np->parent; parent = np->parent;
err = 0; err = 0;
if (!(parent->flags & PARENT_FLGS_CLS_HWINIT)) { if (!(parent->flags & PARENT_FLGS_CLS_HWINIT)) {
netif_printk(np, probe, KERN_DEBUG, np->dev,
"%s() Initting hw on port %u\n",
__func__, np->port);
if (np->parent->plat_type != PLAT_TYPE_NIU) { if (np->parent->plat_type != PLAT_TYPE_NIU) {
fflp_reset(np); fflp_reset(np);
fflp_set_timings(np); fflp_set_timings(np);
...@@ -3248,8 +3245,6 @@ static int fflp_early_init(struct niu *np) ...@@ -3248,8 +3245,6 @@ static int fflp_early_init(struct niu *np)
vlan_tbl_clear(np); vlan_tbl_clear(np);
netif_printk(np, probe, KERN_DEBUG, np->dev,
"%s() Success\n", __func__);
parent->flags |= PARENT_FLGS_CLS_HWINIT; parent->flags |= PARENT_FLGS_CLS_HWINIT;
} }
out: out:
...@@ -4246,13 +4241,11 @@ static irqreturn_t niu_interrupt(int irq, void *dev_id) ...@@ -4246,13 +4241,11 @@ static irqreturn_t niu_interrupt(int irq, void *dev_id)
v2 = nr64(LDSV2(ldg)); v2 = nr64(LDSV2(ldg));
if (netif_msg_intr(np)) if (netif_msg_intr(np))
pr_cont(" v0[%llx] v1[%llx] v2[%llx]", pr_cont(" v0[%llx] v1[%llx] v2[%llx]\n",
(unsigned long long) v0, (unsigned long long) v0,
(unsigned long long) v1, (unsigned long long) v1,
(unsigned long long) v2); (unsigned long long) v2);
pr_cont("\n");
if (unlikely(!v0 && !v1 && !v2)) { if (unlikely(!v0 && !v1 && !v2)) {
spin_unlock_irqrestore(&np->lock, flags); spin_unlock_irqrestore(&np->lock, flags);
return IRQ_NONE; return IRQ_NONE;
...@@ -8577,9 +8570,6 @@ static int __devinit niu_get_and_validate_port(struct niu *np) ...@@ -8577,9 +8570,6 @@ static int __devinit niu_get_and_validate_port(struct niu *np)
} }
} }
netif_printk(np, probe, KERN_DEBUG, np->dev,
"%s() port[%d] num_ports[%d]\n",
__func__, np->port, parent->num_ports);
if (np->port >= parent->num_ports) if (np->port >= parent->num_ports)
return -ENODEV; return -ENODEV;
...@@ -8965,9 +8955,6 @@ static int __devinit niu_probe_ports(struct niu *np) ...@@ -8965,9 +8955,6 @@ static int __devinit niu_probe_ports(struct niu *np)
struct niu_parent *parent = np->parent; struct niu_parent *parent = np->parent;
int err, i; int err, i;
netif_printk(np, probe, KERN_DEBUG, np->dev,
"%s() port_phy[%08x]\n", __func__, parent->port_phy);
if (parent->port_phy == PORT_PHY_UNKNOWN) { if (parent->port_phy == PORT_PHY_UNKNOWN) {
err = walk_phys(np, parent); err = walk_phys(np, parent);
if (err) if (err)
...@@ -8988,10 +8975,6 @@ static int __devinit niu_classifier_swstate_init(struct niu *np) ...@@ -8988,10 +8975,6 @@ static int __devinit niu_classifier_swstate_init(struct niu *np)
{ {
struct niu_classifier *cp = &np->clas; struct niu_classifier *cp = &np->clas;
netif_printk(np, probe, KERN_DEBUG, np->dev,
"%s() num_tcam(%d)\n",
__func__, np->parent->tcam_num_entries);
cp->tcam_top = (u16) np->port; cp->tcam_top = (u16) np->port;
cp->tcam_sz = np->parent->tcam_num_entries / np->parent->num_ports; cp->tcam_sz = np->parent->tcam_num_entries / np->parent->num_ports;
cp->h1_init = 0xffffffff; cp->h1_init = 0xffffffff;
...@@ -9506,9 +9489,6 @@ static struct niu_parent * __devinit niu_new_parent(struct niu *np, ...@@ -9506,9 +9489,6 @@ static struct niu_parent * __devinit niu_new_parent(struct niu *np,
struct niu_parent *p; struct niu_parent *p;
int i; int i;
netif_printk(np, probe, KERN_DEBUG, np->dev,
"%s() Creating new parent\n", __func__);
plat_dev = platform_device_register_simple("niu", niu_parent_index, plat_dev = platform_device_register_simple("niu", niu_parent_index,
NULL, 0); NULL, 0);
if (IS_ERR(plat_dev)) if (IS_ERR(plat_dev))
...@@ -9573,10 +9553,6 @@ static struct niu_parent * __devinit niu_get_parent(struct niu *np, ...@@ -9573,10 +9553,6 @@ static struct niu_parent * __devinit niu_get_parent(struct niu *np,
struct niu_parent *p, *tmp; struct niu_parent *p, *tmp;
int port = np->port; int port = np->port;
netif_printk(np, probe, KERN_DEBUG, np->dev,
"%s() platform_type[%u] port[%u]\n",
__func__, ptype, port);
mutex_lock(&niu_parent_lock); mutex_lock(&niu_parent_lock);
p = NULL; p = NULL;
list_for_each_entry(tmp, &niu_parent_list, list) { list_for_each_entry(tmp, &niu_parent_list, list) {
......
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