]> git.ipfire.org Git - thirdparty/openwrt.git/commitdiff
tegra: bring back workaround for spurious interrupts
authorTomasz Maciej Nowak <tmn505@gmail.com>
Wed, 18 Jun 2025 16:59:29 +0000 (18:59 +0200)
committerHauke Mehrtens <hauke@hauke-m.de>
Sat, 12 Jul 2025 22:48:34 +0000 (00:48 +0200)
Unfortunately they still apear with prolonged serial console usage.
While it's not common to use serial console past initial setup, alas
when the condition is hit the console is almost unusable.

Signed-off-by: Tomasz Maciej Nowak <tmn505@gmail.com>
Link: https://patchwork.ozlabs.org/project/openwrt/patch/20250618170045.473711-2-tmn505@terefe.re/
Signed-off-by: Hauke Mehrtens <hauke@hauke-m.de>
target/linux/tegra/patches-6.6/100-serial8250-on-tegra-hsuart-recover-from-spurious-interrupts-due-to-tegra2-silicon-bug.patch [new file with mode: 0644]

diff --git a/target/linux/tegra/patches-6.6/100-serial8250-on-tegra-hsuart-recover-from-spurious-interrupts-due-to-tegra2-silicon-bug.patch b/target/linux/tegra/patches-6.6/100-serial8250-on-tegra-hsuart-recover-from-spurious-interrupts-due-to-tegra2-silicon-bug.patch
new file mode 100644 (file)
index 0000000..8f01f73
--- /dev/null
@@ -0,0 +1,77 @@
+From patchwork Fri Jul 13 11:32:42 2018
+Content-Type: text/plain; charset="utf-8"
+MIME-Version: 1.0
+Content-Transfer-Encoding: 7bit
+Subject: serial8250 on tegra hsuart: recover from spurious interrupts due to
+ tegra2 silicon bug
+X-Patchwork-Submitter: "David R. Piegdon" <lkml@p23q.org>
+X-Patchwork-Id: 943440
+Message-Id: <4676ea34-69ce-5422-1ded-94218b89f7d9@p23q.org>
+To: linux-tegra@vger.kernel.org
+Date: Fri, 13 Jul 2018 11:32:42 +0000
+From: "David R. Piegdon" <lkml@p23q.org>
+List-Id: <linux-tegra.vger.kernel.org>
+
+Hi,
+a while back I sent a few mails regarding spurious interrupts in the
+UARTA (hsuart) block of the Tegra2 SoC, when using the 8250 driver for
+it instead of the hsuart driver. After going down a pretty deep
+debugging/testing hole, I think I found a patch that fixes the issue. So
+far testing in a reboot-cycle suggests that the error frequency dropped
+from >3% of all reboots to at least <0.05% of all reboots. Tests
+continue to run over the weekend.
+
+The patch below already is a second iteration; the first did not reset
+the MCR or contain the lines below '// clear interrupts'. This resulted
+in no more spurious interrupts, but in a few % of spurious interrupts
+that were recovered the UART block did not receive any characters any
+more. So further resetting was required to fully reacquire operational
+state of the UART block.
+
+I'd love any comments/suggestions on this!
+
+Cheers,
+
+David
+
+--- a/drivers/tty/serial/8250/8250_core.c
++++ b/drivers/tty/serial/8250/8250_core.c
+@@ -134,6 +134,38 @@ static irqreturn_t serial8250_interrupt(
+               if (l == i->head && pass_counter++ > PASS_LIMIT)
+                       break;
++
++#ifdef CONFIG_ARCH_TEGRA_2x_SOC
++              if (!handled && (port->type == PORT_TEGRA)) {
++                      /*
++                       * Fix Tegra 2 CPU silicon bug where sometimes
++                       * "TX holding register empty" interrupts result in a
++                       * bad (metastable?) state in Tegras HSUART IP core.
++                       * Only way to recover seems to be to reset all
++                       * interrupts as well as the TX queue and the MCR.
++                       * But we don't want to loose any outgoing characters,
++                       * so only do it if the RX and TX queues are empty.
++                       */
++                      unsigned char lsr = port->serial_in(port, UART_LSR);
++                      const unsigned char fifo_empty_mask =
++                                              (UART_LSR_TEMT | UART_LSR_THRE);
++                      if (((lsr & (UART_LSR_DR | fifo_empty_mask)) ==
++                                                      fifo_empty_mask)) {
++                              port->serial_out(port, UART_IER, 0);
++                              port->serial_out(port, UART_MCR, 0);
++                              serial8250_clear_and_reinit_fifos(up);
++                              port->serial_out(port, UART_MCR, up->mcr);
++                              port->serial_out(port, UART_IER, up->ier);
++                              // clear interrupts
++                              serial_port_in(port, UART_LSR);
++                              serial_port_in(port, UART_RX);
++                              serial_port_in(port, UART_IIR);
++                              serial_port_in(port, UART_MSR);
++                              up->lsr_saved_flags = 0;
++                              up->msr_saved_flags = 0;
++                      }
++              }
++#endif
+       } while (l != end);
+       spin_unlock(&i->lock);