mirror of
https://github.com/raspberrypi/linux.git
synced 2025-12-26 03:52:41 +00:00
net: ethernet: mtk_eth_soc: add consts for irq index
Use consts instead of fixed integers for accessing IRQ array. Signed-off-by: Frank Wunderlich <frank-w@public-files.de> Reviewed-by: Simon Horman <horms@kernel.org> Reviewed-by: Daniel Golle <daniel@makrotopia.org> Link: https://patch.msgid.link/20250619132125.78368-3-linux@fw-web.de Signed-off-by: Jakub Kicinski <kuba@kernel.org>
This commit is contained in:
committed by
Jakub Kicinski
parent
ee85b483fe
commit
4981901009
@@ -3341,9 +3341,9 @@ static int mtk_get_irqs(struct platform_device *pdev, struct mtk_eth *eth)
|
||||
int i;
|
||||
|
||||
/* future SoCs beginning with MT7988 should use named IRQs in dts */
|
||||
eth->irq[1] = platform_get_irq_byname(pdev, "fe1");
|
||||
eth->irq[2] = platform_get_irq_byname(pdev, "fe2");
|
||||
if (eth->irq[1] >= 0 && eth->irq[2] >= 0)
|
||||
eth->irq[MTK_FE_IRQ_TX] = platform_get_irq_byname(pdev, "fe1");
|
||||
eth->irq[MTK_FE_IRQ_RX] = platform_get_irq_byname(pdev, "fe2");
|
||||
if (eth->irq[MTK_FE_IRQ_TX] >= 0 && eth->irq[MTK_FE_IRQ_RX] >= 0)
|
||||
return 0;
|
||||
|
||||
/* legacy way:
|
||||
@@ -3352,9 +3352,9 @@ static int mtk_get_irqs(struct platform_device *pdev, struct mtk_eth *eth)
|
||||
* On SoCs with non-shared IRQs the first entry is not used,
|
||||
* the second is for TX, and the third is for RX.
|
||||
*/
|
||||
for (i = 0; i < 3; i++) {
|
||||
for (i = 0; i < MTK_FE_IRQ_NUM; i++) {
|
||||
if (MTK_HAS_CAPS(eth->soc->caps, MTK_SHARED_INT) && i > 0)
|
||||
eth->irq[i] = eth->irq[0];
|
||||
eth->irq[i] = eth->irq[MTK_FE_IRQ_SHARED];
|
||||
else
|
||||
eth->irq[i] = platform_get_irq(pdev, i);
|
||||
|
||||
@@ -3420,7 +3420,7 @@ static void mtk_poll_controller(struct net_device *dev)
|
||||
|
||||
mtk_tx_irq_disable(eth, MTK_TX_DONE_INT);
|
||||
mtk_rx_irq_disable(eth, eth->soc->rx.irq_done_mask);
|
||||
mtk_handle_irq_rx(eth->irq[2], dev);
|
||||
mtk_handle_irq_rx(eth->irq[MTK_FE_IRQ_RX], dev);
|
||||
mtk_tx_irq_enable(eth, MTK_TX_DONE_INT);
|
||||
mtk_rx_irq_enable(eth, eth->soc->rx.irq_done_mask);
|
||||
}
|
||||
@@ -4906,7 +4906,7 @@ static int mtk_add_mac(struct mtk_eth *eth, struct device_node *np)
|
||||
eth->netdev[id]->features |= eth->soc->hw_features;
|
||||
eth->netdev[id]->ethtool_ops = &mtk_ethtool_ops;
|
||||
|
||||
eth->netdev[id]->irq = eth->irq[0];
|
||||
eth->netdev[id]->irq = eth->irq[MTK_FE_IRQ_SHARED];
|
||||
eth->netdev[id]->dev.of_node = np;
|
||||
|
||||
if (MTK_HAS_CAPS(eth->soc->caps, MTK_SOC_MT7628))
|
||||
@@ -5183,17 +5183,17 @@ static int mtk_probe(struct platform_device *pdev)
|
||||
}
|
||||
|
||||
if (MTK_HAS_CAPS(eth->soc->caps, MTK_SHARED_INT)) {
|
||||
err = devm_request_irq(eth->dev, eth->irq[0],
|
||||
err = devm_request_irq(eth->dev, eth->irq[MTK_FE_IRQ_SHARED],
|
||||
mtk_handle_irq, 0,
|
||||
dev_name(eth->dev), eth);
|
||||
} else {
|
||||
err = devm_request_irq(eth->dev, eth->irq[1],
|
||||
err = devm_request_irq(eth->dev, eth->irq[MTK_FE_IRQ_TX],
|
||||
mtk_handle_irq_tx, 0,
|
||||
dev_name(eth->dev), eth);
|
||||
if (err)
|
||||
goto err_free_dev;
|
||||
|
||||
err = devm_request_irq(eth->dev, eth->irq[2],
|
||||
err = devm_request_irq(eth->dev, eth->irq[MTK_FE_IRQ_RX],
|
||||
mtk_handle_irq_rx, 0,
|
||||
dev_name(eth->dev), eth);
|
||||
}
|
||||
@@ -5239,7 +5239,7 @@ static int mtk_probe(struct platform_device *pdev)
|
||||
} else
|
||||
netif_info(eth, probe, eth->netdev[i],
|
||||
"mediatek frame engine at 0x%08lx, irq %d\n",
|
||||
eth->netdev[i]->base_addr, eth->irq[0]);
|
||||
eth->netdev[i]->base_addr, eth->irq[MTK_FE_IRQ_SHARED]);
|
||||
}
|
||||
|
||||
/* we run 2 devices on the same DMA ring so we need a dummy device
|
||||
|
||||
@@ -642,6 +642,11 @@
|
||||
|
||||
#define MTK_MAC_FSM(x) (0x1010C + ((x) * 0x100))
|
||||
|
||||
#define MTK_FE_IRQ_SHARED 0
|
||||
#define MTK_FE_IRQ_TX 1
|
||||
#define MTK_FE_IRQ_RX 2
|
||||
#define MTK_FE_IRQ_NUM (MTK_FE_IRQ_RX + 1)
|
||||
|
||||
struct mtk_rx_dma {
|
||||
unsigned int rxd1;
|
||||
unsigned int rxd2;
|
||||
@@ -1292,7 +1297,7 @@ struct mtk_eth {
|
||||
struct net_device *dummy_dev;
|
||||
struct net_device *netdev[MTK_MAX_DEVS];
|
||||
struct mtk_mac *mac[MTK_MAX_DEVS];
|
||||
int irq[3];
|
||||
int irq[MTK_FE_IRQ_NUM];
|
||||
u32 msg_enable;
|
||||
unsigned long sysclk;
|
||||
struct regmap *ethsys;
|
||||
|
||||
Reference in New Issue
Block a user