Re: [PATCH] Remove uneccessary parantheas

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

 



On Saturday, September 25, 2021 9:20:15 PM CEST Michael Estner wrote:
> Fix to be conform with the checkpatch style requirements

Greg K-H's "friendly patch bot" has already been triggered and sent you a 
message. Usually, according to Greg's style :), this is all you get and then 
you are left alone to figure out what you did wrong and eventually submit a 
new version.

However, as this is your very first patch, I would like to give you some 
additional information that may help you.

1) Where is the name of the driver your fixing? You must place it in the 
subject and its position and formatting must comply with this subsystem 
rules. Please, read other patches for staging among those that have already 
been accepted.

2) What language are you using in the subject? "Remove uneccessary 
parantheas" has no meaning in English. Typos can happen, but you have not 
made any typos: your "Subject line" is a completely incomprehensible sequence 
of characters. :)

3) "Fix to be conform[]" (sigh). Fix what? Changelogs (or "Commits messages") 
must be self-contained, so write them like if you expected that the Reviewers 
immediately forget what they read in the "Subject" line. While you are at it, 
please also correct grammar. 

(Finally, it is the Linux kernel maintainers and developers who have "style 
requirements", listed in the "Linux kernel coding style" document, instead 
checkpatch.pl has no requirements (it just checks for style violations)).

Thanks,

Fabio
 
> Signed-off-by: Michael Estner <michaelestner@xxxxxx>
> ---







[Index of Archives]     [Linux Driver Development]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux