1 From 3b9c2f2e0e99bb67c96abcb659b3465efe3bee1f Mon Sep 17 00:00:00 2001
2 From: Malcolm Priestley <tvboxspy@gmail.com>
3 Date: Sun, 24 Mar 2019 18:53:49 +0000
4 Subject: staging: vt6655: Fix interrupt race condition on device start up.
6 From: Malcolm Priestley <tvboxspy@gmail.com>
8 commit 3b9c2f2e0e99bb67c96abcb659b3465efe3bee1f upstream.
10 It appears on some slower systems that the driver can find its way
11 out of the workqueue while the interrupt is disabled by continuous polling
14 Move MACvIntEnable to vnt_interrupt_work so that it is always enabled
15 on all routes out of vnt_interrupt_process.
17 Move MACvIntDisable so that the device doesn't keep polling the system
18 while the workqueue is being processed.
20 Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
21 CC: stable@vger.kernel.org # v4.2+
22 Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
25 drivers/staging/vt6655/device_main.c | 8 ++++----
26 1 file changed, 4 insertions(+), 4 deletions(-)
28 --- a/drivers/staging/vt6655/device_main.c
29 +++ b/drivers/staging/vt6655/device_main.c
30 @@ -1040,8 +1040,6 @@ static void vnt_interrupt_process(struct
34 - MACvIntDisable(priv->PortOffset);
36 spin_lock_irqsave(&priv->lock, flags);
38 /* Read low level stats */
39 @@ -1129,8 +1127,6 @@ static void vnt_interrupt_process(struct
42 spin_unlock_irqrestore(&priv->lock, flags);
44 - MACvIntEnable(priv->PortOffset, IMR_MASK_VALUE);
47 static void vnt_interrupt_work(struct work_struct *work)
48 @@ -1140,6 +1136,8 @@ static void vnt_interrupt_work(struct wo
51 vnt_interrupt_process(priv);
53 + MACvIntEnable(priv->PortOffset, IMR_MASK_VALUE);
56 static irqreturn_t vnt_interrupt(int irq, void *arg)
57 @@ -1148,6 +1146,8 @@ static irqreturn_t vnt_interrupt(int irq
59 schedule_work(&priv->interrupt_work);
61 + MACvIntDisable(priv->PortOffset);