lkml.org 
[lkml]   [2020]   [Feb]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH AUTOSEL 4.14 119/186] tty: synclinkmp: Adjust indentation in several functions
    Date
    From: Nathan Chancellor <natechancellor@gmail.com>

    [ Upstream commit 1feedf61e7265128244f6993f23421f33dd93dbc ]

    Clang warns:

    ../drivers/tty/synclinkmp.c:1456:3: warning: misleading indentation;
    statement is not part of the previous 'if' [-Wmisleading-indentation]
    if (C_CRTSCTS(tty)) {
    ^
    ../drivers/tty/synclinkmp.c:1453:2: note: previous statement is here
    if (I_IXOFF(tty))
    ^
    ../drivers/tty/synclinkmp.c:2473:8: warning: misleading indentation;
    statement is not part of the previous 'if' [-Wmisleading-indentation]
    info->port.tty->hw_stopped = 0;
    ^
    ../drivers/tty/synclinkmp.c:2471:7: note: previous statement is here
    if ( debug_level >= DEBUG_LEVEL_ISR )
    ^
    ../drivers/tty/synclinkmp.c:2482:8: warning: misleading indentation;
    statement is not part of the previous 'if' [-Wmisleading-indentation]
    info->port.tty->hw_stopped = 1;
    ^
    ../drivers/tty/synclinkmp.c:2480:7: note: previous statement is here
    if ( debug_level >= DEBUG_LEVEL_ISR )
    ^
    ../drivers/tty/synclinkmp.c:2809:3: warning: misleading indentation;
    statement is not part of the previous 'if' [-Wmisleading-indentation]
    if (I_BRKINT(info->port.tty) || I_PARMRK(info->port.tty))
    ^
    ../drivers/tty/synclinkmp.c:2807:2: note: previous statement is here
    if (I_INPCK(info->port.tty))
    ^
    ../drivers/tty/synclinkmp.c:3246:3: warning: misleading indentation;
    statement is not part of the previous 'else' [-Wmisleading-indentation]
    set_signals(info);
    ^
    ../drivers/tty/synclinkmp.c:3244:2: note: previous statement is here
    else
    ^
    5 warnings generated.

    The indentation on these lines is not at all consistent, tabs and spaces
    are mixed together. Convert to just using tabs to be consistent with the
    Linux kernel coding style and eliminate these warnings from clang.

    Link: https://github.com/ClangBuiltLinux/linux/issues/823
    Signed-off-by: Nathan Chancellor <natechancellor@gmail.com>
    Link: https://lore.kernel.org/r/20191218024720.3528-1-natechancellor@gmail.com
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    drivers/tty/synclinkmp.c | 24 ++++++++++++------------
    1 file changed, 12 insertions(+), 12 deletions(-)

    diff --git a/drivers/tty/synclinkmp.c b/drivers/tty/synclinkmp.c
    index 4fed9e7b281f0..3c9e314406b4e 100644
    --- a/drivers/tty/synclinkmp.c
    +++ b/drivers/tty/synclinkmp.c
    @@ -1467,10 +1467,10 @@ static void throttle(struct tty_struct * tty)
    if (I_IXOFF(tty))
    send_xchar(tty, STOP_CHAR(tty));

    - if (C_CRTSCTS(tty)) {
    + if (C_CRTSCTS(tty)) {
    spin_lock_irqsave(&info->lock,flags);
    info->serial_signals &= ~SerialSignal_RTS;
    - set_signals(info);
    + set_signals(info);
    spin_unlock_irqrestore(&info->lock,flags);
    }
    }
    @@ -1496,10 +1496,10 @@ static void unthrottle(struct tty_struct * tty)
    send_xchar(tty, START_CHAR(tty));
    }

    - if (C_CRTSCTS(tty)) {
    + if (C_CRTSCTS(tty)) {
    spin_lock_irqsave(&info->lock,flags);
    info->serial_signals |= SerialSignal_RTS;
    - set_signals(info);
    + set_signals(info);
    spin_unlock_irqrestore(&info->lock,flags);
    }
    }
    @@ -2484,7 +2484,7 @@ static void isr_io_pin( SLMP_INFO *info, u16 status )
    if (status & SerialSignal_CTS) {
    if ( debug_level >= DEBUG_LEVEL_ISR )
    printk("CTS tx start...");
    - info->port.tty->hw_stopped = 0;
    + info->port.tty->hw_stopped = 0;
    tx_start(info);
    info->pending_bh |= BH_TRANSMIT;
    return;
    @@ -2493,7 +2493,7 @@ static void isr_io_pin( SLMP_INFO *info, u16 status )
    if (!(status & SerialSignal_CTS)) {
    if ( debug_level >= DEBUG_LEVEL_ISR )
    printk("CTS tx stop...");
    - info->port.tty->hw_stopped = 1;
    + info->port.tty->hw_stopped = 1;
    tx_stop(info);
    }
    }
    @@ -2820,8 +2820,8 @@ static void change_params(SLMP_INFO *info)
    info->read_status_mask2 = OVRN;
    if (I_INPCK(info->port.tty))
    info->read_status_mask2 |= PE | FRME;
    - if (I_BRKINT(info->port.tty) || I_PARMRK(info->port.tty))
    - info->read_status_mask1 |= BRKD;
    + if (I_BRKINT(info->port.tty) || I_PARMRK(info->port.tty))
    + info->read_status_mask1 |= BRKD;
    if (I_IGNPAR(info->port.tty))
    info->ignore_status_mask2 |= PE | FRME;
    if (I_IGNBRK(info->port.tty)) {
    @@ -3191,7 +3191,7 @@ static int tiocmget(struct tty_struct *tty)
    unsigned long flags;

    spin_lock_irqsave(&info->lock,flags);
    - get_signals(info);
    + get_signals(info);
    spin_unlock_irqrestore(&info->lock,flags);

    result = ((info->serial_signals & SerialSignal_RTS) ? TIOCM_RTS : 0) |
    @@ -3229,7 +3229,7 @@ static int tiocmset(struct tty_struct *tty,
    info->serial_signals &= ~SerialSignal_DTR;

    spin_lock_irqsave(&info->lock,flags);
    - set_signals(info);
    + set_signals(info);
    spin_unlock_irqrestore(&info->lock,flags);

    return 0;
    @@ -3241,7 +3241,7 @@ static int carrier_raised(struct tty_port *port)
    unsigned long flags;

    spin_lock_irqsave(&info->lock,flags);
    - get_signals(info);
    + get_signals(info);
    spin_unlock_irqrestore(&info->lock,flags);

    return (info->serial_signals & SerialSignal_DCD) ? 1 : 0;
    @@ -3257,7 +3257,7 @@ static void dtr_rts(struct tty_port *port, int on)
    info->serial_signals |= SerialSignal_RTS | SerialSignal_DTR;
    else
    info->serial_signals &= ~(SerialSignal_RTS | SerialSignal_DTR);
    - set_signals(info);
    + set_signals(info);
    spin_unlock_irqrestore(&info->lock,flags);
    }

    --
    2.20.1
    \
     
     \ /
      Last update: 2020-02-14 17:22    [W:3.915 / U:0.164 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site