serial/uart/8250: Add tunable RX interrupt trigger I/F of FIFO buffers
authorYoshihiro YUNOMAE <yoshihiro.yunomae.ez@hitachi.com>
Wed, 16 Jul 2014 01:19:36 +0000 (01:19 +0000)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 18 Jul 2014 01:23:35 +0000 (18:23 -0700)
Add tunable RX interrupt trigger I/F of FIFO buffers.

Serial devices are used as not only message communication devices but control
or sending communication devices. For the latter uses, normally small data
will be exchanged, so user applications want to receive data unit as soon as
possible for real-time tendency. If we have a sensor which sends a 1 byte data
each time and must control a device based on the sensor feedback, the RX
interrupt should be triggered for each data.

According to HW specification of serial UART devices, RX interrupt trigger
can be changed, but the trigger is hard-coded. For example, RX interrupt trigger
in 16550A can be set to 1, 4, 8, or 14 bytes for HW, but current driver sets
the trigger to only 8bytes.

This patch makes some devices change RX interrupt trigger from userland.

<How to use>
- Read current setting
 # cat /sys/class/tty/ttyS0/rx_trig_bytes
 8

- Write user setting
 # echo 1 > /sys/class/tty/ttyS0/rx_trig_bytes
 # cat /sys/class/tty/ttyS0/rx_trig_bytes
 1

<Support uart devices>
- 16550A and Tegra (1, 4, 8, or 14 bytes)
- 16650V2 (8, 16, 24, or 28 bytes)
- 16654 (8, 16, 56, or 60 bytes)
- 16750 (1, 16, 32, or 56 bytes)

<Change log>
Changes in V9:
 - Use attr_group instead of dev_spec_attr_group of uart_port structure

Changes in V8:
 - Divide this patch from V7's patch based on Greg's comment

Changes in V7:
 - Add Documentation
 - Change I/F name from rx_int_trig to rx_trig_bytes because the name
   rx_int_trig is hard to understand how users specify the value

Changes in V6:
 - Move FCR_RX_TRIG_* definition in 8250.h to include/uapi/linux/serial_reg.h,
   rename those to UART_FCR_R_TRIG_*, and use UART_FCR_TRIGGER_MASK to
   UART_FCR_R_TRIG_BITS()
 - Change following function names:
    convert_fcr2val() => fcr_get_rxtrig_bytes()
    convert_val2rxtrig() => bytes_to_fcr_rxtrig()
 - Fix typo in serial8250_do_set_termios()
 - Delete the verbose error message pr_info() in bytes_to_fcr_rxtrig()
 - Rename *rx_int_trig/rx_trig* to *rxtrig* for several functions or variables
   (but UI remains rx_int_trig)
 - Change the meaningless variable name 'val' to 'bytes' following functions:
    fcr_get_rxtrig_bytes(), bytes_to_fcr_rxtrig(), do_set_rxtrig(),
    do_serial8250_set_rxtrig(), and serial8250_set_attr_rxtrig()
 - Use up->fcr in order to get rxtrig_bytes instead of rx_trig_raw in
   fcr_get_rxtrig_bytes()
 - Use conf_type->rxtrig_bytes[0] instead of switch statement for support check
   in register_dev_spec_attr_grp()
 - Delete the checking whether a user changed FCR or not when minimum buffer
   is needed in serial8250_do_set_termios()

Changes in V5.1:
 - Fix FCR_RX_TRIG_MAX_STATE definition

Changes in V5:
 - Support Tegra, 16650V2, 16654, and 16750
 - Store default FCR value to up->fcr when the port is first created
 - Add rx_trig_byte[] in uart_config[] for each device and use rx_trig_byte[]
   in convert_fcr2val() and convert_val2rxtrig()

Changes in V4:
 - Introduce fifo_bug flag in uart_8250_port structure
   This is enabled only when parity is enabled and UART_BUG_PARITY is enabled
   for up->bugs. If this flag is enabled, user cannot set RX trigger.
 - Return -EOPNOTSUPP when it does not support device at convert_fcr2val() and
   at convert_val2rxtrig()
 - Set the nearest lower RX trigger when users input a meaningless value at
   convert_val2rxtrig()
 - Check whether p->fcr is existing at serial8250_clear_and_reinit_fifos()
 - Set fcr = up->fcr in the begging of serial8250_do_set_termios()

