Re: [PATCH V2 1/2] cpumask: Fix kernel-doc formatting errors in cpumask.h

[Akira Yokosawa <akiyks@gmail.com>]

Hi again, 

Viresh Kumar wrote:
> This fixes various kernel-doc formatting errors in cpumask.h:
> 
> - WARNING: Inline literal start-string without end-string.
> - ERROR: Unexpected indentation.
> - ERROR: Unknown target name: "gfp".
> 
> Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
> ---
>  include/linux/cpumask.h | 65 +++++++++++++++++++++--------------------
>  1 file changed, 34 insertions(+), 31 deletions(-)
> 

Didn't come up in your diff, but you need additional changes shown below:

diff --git a/include/linux/cpumask.h b/include/linux/cpumask.h
index 656d2208467e..a6c1961cc535 100644
--- a/include/linux/cpumask.h
+++ b/include/linux/cpumask.h
@@ -285,10 +285,10 @@ unsigned int cpumask_next_and(int n, const struct cpumask *src1p,
 }
 
 /**
- * cpumask_next_and_wrap - get the next cpu in *src1p & *src2p, starting from
- *                        @n+1. If nothing found, wrap around and start from
+ * cpumask_next_and_wrap - get the next cpu in *@src1p & *@src2p, starting from
+ *                        @n+ 1. If nothing found, wrap around and start from
  *                        the beginning
- * @n: the cpu prior to the place to search (i.e. search starts from @n+1)
+ * @n: the cpu prior to the place to search (i.e. search starts from @n +1)
  * @src1p: the first cpumask pointer
  * @src2p: the second cpumask pointer
  *
@@ -306,9 +306,9 @@ unsigned int cpumask_next_and_wrap(int n, const struct cpumask *src1p,
 }
 
 /**
- * cpumask_next_wrap - get the next cpu in *src, starting from @n+1. If nothing
+ * cpumask_next_wrap - get the next cpu in *@src, starting from @n +1. If nothing
  *                    found, wrap around and start from the beginning
- * @n: the cpu prior to the place to search (i.e. search starts from @n+1)
+ * @n: the cpu prior to the place to search (i.e. search starts from @n +1)
  * @src: cpumask pointer
  *
  * Return: next set bit, wrapped if needed, or >= nr_cpu_ids if @src is empty.

Feel free to squash into your 1/2.
My R-b tag still stands.

        Thanks, Akira