diff options
| author | Andy Shevchenko <andriy.shevchenko@linux.intel.com> | 2020-02-03 15:31:24 +0200 |
|---|---|---|
| committer | Petr Mladek <pmladek@suse.com> | 2020-02-11 10:43:36 +0100 |
| commit | c81ea7203b224cb85c5daf360556304e7212bab6 (patch) | |
| tree | a80909c4090cd5d635a818f765d8b72d23875a23 | |
| parent | a4fe2b4d87c9f2298ae6a641a7a64bc941d079d0 (diff) | |
| download | lwn-c81ea7203b224cb85c5daf360556304e7212bab6.tar.gz lwn-c81ea7203b224cb85c5daf360556304e7212bab6.zip | |
console: Don't perform test for CON_BRL flag
We don't call braille_register_console() without CON_BRL flag set.
And the _braille_unregister_console() already tests for console to have
CON_BRL flag. No need to repeat this in braille_unregister_console().
Drop the repetitive checks from Braille console driver.
Link: http://lkml.kernel.org/r/20200203133130.11591-1-andriy.shevchenko@linux.intel.com
To: linux-kernel@vger.kernel.org
To: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Reviewed-by: Petr Mladek <pmladek@suse.com>
Reviewed-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
Signed-off-by: Petr Mladek <pmladek@suse.com>
| -rw-r--r-- | drivers/accessibility/braille/braille_console.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/drivers/accessibility/braille/braille_console.c b/drivers/accessibility/braille/braille_console.c index 1339c586bf64..a8f7c278b691 100644 --- a/drivers/accessibility/braille/braille_console.c +++ b/drivers/accessibility/braille/braille_console.c @@ -347,8 +347,6 @@ int braille_register_console(struct console *console, int index, { int ret; - if (!(console->flags & CON_BRL)) - return 0; if (!console_options) /* Only support VisioBraille for now */ console_options = "57600o8"; @@ -371,8 +369,6 @@ int braille_unregister_console(struct console *console) { if (braille_co != console) return -EINVAL; - if (!(console->flags & CON_BRL)) - return 0; unregister_keyboard_notifier(&keyboard_notifier_block); unregister_vt_notifier(&vt_notifier_block); braille_co = NULL; |