All of lore.kernel.org
 help / color / mirror / Atom feed
* FAILED: patch "[PATCH] serial: lantiq: add missing interrupt ack" failed to apply to 4.14-stable tree
@ 2023-06-18 12:49 gregkh
  2023-06-19 21:02 ` [PATCH 4.14.y] serial: lantiq: add missing interrupt ack Bernhard Seibold
  2023-06-19 21:10 ` [PATCH 4.14.y v2] " Bernhard Seibold
  0 siblings, 2 replies; 5+ messages in thread
From: gregkh @ 2023-06-18 12:49 UTC (permalink / raw)
  To: mail, gregkh, ilpo.jarvinen, stable; +Cc: stable


The patch below does not apply to the 4.14-stable tree.
If someone wants it applied there, or to any other stable or longterm
tree, then please email the backport, including the original git commit
id to <stable@vger.kernel.org>.

To reproduce the conflict and resubmit, you may use the following commands:

git fetch https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/ linux-4.14.y
git checkout FETCH_HEAD
git cherry-pick -x 306320034e8fbe7ee1cc4f5269c55658b4612048
# <resolve conflicts, build, test, etc.>
git commit -s
git send-email --to '<stable@vger.kernel.org>' --in-reply-to '2023061830-rubbed-stubble-2775@gregkh' --subject-prefix 'PATCH 4.14.y' HEAD^..

Possible dependencies:



thanks,

greg k-h

------------------ original commit in Linus's tree ------------------

From 306320034e8fbe7ee1cc4f5269c55658b4612048 Mon Sep 17 00:00:00 2001
From: Bernhard Seibold <mail@bernhard-seibold.de>
Date: Fri, 2 Jun 2023 15:30:29 +0200
Subject: [PATCH] serial: lantiq: add missing interrupt ack
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Currently, the error interrupt is never acknowledged, so once active it
will stay active indefinitely, causing the handler to be called in an
infinite loop.

Fixes: 2f0fc4159a6a ("SERIAL: Lantiq: Add driver for MIPS Lantiq SOCs.")
Cc: <stable@vger.kernel.org>
Signed-off-by: Bernhard Seibold <mail@bernhard-seibold.de>
Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Message-ID: <20230602133029.546-1-mail@bernhard-seibold.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/tty/serial/lantiq.c b/drivers/tty/serial/lantiq.c
index a58e9277dfad..f1387f1024db 100644
--- a/drivers/tty/serial/lantiq.c
+++ b/drivers/tty/serial/lantiq.c
@@ -250,6 +250,7 @@ lqasc_err_int(int irq, void *_port)
 	struct ltq_uart_port *ltq_port = to_ltq_uart_port(port);
 
 	spin_lock_irqsave(&ltq_port->lock, flags);
+	__raw_writel(ASC_IRNCR_EIR, port->membase + LTQ_ASC_IRNCR);
 	/* clear any pending interrupts */
 	asc_update_bits(0, ASCWHBSTATE_CLRPE | ASCWHBSTATE_CLRFE |
 		ASCWHBSTATE_CLRROE, port->membase + LTQ_ASC_WHBSTATE);


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [PATCH 4.14.y] serial: lantiq: add missing interrupt ack
  2023-06-18 12:49 FAILED: patch "[PATCH] serial: lantiq: add missing interrupt ack" failed to apply to 4.14-stable tree gregkh
@ 2023-06-19 21:02 ` Bernhard Seibold
  2023-06-19 21:10 ` [PATCH 4.14.y v2] " Bernhard Seibold
  1 sibling, 0 replies; 5+ messages in thread
From: Bernhard Seibold @ 2023-06-19 21:02 UTC (permalink / raw)
  To: stable; +Cc: Bernhard Seibold, Ilpo Järvinen, Greg Kroah-Hartman

commit 306320034e8fbe7ee1cc4f5269c55658b4612048 upstream.

Currently, the error interrupt is never acknowledged, so once active it
will stay active indefinitely, causing the handler to be called in an
infinite loop.

