[U-Boot] [PATCH v2 REPOST 1/3] [COSMETIC] checkpatch whitespace cleanups

Simon Glass sjg at chromium.org
Wed Oct 19 03:01:38 CEST 2011


Hi Stephen,

[resend without the context, ooops]

On Tue, Oct 18, 2011 at 2:11 PM, Stephen Warren <swarren at nvidia.com> wrote:
> This avoids the following checkpatch warning in later patches:
>
> ERROR: "(foo*)" should be "(foo *)"
> ERROR: space required before the open brace '{'
> ERROR: space prohibited before that close parenthesis ')'
> ERROR: spaces required around that '||' (ctx:WxV)
> WARNING: space prohibited between function name and open parenthesis '('
> WARNING: line over 80 characters
>
> This fixes all the white-space warnings/errors in my subsequent patch,
> and within this current patch. A number of other checkpatch warnings
> and errors are still present in this patch itself, but are beyond simple
> whitespace fixes, so are not solved by this patch.
>
> v2: New patch
>
> Signed-off-by: Stephen Warren <swarren at nvidia.com>

Tested-by: Simon Glass <sjg at chromium.org>

> ---
>  common/cmd_bootm.c    |  547 ++++++++++++-----------
>  common/cmd_ximg.c     |   72 ++--
>  common/image.c        | 1187 ++++++++++++++++++++++++++-----------------------
>  common/lynxkdi.c      |   34 +-
>  include/image.h       |  230 +++++-----
>  tools/default_image.c |   56 ++--
>  6 files changed, 1101 insertions(+), 1025 deletions(-)
>


More information about the U-Boot mailing list