remove unnecessary initial allocation of vc
authorPranay Kr. Srivastava <pranjas@gmail.com>
Tue, 21 Mar 2017 07:10:23 +0000 (12:40 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 23 Mar 2017 13:16:52 +0000 (14:16 +0100)
This patch removes the unnecessary allocation of
current foreground vc during initialization.

This initialization is already handled in the loop
that follows it for all available virtual consoles.

Signed-off-by: Pranay Kr. Srivastava <pranjas@gmail.com>
Reviewed-by: Samuel Thibault <samuel.thibault@ens-lyon.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/speakup/main.c

index 9e3d9c52255fc92869c8902615459f68b37762bc..d890b9a864de8ac6b06a500d1bbd5d84345062c8 100644 (file)
@@ -2329,7 +2329,6 @@ static int __init speakup_init(void)
 {
        int i;
        long err = 0;
-       struct st_spk_t *first_console;
        struct vc_data *vc = vc_cons[fg_console].d;
        struct var_t *var;
 
@@ -2354,15 +2353,6 @@ static int __init speakup_init(void)
        if (err)
                goto error_virtkeyboard;
 
-       first_console = kzalloc(sizeof(*first_console), GFP_KERNEL);
-       if (!first_console) {
-               err = -ENOMEM;
-               goto error_alloc;
-       }
-
-       speakup_console[vc->vc_num] = first_console;
-       speakup_date(vc);
-
        for (i = 0; i < MAX_NR_CONSOLES; i++)
                if (vc_cons[i].d) {
                        err = speakup_allocate(vc_cons[i].d);
@@ -2424,7 +2414,6 @@ error_kobjects:
        for (i = 0; i < MAX_NR_CONSOLES; i++)
                kfree(speakup_console[i]);
 
-error_alloc:
        speakup_remove_virtual_keyboard();
 
 error_virtkeyboard: