Fixed the following style/coding issues:
*updated ---help to the prefered new help texts which reduces the code/file 
size and fixes the warning messages
*Used else if instead of elese as else is not generally useful after a break or 
return, not sure if this is the acceptable but it resolved the warning messages.
*kstrtoul is used instead of the abosolete simple_strtoul

Signed-off-by: Tamir Suliman <tsuli...@linuxvaults.com>
---
 drivers/staging/speakup/Kconfig    | 43 ++++++++++++++------------------------
 drivers/staging/speakup/keyhelp.c  |  2 +-
 drivers/staging/speakup/kobjects.c |  4 ++--
 3 files changed, 19 insertions(+), 30 deletions(-)

diff --git a/drivers/staging/speakup/Kconfig b/drivers/staging/speakup/Kconfig
index efd6f45..2f282fd 100644
--- a/drivers/staging/speakup/Kconfig
+++ b/drivers/staging/speakup/Kconfig
@@ -3,7 +3,7 @@ menu "Speakup console speech"
 config SPEAKUP
        depends on VT
        tristate "Speakup core"
-       ---help---
+       help
                This is the Speakup screen reader.  Think of it as a
                video console for blind people.  If built in to the
                kernel, it can speak everything on the text console from
@@ -43,7 +43,7 @@ config SPEAKUP
 if SPEAKUP
 config SPEAKUP_SYNTH_ACNTSA
        tristate "Accent SA synthesizer support"
-       ---help---
+       help
                This is the Speakup driver for the Accent SA
                synthesizer.  You can say y to build it into the kernel,
                or m to build it as a module.  See the configuration
@@ -52,7 +52,7 @@ config SPEAKUP_SYNTH_ACNTSA
 config SPEAKUP_SYNTH_ACNTPC
        tristate "Accent PC synthesizer support"
        depends on ISA || COMPILE_TEST
-       ---help---
+       help
                This is the Speakup driver for the accent pc
                synthesizer.  You can say y to build it into the kernel,
                or m to build it as a module.  See the configuration
@@ -60,7 +60,7 @@ config SPEAKUP_SYNTH_ACNTPC
 
 config SPEAKUP_SYNTH_APOLLO
        tristate "Apollo II synthesizer support"
-       ---help---
+       help
                This is the Speakup driver for the Apollo II
                synthesizer.  You can say y to build it into the kernel,
                or m to build it as a module.  See the configuration
@@ -68,7 +68,7 @@ config SPEAKUP_SYNTH_APOLLO
 
 config SPEAKUP_SYNTH_AUDPTR
        tristate "Audapter synthesizer support"
-       ---help---
+       help
                This is the Speakup driver for the Audapter synthesizer.
                 You can say y to build it into the kernel, or m to
                build it as a module.  See the configuration help on the
@@ -76,7 +76,7 @@ config SPEAKUP_SYNTH_AUDPTR
 
 config SPEAKUP_SYNTH_BNS
        tristate "Braille 'n' Speak synthesizer support"
-       ---help---
+       help
                This is the Speakup driver for the Braille 'n' Speak
                synthesizer.  You can say y to build it into the kernel,
                or m to build it as a module.  See the configuration
@@ -84,8 +84,7 @@ config SPEAKUP_SYNTH_BNS
 
 config SPEAKUP_SYNTH_DECTLK
        tristate "DECtalk Express synthesizer support"
-       ---help---
-
+       help
                This is the Speakup driver for the DecTalk Express
                synthesizer.  You can say y to build it into the kernel,
                or m to build it as a module.  See the configuration
@@ -93,8 +92,7 @@ config SPEAKUP_SYNTH_DECTLK
 
 config SPEAKUP_SYNTH_DECEXT
        tristate "DECtalk External (old) synthesizer support"
-       ---help---
-
+       help
                This is the Speakup driver for the DecTalk External
                (old) synthesizer.  You can say y to build it into the
                kernel, or m to build it as a module.  See the
@@ -105,13 +103,11 @@ config SPEAKUP_SYNTH_DECPC
        depends on m
        depends on ISA || COMPILE_TEST
        tristate "DECtalk PC (big ISA card) synthesizer support"
-       ---help---
-
+       help
                This is the Speakup driver for the DecTalk PC (full
                length ISA) synthesizer.  You can say m to build it as
                a module.  See the configuration help on the Speakup
                choice above for more info.
-
                In order to use the DecTalk PC driver, you must download
                the dec_pc.tgz file from linux-speakup.org.  It is in
                the pub/linux/goodies directory.  The dec_pc.tgz file
