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


Groups > linux.kernel > #1290658 > unrolled thread

[PATCH] Staging: comedi: pcmcia: fixed a line with over 80 chars

Started byPhilippe Loctaux <phil@philippeloctaux.com>
First post2015-12-13 19:10 +0100
Last post2015-12-19 20:30 +0100
Articles 3 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] Staging: comedi: pcmcia: fixed a line with over 80 chars Philippe Loctaux <phil@philippeloctaux.com> - 2015-12-13 19:10 +0100
    Re: [PATCH] Staging: comedi: pcmcia: fixed a line with over 80 chars Greg KH <gregkh@linuxfoundation.org> - 2015-12-19 01:50 +0100
      Re: [PATCH] Staging: comedi: pcmcia: fixed a line with over 80 chars Philippe Loctaux <phil@philippeloctaux.com> - 2015-12-19 20:30 +0100

#1290658 — [PATCH] Staging: comedi: pcmcia: fixed a line with over 80 chars

FromPhilippe Loctaux <phil@philippeloctaux.com>
Date2015-12-13 19:10 +0100
Subject[PATCH] Staging: comedi: pcmcia: fixed a line with over 80 chars
Message-ID<qFjfP-3k8-1@gated-at.bofh.it>
This patch fixes the checkpatch.pl warning:

WARNING: line over 80 characters

Signed-off-by: Philippe Loctaux <phil@philippeloctaux.com>
---
 drivers/staging/comedi/comedi_pcmcia.h | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/comedi/comedi_pcmcia.h b/drivers/staging/comedi/comedi_pcmcia.h
index 5d3db2b..245e6bd 100644
--- a/drivers/staging/comedi/comedi_pcmcia.h
+++ b/drivers/staging/comedi/comedi_pcmcia.h
@@ -38,8 +38,10 @@ int comedi_pcmcia_driver_register(struct comedi_driver *,
 void comedi_pcmcia_driver_unregister(struct comedi_driver *,
 				     struct pcmcia_driver *);
 
-/**
- * module_comedi_pcmcia_driver() - Helper macro for registering a comedi PCMCIA driver
+/*
+ * module_comedi_pcmcia_driver()
+ * Helper macro for registering a comedi PCMCIA driver
+ *
  * @__comedi_driver: comedi_driver struct
  * @__pcmcia_driver: pcmcia_driver struct
  *
-- 
2.6.4

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]


#1295249

FromGreg KH <gregkh@linuxfoundation.org>
Date2015-12-19 01:50 +0100
Message-ID<qHdSF-4si-1@gated-at.bofh.it>
In reply to#1290658
On Sun, Dec 13, 2015 at 05:40:34PM +0100, Philippe Loctaux wrote:
> This patch fixes the checkpatch.pl warning:
> 
> WARNING: line over 80 characters
> 
> Signed-off-by: Philippe Loctaux <phil@philippeloctaux.com>
> ---
>  drivers/staging/comedi/comedi_pcmcia.h | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/staging/comedi/comedi_pcmcia.h b/drivers/staging/comedi/comedi_pcmcia.h
> index 5d3db2b..245e6bd 100644
> --- a/drivers/staging/comedi/comedi_pcmcia.h
> +++ b/drivers/staging/comedi/comedi_pcmcia.h
> @@ -38,8 +38,10 @@ int comedi_pcmcia_driver_register(struct comedi_driver *,
>  void comedi_pcmcia_driver_unregister(struct comedi_driver *,
>  				     struct pcmcia_driver *);
>  
> -/**
> - * module_comedi_pcmcia_driver() - Helper macro for registering a comedi PCMCIA driver
> +/*
> + * module_comedi_pcmcia_driver()
> + * Helper macro for registering a comedi PCMCIA driver

No, you just broke the kernel-doc formatting here :(
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

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


#1295435

FromPhilippe Loctaux <phil@philippeloctaux.com>
Date2015-12-19 20:30 +0100
Message-ID<qHvmx-7hd-3@gated-at.bofh.it>
In reply to#1295249
Sorry I didn't read the doc correctly!

--
Philippe Loctaux
phil@philippeloctaux.com

On Fri, Dec 18, 2015 at 03:00:42PM -0800, Greg KH wrote:
> On Sun, Dec 13, 2015 at 05:40:34PM +0100, Philippe Loctaux wrote:
> > This patch fixes the checkpatch.pl warning:
> > 
> > WARNING: line over 80 characters
> > 
> > Signed-off-by: Philippe Loctaux <phil@philippeloctaux.com>
> > ---
> >  drivers/staging/comedi/comedi_pcmcia.h | 6 ++++--
> >  1 file changed, 4 insertions(+), 2 deletions(-)
> > 
> > diff --git a/drivers/staging/comedi/comedi_pcmcia.h b/drivers/staging/comedi/comedi_pcmcia.h
> > index 5d3db2b..245e6bd 100644
> > --- a/drivers/staging/comedi/comedi_pcmcia.h
> > +++ b/drivers/staging/comedi/comedi_pcmcia.h
> > @@ -38,8 +38,10 @@ int comedi_pcmcia_driver_register(struct comedi_driver *,
> >  void comedi_pcmcia_driver_unregister(struct comedi_driver *,
> >  				     struct pcmcia_driver *);
> >  
> > -/**
> > - * module_comedi_pcmcia_driver() - Helper macro for registering a comedi PCMCIA driver
> > +/*
> > + * module_comedi_pcmcia_driver()
> > + * Helper macro for registering a comedi PCMCIA driver
> 
> No, you just broke the kernel-doc formatting here :(
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web