]> Pileus Git - ~andy/linux/commitdiff
Drivers: tty: n_gsm.c: fixed 7 errors & 6 warnings that checkpatch complained
authorAldo Iljazi <neonsync1@gmail.com>
Mon, 8 Jul 2013 19:28:00 +0000 (22:28 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 24 Jul 2013 22:21:13 +0000 (15:21 -0700)
Specifically:
n_gsm.c:810: ERROR: space required before the open parenthesis '('
n_gsm.c:830: WARNING: line over 80 characters
n_gsm.c:971: ERROR: trailing whitespace
n_gsm.c:984: ERROR: code indent should use tabs where possible
n_gsm.c:984: WARNING: please, no space before tabs
n_gsm.c:984: WARNING: please, no spaces at the start of a line
n_gsm.c:1141: WARNING: space prohibited before semicolon
n_gsm.c:1743: ERROR: space required before the open brace '{'
n_gsm.c:1744: WARNING: line over 80 characters
n_gsm.c:1745: ERROR: code indent should use tabs where possible
n_gsm.c:1746: ERROR: code indent should use tabs where possible
n_gsm.c:2908: WARNING: line over 80 characters
n_gsm.c:2912: ERROR: trailing whitespace

Signed-off-by: Aldo Iljazi <neonsync1@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/tty/n_gsm.c

index 642239015b46bbe9e18844f186a31296a343d5f9..c0f76da553042db2f3eadb91cfb9ff08bc60e43d 100644 (file)
@@ -807,7 +807,7 @@ static int gsm_dlci_data_output(struct gsm_mux *gsm, struct gsm_dlci *dlci)
        int h = dlci->adaption - 1;
 
        total_size = 0;
-       while(1) {
+       while (1) {
                len = kfifo_len(dlci->fifo);
                if (len == 0)
                        return total_size;
@@ -827,8 +827,8 @@ static int gsm_dlci_data_output(struct gsm_mux *gsm, struct gsm_dlci *dlci)
                switch (dlci->adaption) {
                case 1: /* Unstructured */
                        break;
-               case 2: /* Unstructed with modem bits. Always one byte as we never
-                          send inline break data */
+               case 2: /* Unstructed with modem bits.
+               Always one byte as we never send inline break data */
                        *dp++ = gsm_encode_modem(dlci);
                        break;
                }
@@ -968,7 +968,7 @@ static void gsm_dlci_data_kick(struct gsm_dlci *dlci)
        unsigned long flags;
        int sweep;
 
-       if (dlci->constipated) 
+       if (dlci->constipated)
                return;
 
        spin_lock_irqsave(&dlci->gsm->tx_lock, flags);
@@ -981,7 +981,7 @@ static void gsm_dlci_data_kick(struct gsm_dlci *dlci)
                        gsm_dlci_data_output(dlci->gsm, dlci);
        }
        if (sweep)
-               gsm_dlci_data_sweep(dlci->gsm);
+               gsm_dlci_data_sweep(dlci->gsm);
        spin_unlock_irqrestore(&dlci->gsm->tx_lock, flags);
 }
 
@@ -1138,7 +1138,7 @@ static void gsm_control_modem(struct gsm_mux *gsm, u8 *data, int clen)
 static void gsm_control_rls(struct gsm_mux *gsm, u8 *data, int clen)
 {
        struct tty_port *port;
-       unsigned int addr = 0 ;
+       unsigned int addr = 0;
        u8 bits;
        int len = clen;
        u8 *dp = data;
@@ -1740,10 +1740,11 @@ static void gsm_queue(struct gsm_mux *gsm)
 
        if ((gsm->control & ~PF) == UI)
                gsm->fcs = gsm_fcs_add_block(gsm->fcs, gsm->buf, gsm->len);
-       if (gsm->encoding == 0){
-               /* WARNING: gsm->received_fcs is used for gsm->encoding = 0 only.
-                           In this case it contain the last piece of data
-                           required to generate final CRC */
+       if (gsm->encoding == 0) {
+               /* WARNING: gsm->received_fcs is used for
+               gsm->encoding = 0 only.
+               In this case it contain the last piece of data
+               required to generate final CRC */
                gsm->fcs = gsm_fcs_add(gsm->fcs, gsm->received_fcs);
        }
        if (gsm->fcs != GOOD_FCS) {
@@ -2904,9 +2905,11 @@ static int gsmtty_install(struct tty_driver *driver, struct tty_struct *tty)
        gsm = gsm_mux[mux];
        if (gsm->dead)
                return -EL2HLT;
-       /* If DLCI 0 is not yet fully open return an error. This is ok from a locking
-          perspective as we don't have to worry about this if DLCI0 is lost */
-       if (gsm->dlci[0] && gsm->dlci[0]->state != DLCI_OPEN) 
+       /* If DLCI 0 is not yet fully open return an error.
+       This is ok from a locking
+       perspective as we don't have to worry about this
+       if DLCI0 is lost */
+       if (gsm->dlci[0] && gsm->dlci[0]->state != DLCI_OPEN)
                return -EL2NSYNC;
        dlci = gsm->dlci[line];
        if (dlci == NULL) {