Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1496090 > unrolled thread

[PATCH] Staging:dgnc:dgnc_neo: fixed 80 character line limit coding style issue

Started byNadim Almas <nadim.902@gmail.com>
First post2016-10-06 00:00 +0200
Last post2016-10-06 18:10 +0200
Articles 5 — 3 participants

Back to article view | Back to linux.kernel

Contents

  [PATCH] Staging:dgnc:dgnc_neo: fixed 80 character line limit coding  style issue Nadim Almas <nadim.902@gmail.com> - 2016-10-06 00:00 +0200
    Re: [PATCH] Staging:dgnc:dgnc_neo: fixed 80 character line limit  coding style issue Greg KH <gregkh@linuxfoundation.org> - 2016-10-06 07:30 +0200
      Re: [PATCH] Staging:dgnc:dgnc_neo: fixed 80 character line limit  coding style issue Nadim Almas <nadim.902@gmail.com> - 2016-10-06 10:30 +0200
        Re: [PATCH] Staging:dgnc:dgnc_neo: fixed 80 character line limit  coding style issue Nadim Almas <nadim.902@gmail.com> - 2016-10-06 17:30 +0200
          Re: [PATCH] Staging:dgnc:dgnc_neo: fixed 80 character line limit  coding style issue Joe Perches <coupons@perches.com> - 2016-10-06 18:10 +0200

#1496090 — [PATCH] Staging:dgnc:dgnc_neo: fixed 80 character line limit coding style issue

FromNadim Almas <nadim.902@gmail.com>
Date2016-10-06 00:00 +0200
Subject[PATCH] Staging:dgnc:dgnc_neo: fixed 80 character line limit coding style issue
Message-ID<sp2om-7M2-13@gated-at.bofh.it>
Fixed coding style issue

Signed-off-by: Nadim Almas <nadim.902@gmail.com>
---
 drivers/staging/dgnc/dgnc_neo.h | 18 ++++++++++++------
 1 file changed, 12 insertions(+), 6 deletions(-)

diff --git a/drivers/staging/dgnc/dgnc_neo.h b/drivers/staging/dgnc/dgnc_neo.h
index abddd48..65994e3 100644
--- a/drivers/staging/dgnc/dgnc_neo.h
+++ b/drivers/staging/dgnc/dgnc_neo.h
@@ -30,7 +30,8 @@
 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,7 +109,8 @@ 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 */
 
 /*
@@ -119,8 +121,10 @@ struct neo_uart_struct {
 #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_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 +136,10 @@ 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 whether chip saw an */
+						/* incoming XOFF char  */
+#define UART_17158_XON_DETECT	0x2	/* Indicates whether chip saw an */
+							/* incoming XON char */
 
 #define UART_17158_IER_RSVD1	0x10	/* Reserved by Exar */
 #define UART_17158_IER_XOFF	0x20	/* Xoff Interrupt Enable */
-- 
2.7.4

[toc] | [next] | [standalone]

#1496166 — Re: [PATCH] Staging:dgnc:dgnc_neo: fixed 80 character line limit coding style issue

FromGreg KH <gregkh@linuxfoundation.org>
Date2016-10-06 07:30 +0200
SubjectRe: [PATCH] Staging:dgnc:dgnc_neo: fixed 80 character line limit coding style issue
Message-ID<sp9pL-47Q-15@gated-at.bofh.it>
In reply to#1496090
On Wed, Oct 05, 2016 at 02:53:58PM -0700, Nadim Almas wrote:
> Fixed coding style issue
> 
> Signed-off-by: Nadim Almas <nadim.902@gmail.com>
> ---
>  drivers/staging/dgnc/dgnc_neo.h | 18 ++++++++++++------
>  1 file changed, 12 insertions(+), 6 deletions(-)
> 
> diff --git a/drivers/staging/dgnc/dgnc_neo.h b/drivers/staging/dgnc/dgnc_neo.h
> index abddd48..65994e3 100644
> --- a/drivers/staging/dgnc/dgnc_neo.h
> +++ b/drivers/staging/dgnc/dgnc_neo.h
> @@ -30,7 +30,8 @@
>  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 */

Does that really look better now than it did before?

I don't think so :(

sorry,

greg k-h

[toc] | [prev] | [next] | [standalone]

#1496217 — Re: [PATCH] Staging:dgnc:dgnc_neo: fixed 80 character line limit coding style issue

FromNadim Almas <nadim.902@gmail.com>
Date2016-10-06 10:30 +0200
SubjectRe: [PATCH] Staging:dgnc:dgnc_neo: fixed 80 character line limit coding style issue
Message-ID<spcdX-60e-15@gated-at.bofh.it>
In reply to#1496166
May be it is not looking better than yours but it removed 6 warning of
the file which is shown by running "perl scripts/checkpatch.pl -f
drivers/staging/dgnc/* | less" commend

On Thu, Oct 6, 2016 at 10:51 AM, Greg KH <gregkh@linuxfoundation.org> wrote:
> On Wed, Oct 05, 2016 at 02:53:58PM -0700, Nadim Almas wrote:
>> Fixed coding style issue
>>
>> Signed-off-by: Nadim Almas <nadim.902@gmail.com>
>> ---
>>  drivers/staging/dgnc/dgnc_neo.h | 18 ++++++++++++------
>>  1 file changed, 12 insertions(+), 6 deletions(-)
>>
>> diff --git a/drivers/staging/dgnc/dgnc_neo.h b/drivers/staging/dgnc/dgnc_neo.h
>> index abddd48..65994e3 100644
>> --- a/drivers/staging/dgnc/dgnc_neo.h
>> +++ b/drivers/staging/dgnc/dgnc_neo.h
>> @@ -30,7 +30,8 @@
>>  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 */
>
> Does that really look better now than it did before?
>
> I don't think so :(
>
> sorry,
>
> greg k-h

