diff mbox series

[SRU,F,PULL,v2,15/23] Revert "UBUNTU: SAUCE: mlxbf-gige: cleanups from review"

Message ID 20210709190830.5405-16-asmaa@nvidia.com
State New
Headers show
Series Cherry-pick the upstreamed mlxbf-gige driver | expand

Commit Message

Asmaa Mnebhi July 9, 2021, 7:08 p.m. UTC
BugLink: https://bugs.launchpad.net/bugs/1934923

This reverts commit 3a4caceacdce4bda2c60220e6a2d22a1243a1121.

Signed-off-by: Asmaa Mnebhi <asmaa@nvidia.com>
---
 .../mellanox/mlxbf_gige/mlxbf_gige_main.c     | 28 ++++++++++---------
 1 file changed, 15 insertions(+), 13 deletions(-)
diff mbox series

Patch

diff --git a/drivers/net/ethernet/mellanox/mlxbf_gige/mlxbf_gige_main.c b/drivers/net/ethernet/mellanox/mlxbf_gige/mlxbf_gige_main.c
index d650e9c0a57f..d074bf9ef5c2 100644
--- a/drivers/net/ethernet/mellanox/mlxbf_gige/mlxbf_gige_main.c
+++ b/drivers/net/ethernet/mellanox/mlxbf_gige/mlxbf_gige_main.c
@@ -18,7 +18,7 @@ 
 #include "mlxbf_gige_regs.h"
 
 #define DRV_NAME    "mlxbf_gige"
-#define DRV_VERSION "1.2"
+#define DRV_VERSION "1.1"
 
 static void mlxbf_gige_set_mac_rx_filter(struct mlxbf_gige *priv,
 					 unsigned int index, u64 dmac)
@@ -36,14 +36,16 @@  static void mlxbf_gige_set_mac_rx_filter(struct mlxbf_gige *priv,
 	writeq(control, base + MLXBF_GIGE_CONTROL);
 }
 
-static void mlxbf_gige_get_mac_rx_filter(struct mlxbf_gige *priv,
-					 unsigned int index, u64 *dmac)
+static int mlxbf_gige_get_mac_rx_filter(struct mlxbf_gige *priv,
+					unsigned int index, u64 *dmac)
 {
 	void __iomem *base = priv->base;
 
 	/* Read destination MAC from specified MAC RX filter */
 	*dmac = readq(base + MLXBF_GIGE_RX_MAC_FILTER +
 		      (index * MLXBF_GIGE_RX_MAC_FILTER_STRIDE));
+
+	return 0;
 }
 
 static void mlxbf_gige_enable_promisc(struct mlxbf_gige *priv)
@@ -252,9 +254,9 @@  static void mlxbf_gige_rx_deinit(struct mlxbf_gige *priv)
 	dma_free_coherent(priv->dev, size,
 			  priv->rx_cqe_base, priv->rx_cqe_base_dma);
 
-	priv->rx_wqe_base = NULL;
+	priv->rx_wqe_base = 0;
 	priv->rx_wqe_base_dma = 0;
-	priv->rx_cqe_base = NULL;
+	priv->rx_cqe_base = 0;
 	priv->rx_cqe_base_dma = 0;
 	writeq(0, priv->base + MLXBF_GIGE_RX_WQ_BASE);
 	writeq(0, priv->base + MLXBF_GIGE_RX_CQ_BASE);
@@ -288,11 +290,11 @@  static void mlxbf_gige_tx_deinit(struct mlxbf_gige *priv)
 	dma_free_coherent(priv->dev, MLXBF_GIGE_TX_CC_SZ,
 			  priv->tx_cc, priv->tx_cc_dma);
 
-	priv->tx_wqe_base = NULL;
+	priv->tx_wqe_base = 0;
 	priv->tx_wqe_base_dma = 0;
-	priv->tx_cc = NULL;
+	priv->tx_cc = 0;
 	priv->tx_cc_dma = 0;
-	priv->tx_wqe_next = NULL;
+	priv->tx_wqe_next = 0;
 	writeq(0, priv->base + MLXBF_GIGE_TX_WQ_BASE);
 	writeq(0, priv->base + MLXBF_GIGE_TX_CI_UPDATE_ADDRESS);
 }
@@ -312,7 +314,7 @@  static void mlxbf_gige_get_regs(struct net_device *netdev,
 				struct ethtool_regs *regs, void *p)
 {
 	struct mlxbf_gige *priv = netdev_priv(netdev);
-	__be64 *buff = p;
+	u64 *buff = p;
 	int reg;
 
 	regs->version = MLXBF_GIGE_REGS_VERSION;
@@ -1054,9 +1056,10 @@  static void mlxbf_gige_initial_mac(struct mlxbf_gige *priv)
 {
 	u8 mac[ETH_ALEN];
 	u64 local_mac;
+	int status;
 
-	mlxbf_gige_get_mac_rx_filter(priv, MLXBF_GIGE_LOCAL_MAC_FILTER_IDX,
-				     &local_mac);
+	status = mlxbf_gige_get_mac_rx_filter(priv, MLXBF_GIGE_LOCAL_MAC_FILTER_IDX,
+					      &local_mac);
 	mlxbf_gige_u64_to_mac(mac, local_mac);
 
 	if (is_valid_ether_addr(mac)) {
@@ -1189,7 +1192,6 @@  static int mlxbf_gige_probe(struct platform_device *pdev)
 	err = register_netdev(netdev);
 	if (err) {
 		dev_err(&pdev->dev, "Failed to register netdev\n");
-		phy_disconnect(phydev);
 		return err;
 	}
 
@@ -1200,8 +1202,8 @@  static int mlxbf_gige_remove(struct platform_device *pdev)
 {
 	struct mlxbf_gige *priv = platform_get_drvdata(pdev);
 
-	unregister_netdev(priv->netdev);
 	phy_disconnect(priv->netdev->phydev);
+	unregister_netdev(priv->netdev);
 	mlxbf_gige_mdio_remove(priv);
 	platform_set_drvdata(pdev, NULL);