diff options
| author | Jiri Slaby <jslaby@suse.cz> | 2021-06-18 08:14:24 +0200 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2021-06-18 13:09:59 +0200 |
| commit | a93963e41ffeff71a735f4fab95d2cbacd9da7a8 (patch) | |
| tree | 5ba4bdabd20fd1f903eb14b3d12c0de321e3ba53 /drivers/tty/mxser.c | |
| parent | 6de6e5c4becb5494d09d44ce4db1c54dc43afbae (diff) | |
| download | linux-a93963e41ffeff71a735f4fab95d2cbacd9da7a8.tar.gz | |
mxser: correct types for uart variables
In struct mxser_port, all the UART registers and masks, and the xon/xoff character are stored into int. Let's re-type all these as UART registers are 8bit (u8) and xon/xoff character is an unsigned char (cc_t in ktermios). This save some bytes in memory, but more importantly the change makes it what it really is. Signed-off-by: Jiri Slaby <jslaby@suse.cz> Link: https://lore.kernel.org/r/20210618061516.662-19-jslaby@suse.cz Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/tty/mxser.c')
| -rw-r--r-- | drivers/tty/mxser.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/drivers/tty/mxser.c b/drivers/tty/mxser.c index 1b8032c9351e..45f36d92c5ea 100644 --- a/drivers/tty/mxser.c +++ b/drivers/tty/mxser.c @@ -335,9 +335,9 @@ struct mxser_port { int baud_base; /* max. speed */ int type; /* UART type */ - int x_char; /* xon/xoff character */ - int IER; /* Interrupt Enable Register */ - int MCR; /* Modem control register */ + unsigned char x_char; /* xon/xoff character */ + u8 IER; /* Interrupt Enable Register */ + u8 MCR; /* Modem control register */ unsigned char stop_rx; unsigned char ldisc_stop_rx; @@ -348,8 +348,8 @@ struct mxser_port { struct async_icount icount; /* kernel counters for 4 input interrupts */ unsigned int timeout; - int read_status_mask; - int ignore_status_mask; + u8 read_status_mask; + u8 ignore_status_mask; u8 xmit_fifo_size; int xmit_head; int xmit_tail; |