1 From 74920ba156136a58dd3411c02d429d4f31497dc0 Mon Sep 17 00:00:00 2001
2 From: Marc Kleine-Budde <mkl@pengutronix.de>
3 Date: Fri, 1 Mar 2019 15:38:05 +0100
4 Subject: [PATCH] can: flexcan: flexcan_read_reg_iflag_rx(): optimize reading
6 The flexcan IP core has up to 64 mailboxes, each one has a corresponding
7 interrupt bit in the iflag1 or iflag2 registers and a mask bit in the
8 imask1 or imask2 registers.
10 In the timestamp (i.e. non FIFO) mode the driver needs to mask all non RX
11 interrupt sources, it uses the precomputed value rx_mask of struct flexcan_priv
14 In certain use cases, for example the CANFD mode, the contents of the iflag2
15 register is completely masked.
17 This patch optimizes the flexcan_read_reg_iflag_rx() function by not reading
18 the iflag1 or iflag2 register if the contents is masked.
20 Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
22 drivers/net/can/flexcan.c | 28 +++++++++++++++++-----------
23 1 file changed, 17 insertions(+), 11 deletions(-)
25 --- a/drivers/net/can/flexcan.c
26 +++ b/drivers/net/can/flexcan.c
27 @@ -779,6 +779,23 @@ static void flexcan_irq_state(struct net
28 dev->stats.rx_fifo_errors++;
31 +static inline u64 flexcan_read64_mask(struct flexcan_priv *priv, void __iomem *addr, u64 mask)
35 + if (upper_32_bits(mask))
36 + reg = (u64)priv->read(addr - 4) << 32;
37 + if (lower_32_bits(mask))
38 + reg |= priv->read(addr);
43 +static inline u64 flexcan_read_reg_iflag_rx(struct flexcan_priv *priv)
45 + return flexcan_read64_mask(priv, &priv->regs->iflag1, priv->rx_mask);
48 static inline struct flexcan_priv *rx_offload_to_priv(struct can_rx_offload *offload)
50 return container_of(offload, struct flexcan_priv, offload);
51 @@ -873,17 +890,6 @@ static struct sk_buff *flexcan_mailbox_r
55 -static inline u64 flexcan_read_reg_iflag_rx(struct flexcan_priv *priv)
57 - struct flexcan_regs __iomem *regs = priv->regs;
60 - iflag = (u64)priv->read(®s->iflag2) << 32 |
61 - priv->read(®s->iflag1);
63 - return iflag & priv->rx_mask;
66 static irqreturn_t flexcan_irq(int irq, void *dev_id)
68 struct net_device *dev = dev_id;