[U-Boot] [PATCH 5/6] sh: sh4: Add CONFIG_SH4 definition to config.mk of SH4
Masahiro Yamada
yamada.m at jp.panasonic.com
Wed Jan 8 07:51:42 CET 2014
Hi Nobuhiro.
> diff --git a/arch/sh/cpu/sh4/config.mk b/arch/sh/cpu/sh4/config.mk
> index c357557..753580b 100644
> --- a/arch/sh/cpu/sh4/config.mk
> +++ b/arch/sh/cpu/sh4/config.mk
> @@ -8,5 +8,5 @@
> # SPDX-License-Identifier: GPL-2.0+
> #
> #
> -PLATFORM_CPPFLAGS += -m4-nofpu
> +PLATFORM_CPPFLAGS += -DCONFIG_SH4 -m4-nofpu
> PLATFORM_RELFLAGS += -ffixed-r13
Basically it looks good to me.
But I have some comments.
Before this patch series,
include/configs/r7780mp.h
include/configs/sh7752evb.h
include/configs/sh7753evb.h
include/configs/sh7757lcr.h
include/configs/sh7785lcr.h
defined not CONFIG_SH4, but CONFIG_SH4A.
Now you are adding "CONFIG_SH4" to them too.
If we do not have to distinguish SH4A from SH4,
can we delete "CONFIG_SH4A"?
It looks like we can re-write
#if defined(CONFIG_SH4) || defined(CONFIG_SH4A)
as
#if defined(CONFIG_SH4)
in /arch/sh/include/asm/cache.h and /include/sh_tmu.h
Best Regards
Masahiro Yamada
More information about the U-Boot
mailing list