linux-kernel - [PATCH] can: m_can: m_can_chip_config(): mask timestamp wraparound IRQ

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Message-Id: <20240911-can-m_can-mask-timestamp-wraparound-irq-v1-1-0155b70dc827@pengutronix.de>
Date: Wed, 11 Sep 2024 10:51:30 +0200
From: Marc Kleine-Budde <mkl@...gutronix.de>
To: Chandrasekar Ramakrishnan <rcsekar@...sung.com>, 
 Vincent Mailhol <mailhol.vincent@...adoo.fr>, 
 "David S. Miller" <davem@...emloft.net>, Eric Dumazet <edumazet@...gle.com>, 
 Jakub Kicinski <kuba@...nel.org>, Paolo Abeni <pabeni@...hat.com>
Cc: kernel@...gutronix.de, linux-can@...r.kernel.org, 
 netdev@...r.kernel.org, linux-kernel@...r.kernel.org, 
 Jake Hamby <Jake.Hamby@...edyne.com>, 
 Marc Kleine-Budde <mkl@...gutronix.de>
Subject: [PATCH] can: m_can: m_can_chip_config(): mask timestamp wraparound
 IRQ

From: Jake Hamby <Jake.Hamby@...edyne.com>

On the Microchip SAMA7G54 MPU the IR_TSW (timestamp wraparound) fires
at about 1 Hz, but the driver doesn't care about it. Add it to the
list of interrupts to disable in m_can_chip_config to reduce unneeded
wakeups.

Signed-off-by: Jake Hamby <Jake.Hamby@...edyne.com>
Signed-off-by: Marc Kleine-Budde <mkl@...gutronix.de>
---
Hello,

this hunk was originally part of Jake's original patch

| https://patch.msgid.link/DM8PR14MB5221D9DD3A7F2130EF161AF7EF9E2@DM8PR14MB5221.namprd14.prod.outlook.com

I've split it into a separate patch.

regards,
Marc
---
 drivers/net/can/m_can/m_can.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/net/can/m_can/m_can.c b/drivers/net/can/m_can/m_can.c
index 012c3d22b01dd3d8558f2a40448770ca1da1aa1e..a7b3bc439ae596527493a73d62b4b7a120ae4e49 100644
--- a/drivers/net/can/m_can/m_can.c
+++ b/drivers/net/can/m_can/m_can.c
@@ -1434,7 +1434,8 @@ static int m_can_chip_config(struct net_device *dev)

 	/* Disable unused interrupts */
 	interrupts &= ~(IR_ARA | IR_ELO | IR_DRX | IR_TEFF | IR_TFE | IR_TCF |
-			IR_HPM | IR_RF1F | IR_RF1W | IR_RF1N | IR_RF0F);
+			IR_HPM | IR_RF1F | IR_RF1W | IR_RF1N | IR_RF0F |
+			IR_TSW);

 	err = m_can_config_enable(cdev);
 	if (err)

---
base-commit: f3b6129b7d252b2fbdcac2e0005abc6804dc287c
change-id: 20240911-can-m_can-mask-timestamp-wraparound-irq-d3a26c048b5d

Best regards,
-- 
Marc Kleine-Budde <mkl@...gutronix.de>