From: walter harms <wharms@bfs.de>
To: kernel-janitors@vger.kernel.org
Subject: Re: [PATCH 1/2] staging: dgnc: fix 'line over 80 characters'
Date: Fri, 25 Mar 2016 08:55:41 +0000 [thread overview]
Message-ID: <56F4FD0D.6060905@bfs.de> (raw)
In-Reply-To: <20160325024406.GA23833@SEL-JYOUN-D1>
Am 25.03.2016 03:44, schrieb Daeseok Youn:
> fix checkpatch.pl warning about 'line over 80 characters'
> in dgnc_neo.c
>
> Signed-off-by: Daeseok Youn <daeseok.youn@gmail.com>
> ---
> drivers/staging/dgnc/dgnc_neo.c | 66 +++++++++++++++++++++++++++--------------
> 1 file changed, 44 insertions(+), 22 deletions(-)
>
> diff --git a/drivers/staging/dgnc/dgnc_neo.c b/drivers/staging/dgnc/dgnc_neo.c
> index 31ac437..10b596f 100644
> --- a/drivers/staging/dgnc/dgnc_neo.c
> +++ b/drivers/staging/dgnc/dgnc_neo.c
> @@ -77,7 +77,8 @@ struct board_ops dgnc_neo_ops = {
> .send_immediate_char = neo_send_immediate_char
> };
>
> -static uint dgnc_offset_table[8] = { 0x01, 0x02, 0x04, 0x08, 0x10, 0x20, 0x40, 0x80 };
> +static uint dgnc_offset_table[8] = { 0x01, 0x02, 0x04, 0x08,
> + 0x10, 0x20, 0x40, 0x80 };
>
This table simple reads 1<<n maybe that can be replaced entierly ?
re,
wh
> /*
> * This function allows calls to ensure that all outstanding
> @@ -116,7 +117,8 @@ static inline void neo_set_cts_flow_control(struct channel_t *ch)
> writeb(efr, &ch->ch_neo_uart->efr);
>
> /* Turn on table D, with 8 char hi/low watermarks */
> - writeb((UART_17158_FCTR_TRGD | UART_17158_FCTR_RTS_4DELAY), &ch->ch_neo_uart->fctr);
> + writeb((UART_17158_FCTR_TRGD | UART_17158_FCTR_RTS_4DELAY),
> + &ch->ch_neo_uart->fctr);
>
> /* Feed the UART our trigger levels */
> writeb(8, &ch->ch_neo_uart->tfifo);
> @@ -150,7 +152,8 @@ static inline void neo_set_rts_flow_control(struct channel_t *ch)
> /* Turn on UART enhanced bits */
> writeb(efr, &ch->ch_neo_uart->efr);
>
> - writeb((UART_17158_FCTR_TRGD | UART_17158_FCTR_RTS_4DELAY), &ch->ch_neo_uart->fctr);
> + writeb((UART_17158_FCTR_TRGD | UART_17158_FCTR_RTS_4DELAY),
> + &ch->ch_neo_uart->fctr);
> ch->ch_r_watermark = 4;
>
> writeb(32, &ch->ch_neo_uart->rfifo);
> @@ -187,7 +190,8 @@ static inline void neo_set_ixon_flow_control(struct channel_t *ch)
> /* Turn on UART enhanced bits */
> writeb(efr, &ch->ch_neo_uart->efr);
>
> - writeb((UART_17158_FCTR_TRGD | UART_17158_FCTR_RTS_8DELAY), &ch->ch_neo_uart->fctr);
> + writeb((UART_17158_FCTR_TRGD | UART_17158_FCTR_RTS_8DELAY),
> + &ch->ch_neo_uart->fctr);
> ch->ch_r_watermark = 4;
>
> writeb(32, &ch->ch_neo_uart->rfifo);
> @@ -225,7 +229,8 @@ static inline void neo_set_ixoff_flow_control(struct channel_t *ch)
> writeb(efr, &ch->ch_neo_uart->efr);
>
> /* Turn on table D, with 8 char hi/low watermarks */
> - writeb((UART_17158_FCTR_TRGD | UART_17158_FCTR_RTS_8DELAY), &ch->ch_neo_uart->fctr);
> + writeb((UART_17158_FCTR_TRGD | UART_17158_FCTR_RTS_8DELAY),
> + &ch->ch_neo_uart->fctr);
>
> writeb(8, &ch->ch_neo_uart->tfifo);
> ch->ch_t_tlevel = 8;
> @@ -265,7 +270,8 @@ static inline void neo_set_no_input_flow_control(struct channel_t *ch)
> writeb(efr, &ch->ch_neo_uart->efr);
>
> /* Turn on table D, with 8 char hi/low watermarks */
> - writeb((UART_17158_FCTR_TRGD | UART_17158_FCTR_RTS_8DELAY), &ch->ch_neo_uart->fctr);
> + writeb((UART_17158_FCTR_TRGD | UART_17158_FCTR_RTS_8DELAY),
> + &ch->ch_neo_uart->fctr);
>
> ch->ch_r_watermark = 0;
>
> @@ -302,7 +308,8 @@ static inline void neo_set_no_output_flow_control(struct channel_t *ch)
> writeb(efr, &ch->ch_neo_uart->efr);
>
> /* Turn on table D, with 8 char hi/low watermarks */
> - writeb((UART_17158_FCTR_TRGD | UART_17158_FCTR_RTS_8DELAY), &ch->ch_neo_uart->fctr);
> + writeb((UART_17158_FCTR_TRGD | UART_17158_FCTR_RTS_8DELAY),
> + &ch->ch_neo_uart->fctr);
>
> ch->ch_r_watermark = 0;
>
> @@ -321,7 +328,8 @@ static inline void neo_set_no_output_flow_control(struct channel_t *ch)
> static inline void neo_set_new_start_stop_chars(struct channel_t *ch)
> {
> /* if hardware flow control is set, then skip this whole thing */
> - if (ch->ch_digi.digi_flags & (CTSPACE | RTSPACE) || ch->ch_c_cflag & CRTSCTS)
> + if (ch->ch_digi.digi_flags & (CTSPACE | RTSPACE) ||
> + ch->ch_c_cflag & CRTSCTS)
> return;
>
> /* Tell UART what start/stop chars it should be looking for */
> @@ -393,7 +401,8 @@ static inline void neo_parse_isr(struct dgnc_board *brd, uint port)
> break;
>
> /*
> - * Yank off the upper 2 bits, which just show that the FIFO's are enabled.
> + * Yank off the upper 2 bits,
> + * which just show that the FIFO's are enabled.
> */
> isr &= ~(UART_17158_IIR_FIFO_ENABLED);
>
> @@ -666,7 +675,8 @@ static void neo_param(struct tty_struct *tty)
> };
>
> /* Only use the TXPrint baud rate if the terminal unit is NOT open */
> - if (!(ch->ch_tun.un_flags & UN_ISOPEN) && (un->un_type = DGNC_PRINT))
> + if (!(ch->ch_tun.un_flags & UN_ISOPEN) &&
> + (un->un_type = DGNC_PRINT))
> baud = C_BAUD(ch->ch_pun.un_tty) & 0xff;
> else
> baud = C_BAUD(ch->ch_tun.un_tty) & 0xff;
> @@ -679,7 +689,8 @@ static void neo_param(struct tty_struct *tty)
>
> jindex = baud;
>
> - if ((iindex >= 0) && (iindex < 4) && (jindex >= 0) && (jindex < 16))
> + if ((iindex >= 0) && (iindex < 4) &&
> + (jindex >= 0) && (jindex < 16))
> baud = bauds[iindex][jindex];
> else
> baud = 0;
> @@ -787,7 +798,8 @@ static void neo_param(struct tty_struct *tty)
> neo_set_cts_flow_control(ch);
> } else if (ch->ch_c_iflag & IXON) {
> /* If start/stop is set to disable, then we should disable flow control */
> - if ((ch->ch_startc = _POSIX_VDISABLE) || (ch->ch_stopc = _POSIX_VDISABLE))
> + if ((ch->ch_startc = _POSIX_VDISABLE) ||
> + (ch->ch_stopc = _POSIX_VDISABLE))
> neo_set_no_output_flow_control(ch);
> else
> neo_set_ixon_flow_control(ch);
> @@ -799,7 +811,8 @@ static void neo_param(struct tty_struct *tty)
> neo_set_rts_flow_control(ch);
> } else if (ch->ch_c_iflag & IXOFF) {
> /* If start/stop is set to disable, then we should disable flow control */
> - if ((ch->ch_startc = _POSIX_VDISABLE) || (ch->ch_stopc = _POSIX_VDISABLE))
> + if ((ch->ch_startc = _POSIX_VDISABLE) ||
> + (ch->ch_stopc = _POSIX_VDISABLE))
> neo_set_no_input_flow_control(ch);
> else
> neo_set_ixoff_flow_control(ch);
> @@ -1172,7 +1185,8 @@ static void neo_copy_data_from_uart_to_queue(struct channel_t *ch)
> linestatus = 0;
>
> /* Copy data from uart to the queue */
> - memcpy_fromio(ch->ch_rqueue + head, &ch->ch_neo_uart->txrxburst, n);
> + memcpy_fromio(ch->ch_rqueue + head,
> + &ch->ch_neo_uart->txrxburst, n);
>
> /*
> * Since RX_FIFO_DATA_ERROR was 0, we are guaranteed
> @@ -1225,7 +1239,8 @@ static void neo_copy_data_from_uart_to_queue(struct channel_t *ch)
> * we don't miss our TX FIFO emptys.
> */
> if (linestatus & (UART_LSR_THRE | UART_17158_TX_AND_FIFO_CLR)) {
> - linestatus &= ~(UART_LSR_THRE | UART_17158_TX_AND_FIFO_CLR);
> + linestatus &= ~(UART_LSR_THRE |
> + UART_17158_TX_AND_FIFO_CLR);
> ch->ch_flags |= (CH_TX_FIFO_EMPTY | CH_TX_FIFO_LWM);
> }
>
> @@ -1255,7 +1270,8 @@ static void neo_copy_data_from_uart_to_queue(struct channel_t *ch)
> qleft++;
> }
>
> - memcpy_fromio(ch->ch_rqueue + head, &ch->ch_neo_uart->txrxburst, 1);
> + memcpy_fromio(ch->ch_rqueue + head,
> + &ch->ch_neo_uart->txrxburst, 1);
> ch->ch_equeue[head] = (unsigned char)linestatus;
>
> /* Ditch any remaining linestatus value. */
> @@ -1328,7 +1344,8 @@ static void neo_flush_uart_write(struct channel_t *ch)
> if (!ch || ch->magic != DGNC_CHANNEL_MAGIC)
> return;
>
> - writeb((UART_FCR_ENABLE_FIFO | UART_FCR_CLEAR_XMIT), &ch->ch_neo_uart->isr_fcr);
> + writeb((UART_FCR_ENABLE_FIFO | UART_FCR_CLEAR_XMIT),
> + &ch->ch_neo_uart->isr_fcr);
> neo_pci_posting_flush(ch->ch_bd);
>
> for (i = 0; i < 10; i++) {
> @@ -1356,7 +1373,8 @@ static void neo_flush_uart_read(struct channel_t *ch)
> if (!ch || ch->magic != DGNC_CHANNEL_MAGIC)
> return;
>
> - writeb((UART_FCR_ENABLE_FIFO | UART_FCR_CLEAR_RCVR), &ch->ch_neo_uart->isr_fcr);
> + writeb((UART_FCR_ENABLE_FIFO | UART_FCR_CLEAR_RCVR),
> + &ch->ch_neo_uart->isr_fcr);
> neo_pci_posting_flush(ch->ch_bd);
>
> for (i = 0; i < 10; i++) {
> @@ -1427,7 +1445,8 @@ static void neo_copy_data_from_queue_to_uart(struct channel_t *ch)
> ch->ch_tun.un_flags |= (UN_EMPTY);
> }
>
> - writeb(ch->ch_wqueue[ch->ch_w_tail], &ch->ch_neo_uart->txrx);
> + writeb(ch->ch_wqueue[ch->ch_w_tail],
> + &ch->ch_neo_uart->txrx);
> ch->ch_w_tail++;
> ch->ch_w_tail &= WQUEUEMASK;
> ch->ch_txcount++;
> @@ -1494,7 +1513,8 @@ static void neo_copy_data_from_queue_to_uart(struct channel_t *ch)
> ch->ch_tun.un_flags |= (UN_EMPTY);
> }
>
> - memcpy_toio(&ch->ch_neo_uart->txrxburst, ch->ch_wqueue + tail, s);
> + memcpy_toio(&ch->ch_neo_uart->txrxburst,
> + ch->ch_wqueue + tail, s);
>
> /* Add and flip queue if needed */
> tail = (tail + s) & WQUEUEMASK;
> @@ -1628,7 +1648,8 @@ static void neo_uart_init(struct channel_t *ch)
>
> /* Clear out UART and FIFO */
> readb(&ch->ch_neo_uart->txrx);
> - writeb((UART_FCR_ENABLE_FIFO | UART_FCR_CLEAR_RCVR | UART_FCR_CLEAR_XMIT), &ch->ch_neo_uart->isr_fcr);
> + writeb((UART_FCR_ENABLE_FIFO | UART_FCR_CLEAR_RCVR | UART_FCR_CLEAR_XMIT),
> + &ch->ch_neo_uart->isr_fcr);
> readb(&ch->ch_neo_uart->lsr);
> readb(&ch->ch_neo_uart->msr);
>
> @@ -1725,7 +1746,8 @@ static void neo_send_immediate_char(struct channel_t *ch, unsigned char c)
> neo_pci_posting_flush(ch->ch_bd);
> }
>
> -static unsigned int neo_read_eeprom(unsigned char __iomem *base, unsigned int address)
> +static unsigned int neo_read_eeprom(unsigned char __iomem *base,
> + unsigned int address)
> {
> unsigned int enable;
> unsigned int bits;
next prev parent reply other threads:[~2016-03-25 8:55 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-03-25 2:44 [PATCH 1/2] staging: dgnc: fix 'line over 80 characters' Daeseok Youn
2016-03-25 3:19 ` Joe Perches
2016-03-25 5:42 ` DaeSeok Youn
2016-03-25 5:56 ` Joe Perches
2016-03-25 8:55 ` walter harms [this message]
2016-03-25 9:14 ` DaeSeok Youn
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=56F4FD0D.6060905@bfs.de \
--to=wharms@bfs.de \
--cc=kernel-janitors@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox