On Tue, 2021-09-14 at 11:14 +0200, 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 the couple
> of
> remaining users.
>
> Signed-off-by: Jiri Slaby <jslaby@xxxxxxx>
> Cc: Arnd Bergmann <arnd@xxxxxxxx>
> Cc: Samuel Iglesias Gonsalvez <siglesias@xxxxxxxxxx>
> Cc: Jens Taprogge <jens.taprogge@xxxxxxxxxxxx>
> Cc: Scott Branden <scott.branden@xxxxxxxxxxxx>
> Cc: Ulf Hansson <ulf.hansson@xxxxxxxxxx>
> Cc: "David S. Miller" <davem@xxxxxxxxxxxxx>
> Cc: Jakub Kicinski <kuba@xxxxxxxxxx>
> Cc: Marcel Holtmann <marcel@xxxxxxxxxxxx>
> Cc: Johan Hedberg <johan.hedberg@xxxxxxxxx>
> Cc: Luiz Augusto von Dentz <luiz.dentz@xxxxxxxxx>
> ---
> drivers/char/pcmcia/synclink_cs.c | 2 +-
> drivers/ipack/devices/ipoctal.c | 2 +-
> drivers/misc/bcm-vk/bcm_vk_tty.c | 2 +-
> drivers/mmc/core/sdio_uart.c | 2 +-
> drivers/net/usb/hso.c | 2 +-
> net/bluetooth/rfcomm/tty.c | 4 ++--
> 6 files changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/char/pcmcia/synclink_cs.c
> b/drivers/char/pcmcia/synclink_cs.c
> index 78baba55a8b5..29a7a89ee58d 100644
> --- a/drivers/char/pcmcia/synclink_cs.c
> +++ b/drivers/char/pcmcia/synclink_cs.c
> @@ -952,7 +952,7 @@ static void rx_ready_async(MGSLPC_INFO *info, int
> tcd)
> }
>
> if (work)
> - tty_flip_buffer_push(port);
> + tty_schedule_flip(port);
> }
>
>
> diff --git a/drivers/ipack/devices/ipoctal.c
> b/drivers/ipack/devices/ipoctal.c
> index c14e65a5d38f..714e2b8c551f 100644
> --- a/drivers/ipack/devices/ipoctal.c
> +++ b/drivers/ipack/devices/ipoctal.c
> @@ -184,7 +184,7 @@ static void ipoctal_irq_rx(struct ipoctal_channel
> *channel, u8 sr)
> sr = ioread8(&channel->regs->r.sr);
> } while (isr & channel->isr_rx_rdy_mask);
>
> - tty_flip_buffer_push(port);
> + tty_schedule_flip(port);
> }
>
For ipoctal changes:
Acked-by: Samuel Iglesias Gonsálvez <siglesias@xxxxxxxxxx>
Sam
> static void ipoctal_irq_tx(struct ipoctal_channel *channel)
> diff --git a/drivers/misc/bcm-vk/bcm_vk_tty.c b/drivers/misc/bcm-
> vk/bcm_vk_tty.c
> index 1b6076a89ca6..328109a719be 100644
> --- a/drivers/misc/bcm-vk/bcm_vk_tty.c
> +++ b/drivers/misc/bcm-vk/bcm_vk_tty.c
> @@ -114,7 +114,7 @@ static void bcm_vk_tty_wq_handler(struct
> work_struct *work)
> }
>
> if (count) {
> - tty_flip_buffer_push(&vktty->port);
> + tty_schedule_flip(&vktty->port);
>
> /* Update read offset from shadow register to
> card */
> vkwrite32(vk, vktty->rd, BAR_1,
> diff --git a/drivers/mmc/core/sdio_uart.c
> b/drivers/mmc/core/sdio_uart.c
> index 04c0823e0359..55fc67bd0937 100644
> --- a/drivers/mmc/core/sdio_uart.c
> +++ b/drivers/mmc/core/sdio_uart.c
> @@ -418,7 +418,7 @@ static void sdio_uart_receive_chars(struct
> sdio_uart_port *port,
> *status = sdio_in(port, UART_LSR);
> } while ((*status & UART_LSR_DR) && (max_count-- > 0));
>
> - tty_flip_buffer_push(&port->port);
> + tty_schedule_flip(&port->port);
> }
>
> static void sdio_uart_transmit_chars(struct sdio_uart_port *port)
> diff --git a/drivers/net/usb/hso.c b/drivers/net/usb/hso.c
> index 7dc1ef3f93c3..55ea64b162e1 100644
> --- a/drivers/net/usb/hso.c
> +++ b/drivers/net/usb/hso.c
> @@ -2018,7 +2018,7 @@ static int put_rxbuf_data(struct urb *urb,
> struct hso_serial *serial)
> if (count >= urb->actual_length) {
> tty_insert_flip_string(&serial->port, urb-
> >transfer_buffer,
> urb->actual_length);
> - tty_flip_buffer_push(&serial->port);
> + tty_schedule_flip(&serial->port);
> } else {
> dev_warn(&serial->parent->usb->dev,
> "dropping data, %d bytes lost\n", urb-
> >actual_length);
> diff --git a/net/bluetooth/rfcomm/tty.c b/net/bluetooth/rfcomm/tty.c
> index ebd78fdbd6e8..2a09637bdb00 100644
> --- a/net/bluetooth/rfcomm/tty.c
> +++ b/net/bluetooth/rfcomm/tty.c
> @@ -618,7 +618,7 @@ static void rfcomm_dev_data_ready(struct
> rfcomm_dlc *dlc, struct sk_buff *skb)
> BT_DBG("dlc %p len %d", dlc, skb->len);
>
> tty_insert_flip_string(&dev->port, skb->data, skb->len);
> - tty_flip_buffer_push(&dev->port);
> + tty_schedule_flip(&dev->port);
>
> kfree_skb(skb);
> }
> @@ -677,7 +677,7 @@ static void rfcomm_tty_copy_pending(struct
> rfcomm_dev *dev)
> rfcomm_dlc_unlock(dev->dlc);
>
> if (inserted > 0)
> - tty_flip_buffer_push(&dev->port);
> + tty_schedule_flip(&dev->port);
> }
>
> /* do the reverse of install, clearing the tty fields and releasing
> the
Attachment:
signature.asc
Description: This is a digitally signed message part