Changes in V3:
 - Change I/F from ioctl(2) to sysfs(rx_int_trig)

Changed in V2:
 - Use _IOW for TIOCSFIFORTRIG definition
 - Pass the interrupt trigger value itself

Signed-off-by: Yoshihiro YUNOMAE <yoshihiro.yunomae.ez@hitachi.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Documentation/ABI/testing/sysfs-tty
drivers/tty/serial/8250/8250.h
drivers/tty/serial/8250/8250_core.c
include/linux/serial_8250.h
include/uapi/linux/serial_reg.h

index ad22fb0ee765b792a3bc0ab7f9d2995c69f1d5d0..9eb3c2b6b0406eeec340f31a47f837ea4f542258 100644 (file)
@@ -138,3 +138,19 @@ Description:
 
                 These sysfs values expose the TIOCGSERIAL interface via
                 sysfs rather than via ioctls.
+
+What:          /sys/class/tty/ttyS0/rx_trig_bytes
+Date:          May 2014
+Contact:       Yoshihiro YUNOMAE <yoshihiro.yunomae.ez@hitachi.com>
+Description:
+                Shows current RX interrupt trigger bytes or sets the
+                user specified value to change it for the FIFO buffer.
+                Users can show or set this value regardless of opening the
+                serial device file or not.
+
+                The RX trigger can be set one of four kinds of values for UART
+                serials. When users input a meaning less value to this I/F,
+                the RX trigger is changed to the nearest lower value for the
+                device specification. For example, when user sets 7bytes on
+                16550A, which has 1/4/8/14 bytes trigger, the RX trigger is
+                automatically changed to 4 bytes.
index 1ebf8538b4fa8aa31eacd34bc9c8cb2c2159770f..1b08c918cd5183ea1572183102722616de6f399c 100644 (file)
@@ -12,6 +12,7 @@
  */
 
 #include <linux/serial_8250.h>
+#include <linux/serial_reg.h>
 #include <linux/dmaengine.h>
 
 struct uart_8250_dma {
@@ -60,6 +61,7 @@ struct serial8250_config {
        unsigned short  fifo_size;
        unsigned short  tx_loadsz;
        unsigned char   fcr;
+       unsigned char   rxtrig_bytes[UART_FCR_R_TRIG_MAX_STATE];
        unsigned int    flags;
 };
 
index 0da01458816e9a40f5eb5b35f3e83d17b419430a..1d42dba6121d029eb9e900df9488b0d3a30f54dd 100644 (file)
@@ -31,7 +31,6 @@
 #include <linux/tty.h>
 #include <linux/ratelimit.h>
 #include <linux/tty_flip.h>
-#include <linux/serial_reg.h>
 #include <linux/serial_core.h>
 #include <linux/serial.h>
 #include <linux/serial_8250.h>
@@ -161,6 +160,7 @@ static const struct serial8250_config uart_config[] = {
                .fifo_size      = 16,
                .tx_loadsz      = 16,
                .fcr            = UART_FCR_ENABLE_FIFO | UART_FCR_R_TRIG_10,
+               .rxtrig_bytes   = {1, 4, 8, 14},
                .flags          = UART_CAP_FIFO,
        },
        [PORT_CIRRUS] = {
@@ -180,6 +180,7 @@ static const struct serial8250_config uart_config[] = {
                .tx_loadsz      = 16,
                .fcr            = UART_FCR_ENABLE_FIFO | UART_FCR_R_TRIG_01 |
                                  UART_FCR_T_TRIG_00,
+               .rxtrig_bytes   = {8, 16, 24, 28},
                .flags          = UART_CAP_FIFO | UART_CAP_EFR | UART_CAP_SLEEP,
        },
        [PORT_16750] = {
@@ -188,6 +189,7 @@ static const struct serial8250_config uart_config[] = {
                .tx_loadsz      = 64,
                .fcr            = UART_FCR_ENABLE_FIFO | UART_FCR_R_TRIG_10 |
                                  UART_FCR7_64BYTE,
+               .rxtrig_bytes   = {1, 16, 32, 56},
                .flags          = UART_CAP_FIFO | UART_CAP_SLEEP | UART_CAP_AFE,
        },
        [PORT_STARTECH] = {
@@ -209,6 +211,7 @@ static const struct serial8250_config uart_config[] = {
                .tx_loadsz      = 32,
                .fcr            = UART_FCR_ENABLE_FIFO | UART_FCR_R_TRIG_01 |
                                  UART_FCR_T_TRIG_10,
+               .rxtrig_bytes   = {8, 16, 56, 60},
                .flags          = UART_CAP_FIFO | UART_CAP_EFR | UART_CAP_SLEEP,
        },
        [PORT_16850] = {
@@ -266,6 +269,7 @@ static const struct serial8250_config uart_config[] = {
                .tx_loadsz      = 8,
                .fcr            = UART_FCR_ENABLE_FIFO | UART_FCR_R_TRIG_01 |
                                  UART_FCR_T_TRIG_01,
+               .rxtrig_bytes   = {1, 4, 8, 14},
                .flags          = UART_CAP_FIFO | UART_CAP_RTOIE,
        },
        [PORT_XR17D15X] = {
@@ -530,11 +534,8 @@ static void serial8250_clear_fifos(struct uart_8250_port *p)
 
 void serial8250_clear_and_reinit_fifos(struct uart_8250_port *p)
 {
-       unsigned char fcr;
-
        serial8250_clear_fifos(p);
-       fcr = uart_config[p->port.type].fcr;
-       serial_out(p, UART_FCR, fcr);
+       serial_out(p, UART_FCR, p->fcr);
 }
 EXPORT_SYMBOL_GPL(serial8250_clear_and_reinit_fifos);
 
@@ -2256,10 +2257,9 @@ serial8250_do_set_termios(struct uart_port *port, struct ktermios *termios,
                          struct ktermios *old)
 {
        struct uart_8250_port *up = up_to_u8250p(port);
-       unsigned char cval, fcr = 0;
+       unsigned char cval;
        unsigned long flags;
        unsigned int baud, quot;
-       int fifo_bug = 0;
 
        switch (termios->c_cflag & CSIZE) {
        case CS5:
@@ -2282,7 +2282,7 @@ serial8250_do_set_termios(struct uart_port *port, struct ktermios *termios,
        if (termios->c_cflag & PARENB) {
                cval |= UART_LCR_PARITY;
                if (up->bugs & UART_BUG_PARITY)
-                       fifo_bug = 1;
+                       up->fifo_bug = true;
        }
        if (!(termios->c_cflag & PARODD))
                cval |= UART_LCR_EPAR;
@@ -2306,10 +2306,10 @@ serial8250_do_set_termios(struct uart_port *port, struct ktermios *termios,
                quot++;
 
        if (up->capabilities & UART_CAP_FIFO && port->fifosize > 1) {
-               fcr = uart_config[port->type].fcr;
-               if ((baud < 2400 && !up->dma) || fifo_bug) {
-                       fcr &= ~UART_FCR_TRIGGER_MASK;
-                       fcr |= UART_FCR_TRIGGER_1;
+               /* NOTE: If fifo_bug is not set, a user can set RX_trigger. */
+               if ((baud < 2400 && !up->dma) || up->fifo_bug) {
+                       up->fcr &= ~UART_FCR_TRIGGER_MASK;
+                       up->fcr |= UART_FCR_TRIGGER_1;
                }
        }
 
@@ -2442,15 +2442,15 @@ serial8250_do_set_termios(struct uart_port *port, struct ktermios *termios,
         * is written without DLAB set, this mode will be disabled.
         */
        if (port->type == PORT_16750)
-               serial_port_out(port, UART_FCR, fcr);
+               serial_port_out(port, UART_FCR, up->fcr);
 
        serial_port_out(port, UART_LCR, cval);          /* reset DLAB */
        up->lcr = cval;                                 /* Save LCR */
        if (port->type != PORT_16750) {
                /* emulated UARTs (Lucent Venus 167x) need two steps */
-               if (fcr & UART_FCR_ENABLE_FIFO)
+               if (up->fcr & UART_FCR_ENABLE_FIFO)
                        serial_port_out(port, UART_FCR, UART_FCR_ENABLE_FIFO);
-               serial_port_out(port, UART_FCR, fcr);           /* set fcr */
+               serial_port_out(port, UART_FCR, up->fcr);       /* set fcr */
        }
        serial8250_set_mctrl(port, port->mctrl);
        spin_unlock_irqrestore(&port->lock, flags);
@@ -2640,6 +2640,146 @@ static int serial8250_request_port(struct uart_port *port)
        return ret;
 }
 
+static int fcr_get_rxtrig_bytes(struct uart_8250_port *up)
+{
+       const struct serial8250_config *conf_type = &uart_config[up->port.type];
+       unsigned char bytes;
+
+       bytes = conf_type->rxtrig_bytes[UART_FCR_R_TRIG_BITS(up->fcr)];
+
+       return bytes ? bytes : -EOPNOTSUPP;
+}
+
+static int bytes_to_fcr_rxtrig(struct uart_8250_port *up, unsigned char bytes)
+{
+       const struct serial8250_config *conf_type = &uart_config[up->port.type];
+       int i;
+
+       if (!conf_type->rxtrig_bytes[UART_FCR_R_TRIG_BITS(UART_FCR_R_TRIG_00)])
+               return -EOPNOTSUPP;
+
+       for (i = 1; i < UART_FCR_R_TRIG_MAX_STATE; i++) {
+               if (bytes < conf_type->rxtrig_bytes[i])
+                       /* Use the nearest lower value */
+                       return (--i) << UART_FCR_R_TRIG_SHIFT;
+       }
+
+       return UART_FCR_R_TRIG_11;
+}
+
+static int do_get_rxtrig(struct tty_port *port)
+{
+       struct uart_state *state = container_of(port, struct uart_state, port);
+       struct uart_port *uport = state->uart_port;
+       struct uart_8250_port *up =
+               container_of(uport, struct uart_8250_port, port);
+
+       if (!(up->capabilities & UART_CAP_FIFO) || uport->fifosize <= 1)
+               return -EINVAL;
+
+       return fcr_get_rxtrig_bytes(up);
+}
+
+static int do_serial8250_get_rxtrig(struct tty_port *port)
+{
+       int rxtrig_bytes;
+
+       mutex_lock(&port->mutex);
+       rxtrig_bytes = do_get_rxtrig(port);
+       mutex_unlock(&port->mutex);
+
+       return rxtrig_bytes;
+}
+
+static ssize_t serial8250_get_attr_rx_trig_bytes(struct device *dev,
+       struct device_attribute *attr, char *buf)
+{
+       struct tty_port *port = dev_get_drvdata(dev);
+       int rxtrig_bytes;
+
+       rxtrig_bytes = do_serial8250_get_rxtrig(port);
+       if (rxtrig_bytes < 0)
+               return rxtrig_bytes;
+
+       return snprintf(buf, PAGE_SIZE, "%d\n", rxtrig_bytes);
+}
+
+static int do_set_rxtrig(struct tty_port *port, unsigned char bytes)
+{
+       struct uart_state *state = container_of(port, struct uart_state, port);
+       struct uart_port *uport = state->uart_port;
+       struct uart_8250_port *up =
+               container_of(uport, struct uart_8250_port, port);
+       int rxtrig;
+
+       if (!(up->capabilities & UART_CAP_FIFO) || uport->fifosize <= 1 ||
+           up->fifo_bug)
+               return -EINVAL;
+
+       rxtrig = bytes_to_fcr_rxtrig(up, bytes);
+       if (rxtrig < 0)
+               return rxtrig;
+
+       serial8250_clear_fifos(up);
+       up->fcr &= ~UART_FCR_TRIGGER_MASK;
+       up->fcr |= (unsigned char)rxtrig;
+       serial_out(up, UART_FCR, up->fcr);
+       return 0;
+}
+
+static int do_serial8250_set_rxtrig(struct tty_port *port, unsigned char bytes)
+{
+       int ret;
+
+       mutex_lock(&port->mutex);
+       ret = do_set_rxtrig(port, bytes);
+       mutex_unlock(&port->mutex);
+
+       return ret;
+}
+
+static ssize_t serial8250_set_attr_rx_trig_bytes(struct device *dev,
+       struct device_attribute *attr, const char *buf, size_t count)
+{
+       struct tty_port *port = dev_get_drvdata(dev);
+       unsigned char bytes;
+       int ret;
+
+       if (!count)
+               return -EINVAL;
+
+       ret = kstrtou8(buf, 10, &bytes);
+       if (ret < 0)
+               return ret;
+
+       ret = do_serial8250_set_rxtrig(port, bytes);
+       if (ret < 0)
+               return ret;
+
+       return count;
+}
+
+static DEVICE_ATTR(rx_trig_bytes, S_IRUSR | S_IWUSR | S_IRGRP,
+                  serial8250_get_attr_rx_trig_bytes,
+                  serial8250_set_attr_rx_trig_bytes);
+
+static struct attribute *serial8250_dev_attrs[] = {
+       &dev_attr_rx_trig_bytes.attr,
+       NULL,
+       };
+
+static struct attribute_group serial8250_dev_attr_group = {
+       .attrs = serial8250_dev_attrs,
+       };
+
+static void register_dev_spec_attr_grp(struct uart_8250_port *up)
+{
+       const struct serial8250_config *conf_type = &uart_config[up->port.type];
+
+       if (conf_type->rxtrig_bytes[0])
+               up->port.attr_group = &serial8250_dev_attr_group;
+}
+
 static void serial8250_config_port(struct uart_port *port, int flags)
 {
        struct uart_8250_port *up = up_to_u8250p(port);
@@ -2687,6 +2827,9 @@ static void serial8250_config_port(struct uart_port *port, int flags)
        if ((port->type == PORT_XR17V35X) ||
           (port->type == PORT_XR17D15X))
                port->handle_irq = exar_handle_irq;
+
+       register_dev_spec_attr_grp(up);
+       up->fcr = uart_config[up->port.type].fcr;
 }
 
 static int
index 730ab4b3d686d902e8e4ed871ef96afe47d1e2b0..f93649e22c438369983ec8a393da3f05d69bea76 100644 (file)
@@ -74,8 +74,10 @@ struct uart_8250_port {
        struct list_head        list;           /* ports on this IRQ */
        unsigned short          capabilities;   /* port capabilities */
        unsigned short          bugs;           /* port bugs */
+       bool                    fifo_bug;       /* min RX trigger if enabled */
        unsigned int            tx_loadsz;      /* transmit fifo load size */
        unsigned char           acr;
+       unsigned char           fcr;
        unsigned char           ier;
        unsigned char           lcr;
        unsigned char           mcr;
index 99b47058816a8858c3116f2c76bd50f4a217a3ce..df6c9ab6b0cd0b7d7283b52b2124f713c6b8c796 100644 (file)
 #define UART_FCR6_T_TRIGGER_30 0x30 /* Mask for transmit trigger set at 30 */
 #define UART_FCR7_64BYTE       0x20 /* Go into 64 byte mode (TI16C750) */
 
+#define UART_FCR_R_TRIG_SHIFT          6
+#define UART_FCR_R_TRIG_BITS(x)                \
+       (((x) & UART_FCR_TRIGGER_MASK) >> UART_FCR_R_TRIG_SHIFT)
+#define UART_FCR_R_TRIG_MAX_STATE      4
+
 #define UART_LCR       3       /* Out: Line Control Register */
 /*
  * Note: if the word length is 5 bits (UART_LCR_WLEN5), then setting