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

Wolfgang Denk wd at denx.de
Sat Oct 22 01:13:46 CEST 2011


Dear Stephen Warren,

In message <1318972311-673-1-git-send-email-swarren at nvidia.com> you 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>
> ---
>  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(-)

Applied, thanks.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
When it is incorrect, it is, at least *authoritatively* incorrect.
                                    - Hitchiker's Guide To The Galaxy


More information about the U-Boot mailing list