diff options
author | Jiri Slaby <jslaby@suse.cz> | 2016-03-31 04:08:17 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2016-04-30 12:26:55 -0400 |
commit | 91e74ca5e7ac4ec6c61b84d6618eb5e401f852f0 (patch) | |
tree | 14382b2b319b99bfd6833dbf08ad661e6b2dfb16 | |
parent | 8ede5cce4f0baff77ef63aa3cb3afc65d0317e0b (diff) |
tty: vt, use proper type for default colors
Every user of default_red, default_grn, and default_blu treats them as
unsigned char. So make it really unsigned char.
And indent the initializers and module_param properly.
This saves ~ 100 bytes of data.
Signed-off-by: Jiri Slaby <jslaby@suse.cz>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r-- | drivers/tty/vt/vt.c | 27 | ||||
-rw-r--r-- | include/linux/selection.h | 6 |
2 files changed, 20 insertions, 13 deletions
diff --git a/drivers/tty/vt/vt.c b/drivers/tty/vt/vt.c index 2c71b3bde174..8f9f8ed3ed09 100644 --- a/drivers/tty/vt/vt.c +++ b/drivers/tty/vt/vt.c | |||
@@ -1043,16 +1043,23 @@ const unsigned char color_table[] = { 0, 4, 2, 6, 1, 5, 3, 7, | |||
1043 | 8,12,10,14, 9,13,11,15 }; | 1043 | 8,12,10,14, 9,13,11,15 }; |
1044 | 1044 | ||
1045 | /* the default colour table, for VGA+ colour systems */ | 1045 | /* the default colour table, for VGA+ colour systems */ |
1046 | int default_red[] = {0x00,0xaa,0x00,0xaa,0x00,0xaa,0x00,0xaa, | 1046 | unsigned char default_red[] = { |
1047 | 0x55,0xff,0x55,0xff,0x55,0xff,0x55,0xff}; | 1047 | 0x00, 0xaa, 0x00, 0xaa, 0x00, 0xaa, 0x00, 0xaa, |
1048 | int default_grn[] = {0x00,0x00,0xaa,0x55,0x00,0x00,0xaa,0xaa, | 1048 | 0x55, 0xff, 0x55, 0xff, 0x55, 0xff, 0x55, 0xff |
1049 | 0x55,0x55,0xff,0xff,0x55,0x55,0xff,0xff}; | 1049 | }; |
1050 | int default_blu[] = {0x00,0x00,0x00,0x00,0xaa,0xaa,0xaa,0xaa, | 1050 | module_param_array(default_red, byte, NULL, S_IRUGO | S_IWUSR); |
1051 | 0x55,0x55,0x55,0x55,0xff,0xff,0xff,0xff}; | 1051 | |
1052 | 1052 | unsigned char default_grn[] = { | |
1053 | module_param_array(default_red, int, NULL, S_IRUGO | S_IWUSR); | 1053 | 0x00, 0x00, 0xaa, 0x55, 0x00, 0x00, 0xaa, 0xaa, |
1054 | module_param_array(default_grn, int, NULL, S_IRUGO | S_IWUSR); | 1054 | 0x55, 0x55, 0xff, 0xff, 0x55, 0x55, 0xff, 0xff |
1055 | module_param_array(default_blu, int, NULL, S_IRUGO | S_IWUSR); | 1055 | }; |
1056 | module_param_array(default_grn, byte, NULL, S_IRUGO | S_IWUSR); | ||
1057 | |||
1058 | unsigned char default_blu[] = { | ||
1059 | 0x00, 0x00, 0x00, 0x00, 0xaa, 0xaa, 0xaa, 0xaa, | ||
1060 | 0x55, 0x55, 0x55, 0x55, 0xff, 0xff, 0xff, 0xff | ||
1061 | }; | ||
1062 | module_param_array(default_blu, byte, NULL, S_IRUGO | S_IWUSR); | ||
1056 | 1063 | ||
1057 | /* | 1064 | /* |
1058 | * gotoxy() must verify all boundaries, because the arguments | 1065 | * gotoxy() must verify all boundaries, because the arguments |
diff --git a/include/linux/selection.h b/include/linux/selection.h index 7e6c4450b8a5..8e4624efdb6f 100644 --- a/include/linux/selection.h +++ b/include/linux/selection.h | |||
@@ -25,9 +25,9 @@ extern void mouse_report(struct tty_struct * tty, int butt, int mrx, int mry); | |||
25 | extern int console_blanked; | 25 | extern int console_blanked; |
26 | 26 | ||
27 | extern const unsigned char color_table[]; | 27 | extern const unsigned char color_table[]; |
28 | extern int default_red[]; | 28 | extern unsigned char default_red[]; |
29 | extern int default_grn[]; | 29 | extern unsigned char default_grn[]; |
30 | extern int default_blu[]; | 30 | extern unsigned char default_blu[]; |
31 | 31 | ||
32 | extern unsigned short *screen_pos(struct vc_data *vc, int w_offset, int viewed); | 32 | extern unsigned short *screen_pos(struct vc_data *vc, int w_offset, int viewed); |
33 | extern u16 screen_glyph(struct vc_data *vc, int offset); | 33 | extern u16 screen_glyph(struct vc_data *vc, int offset); |