lkml.org 
[lkml]   [2018]   [Feb]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH] iio: apply coding-style into ms5611_probe function
Date
Hi Rodrigo,

Mostly looking good. Few comments.

Please read:

https://chris.beams.io/posts/git-commit/

1) Capitalize the first letter of commit
2) Use prefix tags to indicate the driver that is changed. Here iio:pressure:ms5611.

If not sure what prefix tags to use please use git log file.c and see the tags used by previous commits.
If not clear from previous commits be creative! :)

3) A better verb describing the change would be Fix instead of Apply.

So, the commit subject should be:

iio:pressure:ms5611: Fix coding style in probe function

4) Please correct all the occurences of the same coding style error (not sure if here is the case).

Other than that it looks good to me. Please send v2.

Do not forget to add under the scissor line of the new patch a small changelog documenting what
has changed since v1.



On Vi, 2018-02-16 at 11:38 -0200, rodrigosiqueira wrote:
> This patch fixes the checkpatch.pl warning and error:
>
> iio/pressure/ms5611.h:66: ERROR: code indent should use tabs where possible
> iio/pressure/ms5611.h:66: WARNING: please, no spaces at the start of a line
> iio/pressure/ms5611.h:66: ERROR: "foo* bar" should be "foo *bar"
>
> Signed-off-by: Rodrigo Siqueira <rodrigosiqueiramelo@gmail.com>
> ---
>

^ this is the scissor line.

thanks,
Daniel.
\
 
 \ /
  Last update: 2018-02-16 14:57    [W:0.034 / U:0.588 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site