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

Groups > linux.kernel > #1228177 > unrolled thread

[PATCH] Staging: comedi: Fix coding style issues

Started byPunit Vara <punitvara@gmail.com>
First post2015-09-18 19:00 +0200
Last post2015-09-20 10:40 +0200
Articles 8 — 4 participants

Back to article view | Back to linux.kernel

Contents

  [PATCH] Staging: comedi: Fix coding style issues Punit Vara <punitvara@gmail.com> - 2015-09-18 19:00 +0200
    Re: [PATCH] Staging: comedi: Fix coding style issues Greg KH <gregkh@linuxfoundation.org> - 2015-09-19 07:10 +0200
    Re: [PATCH] Staging: comedi: Fix coding style issues punit vara <punitvara@gmail.com> - 2015-09-19 18:30 +0200
      Re: [PATCH] Staging: comedi: Fix coding style issues Greg KH <gregkh@linuxfoundation.org> - 2015-09-19 18:40 +0200
        Re: [PATCH] Staging: comedi: Fix coding style issues punit vara <punitvara@gmail.com> - 2015-09-19 20:00 +0200
    Re: [PATCH] Staging: comedi: Fix coding style issues punit vara <punitvara@gmail.com> - 2015-09-19 20:20 +0200
      Re: [PATCH] Staging: comedi: Fix coding style issues Jaime Arrocha <jarr@innercoder.com> - 2015-09-20 03:30 +0200
        Re: [PATCH] Staging: comedi: Fix coding style issues punit vara <punitvara@gmail.com> - 2015-09-20 10:40 +0200

#1228177 — [PATCH] Staging: comedi: Fix coding style issues

FromPunit Vara <punitvara@gmail.com>
Date2015-09-18 19:00 +0200
Subject[PATCH] Staging: comedi: Fix coding style issues
Message-ID<qa7aW-8r0-25@gated-at.bofh.it>
This patch is to comedidev.h file that fixes up following warnings
reported by checkpatch.pl :

-Block comments use a trailing */ on a separate line
-Block comments use * on subsequent lines

Signed-off-by: Punit Vara <punitvara@gmail.com>
---
 drivers/staging/comedi/comedidev.h | 62 ++++++++++++++++++++------------------
 1 file changed, 32 insertions(+), 30 deletions(-)

diff --git a/drivers/staging/comedi/comedidev.h b/drivers/staging/comedi/comedidev.h
index 28a5d3a..5b0ab03 100644
--- a/drivers/staging/comedi/comedidev.h
+++ b/drivers/staging/comedi/comedidev.h
@@ -1,20 +1,20 @@
 /*
-    include/linux/comedidev.h
-    header file for kernel-only structures, variables, and constants
-
-    COMEDI - Linux Control and Measurement Device Interface
-    Copyright (C) 1997-2000 David A. Schleef <ds@schleef.org>
-
-    This program is free software; you can redistribute it and/or modify
-    it under the terms of the GNU General Public License as published by
-    the Free Software Foundation; either version 2 of the License, or
-    (at your option) any later version.
-
-    This program is distributed in the hope that it will be useful,
-    but WITHOUT ANY WARRANTY; without even the implied warranty of
-    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-    GNU General Public License for more details.
-*/
+ * include/linux/comedidev.h
+ * header file for kernel-only structures, variables, and constants
+ *
+ * COMEDI - Linux Control and Measurement Device Interface
+ * Copyright (C) 1997-2000 David A. Schleef <ds@schleef.org>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ */
 
 #ifndef _COMEDIDEV_H
 #define _COMEDIDEV_H
@@ -105,7 +105,7 @@ struct comedi_buf_map {
 	struct kref refcount;
 };
 
-/**
+/*
  * struct comedi_async - control data for asynchronous comedi commands
  * @prealloc_buf:	preallocated buffer
  * @prealloc_bufsz:	buffer size (in bytes)
@@ -215,7 +215,7 @@ struct comedi_async {
 		       unsigned int x);
 };
 
-/**
+/*
  * comedi_async callback "events"
  * @COMEDI_CB_EOS:		end-of-scan
  * @COMEDI_CB_EOA:		end-of-acquisition/output
@@ -262,9 +262,11 @@ struct comedi_device {
 	struct device *class_dev;
 	int minor;
 	unsigned int detach_count;
-	/* hw_dev is passed to dma_alloc_coherent when allocating async buffers
+	/*
+	 * hw_dev is passed to dma_alloc_coherent when allocating async buffers
 	 * for subdevices that have async_dma_dir set to something other than
-	 * DMA_NONE */
+	 * DMA_NONE
+	 */
 	struct device *hw_dev;
 
 	const char *board_name;
