staging/dgnc/dgnc_neo.h Lines less than 80 characters
PATCH 1/1 staging/dgnc/dgnc_neo.h Lines less than 80 characters
Fix checkpatch.pl error: "Lines over 80 characters" to conform to Kernel coding style.
--- drivers/staging/dgnc/dgnc_neo.h | 23 +++++++++++++---------- 1 file changed, 13 insertions(+), 10 deletions(-)
drivers/staging/dgnc/dgnc_neo.h +13 -10
@@ -30,7 +30,7 @@struct neo_uart_struct {u8 txrx;/* WR RHR/THR - Holding Reg */ u8 ier;/* WR IER - Interrupt Enable Reg */ -u8 isr_fcr;/* WR ISR/FCR - Interrupt Status Reg/Fifo Control Reg */ +u8 isr_fcr;/* WR ISR/FCR - Interrupt Status Reg/Fifo Control Reg */ u8 lcr;/* WR LCR - Line Control Reg */ u8 mcr;/* WR MCR - Modem Control Reg */ u8 lsr;/* WR LSR - Line Status Reg */
@@ -108,19 +108,21 @@ struct neo_uart_struct {/* 17158 Extended IIR's */ #define UART_17158_IIR_RDI_TIMEOUT 0x0C/* Receiver data TIMEOUT */ #define UART_17158_IIR_XONXOFF 0x10/* Received an XON/XOFF char */ -#define UART_17158_IIR_HWFLOW_STATE_CHANGE 0x20/* CTS/DSR or RTS/DTR state change */ +#define UART_17158_IIR_HWFLOW_STATE_CHANGE 0x20 +/* CTS/DSR or RTS/DTR state change */ #define UART_17158_IIR_FIFO_ENABLED 0xC0/* 16550 FIFOs are Enabled */ /* * These are the extended interrupts that get sent * back to us from the UART's 32bit interrupt register */ -#define UART_17158_RX_LINE_STATUS 0x1/* RX Ready */ -#define UART_17158_RXRDY_TIMEOUT 0x2/* RX Ready Timeout */ -#define UART_17158_TXRDY 0x3/* TX Ready */ -#define UART_17158_MSR 0x4/* Modem State Change */ -#define UART_17158_TX_AND_FIFO_CLR 0x40/* Transmitter Holding Reg Empty */ -#define UART_17158_RX_FIFO_DATA_ERROR 0x80/* UART detected an RX FIFO Data error */ +#define UART_17158_RX_LINE_STATUS 0x1/* RX Ready */ +#define UART_17158_RXRDY_TIMEOUT 0x2/* RX Ready Timeout */ +#define UART_17158_TXRDY 0x3/* TX Ready */ +#define UART_17158_MSR 0x4/* Modem State Change */ +#define UART_17158_TX_AND_FIFO_CLR 0x40/* Transmitter Holding Reg Empty */ +#define UART_17158_RX_FIFO_DATA_ERROR 0x80 +/* UART detected an RX FIFO Data error */ /* * These are the EXTENDED definitions for the 17C158's Interrupt
@@ -132,8 +134,9 @@ struct neo_uart_struct {#define UART_17158_EFR_RTSDTR 0x40/* Auto RTS/DTR Flow Control Enable */ #define UART_17158_EFR_CTSDSR 0x80/* Auto CTS/DSR Flow COntrol Enable */ -#define UART_17158_XOFF_DETECT 0x1/* Indicates whether chip saw an incoming XOFF char */ -#define UART_17158_XON_DETECT 0x2/* Indicates whether chip saw an incoming XON char */ +#define UART_17158_XOFF_DETECT 0x1 +/*Indicates if chip saw incoming XOFF char */ +#define UART_17158_XON_DETECT 0x2/*Indicates if chip saw incoming XON char */ #define UART_17158_IER_RSVD1 0x10/* Reserved by Exar */ #define UART_17158_IER_XOFF 0x20/* Xoff Interrupt Enable */
Signature · 1 line
2.7.4
Quoted 13 lines · Keila Novo
Fix checkpatch.pl error: "Lines over 80 characters" to conform to Kernel coding style.
Signed-off-by Keila Novo <[email protected]>--- drivers/staging/dgnc/dgnc_neo.h | 23 +++++++++++++---------- 1 file changed, 13 insertions(+), 10 deletions(-)drivers/staging/dgnc/dgnc_neo.h +0
-0
drivers/staging/dgnc/dgnc_neo.h line 36
Quoted 8 lines
drivers/staging/dgnc/dgnc_neo.h +1
-1@@ -30,7 +30,7 @@struct neo_uart_struct {u8 txrx;/* WR RHR/THR - Holding Reg */ u8 ier;/* WR IER - Interrupt Enable Reg */ -u8 isr_fcr;/* WR ISR/FCR - Interrupt Status Reg/Fifo Control Reg */ +u8 isr_fcr;/* WR ISR/FCR - Interrupt Status Reg/Fifo Control Reg */ u8 lcr;/* WR LCR - Line Control Reg */ u8 mcr;/* WR MCR - Modem Control Reg */ u8 lsr;/* WR LSR - Line Status Reg */
The rsult does not look very nice. It would be betterto put a space after the ; in the line that you changed, and then line all the other lines up with that.
julia
Quoted 51 lines
drivers/staging/dgnc/dgnc_neo.h +12
-10@@ -108,19 +108,21 @@ struct neo_uart_struct {/* 17158 Extended IIR's */ #define UART_17158_IIR_RDI_TIMEOUT 0x0C/* Receiver data TIMEOUT */ #define UART_17158_IIR_XONXOFF 0x10/* Received an XON/XOFF char */ -#define UART_17158_IIR_HWFLOW_STATE_CHANGE 0x20/* CTS/DSR or RTS/DTR state change */ +#define UART_17158_IIR_HWFLOW_STATE_CHANGE 0x20 +/* CTS/DSR or RTS/DTR state change */ #define UART_17158_IIR_FIFO_ENABLED 0xC0/* 16550 FIFOs are Enabled */ /* * These are the extended interrupts that get sent * back to us from the UART's 32bit interrupt register */ -#define UART_17158_RX_LINE_STATUS 0x1/* RX Ready */ -#define UART_17158_RXRDY_TIMEOUT 0x2/* RX Ready Timeout */ -#define UART_17158_TXRDY 0x3/* TX Ready */ -#define UART_17158_MSR 0x4/* Modem State Change */ -#define UART_17158_TX_AND_FIFO_CLR 0x40/* Transmitter Holding Reg Empty */ -#define UART_17158_RX_FIFO_DATA_ERROR 0x80/* UART detected an RX FIFO Data error */ +#define UART_17158_RX_LINE_STATUS 0x1/* RX Ready */ +#define UART_17158_RXRDY_TIMEOUT 0x2/* RX Ready Timeout */ +#define UART_17158_TXRDY 0x3/* TX Ready */ +#define UART_17158_MSR 0x4/* Modem State Change */ +#define UART_17158_TX_AND_FIFO_CLR 0x40/* Transmitter Holding Reg Empty */ +#define UART_17158_RX_FIFO_DATA_ERROR 0x80 +/* UART detected an RX FIFO Data error */ /* * These are the EXTENDED definitions for the 17C158's Interrupt @@ -132,8 +134,9 @@ struct neo_uart_struct {#define UART_17158_EFR_RTSDTR 0x40/* Auto RTS/DTR Flow Control Enable */ #define UART_17158_EFR_CTSDSR 0x80/* Auto CTS/DSR Flow COntrol Enable */ -#define UART_17158_XOFF_DETECT 0x1/* Indicates whether chip saw an incoming XOFF char */ -#define UART_17158_XON_DETECT 0x2/* Indicates whether chip saw an incoming XON char */ +#define UART_17158_XOFF_DETECT 0x1 +/*Indicates if chip saw incoming XOFF char */ +#define UART_17158_XON_DETECT 0x2/*Indicates if chip saw incoming XON char */ #define UART_17158_IER_RSVD1 0x10/* Reserved by Exar */ #define UART_17158_IER_XOFF 0x20/* Xoff Interrupt Enable */ --2.7.4
--
You received this message because you are subscribed to the Google Groups "outreachy-kernel" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected].
To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/20161007224905.GA16609%40kheixx-VB.
For more options, visit https://groups.google.com/d/optout.