uml: driver formatting fixes

Fix a bunch of formatting violations in the drivers:
	return(n) -> return n
	whitespace fixes
	emacs formatting comment removal
	breaking if(foo) return(n) into two lines

There are also a couple of errno use bugs:
	using errno in a printk when the failure put errno into a local variable
	saving errno after a printk, which can change it

Signed-off-by: Jeff Dike <jdike@linux.intel.com>
Cc: Paolo 'Blaisorblade' Giarrusso <blaisorblade@yahoo.it>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>

2 files changed, 30 insertions, 46 deletions

diff --git a/arch/um/os-Linux/drivers/ethertap_user.c b/arch/um/os-Linux/drivers/ethertap_user.c
index 863981ba1468..f3ad0dac7cdc 100644
--- a/arch/um/os-Linux/drivers/ethertap_user.c
+++ b/arch/um/os-Linux/drivers/ethertap_user.c
@@ -121,7 +121,7 @@ static int etap_tramp(char *dev, char *gate, int control_me,
         n = os_read_file(control_me, &c, sizeof(c));
         if(n != sizeof(c)){
                 printk("etap_tramp : read of status failed, err = %d\n", -n);
-                return(-EINVAL);
+                return -EINVAL;
         }
         if(c != 1){
                 printk("etap_tramp : uml_net failed\n");
@@ -132,7 +132,7 @@ static int etap_tramp(char *dev, char *gate, int control_me,
                 else if(!WIFEXITED(status) || (WEXITSTATUS(status) != 1))
                         printk("uml_net didn't exit with status 1\n");
         }
-        return(err);
+        return err;
 }
 
 static int etap_open(void *data)
@@ -142,20 +142,21 @@ static int etap_open(void *data)
         int data_fds[2], control_fds[2], err, output_len;
 
         err = tap_open_common(pri->dev, pri->gate_addr);
-        if(err) return(err);
+        if(err)
+                return err;
 
         err = os_pipe(data_fds, 0, 0);
         if(err < 0){
                 printk("data os_pipe failed - err = %d\n", -err);
-                return(err);
+                return err;
         }
 
         err = os_pipe(control_fds, 1, 0);
         if(err < 0){
                 printk("control os_pipe failed - err = %d\n", -err);
-                return(err);
+                return err;
         }
-        
+
         err = etap_tramp(pri->dev_name, pri->gate_addr, control_fds[0], 
                          control_fds[1], data_fds[0], data_fds[1]);
         output_len = page_size();
@@ -171,13 +172,13 @@ static int etap_open(void *data)
 
         if(err < 0){
                 printk("etap_tramp failed - err = %d\n", -err);
-                return(err);
+                return err;
         }
 
         pri->data_fd = data_fds[0];
         pri->control_fd = control_fds[0];
         iter_addresses(pri->dev, etap_open_addr, &pri->control_fd);
-        return(data_fds[0]);
+        return data_fds[0];
 }
 
 static void etap_close(int fd, void *data)
