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; > } > > _______________________________________________ greybus-dev mailing list greybus-dev@xxxxxxxxxxxxxxxx https://lists.linaro.org/mailman/listinfo/greybus-dev