On Jun 6, 2008, at 9:33 AM, Wolfgang Denk wrote:

In message <[EMAIL PROTECTED]> you wrote:

As I see it, I have to define DTS_FLAGS manually

$ export DTS_FLAGS="-R4 -S0x3000"
$ make ARCH=powerpc tqm8548.dtb

to get a working blob. Is that the intended behavior?

this is normal.  Andy Fleming has posted a patch to u-boot to help
improve the situation.  I'm not sure if WD has picked it up for 1.3.4
yet.

I will. But wouldn't it make sense to allow at least for  some  level
of  compatibility with older versions of U-Boot? Adding these options
as default would probably not hurt?

Agreed. I didn't realize we could actually do this from the kernel build system.

I'm sure others will have their two cents to add.

But something like:

diff --git a/arch/powerpc/boot/Makefile b/arch/powerpc/boot/Makefile
index c40fb82..52db85a 100644
--- a/arch/powerpc/boot/Makefile
+++ b/arch/powerpc/boot/Makefile
@@ -29,6 +29,8 @@ ifdef CONFIG_DEBUG_INFO
 BOOTCFLAGS     += -g
 endif

+DTS_FLAGS      ?= -R4 -S 0x3000
+
 ifeq ($(call cc-option-yn, -fstack-protector),y)
 BOOTCFLAGS     += -fno-stack-protector
 endif


- k
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev

Reply via email to