Commit d1a65e21 authored by David S. Miller's avatar David S. Miller

Merge branch 'NCSI-silence-warning-messages'

Joel Stanley says:

====================
Slience NCSI logging

v2:
  Fix indent issue and commit message based on Joe's feedback
  Add Sam's acks

Here are three changes to silence unnecessary warnings in the ncsi code.

The final patch adds Sam as the maintainer for NCSI.
====================
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parents c51818d5 01a21986
...@@ -9756,6 +9756,11 @@ L: linux-scsi@vger.kernel.org ...@@ -9756,6 +9756,11 @@ L: linux-scsi@vger.kernel.org
S: Maintained S: Maintained
F: drivers/scsi/NCR_D700.* F: drivers/scsi/NCR_D700.*
NCSI LIBRARY:
M: Samuel Mendoza-Jonas <sam@mendozajonas.com>
S: Maintained
F: net/ncsi/
NCT6775 HARDWARE MONITOR DRIVER NCT6775 HARDWARE MONITOR DRIVER
M: Guenter Roeck <linux@roeck-us.net> M: Guenter Roeck <linux@roeck-us.net>
L: linux-hwmon@vger.kernel.org L: linux-hwmon@vger.kernel.org
......
...@@ -1735,8 +1735,8 @@ static void ftgmac100_ncsi_handler(struct ncsi_dev *nd) ...@@ -1735,8 +1735,8 @@ static void ftgmac100_ncsi_handler(struct ncsi_dev *nd)
if (unlikely(nd->state != ncsi_dev_state_functional)) if (unlikely(nd->state != ncsi_dev_state_functional))
return; return;
netdev_info(nd->dev, "NCSI interface %s\n", netdev_dbg(nd->dev, "NCSI interface %s\n",
nd->link_up ? "up" : "down"); nd->link_up ? "up" : "down");
} }
static void ftgmac100_setup_clk(struct ftgmac100 *priv) static void ftgmac100_setup_clk(struct ftgmac100 *priv)
......
...@@ -73,8 +73,8 @@ static int ncsi_aen_handler_lsc(struct ncsi_dev_priv *ndp, ...@@ -73,8 +73,8 @@ static int ncsi_aen_handler_lsc(struct ncsi_dev_priv *ndp,
ncm->data[2] = data; ncm->data[2] = data;
ncm->data[4] = ntohl(lsc->oem_status); ncm->data[4] = ntohl(lsc->oem_status);
netdev_info(ndp->ndev.dev, "NCSI: LSC AEN - channel %u state %s\n", netdev_dbg(ndp->ndev.dev, "NCSI: LSC AEN - channel %u state %s\n",
nc->id, data & 0x1 ? "up" : "down"); nc->id, data & 0x1 ? "up" : "down");
chained = !list_empty(&nc->link); chained = !list_empty(&nc->link);
state = nc->state; state = nc->state;
...@@ -148,9 +148,9 @@ static int ncsi_aen_handler_hncdsc(struct ncsi_dev_priv *ndp, ...@@ -148,9 +148,9 @@ static int ncsi_aen_handler_hncdsc(struct ncsi_dev_priv *ndp,
hncdsc = (struct ncsi_aen_hncdsc_pkt *)h; hncdsc = (struct ncsi_aen_hncdsc_pkt *)h;
ncm->data[3] = ntohl(hncdsc->status); ncm->data[3] = ntohl(hncdsc->status);
spin_unlock_irqrestore(&nc->lock, flags); spin_unlock_irqrestore(&nc->lock, flags);
netdev_printk(KERN_DEBUG, ndp->ndev.dev, netdev_dbg(ndp->ndev.dev,
"NCSI: host driver %srunning on channel %u\n", "NCSI: host driver %srunning on channel %u\n",
ncm->data[3] & 0x1 ? "" : "not ", nc->id); ncm->data[3] & 0x1 ? "" : "not ", nc->id);
return 0; return 0;
} }
......
...@@ -788,8 +788,8 @@ static void ncsi_configure_channel(struct ncsi_dev_priv *ndp) ...@@ -788,8 +788,8 @@ static void ncsi_configure_channel(struct ncsi_dev_priv *ndp)
} }
break; break;
case ncsi_dev_state_config_done: case ncsi_dev_state_config_done:
netdev_printk(KERN_DEBUG, ndp->ndev.dev, netdev_dbg(ndp->ndev.dev, "NCSI: channel %u config done\n",
"NCSI: channel %u config done\n", nc->id); nc->id);
spin_lock_irqsave(&nc->lock, flags); spin_lock_irqsave(&nc->lock, flags);
if (nc->reconfigure_needed) { if (nc->reconfigure_needed) {
/* This channel's configuration has been updated /* This channel's configuration has been updated
...@@ -804,8 +804,7 @@ static void ncsi_configure_channel(struct ncsi_dev_priv *ndp) ...@@ -804,8 +804,7 @@ static void ncsi_configure_channel(struct ncsi_dev_priv *ndp)
list_add_tail_rcu(&nc->link, &ndp->channel_queue); list_add_tail_rcu(&nc->link, &ndp->channel_queue);
spin_unlock_irqrestore(&ndp->lock, flags); spin_unlock_irqrestore(&ndp->lock, flags);
netdev_printk(KERN_DEBUG, dev, netdev_dbg(dev, "Dirty NCSI channel state reset\n");
"Dirty NCSI channel state reset\n");
ncsi_process_next_channel(ndp); ncsi_process_next_channel(ndp);
break; break;
} }
...@@ -816,9 +815,9 @@ static void ncsi_configure_channel(struct ncsi_dev_priv *ndp) ...@@ -816,9 +815,9 @@ static void ncsi_configure_channel(struct ncsi_dev_priv *ndp)
} else { } else {
hot_nc = NULL; hot_nc = NULL;
nc->state = NCSI_CHANNEL_INACTIVE; nc->state = NCSI_CHANNEL_INACTIVE;
netdev_warn(ndp->ndev.dev, netdev_dbg(ndp->ndev.dev,
"NCSI: channel %u link down after config\n", "NCSI: channel %u link down after config\n",
nc->id); nc->id);
} }
spin_unlock_irqrestore(&nc->lock, flags); spin_unlock_irqrestore(&nc->lock, flags);
...@@ -908,9 +907,9 @@ static int ncsi_choose_active_channel(struct ncsi_dev_priv *ndp) ...@@ -908,9 +907,9 @@ static int ncsi_choose_active_channel(struct ncsi_dev_priv *ndp)
} }
ncm = &found->modes[NCSI_MODE_LINK]; ncm = &found->modes[NCSI_MODE_LINK];
netdev_printk(KERN_DEBUG, ndp->ndev.dev, netdev_dbg(ndp->ndev.dev,
"NCSI: Channel %u added to queue (link %s)\n", "NCSI: Channel %u added to queue (link %s)\n",
found->id, ncm->data[2] & 0x1 ? "up" : "down"); found->id, ncm->data[2] & 0x1 ? "up" : "down");
out: out:
spin_lock_irqsave(&ndp->lock, flags); spin_lock_irqsave(&ndp->lock, flags);
...@@ -1199,14 +1198,14 @@ int ncsi_process_next_channel(struct ncsi_dev_priv *ndp) ...@@ -1199,14 +1198,14 @@ int ncsi_process_next_channel(struct ncsi_dev_priv *ndp)
switch (old_state) { switch (old_state) {
case NCSI_CHANNEL_INACTIVE: case NCSI_CHANNEL_INACTIVE:
ndp->ndev.state = ncsi_dev_state_config; ndp->ndev.state = ncsi_dev_state_config;
netdev_info(ndp->ndev.dev, "NCSI: configuring channel %u\n", netdev_dbg(ndp->ndev.dev, "NCSI: configuring channel %u\n",
nc->id); nc->id);
ncsi_configure_channel(ndp); ncsi_configure_channel(ndp);
break; break;
case NCSI_CHANNEL_ACTIVE: case NCSI_CHANNEL_ACTIVE:
ndp->ndev.state = ncsi_dev_state_suspend; ndp->ndev.state = ncsi_dev_state_suspend;
netdev_info(ndp->ndev.dev, "NCSI: suspending channel %u\n", netdev_dbg(ndp->ndev.dev, "NCSI: suspending channel %u\n",
nc->id); nc->id);
ncsi_suspend_channel(ndp); ncsi_suspend_channel(ndp);
break; break;
default: default:
...@@ -1226,8 +1225,6 @@ int ncsi_process_next_channel(struct ncsi_dev_priv *ndp) ...@@ -1226,8 +1225,6 @@ int ncsi_process_next_channel(struct ncsi_dev_priv *ndp)
return ncsi_choose_active_channel(ndp); return ncsi_choose_active_channel(ndp);
} }
netdev_printk(KERN_DEBUG, ndp->ndev.dev,
"NCSI: No more channels to process\n");
ncsi_report_link(ndp, false); ncsi_report_link(ndp, false);
return -ENODEV; return -ENODEV;
} }
...@@ -1318,9 +1315,9 @@ static int ncsi_kick_channels(struct ncsi_dev_priv *ndp) ...@@ -1318,9 +1315,9 @@ static int ncsi_kick_channels(struct ncsi_dev_priv *ndp)
if ((ndp->ndev.state & 0xff00) == if ((ndp->ndev.state & 0xff00) ==
ncsi_dev_state_config || ncsi_dev_state_config ||
!list_empty(&nc->link)) { !list_empty(&nc->link)) {
netdev_printk(KERN_DEBUG, nd->dev, netdev_dbg(nd->dev,
"NCSI: channel %p marked dirty\n", "NCSI: channel %p marked dirty\n",
nc); nc);
nc->reconfigure_needed = true; nc->reconfigure_needed = true;
} }
spin_unlock_irqrestore(&nc->lock, flags); spin_unlock_irqrestore(&nc->lock, flags);
...@@ -1338,8 +1335,7 @@ static int ncsi_kick_channels(struct ncsi_dev_priv *ndp) ...@@ -1338,8 +1335,7 @@ static int ncsi_kick_channels(struct ncsi_dev_priv *ndp)
list_add_tail_rcu(&nc->link, &ndp->channel_queue); list_add_tail_rcu(&nc->link, &ndp->channel_queue);
spin_unlock_irqrestore(&ndp->lock, flags); spin_unlock_irqrestore(&ndp->lock, flags);
netdev_printk(KERN_DEBUG, nd->dev, netdev_dbg(nd->dev, "NCSI: kicked channel %p\n", nc);
"NCSI: kicked channel %p\n", nc);
n++; n++;
} }
} }
...@@ -1370,8 +1366,8 @@ int ncsi_vlan_rx_add_vid(struct net_device *dev, __be16 proto, u16 vid) ...@@ -1370,8 +1366,8 @@ int ncsi_vlan_rx_add_vid(struct net_device *dev, __be16 proto, u16 vid)
list_for_each_entry_rcu(vlan, &ndp->vlan_vids, list) { list_for_each_entry_rcu(vlan, &ndp->vlan_vids, list) {
n_vids++; n_vids++;
if (vlan->vid == vid) { if (vlan->vid == vid) {
netdev_printk(KERN_DEBUG, dev, netdev_dbg(dev, "NCSI: vid %u already registered\n",
"NCSI: vid %u already registered\n", vid); vid);
return 0; return 0;
} }
} }
...@@ -1390,7 +1386,7 @@ int ncsi_vlan_rx_add_vid(struct net_device *dev, __be16 proto, u16 vid) ...@@ -1390,7 +1386,7 @@ int ncsi_vlan_rx_add_vid(struct net_device *dev, __be16 proto, u16 vid)
vlan->vid = vid; vlan->vid = vid;
list_add_rcu(&vlan->list, &ndp->vlan_vids); list_add_rcu(&vlan->list, &ndp->vlan_vids);
netdev_printk(KERN_DEBUG, dev, "NCSI: Added new vid %u\n", vid); netdev_dbg(dev, "NCSI: Added new vid %u\n", vid);
found = ncsi_kick_channels(ndp) != 0; found = ncsi_kick_channels(ndp) != 0;
...@@ -1419,8 +1415,7 @@ int ncsi_vlan_rx_kill_vid(struct net_device *dev, __be16 proto, u16 vid) ...@@ -1419,8 +1415,7 @@ int ncsi_vlan_rx_kill_vid(struct net_device *dev, __be16 proto, u16 vid)
/* Remove the VLAN id from our internal list */ /* Remove the VLAN id from our internal list */
list_for_each_entry_safe(vlan, tmp, &ndp->vlan_vids, list) list_for_each_entry_safe(vlan, tmp, &ndp->vlan_vids, list)
if (vlan->vid == vid) { if (vlan->vid == vid) {
netdev_printk(KERN_DEBUG, dev, netdev_dbg(dev, "NCSI: vid %u found, removing\n", vid);
"NCSI: vid %u found, removing\n", vid);
list_del_rcu(&vlan->list); list_del_rcu(&vlan->list);
found = true; found = true;
kfree(vlan); kfree(vlan);
...@@ -1547,7 +1542,7 @@ void ncsi_stop_dev(struct ncsi_dev *nd) ...@@ -1547,7 +1542,7 @@ void ncsi_stop_dev(struct ncsi_dev *nd)
} }
} }
netdev_printk(KERN_DEBUG, ndp->ndev.dev, "NCSI: Stopping device\n"); netdev_dbg(ndp->ndev.dev, "NCSI: Stopping device\n");
ncsi_report_link(ndp, true); ncsi_report_link(ndp, true);
} }
EXPORT_SYMBOL_GPL(ncsi_stop_dev); EXPORT_SYMBOL_GPL(ncsi_stop_dev);
......
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