Skip to content

Commit bd1b5d8

Browse files
dthompsoNipaLocal
authored andcommitted
mlxbf_gige: emit messages during open and probe failures
The open() and probe() functions of the mlxbf_gige driver check for errors during initialization, but do not provide details regarding the errors. The mlxbf_gige driver should provide error details in the kernel log, noting what step of initialization failed. Signed-off-by: David Thompson <[email protected]> Signed-off-by: NipaLocal <nipa@local>
1 parent e69c637 commit bd1b5d8

File tree

1 file changed

+15
-5
lines changed

1 file changed

+15
-5
lines changed

drivers/net/ethernet/mellanox/mlxbf_gige/mlxbf_gige_main.c

Lines changed: 15 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -142,8 +142,10 @@ static int mlxbf_gige_open(struct net_device *netdev)
142142

143143
mlxbf_gige_cache_stats(priv);
144144
err = mlxbf_gige_clean_port(priv);
145-
if (err)
145+
if (err) {
146+
dev_err(priv->dev, "open: clean_port failed, err=0x%x\n", err);
146147
return err;
148+
}
147149

148150
/* Clear driver's valid_polarity to match hardware,
149151
* since the above call to clean_port() resets the
@@ -154,19 +156,25 @@ static int mlxbf_gige_open(struct net_device *netdev)
154156
phy_start(phydev);
155157

156158
err = mlxbf_gige_tx_init(priv);
157-
if (err)
159+
if (err) {
160+
dev_err(priv->dev, "open: tx_init failed, err=0x%x\n", err);
158161
goto phy_deinit;
162+
}
159163
err = mlxbf_gige_rx_init(priv);
160-
if (err)
164+
if (err) {
165+
dev_err(priv->dev, "open: rx_init failed, err=0x%x\n", err);
161166
goto tx_deinit;
167+
}
162168

163169
netif_napi_add(netdev, &priv->napi, mlxbf_gige_poll);
164170
napi_enable(&priv->napi);
165171
netif_start_queue(netdev);
166172

167173
err = mlxbf_gige_request_irqs(priv);
168-
if (err)
174+
if (err) {
175+
dev_err(priv->dev, "open: request_irqs failed, err=0x%x\n", err);
169176
goto napi_deinit;
177+
}
170178

171179
mlxbf_gige_enable_mac_rx_filter(priv, MLXBF_GIGE_BCAST_MAC_FILTER_IDX);
172180
mlxbf_gige_enable_mac_rx_filter(priv, MLXBF_GIGE_LOCAL_MAC_FILTER_IDX);
@@ -418,8 +426,10 @@ static int mlxbf_gige_probe(struct platform_device *pdev)
418426

419427
/* Attach MDIO device */
420428
err = mlxbf_gige_mdio_probe(pdev, priv);
421-
if (err)
429+
if (err) {
430+
dev_err(priv->dev, "probe: mdio_probe failed, err=0x%x\n", err);
422431
return err;
432+
}
423433

424434
priv->base = base;
425435
priv->llu_base = llu_base;

0 commit comments

Comments
 (0)