@@ -391,7 +393,7 @@ static inline unsigned int comedi_offset_munge(struct comedi_subdevice *s,
 	return val ^ s->maxdata ^ (s->maxdata >> 1);
 }
 
-/**
+/*
  * comedi_bytes_per_sample - determine subdevice sample size
  * @s:		comedi_subdevice struct
  *
@@ -405,7 +407,7 @@ static inline unsigned int comedi_bytes_per_sample(struct comedi_subdevice *s)
 	return s->subdev_flags & SDF_LSAMPL ? sizeof(int) : sizeof(short);
 }
 
-/**
+/*
  * comedi_sample_shift - determine log2 of subdevice sample size
  * @s:		comedi_subdevice struct
  *
@@ -421,7 +423,7 @@ static inline unsigned int comedi_sample_shift(struct comedi_subdevice *s)
 	return s->subdev_flags & SDF_LSAMPL ? 2 : 1;
 }
 
-/**
+/*
  * comedi_bytes_to_samples - converts a number of bytes to a number of samples
  * @s:		comedi_subdevice struct
  * @nbytes:	number of bytes
@@ -434,7 +436,7 @@ static inline unsigned int comedi_bytes_to_samples(struct comedi_subdevice *s,
 	return nbytes >> comedi_sample_shift(s);
 }
 
-/**
+/*
  * comedi_samples_to_bytes - converts a number of samples to a number of bytes
  * @s:		comedi_subdevice struct
  * @nsamples:	number of samples
@@ -448,7 +450,7 @@ static inline unsigned int comedi_samples_to_bytes(struct comedi_subdevice *s,
 	return nsamples << comedi_sample_shift(s);
 }
 
-/**
+/*
  * comedi_check_trigger_src() - trivially validate a comedi_cmd trigger source
  * @src: pointer to the trigger source to validate
  * @flags: bitmask of valid TRIG_* for the trigger
@@ -469,7 +471,7 @@ static inline int comedi_check_trigger_src(unsigned int *src,
 	return 0;
 }
 
-/**
+/*
  * comedi_check_trigger_is_unique() - make sure a trigger source is unique
  * @src: the trigger source to check
  */
@@ -481,7 +483,7 @@ static inline int comedi_check_trigger_is_unique(unsigned int src)
 	return 0;
 }
 
-/**
+/*
  * comedi_check_trigger_arg_is() - trivially validate a trigger argument
  * @arg: pointer to the trigger arg to validate
  * @val: the value the argument should be
@@ -496,7 +498,7 @@ static inline int comedi_check_trigger_arg_is(unsigned int *arg,
 	return 0;
 }
 
-/**
+/*
  * comedi_check_trigger_arg_min() - trivially validate a trigger argument
  * @arg: pointer to the trigger arg to validate
  * @val: the minimum value the argument should be
@@ -511,7 +513,7 @@ static inline int comedi_check_trigger_arg_min(unsigned int *arg,
 	return 0;
 }
 
-/**
+/*
  * comedi_check_trigger_arg_max() - trivially validate a trigger argument
  * @arg: pointer to the trigger arg to validate
  * @val: the maximum value the argument should be
@@ -604,7 +606,7 @@ void comedi_auto_unconfig(struct device *);
 int comedi_driver_register(struct comedi_driver *);
 void comedi_driver_unregister(struct comedi_driver *);
 
-/**
+/*
  * module_comedi_driver() - Helper macro for registering a comedi driver
  * @__comedi_driver: comedi_driver struct
  *
-- 
2.5.2

--
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]

#1228398

FromGreg KH <gregkh@linuxfoundation.org>
Date2015-09-19 07:10 +0200
Message-ID<qaizo-8mG-13@gated-at.bofh.it>
In reply to#1228177
On Fri, Sep 18, 2015 at 10:22:59PM +0530, Punit Vara wrote:
> This patch is to comedidev.h file that fixes up following warnings
> reported by checkpatch.pl :
> 
> -Block comments use a trailing */ on a separate line
> -Block comments use * on subsequent lines

