[U-Boot] [Patch V4 1/4] MIPS: Add support for MIPS64 cpus

Wolfgang Denk wd at denx.de
Sat Sep 1 14:42:07 CEST 2012


Dear Zhizhou Zhang,

In message <1346071750-6634-7-git-send-email-etou.zh at gmail.com> you wrote:
> These files are derived from arch/mips/cpu/mips32/*. The main changes are:
> 1, changes ABI o32 to n64, 2, add mips64 build cflags.
> 
> Signed-off-by: Zhizhou Zhang <etou.zh at gmail.com>

Checkpatch reports a number of "unnecessary whitespace before a quoted
newline" warnings; please fix these.

> diff --git a/arch/mips/cpu/mips64/Makefile b/arch/mips/cpu/mips64/Makefile
> new file mode 100644
> index 0000000..f4c88f5
> --- /dev/null
> +++ b/arch/mips/cpu/mips64/Makefile
...
> +sinclude $(obj).depend
> +
> diff --git a/arch/mips/cpu/mips64/cache.S b/arch/mips/cpu/mips64/cache.S

Please do not add trailing empty lines.  Please fix globally.

...
> +++ b/arch/mips/cpu/mips64/config.mk
> @@ -0,0 +1,40 @@
...
> +CONFIG_STANDALONE_LOAD_ADDR ?= 0xFfffFfff80200000 -T mips64.lds

As requested before: please don't mix upper and lower case letters in
a number; use a consistent styile everywhere. PLease fix globally.


As requested before: please make sure your patches are bisectable.
This references file mips64.lds, but doesn;t add it, so it cannot
work.

...
> +#define cache_op(op, addr)						\
> +	__asm__ __volatile__(						\
> +	"	.set	push					\n"	\
> +	"	.set	noreorder				\n"	\
> +	"	.set	mips64\n\t				\n"	\
> +	"	cache	%0, %1					\n"	\
> +	"	.set	pop					\n"	\

Fix checkpatch warnings here (and globally).

> +	while (1) {
> +		cache_op(Hit_Writeback_Inv_D, addr);
> +		cache_op(Hit_Invalidate_I, addr);

We don't allow camel-case identifiers.  Please fix globally.



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
If it has syntax, it isn't user friendly.


More information about the U-Boot mailing list