[toc] | [prev] | [next] | [standalone]

#1496706 — Re: [PATCH] Staging:dgnc:dgnc_neo: fixed 80 character line limit coding style issue

FromNadim Almas <nadim.902@gmail.com>
Date2016-10-06 17:30 +0200
SubjectRe: [PATCH] Staging:dgnc:dgnc_neo: fixed 80 character line limit coding style issue
Message-ID<spiMp-20b-15@gated-at.bofh.it>
In reply to#1496217
May be it is not looking better than it did before but it removed 6 warning of
the file which is shown by running "perl scripts/checkpatch.pl -f
drivers/staging/dgnc/* | less" commend
than why it is not acceptable

On Thu, Oct 6, 2016 at 1:52 PM, Nadim Almas <nadim.902@gmail.com> wrote:
> May be it is not looking better than yours but it removed 6 warning of
> the file which is shown by running "perl scripts/checkpatch.pl -f
> drivers/staging/dgnc/* | less" commend
>
> On Thu, Oct 6, 2016 at 10:51 AM, Greg KH <gregkh@linuxfoundation.org> wrote:
>> On Wed, Oct 05, 2016 at 02:53:58PM -0700, Nadim Almas wrote:
>>> Fixed coding style issue
>>>
>>> Signed-off-by: Nadim Almas <nadim.902@gmail.com>
>>> ---
>>>  drivers/staging/dgnc/dgnc_neo.h | 18 ++++++++++++------
>>>  1 file changed, 12 insertions(+), 6 deletions(-)
>>>
>>> diff --git a/drivers/staging/dgnc/dgnc_neo.h b/drivers/staging/dgnc/dgnc_neo.h
>>> index abddd48..65994e3 100644
>>> --- a/drivers/staging/dgnc/dgnc_neo.h
>>> +++ b/drivers/staging/dgnc/dgnc_neo.h
>>> @@ -30,7 +30,8 @@
>>>  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 */
>>
>> Does that really look better now than it did before?
>>
>> I don't think so :(
>>
>> sorry,
>>
>> greg k-h

[toc] | [prev] | [next] | [standalone]

#1496724 — Re: [PATCH] Staging:dgnc:dgnc_neo: fixed 80 character line limit coding style issue

FromJoe Perches <coupons@perches.com>
Date2016-10-06 18:10 +0200
SubjectRe: [PATCH] Staging:dgnc:dgnc_neo: fixed 80 character line limit coding style issue
Message-ID<spjp7-2A0-9@gated-at.bofh.it>
In reply to#1496706
On Thu, 2016-10-06 at 20:53 +0530, Nadim Almas wrote:
> May be it is not looking better than it did before but it removed 6 warning of
> the file which is shown by running "perl scripts/checkpatch.pl -f
> drivers/staging/dgnc/* | less" commend
> than why it is not acceptable

(please don't top post)

Because it's ugly and checkpatch output messages are not dicta.

If this is really useful, realign all the comments to start on
the same column.

more below...

> On Thu, Oct 6, 2016 at 1:52 PM, Nadim Almas <nadim.902@gmail.com> wrote:
> > May be it is not looking better than yours but it removed 6 warning of
> > the file which is shown by running "perl scripts/checkpatch.pl -f
> > drivers/staging/dgnc/* | less" commend
> > 
> > On Thu, Oct 6, 2016 at 10:51 AM, Greg KH <gregkh@linuxfoundation.org> wrote:
> > > On Wed, Oct 05, 2016 at 02:53:58PM -0700, Nadim Almas wrote:
> > > > Fixed coding style issue
> > > > 
> > > > Signed-off-by: Nadim Almas <nadim.902@gmail.com>
> > > > ---
> > > >  drivers/staging/dgnc/dgnc_neo.h | 18 ++++++++++++------
> > > >  1 file changed, 12 insertions(+), 6 deletions(-)
> > > > 
> > > > diff --git a/drivers/staging/dgnc/dgnc_neo.h b/drivers/staging/dgnc/dgnc_neo.h
> > > > index abddd48..65994e3 100644
> > > > --- a/drivers/staging/dgnc/dgnc_neo.h
> > > > +++ b/drivers/staging/dgnc/dgnc_neo.h
> > > > @@ -30,7 +30,8 @@
> > > >  struct neo_uart_struct {O
> > > >       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 */

Consider

12345678901234567890123456789012345678901234567890123456789012345678901234567890

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 & Fifo Control Reg */

or

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 and
                         *               Fifo Control Reg
			 */

or

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/Fifo Control Reg

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web