Your subject needs to be very specific, please fix up.

thanks,

greg k-h
--
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]

#1228471

Frompunit vara <punitvara@gmail.com>
Date2015-09-19 18:30 +0200
Message-ID<qatbs-6DF-17@gated-at.bofh.it>
In reply to#1228177
For this patch can I put subject like

[PATCH] Staging: comedi: Fix trailing */ and * comment warning

??
Kindly suggest me as this is my first patch and i am trying to
contribute to linux kernel .

On Fri, Sep 18, 2015 at 10:22 PM, Punit Vara <punitvara@gmail.com> wrote:
> This patch is to comedidev.h file that fixes up following warnings
> reported by checkpatch.pl :
>
> -Block comments use a trailing */ on a separate line
> -Block comments use * on subsequent lines
>
> Signed-off-by: Punit Vara <punitvara@gmail.com>
> ---
>  drivers/staging/comedi/comedidev.h | 62 ++++++++++++++++++++------------------
>  1 file changed, 32 insertions(+), 30 deletions(-)
>
> diff --git a/drivers/staging/comedi/comedidev.h b/drivers/staging/comedi/comedidev.h
> index 28a5d3a..5b0ab03 100644
> --- a/drivers/staging/comedi/comedidev.h
> +++ b/drivers/staging/comedi/comedidev.h
> @@ -1,20 +1,20 @@
>  /*
> -    include/linux/comedidev.h
> -    header file for kernel-only structures, variables, and constants
> -
> -    COMEDI - Linux Control and Measurement Device Interface
> -    Copyright (C) 1997-2000 David A. Schleef <ds@schleef.org>
> -
> -    This program is free software; you can redistribute it and/or modify
> -    it under the terms of the GNU General Public License as published by
> -    the Free Software Foundation; either version 2 of the License, or
> -    (at your option) any later version.
> -
> -    This program is distributed in the hope that it will be useful,
> -    but WITHOUT ANY WARRANTY; without even the implied warranty of
> -    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> -    GNU General Public License for more details.
> -*/
> + * include/linux/comedidev.h
> + * header file for kernel-only structures, variables, and constants
> + *
> + * COMEDI - Linux Control and Measurement Device Interface
> + * Copyright (C) 1997-2000 David A. Schleef <ds@schleef.org>
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License as published by
> + * the Free Software Foundation; either version 2 of the License, or
> + * (at your option) any later version.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> + * GNU General Public License for more details.
> + */
>
>  #ifndef _COMEDIDEV_H
>  #define _COMEDIDEV_H
> @@ -105,7 +105,7 @@ struct comedi_buf_map {
>         struct kref refcount;
>  };
>
> -/**
> +/*
>   * struct comedi_async - control data for asynchronous comedi commands
>   * @prealloc_buf:      preallocated buffer
>   * @prealloc_bufsz:    buffer size (in bytes)
> @@ -215,7 +215,7 @@ struct comedi_async {
>                        unsigned int x);
>  };
>
> -/**
> +/*
>   * comedi_async callback "events"
>   * @COMEDI_CB_EOS:             end-of-scan
>   * @COMEDI_CB_EOA:             end-of-acquisition/output
> @@ -262,9 +262,11 @@ struct comedi_device {
>         struct device *class_dev;
>         int minor;
>         unsigned int detach_count;
> -       /* hw_dev is passed to dma_alloc_coherent when allocating async buffers
> +       /*
> +        * hw_dev is passed to dma_alloc_coherent when allocating async buffers
>          * for subdevices that have async_dma_dir set to something other than
> -        * DMA_NONE */
> +        * DMA_NONE
> +        */
>         struct device *hw_dev;
>
>         const char *board_name;
> @@ -391,7 +393,7 @@ static inline unsigned int comedi_offset_munge(struct comedi_subdevice *s,
>         return val ^ s->maxdata ^ (s->maxdata >> 1);
>  }
>
> -/**
> +/*
>   * comedi_bytes_per_sample - determine subdevice sample size
>   * @s:         comedi_subdevice struct
>   *
> @@ -405,7 +407,7 @@ static inline unsigned int comedi_bytes_per_sample(struct comedi_subdevice *s)
>         return s->subdev_flags & SDF_LSAMPL ? sizeof(int) : sizeof(short);
>  }
>
> -/**
> +/*
>   * comedi_sample_shift - determine log2 of subdevice sample size
>   * @s:         comedi_subdevice struct
>   *
> @@ -421,7 +423,7 @@ static inline unsigned int comedi_sample_shift(struct comedi_subdevice *s)
>         return s->subdev_flags & SDF_LSAMPL ? 2 : 1;
>  }
>
> -/**
> +/*
>   * comedi_bytes_to_samples - converts a number of bytes to a number of samples
>   * @s:         comedi_subdevice struct
>   * @nbytes:    number of bytes
> @@ -434,7 +436,7 @@ static inline unsigned int comedi_bytes_to_samples(struct comedi_subdevice *s,
>         return nbytes >> comedi_sample_shift(s);
>  }
>
> -/**
> +/*
>   * comedi_samples_to_bytes - converts a number of samples to a number of bytes
>   * @s:         comedi_subdevice struct
>   * @nsamples:  number of samples
> @@ -448,7 +450,7 @@ static inline unsigned int comedi_samples_to_bytes(struct comedi_subdevice *s,
>         return nsamples << comedi_sample_shift(s);
>  }
>
> -/**
> +/*
>   * comedi_check_trigger_src() - trivially validate a comedi_cmd trigger source
>   * @src: pointer to the trigger source to validate
>   * @flags: bitmask of valid TRIG_* for the trigger
> @@ -469,7 +471,7 @@ static inline int comedi_check_trigger_src(unsigned int *src,
>         return 0;
>  }
>
> -/**
> +/*
>   * comedi_check_trigger_is_unique() - make sure a trigger source is unique
>   * @src: the trigger source to check
>   */
> @@ -481,7 +483,7 @@ static inline int comedi_check_trigger_is_unique(unsigned int src)
>         return 0;
>  }
>
> -/**
> +/*
>   * comedi_check_trigger_arg_is() - trivially validate a trigger argument
>   * @arg: pointer to the trigger arg to validate
>   * @val: the value the argument should be
> @@ -496,7 +498,7 @@ static inline int comedi_check_trigger_arg_is(unsigned int *arg,
>         return 0;
>  }
>
> -/**
> +/*
>   * comedi_check_trigger_arg_min() - trivially validate a trigger argument
>   * @arg: pointer to the trigger arg to validate
>   * @val: the minimum value the argument should be
> @@ -511,7 +513,7 @@ static inline int comedi_check_trigger_arg_min(unsigned int *arg,
>         return 0;
>  }
>
> -/**
> +/*
>   * comedi_check_trigger_arg_max() - trivially validate a trigger argument
>   * @arg: pointer to the trigger arg to validate
>   * @val: the maximum value the argument should be
> @@ -604,7 +606,7 @@ void comedi_auto_unconfig(struct device *);
>  int comedi_driver_register(struct comedi_driver *);
>  void comedi_driver_unregister(struct comedi_driver *);
>
> -/**
> +/*
>   * module_comedi_driver() - Helper macro for registering a comedi driver
>   * @__comedi_driver: comedi_driver struct
>   *
> --
> 2.5.2
>



-- 
Regards,
Punit Vara
9033553488
--
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]

#1228473

FromGreg KH <gregkh@linuxfoundation.org>
Date2015-09-19 18:40 +0200
Message-ID<qatl8-6Oz-7@gated-at.bofh.it>
In reply to#1228471
A: No.
Q: Should I include quotations after my reply?

http://daringfireball.net/2007/07/on_top

On Sat, Sep 19, 2015 at 09:57:43PM +0530, punit vara wrote:
> For this patch can I put subject like
> 
> [PATCH] Staging: comedi: Fix trailing */ and * comment warning
> 
> ??

