Re: [PATCH v2] [hpsa] Fix the wrong chars in comment section

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




在 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@xxxxxxxxxxxxxxxxx>
Reviewed-by: Bart Van Assche <bvanassche@xxxxxxx>

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.



[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]

  Powered by Linux