[PATCH] Staging:dgnc:dgnc_neo: fixed 80 character line limit coding style issue
Joe Perches
coupons at perches.com
Thu Oct 6 15:52:23 UTC 2016
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 at 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 at 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 at 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
More information about the devel
mailing list