On 9/14/21 4:14 AM, Jiri Slaby wrote:
> Since commit a9c3f68f3cd8d (tty: Fix low_latency BUG) in 2014,
> tty_flip_buffer_push() is only a wrapper to tty_schedule_flip(). We are
> going to remove the former, so call the latter directly in
> drivers/staging/.
For "greybus/uart.c", looks good.
Reviewed-by: Alex Elder <elder@xxxxxxxxxx>
>
> Signed-off-by: Jiri Slaby <jslaby@xxxxxxx>
> Cc: David Lin <dtwlin@xxxxxxxxx>
> Cc: Johan Hovold <johan@xxxxxxxxxx>
> Cc: Alex Elder <elder@xxxxxxxxxx>
> Cc: linux-staging@xxxxxxxxxxxxxxx
> Cc: greybus-dev@xxxxxxxxxxxxxxxx
> ---
> drivers/staging/fwserial/fwserial.c | 4 ++--
> drivers/staging/gdm724x/gdm_tty.c | 2 +-
> drivers/staging/greybus/uart.c | 2 +-
> 3 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/staging/fwserial/fwserial.c b/drivers/staging/fwserial/fwserial.c
> index e8fa7f53cd5e..b2d3f95edbc3 100644
> --- a/drivers/staging/fwserial/fwserial.c
> +++ b/drivers/staging/fwserial/fwserial.c
> @@ -518,7 +518,7 @@ static void fwtty_emit_breaks(struct work_struct *work)
> if (c < t)
> break;
> }
> - tty_flip_buffer_push(&port->port);
> + tty_schedule_flip(&port->port);
>
> if (port->mstatus & (UART_LSR_BI << 24))
> schedule_delayed_work(&port->emit_breaks, FREQ_BREAKS);
> @@ -565,7 +565,7 @@ static int fwtty_rx(struct fwtty_port *port, unsigned char *data, size_t len)
>
> c = tty_insert_flip_string_fixed_flag(&port->port, data, TTY_NORMAL, n);
> if (c > 0)
> - tty_flip_buffer_push(&port->port);
> + tty_schedule_flip(&port->port);
> n -= c;
>
> if (n) {
> diff --git a/drivers/staging/gdm724x/gdm_tty.c b/drivers/staging/gdm724x/gdm_tty.c
> index 04df6f9f5403..95a3b4e61fec 100644
> --- a/drivers/staging/gdm724x/gdm_tty.c
> +++ b/drivers/staging/gdm724x/gdm_tty.c
> @@ -129,7 +129,7 @@ static int gdm_tty_recv_complete(void *data,
> if (data && len) {
> if (tty_buffer_request_room(&gdm->port, len) == len) {
> tty_insert_flip_string(&gdm->port, data, len);
> - tty_flip_buffer_push(&gdm->port);
> + tty_schedule_flip(&gdm->port);
> } else {
> return TO_HOST_BUFFER_REQUEST_FAIL;
> }
> diff --git a/drivers/staging/greybus/uart.c b/drivers/staging/greybus/uart.c
> index e6d860a9678e..2bdc2401b652 100644
> --- a/drivers/staging/greybus/uart.c
> +++ b/drivers/staging/greybus/uart.c
> @@ -122,7 +122,7 @@ static int gb_uart_receive_data_handler(struct gb_operation *op)
> recv_data_size, count);
> }
> if (count)
> - tty_flip_buffer_push(port);
> + tty_schedule_flip(port);
> return 0;
> }
>
>
