net: ethernet: ti: am65-cpsw: handle deferred probe with dev_err_probe()
[ Upstream commit 8fbc2f9edce23d19fc09ef5bf8d4eb38be2db0f8 ] Use new dev_err_probe() API to handle deferred probe properly and simplify the code. Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com> Signed-off-by: Jakub Kicinski <kuba@kernel.org> Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:

committed by
Greg Kroah-Hartman

parent
37f0ca73fe
commit
80c81aafc9
@@ -1487,9 +1487,8 @@ static int am65_cpsw_nuss_init_tx_chns(struct am65_cpsw_common *common)
|
|||||||
tx_chn->tx_chn_name,
|
tx_chn->tx_chn_name,
|
||||||
&tx_cfg);
|
&tx_cfg);
|
||||||
if (IS_ERR(tx_chn->tx_chn)) {
|
if (IS_ERR(tx_chn->tx_chn)) {
|
||||||
ret = PTR_ERR(tx_chn->tx_chn);
|
ret = dev_err_probe(dev, PTR_ERR(tx_chn->tx_chn),
|
||||||
dev_err(dev, "Failed to request tx dma channel %d\n",
|
"Failed to request tx dma channel\n");
|
||||||
ret);
|
|
||||||
goto err;
|
goto err;
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -1560,8 +1559,8 @@ static int am65_cpsw_nuss_init_rx_chns(struct am65_cpsw_common *common)
|
|||||||
|
|
||||||
rx_chn->rx_chn = k3_udma_glue_request_rx_chn(dev, "rx", &rx_cfg);
|
rx_chn->rx_chn = k3_udma_glue_request_rx_chn(dev, "rx", &rx_cfg);
|
||||||
if (IS_ERR(rx_chn->rx_chn)) {
|
if (IS_ERR(rx_chn->rx_chn)) {
|
||||||
ret = PTR_ERR(rx_chn->rx_chn);
|
ret = dev_err_probe(dev, PTR_ERR(rx_chn->rx_chn),
|
||||||
dev_err(dev, "Failed to request rx dma channel %d\n", ret);
|
"Failed to request rx dma channel\n");
|
||||||
goto err;
|
goto err;
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -1768,12 +1767,10 @@ static int am65_cpsw_nuss_init_slave_ports(struct am65_cpsw_common *common)
|
|||||||
/* get phy/link info */
|
/* get phy/link info */
|
||||||
if (of_phy_is_fixed_link(port_np)) {
|
if (of_phy_is_fixed_link(port_np)) {
|
||||||
ret = of_phy_register_fixed_link(port_np);
|
ret = of_phy_register_fixed_link(port_np);
|
||||||
if (ret) {
|
if (ret)
|
||||||
if (ret != -EPROBE_DEFER)
|
return dev_err_probe(dev, ret,
|
||||||
dev_err(dev, "%pOF failed to register fixed-link phy: %d\n",
|
"failed to register fixed-link phy %pOF\n",
|
||||||
port_np, ret);
|
port_np);
|
||||||
return ret;
|
|
||||||
}
|
|
||||||
port->slave.phy_node = of_node_get(port_np);
|
port->slave.phy_node = of_node_get(port_np);
|
||||||
} else {
|
} else {
|
||||||
port->slave.phy_node =
|
port->slave.phy_node =
|
||||||
@@ -2062,13 +2059,8 @@ static int am65_cpsw_nuss_probe(struct platform_device *pdev)
|
|||||||
return -ENOMEM;
|
return -ENOMEM;
|
||||||
|
|
||||||
clk = devm_clk_get(dev, "fck");
|
clk = devm_clk_get(dev, "fck");
|
||||||
if (IS_ERR(clk)) {
|
if (IS_ERR(clk))
|
||||||
ret = PTR_ERR(clk);
|
return dev_err_probe(dev, PTR_ERR(clk), "getting fck clock\n");
|
||||||
|
|
||||||
if (ret != -EPROBE_DEFER)
|
|
||||||
dev_err(dev, "error getting fck clock %d\n", ret);
|
|
||||||
return ret;
|
|
||||||
}
|
|
||||||
common->bus_freq = clk_get_rate(clk);
|
common->bus_freq = clk_get_rate(clk);
|
||||||
|
|
||||||
pm_runtime_enable(dev);
|
pm_runtime_enable(dev);
|
||||||
|
Reference in New Issue
Block a user