tty: Remove dead code
authorAndreas Platschek <andi.platschek@gmail.com>
Fri, 26 Jul 2013 04:46:37 +0000 (06:46 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 26 Jul 2013 23:19:48 +0000 (16:19 -0700)
-> The ledptrs[] array is never initialized.
-> There is no place where kbd->ledmode is set to LED_SHOW_MEM therefore the if
   statement does not make much sense.
-> Since LED_SHOW_MEM is not used, it can be removed from the header file as well.

Signed-off-by: Andreas Platschek <andi.platschek@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/tty/vt/keyboard.c
include/linux/kbd_kern.h

index a9af1b9ae160ec1b9bd47d4e9c1dfcabb27a2db9..d0e3a44977074526979a92e638c620aabaf661cb 100644 (file)
@@ -132,12 +132,6 @@ static int shift_state = 0;
 static unsigned char ledstate = 0xff;                  /* undefined */
 static unsigned char ledioctl;
 
-static struct ledptr {
-       unsigned int *addr;
-       unsigned int mask;
-       unsigned char valid:1;
-} ledptrs[3];
-
 /*
  * Notifier list for console keyboard events
  */
@@ -994,24 +988,11 @@ void setledstate(struct kbd_struct *kbd, unsigned int led)
 static inline unsigned char getleds(void)
 {
        struct kbd_struct *kbd = kbd_table + fg_console;
-       unsigned char leds;
-       int i;
 
        if (kbd->ledmode == LED_SHOW_IOCTL)
                return ledioctl;
 
-       leds = kbd->ledflagstate;
-
-       if (kbd->ledmode == LED_SHOW_MEM) {
-               for (i = 0; i < 3; i++)
-                       if (ledptrs[i].valid) {
-                               if (*ledptrs[i].addr & ledptrs[i].mask)
-                                       leds |= (1 << i);
-                               else
-                                       leds &= ~(1 << i);
-                       }
-       }
-       return leds;
+       return kbd->ledflagstate;
 }
 
 static int kbd_update_leds_helper(struct input_handle *handle, void *data)
index b7c8cdc1d4223b4565d7c1ede7e10229621cca97..cbfb171bbcbaaf169726b00a2013683e3ec94a1b 100644 (file)
@@ -36,10 +36,9 @@ struct kbd_struct {
 #define VC_CTRLRLOCK   KG_CTRLR        /* ctrlr lock mode */
        unsigned char slockstate;       /* for `sticky' Shift, Ctrl, etc. */
 
-       unsigned char ledmode:2;        /* one 2-bit value */
+       unsigned char ledmode:1;
 #define LED_SHOW_FLAGS 0        /* traditional state */
 #define LED_SHOW_IOCTL 1        /* only change leds upon ioctl */
-#define LED_SHOW_MEM 2          /* `heartbeat': peek into memory */
 
        unsigned char ledflagstate:4;   /* flags, not lights */
        unsigned char default_ledflagstate:4;