3da861ccca
Refreshed all patches. The following patches were manually changed: * 610-netfilter_match_bypass_default_checks.patch * 611-netfilter_match_bypass_default_table.patch * 802-can-0002-can-rx-offload-fix-long-lines.patch * 802-can-0003-can-rx-offload-can_rx_offload_compare-fix-typo.patch * 802-can-0004-can-rx-offload-can_rx_offload_irq_offload_timestamp-.patch * 802-can-0005-can-rx-offload-can_rx_offload_reset-remove-no-op-fun.patch * 802-can-0006-can-rx-offload-Prepare-for-CAN-FD-support.patch * 802-can-0018-can-flexcan-use-struct-canfd_frame-for-CAN-classic-f.patch The can-dev.ko model was moved in the upstream kernel. Compile-tested on: x86/64, armvirt/64, ath79/generic Runtime-tested on: x86/64, armvirt/64, ath79/generic Signed-off-by: Hauke Mehrtens <hauke@hauke-m.de>
39 lines
1.2 KiB
Diff
39 lines
1.2 KiB
Diff
From 1a92e5a9109963e2491eec111b84b35b4e2adc8f Mon Sep 17 00:00:00 2001
|
|
From: Marc Kleine-Budde <mkl@pengutronix.de>
|
|
Date: Wed, 9 Oct 2019 15:15:37 +0200
|
|
Subject: [PATCH] can: flexcan: flexcan_irq_state(): only read timestamp if
|
|
needed
|
|
|
|
The function flexcan_irq_state() checks the controller for CAN state
|
|
changes and pushes a skb with the new state and a timestamp into the
|
|
rx-offload framework.
|
|
|
|
This patch optimizes the function by only reading the timestamp, if a
|
|
state change is detected.
|
|
|
|
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
|
|
---
|
|
drivers/net/can/flexcan.c | 4 ++--
|
|
1 file changed, 2 insertions(+), 2 deletions(-)
|
|
|
|
--- a/drivers/net/can/flexcan.c
|
|
+++ b/drivers/net/can/flexcan.c
|
|
@@ -751,8 +751,6 @@ static void flexcan_irq_state(struct net
|
|
u32 timestamp;
|
|
int err;
|
|
|
|
- timestamp = priv->read(®s->timer) << 16;
|
|
-
|
|
flt = reg_esr & FLEXCAN_ESR_FLT_CONF_MASK;
|
|
if (likely(flt == FLEXCAN_ESR_FLT_CONF_ACTIVE)) {
|
|
tx_state = unlikely(reg_esr & FLEXCAN_ESR_TX_WRN) ?
|
|
@@ -772,6 +770,8 @@ static void flexcan_irq_state(struct net
|
|
if (likely(new_state == priv->can.state))
|
|
return;
|
|
|
|
+ timestamp = priv->read(®s->timer) << 16;
|
|
+
|
|
skb = alloc_can_err_skb(dev, &cf);
|
|
if (unlikely(!skb))
|
|
return;
|