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@xxxxxxxxxxxx> --- 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
