[PATCH] drivers: net: fec_imx: Fix system halt after FEC reinit

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Fix problem with system hang when trying to re-init ethernet interface
on i.MX devices. The problem is that RX and TX buffer descriptor address
registers are only being set in fec_probe(). When fec_halt() is called
when bringing down the interface, this resets the FEC and values are
lost. As per documentation, these registers should be reconfigured
before re-enabling the FEC. Additionally, move fec_init() call from
probe to open, since the FEC needs to re-initialized after every reset.

Signed-off-by: Anže Lešnik <anze.lesnik@xxxxxxxxx>
Link: https://lore.barebox.org/barebox/c51b2c94-61b7-df44-5d4c-025d18c4b24f@xxxxxxxxx
---
 drivers/net/fec_imx.c | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/drivers/net/fec_imx.c b/drivers/net/fec_imx.c
index 0c2d600d1..673555a48 100644
--- a/drivers/net/fec_imx.c
+++ b/drivers/net/fec_imx.c
@@ -320,6 +320,10 @@ static int fec_init(struct eth_device *dev)
 	/* size of each buffer */
 	writel(FEC_MAX_PKT_SIZE, fec->regs + FEC_EMRBR);
 
+	/* set rx and tx buffer descriptor base address */
+	writel(virt_to_phys(fec->tbd_base), fec->regs + FEC_ETDSR);
+	writel(virt_to_phys(fec->rbd_base), fec->regs + FEC_ERDSR);
+
 	return 0;
 }
 
@@ -359,6 +363,8 @@ static int fec_open(struct eth_device *edev)
 	if (fec->phy_init)
 		fec->phy_init(edev->phydev);
 
+	fec_init(edev);
+
 	/*
 	 * Initialize RxBD/TxBD rings
 	 */
@@ -839,9 +845,6 @@ static int fec_probe(struct device_d *dev)
 	base += FEC_RBD_NUM * sizeof(struct buffer_descriptor);
 	fec->tbd_base = base;
 
-	writel(virt_to_phys(fec->tbd_base), fec->regs + FEC_ETDSR);
-	writel(virt_to_phys(fec->rbd_base), fec->regs + FEC_ERDSR);
-
 	ret = fec_alloc_receive_packets(fec, FEC_RBD_NUM, FEC_MAX_PKT_SIZE);
 	if (ret < 0)
 		goto free_xbd;
@@ -861,8 +864,6 @@ static int fec_probe(struct device_d *dev)
 	if (ret)
 		goto free_receive_packets;
 
-	fec_init(edev);
-
 	fec->miibus.read = fec_miibus_read;
 	fec->miibus.write = fec_miibus_write;
 
-- 
2.37.0





[Index of Archives]     [Linux Embedded]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux