Staging: otus: fix some sparse warnings
authorThiago Farina <tfransosi@gmail.com>
Sun, 22 Nov 2009 22:00:00 +0000 (17:00 -0500)
committerGreg Kroah-Hartman <gregkh@suse.de>
Thu, 4 Mar 2010 00:42:27 +0000 (16:42 -0800)
*apdbg.c: use NULL pointer instead of 0 interger. Also make two functions private.

Signed-off-by: Thiago Farina <tfransosi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/otus/apdbg.c

index 0eb93f19958a585a888dfab691f6b1193269935e..b59028e7e33c607ff4555b48a0b5aaa808d2df43 100644 (file)
@@ -90,8 +90,27 @@ struct zdap_ioctl {
 
 #endif
 
-char hex(char);
-unsigned char asctohex(char *str);
+static char hex(char v)
+{
+       if (isdigit(v))
+               return v - '0';
+       else if (isxdigit(v))
+               return tolower(v) - 'a' + 10;
+       else
+               return 0;
+}
+
+static unsigned char asctohex(char *str)
+{
+       unsigned char value;
+
+       value = hex(*str) & 0x0f;
+       value = value << 4;
+       str++;
+       value |= hex(*str) & 0x0f;
+
+       return value;
+}
 
 char *prgname;
 
@@ -109,10 +128,10 @@ int set_ioctl(int sock, struct ifreq *req)
 
 int read_reg(int sock, struct ifreq *req)
 {
-       struct zdap_ioctl *zdreq = 0;
+       struct zdap_ioctl *zdreq = NULL;
 
        if (!set_ioctl(sock, req))
-                       return -1;
+               return -1;
 
        /*
         * zdreq = (struct zdap_ioctl *)req->ifr_data;
@@ -125,7 +144,7 @@ int read_reg(int sock, struct ifreq *req)
 
 int read_mem(int sock, struct ifreq *req)
 {
-       struct zdap_ioctl *zdreq = 0;
+       struct zdap_ioctl *zdreq = NULL;
        int i;
 
        if (!set_ioctl(sock, req))
@@ -368,7 +387,7 @@ int main(int argc, char **argv)
 
                zdreq.addr = addr;
                zdreq.cmd = ZM_IOCTL_SET_PIBSS_MODE;
-       } else  {
+       } else {
                fprintf(stderr, "error action\n");
                exit(1);
        }
@@ -380,25 +399,3 @@ fail:
        exit(0);
 }
 
-unsigned char asctohex(char *str)
-{
-       unsigned char value;
-
-       value = hex(*str) & 0x0f;
-       value = value << 4;
-       str++;
-       value |= hex(*str) & 0x0f;
-
-       return value;
-}
-
-char hex(char v)
-{
-       if (isdigit(v))
-               return v - '0';
-       else if (isxdigit(v))
-               return tolower(v) - 'a' + 10;
-       else
-               return 0;
-}
-