diff options
| author | Jiri Slaby (SUSE) <jirislaby@kernel.org> | 2023-08-16 12:55:21 +0200 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2023-08-22 14:58:15 +0200 |
| commit | d4d13ff3ac78b41bfaefd5ad1efd72cfbf43f288 (patch) | |
| tree | 0ca4cadf3eb2f9c7030f9f5024fdf02618617d22 /include/linux/tty_buffer.h | |
| parent | 2f4926723ac7343ae11861d0ba2aaa8e04bd3ca1 (diff) | |
tty: tty_buffer: switch data type to u8
There is no reason to have tty_buffer::data typed as unsigned long.
Switch to u8, but preserve the ulong alignment using __aligned.
This allows for the cast removal from char_buf_ptr(). And for use of
struct_size() in the allocation in tty_buffer_alloc() -- in the next
patch.
Signed-off-by: "Jiri Slaby (SUSE)" <jirislaby@kernel.org>
Link: https://lore.kernel.org/r/20230816105530.3335-2-jirislaby@kernel.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'include/linux/tty_buffer.h')
| -rw-r--r-- | include/linux/tty_buffer.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/include/linux/tty_buffer.h b/include/linux/tty_buffer.h index e45cba81d0e9..31125e3be3c5 100644 --- a/include/linux/tty_buffer.h +++ b/include/linux/tty_buffer.h @@ -19,12 +19,12 @@ struct tty_buffer { unsigned int read; bool flags; /* Data points here */ - unsigned long data[]; + u8 data[] __aligned(sizeof(unsigned long)); }; static inline u8 *char_buf_ptr(struct tty_buffer *b, unsigned int ofs) { - return ((u8 *)b->data) + ofs; + return b->data + ofs; } static inline u8 *flag_buf_ptr(struct tty_buffer *b, unsigned int ofs) |