Fixes: 2f0fc4159a6a ("SERIAL: Lantiq: Add driver for MIPS Lantiq SOCs.")
Cc: <stable@vger.kernel.org>
Signed-off-by: Bernhard Seibold <mail@bernhard-seibold.de>
Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Message-ID: <20230602133029.546-1-mail@bernhard-seibold.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
---
 drivers/tty/serial/lantiq.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/tty/serial/lantiq.c b/drivers/tty/serial/lantiq.c
index 22df94f107e5..345d8b3d4319 100644
--- a/drivers/tty/serial/lantiq.c
+++ b/drivers/tty/serial/lantiq.c
@@ -1,3 +1,4 @@
+
 /*
  *  Based on drivers/char/serial.c, by Linus Torvalds, Theodore Ts'o.
  *
@@ -263,6 +264,7 @@ lqasc_err_int(int irq, void *_port)
 	unsigned long flags;
 	struct uart_port *port = (struct uart_port *)_port;
 	spin_lock_irqsave(&ltq_asc_lock, flags);
+	__raw_writel(ASC_IRNCR_EIR, port->membase + LTQ_ASC_IRNCR);
 	/* clear any pending interrupts */
 	ltq_w32_mask(0, ASCWHBSTATE_CLRPE | ASCWHBSTATE_CLRFE |
 		ASCWHBSTATE_CLRROE, port->membase + LTQ_ASC_WHBSTATE);
-- 
2.41.0


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [PATCH 4.14.y v2] serial: lantiq: add missing interrupt ack
  2023-06-18 12:49 FAILED: patch "[PATCH] serial: lantiq: add missing interrupt ack" failed to apply to 4.14-stable tree gregkh
  2023-06-19 21:02 ` [PATCH 4.14.y] serial: lantiq: add missing interrupt ack Bernhard Seibold
@ 2023-06-19 21:10 ` Bernhard Seibold
  2023-06-21 18:33   ` Greg Kroah-Hartman
  1 sibling, 1 reply; 5+ messages in thread
From: Bernhard Seibold @ 2023-06-19 21:10 UTC (permalink / raw)
  To: stable; +Cc: Bernhard Seibold, Ilpo Järvinen, Greg Kroah-Hartman

commit 306320034e8fbe7ee1cc4f5269c55658b4612048 upstream.

Currently, the error interrupt is never acknowledged, so once active it
will stay active indefinitely, causing the handler to be called in an
infinite loop.

Fixes: 2f0fc4159a6a ("SERIAL: Lantiq: Add driver for MIPS Lantiq SOCs.")
Cc: <stable@vger.kernel.org>
Signed-off-by: Bernhard Seibold <mail@bernhard-seibold.de>
Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Message-ID: <20230602133029.546-1-mail@bernhard-seibold.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
---
 drivers/tty/serial/lantiq.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/tty/serial/lantiq.c b/drivers/tty/serial/lantiq.c
index 22df94f107e5..195b85176914 100644
--- a/drivers/tty/serial/lantiq.c
+++ b/drivers/tty/serial/lantiq.c
@@ -1,3 +1,4 @@
+
 /*
  *  Based on drivers/char/serial.c, by Linus Torvalds, Theodore Ts'o.
  *
@@ -263,6 +264,7 @@ lqasc_err_int(int irq, void *_port)
 	unsigned long flags;
 	struct uart_port *port = (struct uart_port *)_port;
 	spin_lock_irqsave(&ltq_asc_lock, flags);
+	ltq_w32(ASC_IRNCR_EIR, port->membase + LTQ_ASC_IRNCR);
 	/* clear any pending interrupts */
 	ltq_w32_mask(0, ASCWHBSTATE_CLRPE | ASCWHBSTATE_CLRFE |
 		ASCWHBSTATE_CLRROE, port->membase + LTQ_ASC_WHBSTATE);
-- 
2.41.0


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH 4.14.y v2] serial: lantiq: add missing interrupt ack
  2023-06-19 21:10 ` [PATCH 4.14.y v2] " Bernhard Seibold
@ 2023-06-21 18:33   ` Greg Kroah-Hartman
  2023-06-21 19:54     ` Bernhard Seibold
  0 siblings, 1 reply; 5+ messages in thread
