From mboxrd@z Thu Jan 1 00:00:00 1970 From: Wolfgang Denk Date: Sat, 22 Oct 2011 01:13:46 +0200 Subject: [U-Boot] [PATCH v2 REPOST 1/3] [COSMETIC] checkpatch whitespace cleanups In-Reply-To: <1318972311-673-1-git-send-email-swarren@nvidia.com> References: <1318972311-673-1-git-send-email-swarren@nvidia.com> Message-ID: <20111021231346.51E1318AE81C@gemini.denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Dear Stephen Warren, In message <1318972311-673-1-git-send-email-swarren@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 > --- > 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