* [PATCH v6 01/14] serial: liteuart: use KBUILD_MODNAME as driver name
2022-11-23 13:04 [PATCH v6 00/14] serial: liteuart: add IRQ support Gabriel Somlo
@ 2022-11-23 13:04 ` Gabriel Somlo
2022-11-23 13:04 ` [PATCH v6 02/14] serial: liteuart: use bit number macros Gabriel Somlo
` (13 subsequent siblings)
14 siblings, 0 replies; 16+ messages in thread
From: Gabriel Somlo @ 2022-11-23 13:04 UTC (permalink / raw)
To: linux-kernel
Cc: linux-serial, gregkh, jirislaby, kgugala, mholenko, joel,
david.abdurachmanov, florent, geert, ilpo.jarvinen
Replace hard-coded instances of "liteuart" with KBUILD_MODNAME.
Signed-off-by: Gabriel Somlo <gsomlo@gmail.com>
Reviewed-by: Geert Uytterhoeven <geert@linux-m68k.org>
---
drivers/tty/serial/liteuart.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/tty/serial/liteuart.c b/drivers/tty/serial/liteuart.c
index 062812fe1b09..db898751ffe3 100644
--- a/drivers/tty/serial/liteuart.c
+++ b/drivers/tty/serial/liteuart.c
@@ -57,7 +57,7 @@ static struct console liteuart_console;
static struct uart_driver liteuart_driver = {
.owner = THIS_MODULE,
- .driver_name = "liteuart",
+ .driver_name = KBUILD_MODNAME,
.dev_name = "ttyLXU",
.major = 0,
.minor = 0,
@@ -321,7 +321,7 @@ static struct platform_driver liteuart_platform_driver = {
.probe = liteuart_probe,
.remove = liteuart_remove,
.driver = {
- .name = "liteuart",
+ .name = KBUILD_MODNAME,
.of_match_table = liteuart_of_match,
},
};
@@ -367,7 +367,7 @@ static int liteuart_console_setup(struct console *co, char *options)
}
static struct console liteuart_console = {
- .name = "liteuart",
+ .name = KBUILD_MODNAME,
.write = liteuart_console_write,
.device = uart_console_device,
.setup = liteuart_console_setup,
--
2.38.1
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH v6 02/14] serial: liteuart: use bit number macros
2022-11-23 13:04 [PATCH v6 00/14] serial: liteuart: add IRQ support Gabriel Somlo
2022-11-23 13:04 ` [PATCH v6 01/14] serial: liteuart: use KBUILD_MODNAME as driver name Gabriel Somlo
@ 2022-11-23 13:04 ` Gabriel Somlo
2022-11-23 13:04 ` [PATCH v6 03/14] serial: liteuart: remove unused uart_ops stubs Gabriel Somlo
` (12 subsequent siblings)
14 siblings, 0 replies; 16+ messages in thread
From: Gabriel Somlo @ 2022-11-23 13:04 UTC (permalink / raw)
To: linux-kernel
Cc: linux-serial, gregkh, jirislaby, kgugala, mholenko, joel,
david.abdurachmanov, florent, geert, ilpo.jarvinen
Replace magic bit constants (e.g., 1, 2, 4) with BIT(x) expressions.
Signed-off-by: Gabriel Somlo <gsomlo@gmail.com>
Reviewed-by: Geert Uytterhoeven <geert@linux-m68k.org>
---
drivers/tty/serial/liteuart.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/drivers/tty/serial/liteuart.c b/drivers/tty/serial/liteuart.c
index db898751ffe3..18c1eb315ee9 100644
--- a/drivers/tty/serial/liteuart.c
+++ b/drivers/tty/serial/liteuart.c
@@ -5,6 +5,7 @@
* Copyright (C) 2019-2020 Antmicro <www.antmicro.com>
*/
+#include <linux/bits.h>
#include <linux/console.h>
#include <linux/litex.h>
#include <linux/module.h>
@@ -38,8 +39,8 @@
#define OFF_EV_ENABLE 0x14
/* events */
-#define EV_TX 0x1
-#define EV_RX 0x2
+#define EV_TX BIT(0)
+#define EV_RX BIT(1)
struct liteuart_port {
struct uart_port port;
--
2.38.1
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH v6 03/14] serial: liteuart: remove unused uart_ops stubs
2022-11-23 13:04 [PATCH v6 00/14] serial: liteuart: add IRQ support Gabriel Somlo
2022-11-23 13:04 ` [PATCH v6 01/14] serial: liteuart: use KBUILD_MODNAME as driver name Gabriel Somlo
2022-11-23 13:04 ` [PATCH v6 02/14] serial: liteuart: use bit number macros Gabriel Somlo
@ 2022-11-23 13:04 ` Gabriel Somlo
2022-11-23 13:04 ` [PATCH v6 04/14] serial: liteuart: don't set unused port fields Gabriel Somlo
` (11 subsequent siblings)
14 siblings, 0 replies; 16+ messages in thread
From: Gabriel Somlo @ 2022-11-23 13:04 UTC (permalink / raw)
To: linux-kernel
Cc: linux-serial, gregkh, jirislaby, kgugala, mholenko, joel,
david.abdurachmanov, florent, geert, ilpo.jarvinen
Remove stub uart_ops methods that are not called unconditionally
from serial_core.
Signed-off-by: Gabriel Somlo <gsomlo@gmail.com>
Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Reviewed-by: Geert Uytterhoeven <geert@linux-m68k.org>
---
drivers/tty/serial/liteuart.c | 17 -----------------
1 file changed, 17 deletions(-)
diff --git a/drivers/tty/serial/liteuart.c b/drivers/tty/serial/liteuart.c
index 18c1eb315ee9..989a4f8d5bd4 100644
--- a/drivers/tty/serial/liteuart.c
+++ b/drivers/tty/serial/liteuart.c
@@ -154,11 +154,6 @@ static void liteuart_stop_rx(struct uart_port *port)
del_timer(&uart->timer);
}
-static void liteuart_break_ctl(struct uart_port *port, int break_state)
-{
- /* LiteUART doesn't support sending break signal */
-}
-
static int liteuart_startup(struct uart_port *port)
{
struct liteuart_port *uart = to_liteuart_port(port);
@@ -197,15 +192,6 @@ static const char *liteuart_type(struct uart_port *port)
return "liteuart";
}
-static void liteuart_release_port(struct uart_port *port)
-{
-}
-
-static int liteuart_request_port(struct uart_port *port)
-{
- return 0;
-}
-
static void liteuart_config_port(struct uart_port *port, int flags)
{
/*
@@ -232,13 +218,10 @@ static const struct uart_ops liteuart_ops = {
.stop_tx = liteuart_stop_tx,
.start_tx = liteuart_start_tx,
.stop_rx = liteuart_stop_rx,
- .break_ctl = liteuart_break_ctl,
.startup = liteuart_startup,
.shutdown = liteuart_shutdown,
.set_termios = liteuart_set_termios,
.type = liteuart_type,
- .release_port = liteuart_release_port,
- .request_port = liteuart_request_port,
.config_port = liteuart_config_port,
.verify_port = liteuart_verify_port,
};
--
2.38.1
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH v6 04/14] serial: liteuart: don't set unused port fields
2022-11-23 13:04 [PATCH v6 00/14] serial: liteuart: add IRQ support Gabriel Somlo
` (2 preceding siblings ...)
2022-11-23 13:04 ` [PATCH v6 03/14] serial: liteuart: remove unused uart_ops stubs Gabriel Somlo
@ 2022-11-23 13:04 ` Gabriel Somlo
2022-11-23 13:04 ` [PATCH v6 05/14] serial: liteuart: minor style fix in liteuart_init() Gabriel Somlo
` (10 subsequent siblings)
14 siblings, 0 replies; 16+ messages in thread
From: Gabriel Somlo @ 2022-11-23 13:04 UTC (permalink / raw)
To: linux-kernel
Cc: linux-serial, gregkh, jirislaby, kgugala, mholenko, joel,
david.abdurachmanov, florent, geert, ilpo.jarvinen
Remove regshift and iobase port fields, since they are unused
by the driver.
Signed-off-by: Gabriel Somlo <gsomlo@gmail.com>
Reviewed-by: Geert Uytterhoeven <geert@linux-m68k.org>
---
drivers/tty/serial/liteuart.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/tty/serial/liteuart.c b/drivers/tty/serial/liteuart.c
index 989a4f8d5bd4..c6eb7eba5af8 100644
--- a/drivers/tty/serial/liteuart.c
+++ b/drivers/tty/serial/liteuart.c
@@ -263,9 +263,7 @@ static int liteuart_probe(struct platform_device *pdev)
port->iotype = UPIO_MEM;
port->flags = UPF_BOOT_AUTOCONF;
port->ops = &liteuart_ops;
- port->regshift = 2;
port->fifosize = 16;
- port->iobase = 1;
port->type = PORT_UNKNOWN;
port->line = dev_id;
spin_lock_init(&port->lock);
--
2.38.1
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH v6 05/14] serial: liteuart: minor style fix in liteuart_init()
2022-11-23 13:04 [PATCH v6 00/14] serial: liteuart: add IRQ support Gabriel Somlo
` (3 preceding siblings ...)
2022-11-23 13:04 ` [PATCH v6 04/14] serial: liteuart: don't set unused port fields Gabriel Somlo
@ 2022-11-23 13:04 ` Gabriel Somlo
2022-11-23 13:04 ` [PATCH v6 06/14] serial: liteuart: move tty_flip_buffer_push() out of rx loop Gabriel Somlo
` (9 subsequent siblings)
14 siblings, 0 replies; 16+ messages in thread
From: Gabriel Somlo @ 2022-11-23 13:04 UTC (permalink / raw)
To: linux-kernel
Cc: linux-serial, gregkh, jirislaby, kgugala, mholenko, joel,
david.abdurachmanov, florent, geert, ilpo.jarvinen
Signed-off-by: Gabriel Somlo <gsomlo@gmail.com>
Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Reviewed-by: Geert Uytterhoeven <geert@linux-m68k.org>
---
drivers/tty/serial/liteuart.c | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/tty/serial/liteuart.c b/drivers/tty/serial/liteuart.c
index c6eb7eba5af8..1e3429bcc2ad 100644
--- a/drivers/tty/serial/liteuart.c
+++ b/drivers/tty/serial/liteuart.c
@@ -397,12 +397,10 @@ static int __init liteuart_init(void)
return res;
res = platform_driver_register(&liteuart_platform_driver);
- if (res) {
+ if (res)
uart_unregister_driver(&liteuart_driver);
- return res;
- }
- return 0;
+ return res;
}
static void __exit liteuart_exit(void)
--
2.38.1
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH v6 06/14] serial: liteuart: move tty_flip_buffer_push() out of rx loop
2022-11-23 13:04 [PATCH v6 00/14] serial: liteuart: add IRQ support Gabriel Somlo
` (4 preceding siblings ...)
2022-11-23 13:04 ` [PATCH v6 05/14] serial: liteuart: minor style fix in liteuart_init() Gabriel Somlo
@ 2022-11-23 13:04 ` Gabriel Somlo
2022-11-23 13:04 ` [PATCH v6 07/14] serial: liteuart: rx loop should only ack rx events Gabriel Somlo
` (8 subsequent siblings)
14 siblings, 0 replies; 16+ messages in thread
From: Gabriel Somlo @ 2022-11-23 13:04 UTC (permalink / raw)
To: linux-kernel
Cc: linux-serial, gregkh, jirislaby, kgugala, mholenko, joel,
david.abdurachmanov, florent, geert, ilpo.jarvinen
Calling tty_flip_buffer_push() for each individual received character
is overkill. Move it out of the rx loop, and only call it once per
set of characters received together.
Signed-off-by: Gabriel Somlo <gsomlo@gmail.com>
Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Reviewed-by: Geert Uytterhoeven <geert@linux-m68k.org>
---
drivers/tty/serial/liteuart.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/tty/serial/liteuart.c b/drivers/tty/serial/liteuart.c
index 1e3429bcc2ad..81a86c5eb393 100644
--- a/drivers/tty/serial/liteuart.c
+++ b/drivers/tty/serial/liteuart.c
@@ -87,10 +87,10 @@ static void liteuart_timer(struct timer_list *t)
/* no overflow bits in status */
if (!(uart_handle_sysrq_char(port, ch)))
uart_insert_char(port, status, 0, ch, flg);
-
- tty_flip_buffer_push(&port->state->port);
}
+ tty_flip_buffer_push(&port->state->port);
+
mod_timer(&uart->timer, jiffies + uart_poll_timeout(port));
}
--
2.38.1
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH v6 07/14] serial: liteuart: rx loop should only ack rx events
2022-11-23 13:04 [PATCH v6 00/14] serial: liteuart: add IRQ support Gabriel Somlo
` (5 preceding siblings ...)
2022-11-23 13:04 ` [PATCH v6 06/14] serial: liteuart: move tty_flip_buffer_push() out of rx loop Gabriel Somlo
@ 2022-11-23 13:04 ` Gabriel Somlo
2022-11-23 13:04 ` [PATCH v6 08/14] serial: liteuart: simplify passing of uart_insert_char() flag Gabriel Somlo
` (7 subsequent siblings)
14 siblings, 0 replies; 16+ messages in thread
From: Gabriel Somlo @ 2022-11-23 13:04 UTC (permalink / raw)
To: linux-kernel
Cc: linux-serial, gregkh, jirislaby, kgugala, mholenko, joel,
david.abdurachmanov, florent, geert, ilpo.jarvinen
While receiving characters, it is necessary to acknowledge each one
by writing to the EV_PENDING register's EV_RX bit. Ensure we do not
also gratuitously set the EV_TX bit in the process.
Signed-off-by: Gabriel Somlo <gsomlo@gmail.com>
---
drivers/tty/serial/liteuart.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/tty/serial/liteuart.c b/drivers/tty/serial/liteuart.c
index 81a86c5eb393..c90ab65fbdcf 100644
--- a/drivers/tty/serial/liteuart.c
+++ b/drivers/tty/serial/liteuart.c
@@ -82,7 +82,7 @@ static void liteuart_timer(struct timer_list *t)
port->icount.rx++;
/* necessary for RXEMPTY to refresh its value */
- litex_write8(membase + OFF_EV_PENDING, EV_TX | EV_RX);
+ litex_write8(membase + OFF_EV_PENDING, EV_RX);
/* no overflow bits in status */
if (!(uart_handle_sysrq_char(port, ch)))
--
2.38.1
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH v6 08/14] serial: liteuart: simplify passing of uart_insert_char() flag
2022-11-23 13:04 [PATCH v6 00/14] serial: liteuart: add IRQ support Gabriel Somlo
` (6 preceding siblings ...)
2022-11-23 13:04 ` [PATCH v6 07/14] serial: liteuart: rx loop should only ack rx events Gabriel Somlo
@ 2022-11-23 13:04 ` Gabriel Somlo
2022-11-23 13:04 ` [PATCH v6 09/14] serial: liteuart: clean up rx loop variables Gabriel Somlo
` (6 subsequent siblings)
14 siblings, 0 replies; 16+ messages in thread
From: Gabriel Somlo @ 2022-11-23 13:04 UTC (permalink / raw)
To: linux-kernel
Cc: linux-serial, gregkh, jirislaby, kgugala, mholenko, joel,
david.abdurachmanov, florent, geert, ilpo.jarvinen
Simply provide the hard-coded TTY_NORMAL flag to uart_insert_char()
directly -- no need to dedicate a variable for that exclusive purpose.
Signed-off-by: Gabriel Somlo <gsomlo@gmail.com>
Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Reviewed-by: Geert Uytterhoeven <geert@linux-m68k.org>
---
drivers/tty/serial/liteuart.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/tty/serial/liteuart.c b/drivers/tty/serial/liteuart.c
index c90ab65fbdcf..81aa7c1da73c 100644
--- a/drivers/tty/serial/liteuart.c
+++ b/drivers/tty/serial/liteuart.c
@@ -73,7 +73,6 @@ static void liteuart_timer(struct timer_list *t)
struct liteuart_port *uart = from_timer(uart, t, timer);
struct uart_port *port = &uart->port;
unsigned char __iomem *membase = port->membase;
- unsigned int flg = TTY_NORMAL;
int ch;
unsigned long status;
@@ -86,7 +85,7 @@ static void liteuart_timer(struct timer_list *t)
/* no overflow bits in status */
if (!(uart_handle_sysrq_char(port, ch)))
- uart_insert_char(port, status, 0, ch, flg);
+ uart_insert_char(port, status, 0, ch, TTY_NORMAL);
}
tty_flip_buffer_push(&port->state->port);
--
2.38.1
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH v6 09/14] serial: liteuart: clean up rx loop variables
2022-11-23 13:04 [PATCH v6 00/14] serial: liteuart: add IRQ support Gabriel Somlo
` (7 preceding siblings ...)
2022-11-23 13:04 ` [PATCH v6 08/14] serial: liteuart: simplify passing of uart_insert_char() flag Gabriel Somlo
@ 2022-11-23 13:04 ` Gabriel Somlo
2022-11-23 13:04 ` [PATCH v6 10/14] serial: liteuart: separate rx loop from poll timer Gabriel Somlo
` (5 subsequent siblings)
14 siblings, 0 replies; 16+ messages in thread
From: Gabriel Somlo @ 2022-11-23 13:04 UTC (permalink / raw)
To: linux-kernel
Cc: linux-serial, gregkh, jirislaby, kgugala, mholenko, joel,
david.abdurachmanov, florent, geert, ilpo.jarvinen
The `status` variable will always be `1` when passed into the call
to `uart_insert_char()`, so it can be eliminated altogether.
Use `u8` as the type for `ch`, as it matches the return type of
the `litex_read8()` call which produces its value.
Signed-off-by: Gabriel Somlo <gsomlo@gmail.com>
Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
---
Changes from v5:
- removing `status` variable altogether
- using `u8` for `ch`, matching the type of the call producing it
rather than that of the calls that consume its value
drivers/tty/serial/liteuart.c | 7 +++----
1 file changed, 3 insertions(+), 4 deletions(-)
diff --git a/drivers/tty/serial/liteuart.c b/drivers/tty/serial/liteuart.c
index 81aa7c1da73c..62bfd2ed9051 100644
--- a/drivers/tty/serial/liteuart.c
+++ b/drivers/tty/serial/liteuart.c
@@ -73,10 +73,9 @@ static void liteuart_timer(struct timer_list *t)
struct liteuart_port *uart = from_timer(uart, t, timer);
struct uart_port *port = &uart->port;
unsigned char __iomem *membase = port->membase;
- int ch;
- unsigned long status;
+ u8 ch;
- while ((status = !litex_read8(membase + OFF_RXEMPTY)) == 1) {
+ while (!litex_read8(membase + OFF_RXEMPTY)) {
ch = litex_read8(membase + OFF_RXTX);
port->icount.rx++;
@@ -85,7 +84,7 @@ static void liteuart_timer(struct timer_list *t)
/* no overflow bits in status */
if (!(uart_handle_sysrq_char(port, ch)))
- uart_insert_char(port, status, 0, ch, TTY_NORMAL);
+ uart_insert_char(port, 1, 0, ch, TTY_NORMAL);
}
tty_flip_buffer_push(&port->state->port);
--
2.38.1
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH v6 10/14] serial: liteuart: separate rx loop from poll timer
2022-11-23 13:04 [PATCH v6 00/14] serial: liteuart: add IRQ support Gabriel Somlo
` (8 preceding siblings ...)
2022-11-23 13:04 ` [PATCH v6 09/14] serial: liteuart: clean up rx loop variables Gabriel Somlo
@ 2022-11-23 13:04 ` Gabriel Somlo
2022-11-23 13:04 ` [PATCH v6 11/14] serial: liteuart: move function definitions Gabriel Somlo
` (4 subsequent siblings)
14 siblings, 0 replies; 16+ messages in thread
From: Gabriel Somlo @ 2022-11-23 13:04 UTC (permalink / raw)
To: linux-kernel
Cc: linux-serial, gregkh, jirislaby, kgugala, mholenko, joel,
david.abdurachmanov, florent, geert, ilpo.jarvinen
Convert the rx loop into its own dedicated function, and (for now)
call it from the poll timer. This is in preparation for adding irq
support to the receive path.
Signed-off-by: Gabriel Somlo <gsomlo@gmail.com>
Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
---
drivers/tty/serial/liteuart.c | 12 +++++++++---
1 file changed, 9 insertions(+), 3 deletions(-)
diff --git a/drivers/tty/serial/liteuart.c b/drivers/tty/serial/liteuart.c
index 62bfd2ed9051..ab6837f3e40d 100644
--- a/drivers/tty/serial/liteuart.c
+++ b/drivers/tty/serial/liteuart.c
@@ -68,10 +68,8 @@ static struct uart_driver liteuart_driver = {
#endif
};
-static void liteuart_timer(struct timer_list *t)
+static void liteuart_rx_chars(struct uart_port *port)
{
- struct liteuart_port *uart = from_timer(uart, t, timer);
- struct uart_port *port = &uart->port;
unsigned char __iomem *membase = port->membase;
u8 ch;
@@ -88,6 +86,14 @@ static void liteuart_timer(struct timer_list *t)
}
tty_flip_buffer_push(&port->state->port);
+}
+
+static void liteuart_timer(struct timer_list *t)
+{
+ struct liteuart_port *uart = from_timer(uart, t, timer);
+ struct uart_port *port = &uart->port;
+
+ liteuart_rx_chars(port);
mod_timer(&uart->timer, jiffies + uart_poll_timeout(port));
}
--
2.38.1
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH v6 11/14] serial: liteuart: move function definitions
2022-11-23 13:04 [PATCH v6 00/14] serial: liteuart: add IRQ support Gabriel Somlo
` (9 preceding siblings ...)
2022-11-23 13:04 ` [PATCH v6 10/14] serial: liteuart: separate rx loop from poll timer Gabriel Somlo
@ 2022-11-23 13:04 ` Gabriel Somlo
2022-11-23 13:04 ` [PATCH v6 12/14] serial: liteuart: add IRQ support for the RX path Gabriel Somlo
` (3 subsequent siblings)
14 siblings, 0 replies; 16+ messages in thread
From: Gabriel Somlo @ 2022-11-23 13:04 UTC (permalink / raw)
To: linux-kernel
Cc: linux-serial, gregkh, jirislaby, kgugala, mholenko, joel,
david.abdurachmanov, florent, geert, ilpo.jarvinen
Move definitions for liteuart_[stop|start]_tx(), liteuart_stop_rx(),
and liteuart_putchar() to a more convenient location in preparation
for adding IRQ support. This patch contains no functional changes.
Signed-off-by: Gabriel Somlo <gsomlo@gmail.com>
Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Reviewed-by: Geert Uytterhoeven <geert@linux-m68k.org>
---
drivers/tty/serial/liteuart.c | 82 +++++++++++++++++------------------
1 file changed, 41 insertions(+), 41 deletions(-)
diff --git a/drivers/tty/serial/liteuart.c b/drivers/tty/serial/liteuart.c
index ab6837f3e40d..0b9d96e5efcf 100644
--- a/drivers/tty/serial/liteuart.c
+++ b/drivers/tty/serial/liteuart.c
@@ -68,6 +68,47 @@ static struct uart_driver liteuart_driver = {
#endif
};
+static void liteuart_putchar(struct uart_port *port, unsigned char ch)
+{
+ while (litex_read8(port->membase + OFF_TXFULL))
+ cpu_relax();
+
+ litex_write8(port->membase + OFF_RXTX, ch);
+}
+
+static void liteuart_stop_tx(struct uart_port *port)
+{
+}
+
+static void liteuart_start_tx(struct uart_port *port)
+{
+ struct circ_buf *xmit = &port->state->xmit;
+ unsigned char ch;
+
+ if (unlikely(port->x_char)) {
+ litex_write8(port->membase + OFF_RXTX, port->x_char);
+ port->icount.tx++;
+ port->x_char = 0;
+ } else if (!uart_circ_empty(xmit)) {
+ while (xmit->head != xmit->tail) {
+ ch = xmit->buf[xmit->tail];
+ uart_xmit_advance(port, 1);
+ liteuart_putchar(port, ch);
+ }
+ }
+
+ if (uart_circ_chars_pending(xmit) < WAKEUP_CHARS)
+ uart_write_wakeup(port);
+}
+
+static void liteuart_stop_rx(struct uart_port *port)
+{
+ struct liteuart_port *uart = to_liteuart_port(port);
+
+ /* just delete timer */
+ del_timer(&uart->timer);
+}
+
static void liteuart_rx_chars(struct uart_port *port)
{
unsigned char __iomem *membase = port->membase;
@@ -98,14 +139,6 @@ static void liteuart_timer(struct timer_list *t)
mod_timer(&uart->timer, jiffies + uart_poll_timeout(port));
}
-static void liteuart_putchar(struct uart_port *port, unsigned char ch)
-{
- while (litex_read8(port->membase + OFF_TXFULL))
- cpu_relax();
-
- litex_write8(port->membase + OFF_RXTX, ch);
-}
-
static unsigned int liteuart_tx_empty(struct uart_port *port)
{
/* not really tx empty, just checking if tx is not full */
@@ -125,39 +158,6 @@ static unsigned int liteuart_get_mctrl(struct uart_port *port)
return TIOCM_CTS | TIOCM_DSR | TIOCM_CAR;
}
-static void liteuart_stop_tx(struct uart_port *port)
-{
-}
-
-static void liteuart_start_tx(struct uart_port *port)
-{
- struct circ_buf *xmit = &port->state->xmit;
- unsigned char ch;
-
- if (unlikely(port->x_char)) {
- litex_write8(port->membase + OFF_RXTX, port->x_char);
- port->icount.tx++;
- port->x_char = 0;
- } else if (!uart_circ_empty(xmit)) {
- while (xmit->head != xmit->tail) {
- ch = xmit->buf[xmit->tail];
- uart_xmit_advance(port, 1);
- liteuart_putchar(port, ch);
- }
- }
-
- if (uart_circ_chars_pending(xmit) < WAKEUP_CHARS)
- uart_write_wakeup(port);
-}
-
-static void liteuart_stop_rx(struct uart_port *port)
-{
- struct liteuart_port *uart = to_liteuart_port(port);
-
- /* just delete timer */
- del_timer(&uart->timer);
-}
-
static int liteuart_startup(struct uart_port *port)
{
struct liteuart_port *uart = to_liteuart_port(port);
--
2.38.1
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH v6 12/14] serial: liteuart: add IRQ support for the RX path
2022-11-23 13:04 [PATCH v6 00/14] serial: liteuart: add IRQ support Gabriel Somlo
` (10 preceding siblings ...)
2022-11-23 13:04 ` [PATCH v6 11/14] serial: liteuart: move function definitions Gabriel Somlo
@ 2022-11-23 13:04 ` Gabriel Somlo
2022-11-23 13:04 ` [PATCH v6 13/14] serial: liteuart: add IRQ support for the TX path Gabriel Somlo
` (2 subsequent siblings)
14 siblings, 0 replies; 16+ messages in thread
From: Gabriel Somlo @ 2022-11-23 13:04 UTC (permalink / raw)
To: linux-kernel
Cc: linux-serial, gregkh, jirislaby, kgugala, mholenko, joel,
david.abdurachmanov, florent, geert, ilpo.jarvinen
Add support for IRQ-driven RX. Support for the TX path will be added
in a separate commit.
Signed-off-by: Gabriel Somlo <gsomlo@gmail.com>
Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
---
Changes from v5:
- s/dev_err/dev_warn/ when falling back to polling in `liteuart_startup()`
- use `spin_[lock_irqsave|unlock_irqrestore]()` in the interrupt handler,
since the same code may also be called in "serving_softirq" context when
using a poll timer instead of hardware interrupts
> Changes from v4:
> - using dev_err() instead of a combo of pr_err() and pr_fmt()
> - dropped "get irq" comment in probe()
>
> Changes from v3:
> - add shadow irq register to support polling mode and avoid reading
> hardware mmio irq register to learn which irq flags are enabled
> - this also simplifies both liteuart_interrupt() and liteuart_startup()
drivers/tty/serial/liteuart.c | 81 ++++++++++++++++++++++++++++++++---
1 file changed, 74 insertions(+), 7 deletions(-)
diff --git a/drivers/tty/serial/liteuart.c b/drivers/tty/serial/liteuart.c
index 0b9d96e5efcf..8685c97d391e 100644
--- a/drivers/tty/serial/liteuart.c
+++ b/drivers/tty/serial/liteuart.c
@@ -7,6 +7,7 @@
#include <linux/bits.h>
#include <linux/console.h>
+#include <linux/interrupt.h>
#include <linux/litex.h>
#include <linux/module.h>
#include <linux/of.h>
@@ -46,6 +47,7 @@ struct liteuart_port {
struct uart_port port;
struct timer_list timer;
u32 id;
+ u8 irq_reg;
};
#define to_liteuart_port(port) container_of(port, struct liteuart_port, port)
@@ -76,6 +78,19 @@ static void liteuart_putchar(struct uart_port *port, unsigned char ch)
litex_write8(port->membase + OFF_RXTX, ch);
}
+static void liteuart_update_irq_reg(struct uart_port *port, bool set, u8 mask)
+{
+ struct liteuart_port *uart = to_liteuart_port(port);
+
+ if (set)
+ uart->irq_reg |= mask;
+ else
+ uart->irq_reg &= ~mask;
+
+ if (port->irq)
+ litex_write8(port->membase + OFF_EV_ENABLE, uart->irq_reg);
+}
+
static void liteuart_stop_tx(struct uart_port *port)
{
}
@@ -129,13 +144,32 @@ static void liteuart_rx_chars(struct uart_port *port)
tty_flip_buffer_push(&port->state->port);
}
+static irqreturn_t liteuart_interrupt(int irq, void *data)
+{
+ struct liteuart_port *uart = data;
+ struct uart_port *port = &uart->port;
+ unsigned long flags;
+ u8 isr;
+
+ /*
+ * if polling, the context would be "in_serving_softirq", so use
+ * irq[save|restore] spin_lock variants to cover all possibilities
+ */
+ spin_lock_irqsave(&port->lock, flags);
+ isr = litex_read8(port->membase + OFF_EV_PENDING) & uart->irq_reg;
+ if (isr & EV_RX)
+ liteuart_rx_chars(port);
+ spin_unlock_irqrestore(&port->lock, flags);
+
+ return IRQ_RETVAL(isr);
+}
+
static void liteuart_timer(struct timer_list *t)
{
struct liteuart_port *uart = from_timer(uart, t, timer);
struct uart_port *port = &uart->port;
- liteuart_rx_chars(port);
-
+ liteuart_interrupt(0, port);
mod_timer(&uart->timer, jiffies + uart_poll_timeout(port));
}
@@ -161,19 +195,46 @@ static unsigned int liteuart_get_mctrl(struct uart_port *port)
static int liteuart_startup(struct uart_port *port)
{
struct liteuart_port *uart = to_liteuart_port(port);
+ unsigned long flags;
+ int ret;
- /* disable events */
- litex_write8(port->membase + OFF_EV_ENABLE, 0);
+ if (port->irq) {
+ ret = request_irq(port->irq, liteuart_interrupt, 0,
+ KBUILD_MODNAME, uart);
+ if (ret) {
+ dev_warn(port->dev,
+ "line %d irq %d failed: switch to polling\n",
+ port->line, port->irq);
+ port->irq = 0;
+ }
+ }
- /* prepare timer for polling */
- timer_setup(&uart->timer, liteuart_timer, 0);
- mod_timer(&uart->timer, jiffies + uart_poll_timeout(port));
+ spin_lock_irqsave(&port->lock, flags);
+ /* only enabling rx irqs during startup */
+ liteuart_update_irq_reg(port, true, EV_RX);
+ spin_unlock_irqrestore(&port->lock, flags);
+
+ if (!port->irq) {
+ timer_setup(&uart->timer, liteuart_timer, 0);
+ mod_timer(&uart->timer, jiffies + uart_poll_timeout(port));
+ }
return 0;
}
static void liteuart_shutdown(struct uart_port *port)
{
+ struct liteuart_port *uart = to_liteuart_port(port);
+ unsigned long flags;
+
+ spin_lock_irqsave(&port->lock, flags);
+ liteuart_update_irq_reg(port, false, EV_RX | EV_TX);
+ spin_unlock_irqrestore(&port->lock, flags);
+
+ if (port->irq)
+ free_irq(port->irq, port);
+ else
+ del_timer_sync(&uart->timer);
}
static void liteuart_set_termios(struct uart_port *port, struct ktermios *new,
@@ -262,6 +323,12 @@ static int liteuart_probe(struct platform_device *pdev)
goto err_erase_id;
}
+ ret = platform_get_irq_optional(pdev, 0);
+ if (ret < 0 && ret != -ENXIO)
+ return ret;
+ if (ret > 0)
+ port->irq = ret;
+
/* values not from device tree */
port->dev = &pdev->dev;
port->iotype = UPIO_MEM;
--
2.38.1
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH v6 13/14] serial: liteuart: add IRQ support for the TX path
2022-11-23 13:04 [PATCH v6 00/14] serial: liteuart: add IRQ support Gabriel Somlo
` (11 preceding siblings ...)
2022-11-23 13:04 ` [PATCH v6 12/14] serial: liteuart: add IRQ support for the RX path Gabriel Somlo
@ 2022-11-23 13:04 ` Gabriel Somlo
2022-11-23 13:05 ` [PATCH v6 14/14] serial: liteuart: move polling putchar() function Gabriel Somlo
2022-11-24 7:02 ` [PATCH v6 00/14] serial: liteuart: add IRQ support Jiri Slaby
14 siblings, 0 replies; 16+ messages in thread
From: Gabriel Somlo @ 2022-11-23 13:04 UTC (permalink / raw)
To: linux-kernel
Cc: linux-serial, gregkh, jirislaby, kgugala, mholenko, joel,
david.abdurachmanov, florent, geert, ilpo.jarvinen
Switch the TX path to IRQ-driven operation, while maintaining support
for polling mode via the poll timer.
Signed-off-by: Gabriel Somlo <gsomlo@gmail.com>
Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
---
drivers/tty/serial/liteuart.c | 30 +++++++++++++-----------------
1 file changed, 13 insertions(+), 17 deletions(-)
diff --git a/drivers/tty/serial/liteuart.c b/drivers/tty/serial/liteuart.c
index 8685c97d391e..6e9f58d3957c 100644
--- a/drivers/tty/serial/liteuart.c
+++ b/drivers/tty/serial/liteuart.c
@@ -93,27 +93,12 @@ static void liteuart_update_irq_reg(struct uart_port *port, bool set, u8 mask)
static void liteuart_stop_tx(struct uart_port *port)
{
+ liteuart_update_irq_reg(port, false, EV_TX);
}
static void liteuart_start_tx(struct uart_port *port)
{
- struct circ_buf *xmit = &port->state->xmit;
- unsigned char ch;
-
- if (unlikely(port->x_char)) {
- litex_write8(port->membase + OFF_RXTX, port->x_char);
- port->icount.tx++;
- port->x_char = 0;
- } else if (!uart_circ_empty(xmit)) {
- while (xmit->head != xmit->tail) {
- ch = xmit->buf[xmit->tail];
- uart_xmit_advance(port, 1);
- liteuart_putchar(port, ch);
- }
- }
-
- if (uart_circ_chars_pending(xmit) < WAKEUP_CHARS)
- uart_write_wakeup(port);
+ liteuart_update_irq_reg(port, true, EV_TX);
}
static void liteuart_stop_rx(struct uart_port *port)
@@ -144,6 +129,15 @@ static void liteuart_rx_chars(struct uart_port *port)
tty_flip_buffer_push(&port->state->port);
}
+static void liteuart_tx_chars(struct uart_port *port)
+{
+ u8 ch;
+
+ uart_port_tx(port, ch,
+ !litex_read8(port->membase + OFF_TXFULL),
+ litex_write8(port->membase + OFF_RXTX, ch));
+}
+
static irqreturn_t liteuart_interrupt(int irq, void *data)
{
struct liteuart_port *uart = data;
@@ -159,6 +153,8 @@ static irqreturn_t liteuart_interrupt(int irq, void *data)
isr = litex_read8(port->membase + OFF_EV_PENDING) & uart->irq_reg;
if (isr & EV_RX)
liteuart_rx_chars(port);
+ if (isr & EV_TX)
+ liteuart_tx_chars(port);
spin_unlock_irqrestore(&port->lock, flags);
return IRQ_RETVAL(isr);
--
2.38.1
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH v6 14/14] serial: liteuart: move polling putchar() function
2022-11-23 13:04 [PATCH v6 00/14] serial: liteuart: add IRQ support Gabriel Somlo
` (12 preceding siblings ...)
2022-11-23 13:04 ` [PATCH v6 13/14] serial: liteuart: add IRQ support for the TX path Gabriel Somlo
@ 2022-11-23 13:05 ` Gabriel Somlo
2022-11-24 7:02 ` [PATCH v6 00/14] serial: liteuart: add IRQ support Jiri Slaby
14 siblings, 0 replies; 16+ messages in thread
From: Gabriel Somlo @ 2022-11-23 13:05 UTC (permalink / raw)
To: linux-kernel
Cc: linux-serial, gregkh, jirislaby, kgugala, mholenko, joel,
david.abdurachmanov, florent, geert, ilpo.jarvinen
The polling liteuart_putchar() function is only called from methods
conditionally enabled by CONFIG_SERIAL_LITEUART_CONSOLE. Move its
definition closer to the console code where it is dependent on the
same config option.
Signed-off-by: Gabriel Somlo <gsomlo@gmail.com>
Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
---
drivers/tty/serial/liteuart.c | 16 ++++++++--------
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/drivers/tty/serial/liteuart.c b/drivers/tty/serial/liteuart.c
index 6e9f58d3957c..ef557d59e4c8 100644
--- a/drivers/tty/serial/liteuart.c
+++ b/drivers/tty/serial/liteuart.c
@@ -70,14 +70,6 @@ static struct uart_driver liteuart_driver = {
#endif
};
-static void liteuart_putchar(struct uart_port *port, unsigned char ch)
-{
- while (litex_read8(port->membase + OFF_TXFULL))
- cpu_relax();
-
- litex_write8(port->membase + OFF_RXTX, ch);
-}
-
static void liteuart_update_irq_reg(struct uart_port *port, bool set, u8 mask)
{
struct liteuart_port *uart = to_liteuart_port(port);
@@ -377,6 +369,14 @@ static struct platform_driver liteuart_platform_driver = {
#ifdef CONFIG_SERIAL_LITEUART_CONSOLE
+static void liteuart_putchar(struct uart_port *port, unsigned char ch)
+{
+ while (litex_read8(port->membase + OFF_TXFULL))
+ cpu_relax();
+
+ litex_write8(port->membase + OFF_RXTX, ch);
+}
+
static void liteuart_console_write(struct console *co, const char *s,
unsigned int count)
{
--
2.38.1
^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [PATCH v6 00/14] serial: liteuart: add IRQ support
2022-11-23 13:04 [PATCH v6 00/14] serial: liteuart: add IRQ support Gabriel Somlo
` (13 preceding siblings ...)
2022-11-23 13:05 ` [PATCH v6 14/14] serial: liteuart: move polling putchar() function Gabriel Somlo
@ 2022-11-24 7:02 ` Jiri Slaby
14 siblings, 0 replies; 16+ messages in thread
From: Jiri Slaby @ 2022-11-24 7:02 UTC (permalink / raw)
To: Gabriel Somlo, linux-kernel
Cc: linux-serial, gregkh, kgugala, mholenko, joel,
david.abdurachmanov, florent, geert, ilpo.jarvinen
On 23. 11. 22, 14:04, Gabriel Somlo wrote:
> Add IRQ support to the LiteX LiteUART serial interface
LGTM
Reviewed-by: Jiri Slaby <jirislaby@kernel.org>
thanks,
--
js
suse labs
^ permalink raw reply [flat|nested] 16+ messages in thread