From: Greg Kroah-Hartman @ 2023-06-21 18:33 UTC (permalink / raw)
  To: Bernhard Seibold; +Cc: stable, Ilpo Järvinen

On Mon, Jun 19, 2023 at 11:10:08PM +0200, Bernhard Seibold wrote:
> commit 306320034e8fbe7ee1cc4f5269c55658b4612048 upstream.
> 
> Currently, the error interrupt is never acknowledged, so once active it
> will stay active indefinitely, causing the handler to be called in an
> infinite loop.
> 
> Fixes: 2f0fc4159a6a ("SERIAL: Lantiq: Add driver for MIPS Lantiq SOCs.")
> Cc: <stable@vger.kernel.org>
> Signed-off-by: Bernhard Seibold <mail@bernhard-seibold.de>
> Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
> Message-ID: <20230602133029.546-1-mail@bernhard-seibold.de>
> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> ---
>  drivers/tty/serial/lantiq.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/tty/serial/lantiq.c b/drivers/tty/serial/lantiq.c
> index 22df94f107e5..195b85176914 100644
> --- a/drivers/tty/serial/lantiq.c
> +++ b/drivers/tty/serial/lantiq.c
> @@ -1,3 +1,4 @@
> +
>  /*
>   *  Based on drivers/char/serial.c, by Linus Torvalds, Theodore Ts'o.
>   *

This first chunk was not needed :(

I'll go hand edit it...

greg k-h

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH 4.14.y v2] serial: lantiq: add missing interrupt ack
  2023-06-21 18:33   ` Greg Kroah-Hartman
@ 2023-06-21 19:54     ` Bernhard Seibold
  0 siblings, 0 replies; 5+ messages in thread
From: Bernhard Seibold @ 2023-06-21 19:54 UTC (permalink / raw)
  To: Greg Kroah-Hartman; +Cc: stable, Ilpo Järvinen

On Wed, Jun 21, 2023 at 08:33:05PM +0200, Greg Kroah-Hartman wrote:
> On Mon, Jun 19, 2023 at 11:10:08PM +0200, Bernhard Seibold wrote:
> > commit 306320034e8fbe7ee1cc4f5269c55658b4612048 upstream.
> > 
> > Currently, the error interrupt is never acknowledged, so once active it
> > will stay active indefinitely, causing the handler to be called in an
> > infinite loop.
> > 
> > Fixes: 2f0fc4159a6a ("SERIAL: Lantiq: Add driver for MIPS Lantiq SOCs.")
> > Cc: <stable@vger.kernel.org>
> > Signed-off-by: Bernhard Seibold <mail@bernhard-seibold.de>
> > Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
> > Message-ID: <20230602133029.546-1-mail@bernhard-seibold.de>
> > Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> > ---
> >  drivers/tty/serial/lantiq.c | 2 ++
> >  1 file changed, 2 insertions(+)
> > 
> > diff --git a/drivers/tty/serial/lantiq.c b/drivers/tty/serial/lantiq.c
> > index 22df94f107e5..195b85176914 100644
> > --- a/drivers/tty/serial/lantiq.c
> > +++ b/drivers/tty/serial/lantiq.c
> > @@ -1,3 +1,4 @@
> > +
> >  /*
> >   *  Based on drivers/char/serial.c, by Linus Torvalds, Theodore Ts'o.
> >   *
> 
> This first chunk was not needed :(
> 
> I'll go hand edit it...
> 
> greg k-h

Thanks! Sorry, it was a bit too late at night.

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2023-06-21 19:54 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-06-18 12:49 FAILED: patch "[PATCH] serial: lantiq: add missing interrupt ack" failed to apply to 4.14-stable tree gregkh
2023-06-19 21:02 ` [PATCH 4.14.y] serial: lantiq: add missing interrupt ack Bernhard Seibold
2023-06-19 21:10 ` [PATCH 4.14.y v2] " Bernhard Seibold
2023-06-21 18:33   ` Greg Kroah-Hartman
2023-06-21 19:54     ` Bernhard Seibold

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.