Gitweb:     
http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=181d6f4fac7f01ede95284954ea1231939dca0d9
Commit:     181d6f4fac7f01ede95284954ea1231939dca0d9
Parent:     8f8ecbad09b48e5fe44a8d7f5344e802e9c231c8
Author:     Jiri Slaby <[EMAIL PROTECTED]>
AuthorDate: Sat Feb 10 01:45:34 2007 -0800
Committer:  Linus Torvalds <[EMAIL PROTECTED]>
CommitDate: Sun Feb 11 10:51:30 2007 -0800

    [PATCH] Char: moxa, remove useless variables
    
    Remove temporary or once used variables, that can be defined locally to
    save some bytes.
    
    Signed-off-by: Jiri Slaby <[EMAIL PROTECTED]>
    Signed-off-by: Andrew Morton <[EMAIL PROTECTED]>
    Signed-off-by: Linus Torvalds <[EMAIL PROTECTED]>
---
 drivers/char/moxa.c |   35 ++++++++++++++---------------------
 1 files changed, 14 insertions(+), 21 deletions(-)

diff --git a/drivers/char/moxa.c b/drivers/char/moxa.c
index 0c34bc5..8849d66 100644
--- a/drivers/char/moxa.c
+++ b/drivers/char/moxa.c
@@ -146,8 +146,6 @@ struct moxa_port {
        wait_queue_head_t close_wait;
 
        struct timer_list emptyTimer;
-       struct mxser_mstatus GMStatus;
-       struct moxaq_str temp_queue;
 
        char chkPort;
        char lineCtrl;
@@ -1487,17 +1485,15 @@ int MoxaDriverIoctl(unsigned int cmd, unsigned long 
arg, int port)
                return (0);
        case MOXA_GET_IOQUEUE: {
                struct moxaq_str __user *argm = argp;
-               struct moxa_port *p;
+               struct moxaq_str tmp;
 
                for (i = 0; i < MAX_PORTS; i++, argm++) {
-                       p = &moxa_ports[i];
-                       memset(&p->temp_queue, 0, sizeof(p->temp_queue));
-                       if (p->chkPort) {
-                               p->temp_queue.inq = MoxaPortRxQueue(i);
-                               p->temp_queue.outq = MoxaPortTxQueue(i);
+                       memset(&tmp, 0, sizeof(tmp));
+                       if (moxa_ports[i].chkPort) {
+                               tmp.inq = MoxaPortRxQueue(i);
+                               tmp.outq = MoxaPortTxQueue(i);
                        }
-                       if (copy_to_user(argm, &p->temp_queue,
-                                               sizeof(p->temp_queue)))
+                       if (copy_to_user(argm, &tmp, sizeof(tmp)))
                                return -EFAULT;
                }
                return (0);
@@ -1518,33 +1514,30 @@ int MoxaDriverIoctl(unsigned int cmd, unsigned long 
arg, int port)
                return 0;
        case MOXA_GETMSTATUS: {
                struct mxser_mstatus __user *argm = argp;
+               struct mxser_mstatus tmp;
                struct moxa_port *p;
 
                for (i = 0; i < MAX_PORTS; i++, argm++) {
                        p = &moxa_ports[i];
-                       p->GMStatus.ri = 0;
-                       p->GMStatus.dcd = 0;
-                       p->GMStatus.dsr = 0;
-                       p->GMStatus.cts = 0;
+                       memset(&tmp, 0, sizeof(tmp));
                        if (!p->chkPort) {
                                goto copy;
                        } else {
                                status = MoxaPortLineStatus(p->port);
                                if (status & 1)
-                                       p->GMStatus.cts = 1;
+                                       tmp.cts = 1;
                                if (status & 2)
-                                       p->GMStatus.dsr = 1;
+                                       tmp.dsr = 1;
                                if (status & 4)
-                                       p->GMStatus.dcd = 1;
+                                       tmp.dcd = 1;
                        }
 
                        if (!p->tty || !p->tty->termios)
-                               p->GMStatus.cflag = p->cflag;
+                               tmp.cflag = p->cflag;
                        else
-                               p->GMStatus.cflag = p->tty->termios->c_cflag;
+                               tmp.cflag = p->tty->termios->c_cflag;
 copy:
-                       if (copy_to_user(argm, &p->GMStatus,
-                                               sizeof(p->GMStatus)))
+                       if (copy_to_user(argm, &tmp, sizeof(tmp)))
                                return -EFAULT;
                }
                return 0;
-
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to