@@ -195,7 +196,7 @@ static void etap_close(int fd, void *data)
 
 static int etap_set_mtu(int mtu, void *data)
 {
-        return(mtu);
+        return mtu;
 }
 
 static void etap_add_addr(unsigned char *addr, unsigned char *netmask,
@@ -204,7 +205,8 @@ static void etap_add_addr(unsigned char *addr, unsigned char *netmask,
         struct ethertap_data *pri = data;
 
         tap_check_ips(pri->gate_addr, addr);
-        if(pri->control_fd == -1) return;
+        if(pri->control_fd == -1)
+                return;
         etap_open_addr(addr, netmask, &pri->control_fd);
 }
 
@@ -213,7 +215,8 @@ static void etap_del_addr(unsigned char *addr, unsigned char *netmask,
 {
         struct ethertap_data *pri = data;
 
-        if(pri->control_fd == -1) return;
+        if(pri->control_fd == -1)
+                return;
         etap_close_addr(addr, netmask, &pri->control_fd);
 }
 
@@ -227,14 +230,3 @@ const struct net_user_info ethertap_user_info = {
         .delete_address = etap_del_addr,
         .max_packet     = MAX_PACKET - ETH_HEADER_ETHERTAP
 };
-
-/*
- * Overrides for Emacs so that we follow Linus's tabbing style.
- * Emacs will notice this stuff at the end of the file and automatically
- * adjust the settings for this buffer only.  This must remain at the end
- * of the file.
- * ---------------------------------------------------------------------------
- * Local variables:
- * c-file-style: "linux"
- * End:
- */
diff --git a/arch/um/os-Linux/drivers/tuntap_user.c b/arch/um/os-Linux/drivers/tuntap_user.c
index e846b23f7558..ab63fb6f9992 100644
--- a/arch/um/os-Linux/drivers/tuntap_user.c
+++ b/arch/um/os-Linux/drivers/tuntap_user.c
@@ -37,7 +37,8 @@ static void tuntap_add_addr(unsigned char *addr, unsigned char *netmask,
         struct tuntap_data *pri = data;
 
         tap_check_ips(pri->gate_addr, addr);
-        if((pri->fd == -1) || pri->fixed_config) return;
+        if((pri->fd == -1) || pri->fixed_config)
+                return;
         open_addr(addr, netmask, pri->dev_name);
 }
 
@@ -46,7 +47,8 @@ static void tuntap_del_addr(unsigned char *addr, unsigned char *netmask,
 {
         struct tuntap_data *pri = data;
 
-        if((pri->fd == -1) || pri->fixed_config) return;
+        if((pri->fd == -1) || pri->fixed_config)
+                return;
         close_addr(addr, netmask, pri->dev_name);
 }
 
@@ -58,7 +60,7 @@ struct tuntap_pre_exec_data {
 static void tuntap_pre_exec(void *arg)
 {
         struct tuntap_pre_exec_data *data = arg;
-        
+
         dup2(data->stdout, 1);
         os_close_file(data->close_me);
 }
@@ -83,7 +85,8 @@ static int tuntap_open_tramp(char *gate, int *fd_out, int me, int remote,
 
         pid = run_helper(tuntap_pre_exec, &data, argv, NULL);
 
-        if(pid < 0) return(-pid);
+        if(pid < 0)
+                return -pid;
 
         os_close_file(remote);
 
@@ -114,16 +117,16 @@ static int tuntap_open_tramp(char *gate, int *fd_out, int me, int remote,
         cmsg = CMSG_FIRSTHDR(&msg);
         if(cmsg == NULL){
                 printk("tuntap_open_tramp : didn't receive a message\n");
-                return(-EINVAL);
+                return -EINVAL;
         }
         if((cmsg->cmsg_level != SOL_SOCKET) || 
            (cmsg->cmsg_type != SCM_RIGHTS)){
                 printk("tuntap_open_tramp : didn't receive a descriptor\n");
-                return(-EINVAL);
+                return -EINVAL;
         }
         *fd_out = ((int *) CMSG_DATA(cmsg))[0];
         os_set_exec_close(*fd_out, 1);
-        return(0);
+        return 0;
 }
 
 static int tuntap_open(void *data)
@@ -135,7 +138,7 @@ static int tuntap_open(void *data)
 
         err = tap_open_common(pri->dev, pri->gate_addr);
         if(err < 0)
-                return(err);
+                return err;
 
         if(pri->fixed_config){
                 pri->fd = os_open_file("/dev/net/tun",
@@ -143,7 +146,7 @@ static int tuntap_open(void *data)
                 if(pri->fd < 0){
                         printk("Failed to open /dev/net/tun, err = %d\n",
                                -pri->fd);
-                        return(pri->fd);
+                        return pri->fd;
                 }
                 memset(&ifr, 0, sizeof(ifr));
                 ifr.ifr_flags = IFF_TAP | IFF_NO_PI;
@@ -160,7 +163,7 @@ static int tuntap_open(void *data)
                 if(err < 0){
                         printk("tuntap_open : os_pipe failed - err = %d\n",
                                -err);
-                        return(err);
+                        return err;
                 }
 
                 buffer = get_output_buffer(&len);
@@ -175,7 +178,7 @@ static int tuntap_open(void *data)
                         printk("%s", output);
                         free_output_buffer(buffer);
                         printk("tuntap_open_tramp failed - err = %d\n", -err);
-                        return(err);
+                        return err;
                 }
 
                 pri->dev_name = uml_strdup(buffer);
@@ -187,7 +190,7 @@ static int tuntap_open(void *data)
                 iter_addresses(pri->dev, open_addr, pri->dev_name);
         }
 
-        return(pri->fd);
+        return pri->fd;
 }
 
 static void tuntap_close(int fd, void *data)
@@ -202,7 +205,7 @@ static void tuntap_close(int fd, void *data)
 
 static int tuntap_set_mtu(int mtu, void *data)
 {
-        return(mtu);
+        return mtu;
 }
 
 const struct net_user_info tuntap_user_info = {
@@ -215,14 +218,3 @@ const struct net_user_info tuntap_user_info = {
         .delete_address = tuntap_del_addr,
         .max_packet     = MAX_PACKET
 };
-
-/*
- * Overrides for Emacs so that we follow Linus's tabbing style.
- * Emacs will notice this stuff at the end of the file and automatically
- * adjust the settings for this buffer only.  This must remain at the end
- * of the file.
- * ---------------------------------------------------------------------------
- * Local variables:
- * c-file-style: "linux"
- * End:
- */