[U-Boot] [PATCH 5/8] ARM: moved general function to arm/lib

stefano babic sbabic at denx.de
Fri Oct 7 00:11:39 CEST 2011


Am 06/10/2011 23:30, schrieb Wolfgang Denk:
> Dear Stefano Babic,
> 
> In message <1316166383-11314-5-git-send-email-sbabic at denx.de> you wrote:
>> Functions inside armv7/syslib can be used by other ARM
>> architectures, too. The file is added as part of
>> ARM library.
>>
>> Signed-off-by: Stefano Babic <sbabic at denx.de>
>> CC: Albert ARIBAUD <albert.u.boot at aribaud.net>
>> ---
>>  arch/arm/cpu/armv7/Makefile |    2 -
>>  arch/arm/cpu/armv7/syslib.c |   69 -------------------------------------------
>>  arch/arm/lib/Makefile       |    2 +
>>  arch/arm/lib/syslib.c       |   69 +++++++++++++++++++++++++++++++++++++++++++
>>  4 files changed, 71 insertions(+), 71 deletions(-)
>>  delete mode 100644 arch/arm/cpu/armv7/syslib.c
>>  create mode 100644 arch/arm/lib/syslib.c
> 
> Checkpatch says:
> 
> total: 2 errors, 0 warnings, 85 lines checked
> 
> Please clean up and resubmit.  Thanks.

mmhhh..I do not know if this is correct. The patch is really only a "git
mv", an the original files are only moved. If I run checkpatch, I see:

ERROR: spaces required around that ':' (ctx:VxV)
#164: FILE: arch/arm/lib/syslib.c:37:
+			  "bne 1b":"=r" (loops):"0"(loops));
 			          ^

ERROR: spaces required around that ':' (ctx:VxV)
#164: FILE: arch/arm/lib/syslib.c:37:
+			  "bne 1b":"=r" (loops):"0"(loops));
 			                       ^

total: 2 errors, 0 warnings, 85 lines checked

This is an assembly inline - are we sure we apply the same rules for C
and assembly ?

Best regards,
Stefano Babic

-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
=====================================================================


More information about the U-Boot mailing list