Yes, that would be much better thanks.

greg k-h
--
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]

#1228577

Frompunit vara <punitvara@gmail.com>
Date2015-09-19 20:00 +0200
Message-ID<qauAz-7T-37@gated-at.bofh.it>
In reply to#1228473
[PATCH] Staging: comedi: Fix block comment according to preferred style

[PATCH] Staging: comedi: Fix block comment warning

which would be the better subject for my patch ? anybody can suggest please ?

regards,
punitvara
--
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]

#1228661

Frompunit vara <punitvara@gmail.com>
Date2015-09-19 20:20 +0200
Message-ID<qauTV-Kr-59@gated-at.bofh.it>
In reply to#1228177
On Fri, Sep 18, 2015 at 10:22 PM, Punit Vara <punitvara@gmail.com> wrote:
> This patch is to comedidev.h file that fixes up following warnings
> reported by checkpatch.pl :
>
> -Block comments use a trailing */ on a separate line
> -Block comments use * on subsequent lines
>
> Signed-off-by: Punit Vara <punitvara@gmail.com>
> ---
>  drivers/staging/comedi/comedidev.h | 62 ++++++++++++++++++++------------------
>  1 file changed, 32 insertions(+), 30 deletions(-)
>
> diff --git a/drivers/staging/comedi/comedidev.h b/drivers/staging/comedi/comedidev.h
> index 28a5d3a..5b0ab03 100644
> --- a/drivers/staging/comedi/comedidev.h
> +++ b/drivers/staging/comedi/comedidev.h
> @@ -1,20 +1,20 @@
>  /*
> -    include/linux/comedidev.h
> -    header file for kernel-only structures, variables, and constants
> -
> -    COMEDI - Linux Control and Measurement Device Interface
> -    Copyright (C) 1997-2000 David A. Schleef <ds@schleef.org>
> -
> -    This program is free software; you can redistribute it and/or modify
> -    it under the terms of the GNU General Public License as published by
> -    the Free Software Foundation; either version 2 of the License, or
> -    (at your option) any later version.
> -
> -    This program is distributed in the hope that it will be useful,
> -    but WITHOUT ANY WARRANTY; without even the implied warranty of
> -    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> -    GNU General Public License for more details.
> -*/
> + * include/linux/comedidev.h
> + * header file for kernel-only structures, variables, and constants
> + *
> + * COMEDI - Linux Control and Measurement Device Interface
> + * Copyright (C) 1997-2000 David A. Schleef <ds@schleef.org>
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License as published by
> + * the Free Software Foundation; either version 2 of the License, or
> + * (at your option) any later version.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> + * GNU General Public License for more details.
> + */
>
>  #ifndef _COMEDIDEV_H
>  #define _COMEDIDEV_H
> @@ -105,7 +105,7 @@ struct comedi_buf_map {
>         struct kref refcount;
>  };
>
> -/**
> +/*
>   * struct comedi_async - control data for asynchronous comedi commands
>   * @prealloc_buf:      preallocated buffer
>   * @prealloc_bufsz:    buffer size (in bytes)
> @@ -215,7 +215,7 @@ struct comedi_async {
>                        unsigned int x);
>  };
>
> -/**
> +/*
>   * comedi_async callback "events"
>   * @COMEDI_CB_EOS:             end-of-scan
>   * @COMEDI_CB_EOA:             end-of-acquisition/output
> @@ -262,9 +262,11 @@ struct comedi_device {
>         struct device *class_dev;
>         int minor;
>         unsigned int detach_count;
> -       /* hw_dev is passed to dma_alloc_coherent when allocating async buffers
> +       /*
> +        * hw_dev is passed to dma_alloc_coherent when allocating async buffers
>          * for subdevices that have async_dma_dir set to something other than
> -        * DMA_NONE */
> +        * DMA_NONE
> +        */
>         struct device *hw_dev;
>
>         const char *board_name;
> @@ -391,7 +393,7 @@ static inline unsigned int comedi_offset_munge(struct comedi_subdevice *s,
>         return val ^ s->maxdata ^ (s->maxdata >> 1);
>  }
>
> -/**
> +/*
>   * comedi_bytes_per_sample - determine subdevice sample size
>   * @s:         comedi_subdevice struct
>   *
> @@ -405,7 +407,7 @@ static inline unsigned int comedi_bytes_per_sample(struct comedi_subdevice *s)
>         return s->subdev_flags & SDF_LSAMPL ? sizeof(int) : sizeof(short);
>  }
>
> -/**
> +/*
>   * comedi_sample_shift - determine log2 of subdevice sample size
>   * @s:         comedi_subdevice struct
>   *
> @@ -421,7 +423,7 @@ static inline unsigned int comedi_sample_shift(struct comedi_subdevice *s)
>         return s->subdev_flags & SDF_LSAMPL ? 2 : 1;
>  }
>
> -/**
> +/*
>   * comedi_bytes_to_samples - converts a number of bytes to a number of samples
>   * @s:         comedi_subdevice struct
>   * @nbytes:    number of bytes
> @@ -434,7 +436,7 @@ static inline unsigned int comedi_bytes_to_samples(struct comedi_subdevice *s,
>         return nbytes >> comedi_sample_shift(s);
>  }
>
> -/**
> +/*
>   * comedi_samples_to_bytes - converts a number of samples to a number of bytes
>   * @s:         comedi_subdevice struct
>   * @nsamples:  number of samples
> @@ -448,7 +450,7 @@ static inline unsigned int comedi_samples_to_bytes(struct comedi_subdevice *s,
>         return nsamples << comedi_sample_shift(s);
>  }
>
> -/**
> +/*
>   * comedi_check_trigger_src() - trivially validate a comedi_cmd trigger source
>   * @src: pointer to the trigger source to validate
>   * @flags: bitmask of valid TRIG_* for the trigger
> @@ -469,7 +471,7 @@ static inline int comedi_check_trigger_src(unsigned int *src,
>         return 0;
>  }
>
> -/**
> +/*
>   * comedi_check_trigger_is_unique() - make sure a trigger source is unique
>   * @src: the trigger source to check
>   */
> @@ -481,7 +483,7 @@ static inline int comedi_check_trigger_is_unique(unsigned int src)
>         return 0;
>  }
>
> -/**
> +/*
>   * comedi_check_trigger_arg_is() - trivially validate a trigger argument
>   * @arg: pointer to the trigger arg to validate
>   * @val: the value the argument should be
> @@ -496,7 +498,7 @@ static inline int comedi_check_trigger_arg_is(unsigned int *arg,
>         return 0;
>  }
>
> -/**
> +/*
>   * comedi_check_trigger_arg_min() - trivially validate a trigger argument
>   * @arg: pointer to the trigger arg to validate
>   * @val: the minimum value the argument should be
> @@ -511,7 +513,7 @@ static inline int comedi_check_trigger_arg_min(unsigned int *arg,
>         return 0;
>  }
>
> -/**
> +/*
>   * comedi_check_trigger_arg_max() - trivially validate a trigger argument
>   * @arg: pointer to the trigger arg to validate
>   * @val: the maximum value the argument should be
> @@ -604,7 +606,7 @@ void comedi_auto_unconfig(struct device *);
>  int comedi_driver_register(struct comedi_driver *);
>  void comedi_driver_unregister(struct comedi_driver *);
>
> -/**
> +/*
>   * module_comedi_driver() - Helper macro for registering a comedi driver
>   * @__comedi_driver: comedi_driver struct
>   *
> --
> 2.5.2
>


[PATCH] Staging: comedi: Fix block comment according to preferred style

[PATCH] Staging: comedi: Fix block comment warning

which would be the better subject for my patch ? anybody can suggest please ?
--
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]

#1228807

FromJaime Arrocha <jarr@innercoder.com>
Date2015-09-20 03:30 +0200
Message-ID<qaBC2-1TU-1@gated-at.bofh.it>
In reply to#1228661
On 09/19/2015 01:13 PM, punit vara wrote:
> [PATCH] Staging: comedi: Fix block comment warning

[PATCH] Staging: comedi: Coding style warnings fix for block comments



--
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]

#1228848

Frompunit vara <punitvara@gmail.com>
Date2015-09-20 10:40 +0200
Message-ID<qaIka-33c-23@gated-at.bofh.it>
In reply to#1228807
On Sun, Sep 20, 2015 at 6:50 AM, Jaime Arrocha <jarr@innercoder.com> wrote:
>
> On 09/19/2015 01:13 PM, punit vara wrote:
>>
>> [PATCH] Staging: comedi: Fix block comment warning
>
>
> [PATCH] Staging: comedi: Coding style warnings fix for block comments
>


Thank you for suggestion sir .I would appreciate your help.
--
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