Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1640334

Re: [PATCH] staging: ccree: remove trailing whitespace. switch spaces to tabs

From Greg KH <gregkh@linuxfoundation.org>
Newsgroups linux.kernel
Subject Re: [PATCH] staging: ccree: remove trailing whitespace. switch spaces to tabs
Date 2017-05-12 11:10 +0200
Message-ID <tGeKe-2eI-23@gated-at.bofh.it> (permalink)
References <tFimR-5AC-3@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On Tue, May 09, 2017 at 11:39:56AM -0700, Connor Kelleher wrote:
> ssi_fips.c:
> 
> fixing checkpatch.pl errors:
> 
> ERROR: trailing whitespace
> + * $
> 
> ERROR: trailing whitespace
> + * $
> 
> ERROR: trailing whitespace
> + * $
> 
> ERROR: trailing whitespace
> +This function returns the REE FIPS state.  $
> 
> ERROR: trailing whitespace
> +It should be called by kernel module. $
> 
> ERROR: code indent should use tabs where possible
> +        int rc = 0;$
> 
> ERROR: trailing whitespace
> +This function returns the REE FIPS error.  $
> 
> ERROR: trailing whitespace
> +It should be called by kernel module. $
> 
> ERROR: code indent should use tabs where possible
> +        int rc = 0;$

Those are different "types" of errors, please only do one type of thing
per patch.

thanks,

greg k-h

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH] staging: ccree: remove trailing whitespace. switch spaces to tabs Connor Kelleher <connor.r.kelleher@gmail.com> - 2017-05-09 20:50 +0200
  Re: [PATCH] staging: ccree: remove trailing whitespace. switch  spaces to tabs Greg KH <gregkh@linuxfoundation.org> - 2017-05-12 11:10 +0200
    [PATCH v2 1/2] staging: ccree: switch spaces to tabs Connor Kelleher <connor.r.kelleher@gmail.com> - 2017-05-12 21:20 +0200
      [PATCH v2 2/2] staging: ccree: remove trailing whitespace Connor Kelleher <connor.r.kelleher@gmail.com> - 2017-05-12 21:20 +0200

csiph-web