@@ -127,8 +123,7 @@ config SPEAKUP_SYNTH_DECPC
 config SPEAKUP_SYNTH_DTLK
        tristate "DoubleTalk PC synthesizer support"
        depends on ISA || COMPILE_TEST
-       ---help---
-
+       help
                This is the Speakup driver for the internal DoubleTalk
                PC synthesizer.  You can say y to build it into the
                kernel, or m to build it as a module.  See the
@@ -138,8 +133,7 @@ config SPEAKUP_SYNTH_DTLK
 config SPEAKUP_SYNTH_KEYPC
        tristate "Keynote Gold PC synthesizer support"
        depends on ISA || COMPILE_TEST
-       ---help---
-
+       help
                This is the Speakup driver for the Keynote Gold
                PC synthesizer.  You can say y to build it into the
                kernel, or m to build it as a module.  See the
@@ -148,8 +142,7 @@ config SPEAKUP_SYNTH_KEYPC
 
 config SPEAKUP_SYNTH_LTLK
        tristate "DoubleTalk LT/LiteTalk synthesizer support"
----help---
-
+       help
                This is the Speakup driver for the LiteTalk/DoubleTalk
                LT synthesizer.  You can say y to build it into the
                kernel, or m to build it as a module.  See the
@@ -158,8 +151,7 @@ config SPEAKUP_SYNTH_LTLK
 
 config SPEAKUP_SYNTH_SOFT
        tristate "Userspace software synthesizer support"
-       ---help---
-
+       help
                This is the software synthesizer device node.  It will
                register a device /dev/softsynth which midware programs
                and speech daemons may open and read to provide kernel
@@ -169,8 +161,7 @@ config SPEAKUP_SYNTH_SOFT
 
 config SPEAKUP_SYNTH_SPKOUT
        tristate "Speak Out synthesizer support"
-       ---help---
-
+       help
                This is the Speakup driver for the Speakout synthesizer.
                 You can say y to build it into the kernel, or m to
                build it as a module.  See the configuration help on the
@@ -178,8 +169,7 @@ config SPEAKUP_SYNTH_SPKOUT
 
 config SPEAKUP_SYNTH_TXPRT
        tristate "Transport synthesizer support"
-       ---help---
-
+       help
                This is the Speakup driver for the Transport
                synthesizer.  You can say y to build it into the kernel,
                or m to build it as a module.  See the configuration
@@ -187,8 +177,7 @@ config SPEAKUP_SYNTH_TXPRT
 
 config SPEAKUP_SYNTH_DUMMY
        tristate "Dummy synthesizer driver (for testing)"
-       ---help---
-
+       help
                This is a dummy Speakup driver for plugging a mere serial
                terminal.  This is handy if you want to test speakup but
                don't have the hardware.  You can say y to build it into
diff --git a/drivers/staging/speakup/keyhelp.c 
b/drivers/staging/speakup/keyhelp.c
index 5f1bda3..a1bbe8f 100644
--- a/drivers/staging/speakup/keyhelp.c
+++ b/drivers/staging/speakup/keyhelp.c
@@ -167,7 +167,7 @@ int spk_handle_help(struct vc_data *vc, u_char type, u_char 
ch, u_short key)
                synth_printf("%s\n", spk_msg_get(MSG_HELP_INFO));
                build_key_data(); /* rebuild each time in case new mapping */
                return 1;
-       } else {
+       } else if {
                name = NULL;
                if ((type != KT_SPKUP) && (key > 0) && (key <= num_key_names)) {
                        synth_printf("%s\n",
diff --git a/drivers/staging/speakup/kobjects.c 
b/drivers/staging/speakup/kobjects.c
index f1f9022..a98a0a4 100644
--- a/drivers/staging/speakup/kobjects.c
+++ b/drivers/staging/speakup/kobjects.c
@@ -154,7 +154,7 @@ static ssize_t chars_chartab_store(struct kobject *kobj,
                        continue;
                }
 
-               index = simple_strtoul(cp, &temp, 10);
+               index = simple_kstrtoul(cp, &temp, 10);
                if (index > 255) {
                        rejected++;
                        cp = linefeed + 1;
@@ -787,7 +787,7 @@ static ssize_t message_store_helper(const char *buf, size_t 
count,
                        continue;
                }
 
-               index = simple_strtoul(cp, &temp, 10);
+               index = simple_ktrtoul(cp, &temp, 10);
 
                while ((temp < linefeed) && (*temp == ' ' || *temp == '\t'))
                        temp++;
-- 
1.8.3.1

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to