V4L/DVB (13231): pwc: Use kernel's simple_strtol()
authorAndy Shevchenko <ext-andriy.shevchenko@nokia.com>
Thu, 24 Sep 2009 10:58:09 +0000 (07:58 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Sat, 5 Dec 2009 20:41:01 +0000 (18:41 -0200)
Change own implementation of pwc_atoi() by simple_strtol(x, NULL, 10).

Signed-off-by: Andy Shevchenko <ext-andriy.shevchenko@nokia.com>
Acked-by: Pekka Enberg <penberg@cs.helsinki.fi>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/video/pwc/pwc-if.c

index f976df452a34105367287e7c62a92fba1bd60d3a..89b620f6db7b41d8cc44ba901090b6cef666304a 100644 (file)
@@ -68,6 +68,7 @@
 #endif
 #include <linux/vmalloc.h>
 #include <asm/io.h>
+#include <linux/kernel.h>              /* simple_strtol() */
 
 #include "pwc.h"
 #include "pwc-kiara.h"
@@ -1916,19 +1917,6 @@ disconnect_out:
        unlock_kernel();
 }
 
-/* *grunt* We have to do atoi ourselves :-( */
-static int pwc_atoi(const char *s)
-{
-       int k = 0;
-
-       k = 0;
-       while (*s != '\0' && *s >= '0' && *s <= '9') {
-               k = 10 * k + (*s - '0');
-               s++;
-       }
-       return k;
-}
-
 
 /*
  * Initialization code & module stuff
@@ -2078,13 +2066,16 @@ static int __init usb_pwc_init(void)
                                }
                                else {
                                        /* No type or serial number specified, just a number. */
-                                       device_hint[i].device_node = pwc_atoi(s);
+                                       device_hint[i].device_node =
+                                               simple_strtol(s, NULL, 10);
                                }
                        }
                        else {
                                /* There's a colon, so we have at least a type and a device node */
-                               device_hint[i].type = pwc_atoi(s);
-                               device_hint[i].device_node = pwc_atoi(colon + 1);
+                               device_hint[i].type =
+                                       simple_strtol(s, NULL, 10);
+                               device_hint[i].device_node =
+                                       simple_strtol(colon + 1, NULL, 10);
                                if (*dot != '\0') {
                                        /* There's a serial number as well */
                                        int k;