]> Pileus Git - ~andy/linux/commitdiff
vmxnet3: fix messages printed before registration
authorStephen Hemminger <stephen@networkplumber.org>
Tue, 15 Jan 2013 07:28:28 +0000 (07:28 +0000)
committerDavid S. Miller <davem@davemloft.net>
Wed, 16 Jan 2013 19:40:49 +0000 (14:40 -0500)
This messages that occur during boot time from this device
when netdev_err is called before calling register_netdevice().
Switch to using dev_XXX macros which correlate message with PCI info which
is available.

Rather than fixing the features message, just remove it since
the information is redundant and available through ethtool.

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/vmxnet3/vmxnet3_drv.c

index 99ef9eddfc78c51552d1de347931bf5a74365c5e..7b10d9cb37b3c0ffd9e416f4308882df68e87af3 100644 (file)
@@ -2671,10 +2671,6 @@ vmxnet3_declare_features(struct vmxnet3_adapter *adapter, bool dma64)
        netdev->vlan_features = netdev->hw_features &
                                ~(NETIF_F_HW_VLAN_TX | NETIF_F_HW_VLAN_RX);
        netdev->features = netdev->hw_features | NETIF_F_HW_VLAN_FILTER;
-
-       netdev_info(adapter->netdev,
-               "features: sg csum vlan jf tso tsoIPv6 lro%s\n",
-               dma64 ? " highDMA" : "");
 }
 
 
@@ -2717,7 +2713,7 @@ vmxnet3_acquire_msix_vectors(struct vmxnet3_adapter *adapter,
                        adapter->intr.num_intrs = vectors;
                        return 0;
                } else if (err < 0) {
-                       netdev_err(adapter->netdev,
+                       dev_err(&adapter->netdev->dev,
                                   "Failed to enable MSI-X, error: %d\n", err);
                        vectors = 0;
                } else if (err < vector_threshold) {
@@ -2726,15 +2722,16 @@ vmxnet3_acquire_msix_vectors(struct vmxnet3_adapter *adapter,
                        /* If fails to enable required number of MSI-x vectors
                         * try enabling minimum number of vectors required.
                         */
-                       netdev_err(adapter->netdev,
-                                  "Failed to enable %d MSI-X, trying %d instead\n",
+                       dev_err(&adapter->netdev->dev,
+                               "Failed to enable %d MSI-X, trying %d instead\n",
                                    vectors, vector_threshold);
                        vectors = vector_threshold;
                }
        }
 
-       netdev_info(adapter->netdev,
-                   "Number of MSI-X interrupts which can be allocated are lower than min threshold required.\n");
+       dev_info(&adapter->pdev->dev,
+                "Number of MSI-X interrupts which can be allocated "
+                "is lower than min threshold required.\n");
        return err;
 }
 
@@ -2800,9 +2797,9 @@ vmxnet3_alloc_intr_resources(struct vmxnet3_adapter *adapter)
                        return;
 
                /* If we cannot allocate MSIx vectors use only one rx queue */
-               netdev_info(adapter->netdev,
-                           "Failed to enable MSI-X, error %d . Limiting #rx queues to 1, try MSI.\n",
-                           err);
+               dev_info(&adapter->pdev->dev,
+                        "Failed to enable MSI-X, error %d. "
+                        "Limiting #rx queues to 1, try MSI.\n", err);
 
                adapter->intr.type = VMXNET3_IT_MSI;
        }