[U-Boot] [PATCH 6/7] ARM: bcm283x: merge BCM2835/BCM2836 directories into mach-bcm283x
Masahiro Yamada
yamada.masahiro at socionext.com
Tue Mar 17 04:51:15 CET 2015
Hi Stephen,
2015-03-17 12:39 GMT+09:00 Stephen Warren <swarren at nvidia.com>:
> On 03/16/2015 04:35 AM, Masahiro Yamada wrote:
>> BCM2835 (used on Raspberry Pi) and BCM2836 (used on Raspberry Pi 2)
>> are similar enough. One of the biggest differences is the ARM
>> processor. It is reasonable to collect the source files into a
>> single place, arch/arm/mach-bcm283x/.
>
>> diff --git a/arch/arm/cpu/arm1176/bcm2835/Makefile b/arch/arm/mach-bcm283x/Makefile
>
>> -obj-y := lowlevel_init.o
>> +obj-$(CONFIG_TARGET_RPI) += lowlevel_init.o
>> obj-y += init.o reset.o timer.o mbox.o
>
> Why is only one file conditional; shouldn't they all be either -y or
> -$(CONFIG_TARGET_RPI)?
> _______________________________________________
This patch just moves source files, not changing the behavior.
If you see arch/arm/cpu/armv7/bcm2835/Makefile (it is added by you),
--------------->8------------------
src_dir := ../../arm1176/bcm2835/
obj-y :=
obj-y += $(src_dir)/init.o
obj-y += $(src_dir)/reset.o
obj-y += $(src_dir)/timer.o
obj-y += $(src_dir)/mbox.o
--------------8<------------------
BCM2836 borrows source files from BCM2835, except lowlevel_init.S.
I intended to reflect the same behavior.
Anyway, lowlevel_init is not linked for Raspberry Pi2
because you added CONFIG_SKIP_LOWLEVEL_INIT to include/configs/rpi_2.h
--
Best Regards
Masahiro Yamada
More information about the U-Boot
mailing list