staging: speakup: main: switch multiple assignment for one assignment per line
authorLourdes Pedrajas <lu@pplo.net>
Sun, 15 Mar 2020 22:12:47 +0000 (23:12 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 17 Mar 2020 13:07:55 +0000 (14:07 +0100)
One assignment per line is preferred, instead of multiple assignments.
To make it more readable.
Issue found with checkpatch.

Signed-off-by: Lourdes Pedrajas <lu@pplo.net>
Reviewed-by: Samuel Thibautl <samuel.thibault@ens-lyon.org>
Link: https://lore.kernel.org/r/20200315221247.16629-1-lu@pplo.net
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/speakup/main.c

index 488f253..5c47643 100644 (file)
@@ -2117,7 +2117,8 @@ speakup_key(struct vc_data *vc, int shift_state, int keycode, u_short keysym,
                        spk_keydown = 0;
                        goto out;
                }
-               value = spk_lastkey = pad_chars[value];
+               value = pad_chars[value];
+               spk_lastkey = value;
                spk_keydown++;
                spk_parked &= 0xfe;
                goto no_map;