net: atlantic: add msglevel configuration
authorNikita Danilov <ndanilov@marvell.com>
Thu, 7 Nov 2019 22:41:54 +0000 (22:41 +0000)
committerDavid S. Miller <davem@davemloft.net>
Fri, 8 Nov 2019 03:54:43 +0000 (19:54 -0800)
We add ethtool msglevel configuration and change some
printouts to use netdev_info set of functions.

Signed-off-by: Nikita Danilov <ndanilov@marvell.com>
Signed-off-by: Igor Russkikh <irusskikh@marvell.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/aquantia/atlantic/aq_ethtool.c
drivers/net/ethernet/aquantia/atlantic/aq_nic.c
drivers/net/ethernet/aquantia/atlantic/aq_nic.h

index 3c55cf13cf14d793e3d09a0c299c4acfdf9ee76e..5be27389243078ce19af21b37b572804dff5be88 100644 (file)
@@ -607,6 +607,20 @@ err_exit:
        return err;
 }
 
+static u32 aq_get_msg_level(struct net_device *ndev)
+{
+       struct aq_nic_s *aq_nic = netdev_priv(ndev);
+
+       return aq_nic->msg_enable;
+}
+
+static void aq_set_msg_level(struct net_device *ndev, u32 data)
+{
+       struct aq_nic_s *aq_nic = netdev_priv(ndev);
+
+       aq_nic->msg_enable = data;
+}
+
 const struct ethtool_ops aq_ethtool_ops = {
        .get_link            = aq_ethtool_get_link,
        .get_regs_len        = aq_ethtool_get_regs_len,
@@ -628,6 +642,8 @@ const struct ethtool_ops aq_ethtool_ops = {
        .set_rxfh            = aq_ethtool_set_rss,
        .get_rxnfc           = aq_ethtool_get_rxnfc,
        .set_rxnfc           = aq_ethtool_set_rxnfc,
+       .get_msglevel        = aq_get_msg_level,
+       .set_msglevel        = aq_set_msg_level,
        .get_sset_count      = aq_ethtool_get_sset_count,
        .get_ethtool_stats   = aq_ethtool_stats,
        .get_link_ksettings  = aq_ethtool_get_link_ksettings,
index d5764228cea5515f88a6d04ac600cdf0e7bdef73..8f83e91f81469769be269aa5aab889553c7b9dbe 100644 (file)
@@ -144,9 +144,9 @@ static int aq_nic_update_link_status(struct aq_nic_s *self)
                return err;
 
        if (self->link_status.mbps != self->aq_hw->aq_link_status.mbps) {
-               pr_info("%s: link change old %d new %d\n",
-                       AQ_CFG_DRV_NAME, self->link_status.mbps,
-                       self->aq_hw->aq_link_status.mbps);
+               netdev_info(self->ndev, "%s: link change old %d new %d\n",
+                           AQ_CFG_DRV_NAME, self->link_status.mbps,
+                           self->aq_hw->aq_link_status.mbps);
                aq_nic_update_interrupt_moderation_settings(self);
 
                if (self->aq_ptp) {
@@ -306,6 +306,7 @@ void aq_nic_ndev_init(struct aq_nic_s *self)
        self->ndev->priv_flags = aq_hw_caps->hw_priv_flags;
        self->ndev->priv_flags |= IFF_LIVE_ADDR_CHANGE;
 
+       self->msg_enable = NETIF_MSG_DRV | NETIF_MSG_LINK;
        self->ndev->mtu = aq_nic_cfg->mtu - ETH_HLEN;
        self->ndev->max_mtu = aq_hw_caps->mtu - ETH_FCS_LEN - ETH_HLEN;
 
index ab3176dfc2090e96cdda06fa15af679d3f3e9284..527273502d54001f4fbfdef409670583dafe9b10 100644 (file)
@@ -88,6 +88,7 @@ struct aq_hw_rx_fltrs_s {
 
 struct aq_nic_s {
        atomic_t flags;
+       u32 msg_enable;
        struct aq_vec_s *aq_vec[AQ_CFG_VECS_MAX];
        struct aq_ring_s *aq_ring_tx[AQ_CFG_VECS_MAX * AQ_CFG_TCS_MAX];
        struct aq_hw_s *aq_hw;