[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <20241104084705.5005-1-lucas.liu@siengine.com>
Date: Mon, 4 Nov 2024 16:47:05 +0800
From: "baozhu.liu" <lucas.liu@...ngine.com>
To: <mkl@...gutronix.de>
CC: <wg@...ndegger.com>, <davem@...emloft.net>, <edumazet@...gle.com>,
<kuba@...nel.org>, <linux-can@...r.kernel.org>,
<linux-kernel@...r.kernel.org>, <netdev@...r.kernel.org>,
"baozhu.liu"
<lucas.liu@...ngine.com>
Subject: [PATCH] can: flexcan: simplify the calculation of priv->mb_count
Since mb is a fixed-size two-dimensional array (u8 mb[2][512]),
"priv->mb_count = sizeof(priv->regs->mb)/priv->mb_size;",
this expression calculates mb_count correctly and is more concise.
Signed-off-by: baozhu.liu <lucas.liu@...ngine.com>
---
drivers/net/can/flexcan/flexcan-core.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/net/can/flexcan/flexcan-core.c b/drivers/net/can/flexcan/flexcan-core.c
index 6d638c939..e3a8bad21 100644
--- a/drivers/net/can/flexcan/flexcan-core.c
+++ b/drivers/net/can/flexcan/flexcan-core.c
@@ -1371,8 +1371,7 @@ static int flexcan_rx_offload_setup(struct net_device *dev)
if (priv->devtype_data.quirks & FLEXCAN_QUIRK_NR_MB_16)
priv->mb_count = 16;
else
- priv->mb_count = (sizeof(priv->regs->mb[0]) / priv->mb_size) +
- (sizeof(priv->regs->mb[1]) / priv->mb_size);
+ priv->mb_count = sizeof(priv->regs->mb) / priv->mb_size;
if (priv->devtype_data.quirks & FLEXCAN_QUIRK_USE_RX_MAILBOX)
priv->tx_mb_reserved =
--
2.17.1
Powered by blists - more mailing lists