[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <ee234a02-f218-45ae-9261-5f71ff48e840@kernel.org>
Date: Mon, 17 Mar 2025 08:03:26 +0100
From: Jiri Slaby <jirislaby@...nel.org>
To: gregkh@...uxfoundation.org
Cc: linux-serial@...r.kernel.org, linux-kernel@...r.kernel.org,
Alex Elder <elder@...nel.org>, Andrew Lunn <andrew+netdev@...n.ch>,
Andy Shevchenko <andriy.shevchenko@...ux.intel.com>,
David Lin <dtwlin@...il.com>, "David S. Miller" <davem@...emloft.net>,
Eric Dumazet <edumazet@...gle.com>, greybus-dev@...ts.linaro.org,
Ilpo Järvinen <ilpo.jarvinen@...ux.intel.com>,
Jakub Kicinski <kuba@...nel.org>, Johan Hovold <johan@...nel.org>,
linux-alpha@...r.kernel.org, linux-mmc@...r.kernel.org,
linux-staging@...ts.linux.dev, Matt Turner <mattst88@...il.com>,
netdev@...r.kernel.org, Paolo Abeni <pabeni@...hat.com>,
Richard Henderson <richard.henderson@...aro.org>,
Rob Herring <robh@...nel.org>, sparclinux@...r.kernel.org,
Ulf Hansson <ulf.hansson@...aro.org>
Subject: Re: [PATCH v2 00/31] tty: cleanup no. 99
Cc (sorry):
Alex Elder <elder@...nel.org>
Andrew Lunn <andrew+netdev@...n.ch>
Andy Shevchenko <andriy.shevchenko@...ux.intel.com>
David Lin <dtwlin@...il.com>
"David S. Miller" <davem@...emloft.net>
Eric Dumazet <edumazet@...gle.com>
greybus-dev@...ts.linaro.org
"Ilpo Järvinen" <ilpo.jarvinen@...ux.intel.com>
Jakub Kicinski <kuba@...nel.org>
Johan Hovold <johan@...nel.org>
linux-alpha@...r.kernel.org
linux-mmc@...r.kernel.org
linux-staging@...ts.linux.dev
Matt Turner <mattst88@...il.com>
netdev@...r.kernel.org
Paolo Abeni <pabeni@...hat.com>
Richard Henderson <richard.henderson@...aro.org>
Rob Herring <robh@...nel.org>
sparclinux@...r.kernel.org
Ulf Hansson <ulf.hansson@...aro.org>
On 17. 03. 25, 8:00, Jiri Slaby (SUSE) wrote:
> Hi,
>
> this is (again) a series of cleanup in tty. I am trying to rework
> tty+serial to avoid limitations of devices (so called NR_UART or
> tty_alloc_driver()'s first parameter). And the below popped up while
> crawling through the code. So this is only a prep cleanup.
>
> * many tty flags are now enums
> * many functions were improved for readability
> * quite a few unused or old code dropped
>
> In particular, the runtime behaviour of the kernel before and after the
> changes is supposed to be bug to bug compatible (except moxa's ioctl
> and ISA evils dropped). That is, noone should notice.
>
> [v2]
> * use serial_port_in/out() helpers in 26/30 (and not use serial_in/out())
> * the last patch is new
>
> Jiri Slaby (SUSE) (31):
> tty: convert "TTY Struct Flags" to an enum
> tty: audit: do not use N_TTY_BUF_SIZE
> tty: caif: do not use N_TTY_BUF_SIZE
> tty: move N_TTY_BUF_SIZE to n_tty
> tty: n_tty: use uint for space returned by tty_write_room()
> tty: n_tty: simplify process_output()
> tty: n_tty: clean up process_output_block()
> tty: n_tty: drop n_tty_trace()
> tty: n_tty: extract n_tty_continue_cookie() from n_tty_read()
> tty: n_tty: extract n_tty_wait_for_input()
> tty: n_tty: move more_to_be_read to the end of n_tty_read()
> tty: tty_driver: move TTY macros to the top
> tty: tty_driver: convert "TTY Driver Flags" to an enum
> tty: tty_driver: document both {,__}tty_alloc_driver() properly
> tty: tty_driver: introduce TTY driver sub/types enums
> tty: serdev: drop serdev_controller_ops::write_room()
> tty: mmc: sdio: use bool for cts and remove parentheses
> tty: moxa: drop version dump to logs
> tty: moxa: drop ISA support
> tty: moxa: carve out special ioctls and extra tty_port
> tty: srmcons: fix retval from srmcons_init()
> tty: staging/greybus: pass tty_driver flags to tty_alloc_driver()
> tty: sunsu: drop serial_{in,out}p()
> tty: sunsu: remove unused serial_icr_read()
> serial: remove redundant tty_port_link_device()
> serial: pass struct uart_state to uart_line_info()
> serial: 8250: use serial_port_in/out() helpers
> serial: 8250_rsa: simplify rsa8250_{request/release}_resource()
> serial: 8250_port: do not use goto for UPQ_NO_TXEN_TEST code flow
> serial: 8250_port: simplify serial8250_request_std_resource()
> serial: switch change_irq and change_port to bool in uart_set_info()
>
> Documentation/driver-api/tty/tty_driver.rst | 4 +-
> Documentation/driver-api/tty/tty_struct.rst | 2 +-
> arch/alpha/kernel/srmcons.c | 62 ++---
> drivers/mmc/core/sdio_uart.c | 2 +-
> drivers/net/caif/caif_serial.c | 2 +-
> drivers/staging/greybus/uart.c | 4 +-
> drivers/tty/Kconfig | 2 +-
> drivers/tty/moxa.c | 251 +-------------------
> drivers/tty/n_tty.c | 212 ++++++++---------
> drivers/tty/serdev/core.c | 11 -
> drivers/tty/serdev/serdev-ttyport.c | 9 -
> drivers/tty/serial/8250/8250_dw.c | 16 +-
> drivers/tty/serial/8250/8250_fsl.c | 8 +-
> drivers/tty/serial/8250/8250_omap.c | 2 +-
> drivers/tty/serial/8250/8250_port.c | 59 ++---
> drivers/tty/serial/8250/8250_rsa.c | 21 +-
> drivers/tty/serial/serial_core.c | 10 +-
> drivers/tty/serial/sunsu.c | 178 ++++++--------
> drivers/tty/tty_audit.c | 10 +-
> drivers/tty/tty_io.c | 8 +-
> include/linux/serdev.h | 6 -
> include/linux/tty.h | 53 +++--
> include/linux/tty_driver.h | 180 +++++++-------
> 23 files changed, 412 insertions(+), 700 deletions(-)
>
--
js
suse labs
Powered by blists - more mailing lists