Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1496724
| From | Joe Perches <coupons@perches.com> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH] Staging:dgnc:dgnc_neo: fixed 80 character line limit coding style issue |
| Date | 2016-10-06 18:10 +0200 |
| Message-ID | <spjp7-2A0-9@gated-at.bofh.it> (permalink) |
| References | <sp2om-7M2-13@gated-at.bofh.it> <sp9pL-47Q-15@gated-at.bofh.it> <spcdX-60e-15@gated-at.bofh.it> <spiMp-20b-15@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
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
Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread
[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
csiph-web