lkml.org 
[lkml]   [2022]   [Feb]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH v2] [hpsa] Fix the wrong chars in comment section
From

在 2022/2/16 上午1:37, Bart Van Assche 写道:
> On 2/14/22 17:52, James Wang wrote:
>> These '+' should be redundant.
>>
>> Signed-off-by: James Wang <jnwang@linux.alibaba.com>
>> Reviewed-by: Bart Van Assche <bvanassche@acm.org>
>
> Please do NOT add a Reviewed-by tag if a reviewer has not posted that
> tag himself.
>
> I have the following additional comments on this patch:
> - I am not convinced that this patch is useful since it does not help
>   users of the hpsa driver.

Maybe the patch helps the coding style. ;-)

James.W isn't a expert in storage system,  James.B should be;

but I have to post the issue what I see as a kernel hacker(junior).


> - The comment still does not conform to the Linux kernel coding style.
>   From coding-style.rst:
>
OK; I have run the script to check the code style this patch;

I like to imporve this patch again, without reviewed-by;


James


> The preferred style for long (multi-line) comments is:
>
> .. code-block:: c
>
>     /*
>      * This is the preferred style for multi-line
>      * comments in the Linux kernel source code.
>      * Please use it consistently.
>      *
>      * Description:  A column of asterisks on the left side,
>      * with beginning and ending almost-blank lines.
>      */
>
> Bart.

\
 
 \ /
  Last update: 2022-02-22 08:28    [W:0.053 / U:0.568 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site