From: Alexander Graf <agraf@suse.de>
To: Gerd Hoffmann <kraxel@redhat.com>
Cc: qemu-ppc@nongnu.org, qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH 8/9] PPC: Add u-boot firmware for e500
Date: Fri, 16 May 2014 15:43:30 +0200 [thread overview]
Message-ID: <53761602.4060606@suse.de> (raw)
In-Reply-To: <1400245912.2573.14.camel@nilsson.home.kraxel.org>
On 16.05.14 15:11, Gerd Hoffmann wrote:
> Hi,
>
>> +u-boot.e500:
>> + # examples fail to honor cflags, just remove them
>> + rm -rf u-boot/examples
>> + sed -i 's/examples//g' u-boot/Makefile
>> + # Make linker script biarch aware
>> + sed -i 's/OUTPUT_ARCH(powerpc)/OUTPUT_ARCH(powerpc:common)/' \
>> + u-boot/arch/powerpc/cpu/mpc85xx/u-boot.lds
> Hmm. Mucking with the source tree like this will leave the u-boot git
> tree in a dirty state, which isn't exactly nice. Fixable by sprinkling
> in a "git reset --hard", but that obviously has other downsides, such as
> seriously disturbing people trying to build a u-boot tree with local
> changes.
>
> Any chance to fix that without patching the source tree, by passing
> SUBDIRS="<list-dirs-without-examples>" to $(MAKE) for example?
> Or send patches to u-boot upstream to tackle this?
>
>> + # Compile u-boot
>> + $(MAKE) -C u-boot O=build.e500 qemu-ppce500_config
>> + $(MAKE) -C u-boot CROSS_COMPILE=$(powerpc64_cross_prefix) \
>> + O=build.e500 KCFLAGS=-m32 KAFLAGS=-m32 \
>> + LDFLAGS=-melf32ppc KCPPFLAGS=-m32 V=1 u-boot.bin
>> + strip u-boot/build.e500/u-boot -o ../pc-bios/u-boot.e500
> $(powerpc64_cross_prefix)strip I guess ...
This is getting too messy. I guess we can live without supporting biarch
compilers and just ask people to have a 32bit ppc compiler handy. Then
things become a lot more sane.
Alex
diff --git a/roms/Makefile b/roms/Makefile
index 2721b02..610b534 100644
--- a/roms/Makefile
+++ b/roms/Makefile
@@ -31,6 +31,7 @@ find-cross-gcc = $(firstword $(wildcard $(patsubst
%ld,%gcc,$(call find-cross-ld
find-cross-prefix = $(subst gcc,,$(notdir $(call find-cross-gcc,$(1))))
powerpc64_cross_prefix := $(call find-cross-prefix,powerpc64)
+powerpc_cross_prefix := $(call find-cross-prefix,powerpc)
x86_64_cross_prefix := $(call find-cross-prefix,x86_64)
#
@@ -55,6 +56,7 @@ default:
@echo " efirom -- update nic roms (bios+efi, this needs"
@echo " the EfiRom utility from edk2 /
tianocore)"
@echo " slof -- update slof.bin"
+ @echo " u-boot.e500 -- update u-boot.e500"
bios: build-seabios-config-seabios-128k build-seabios-config-seabios-256k
cp seabios/builds/seabios-128k/bios.bin ../pc-bios/bios.bin
@@ -132,6 +134,12 @@ slof:
$(MAKE) -C SLOF CROSS=$(powerpc64_cross_prefix) qemu
cp SLOF/boot_rom.bin ../pc-bios/slof.bin
+u-boot.e500:
+ $(MAKE) -C u-boot O=build.e500 qemu-ppce500_config
+ $(MAKE) -C u-boot CROSS_COMPILE=$(powerpc_cross_prefix) \
+ O=build.e500
+ $(powerpc_cross_prefix)strip u-boot/build.e500/u-boot -o \
+ ../pc-bios/u-boot.e500
clean:
rm -rf seabios/.config seabios/out seabios/builds
@@ -141,3 +149,4 @@ clean:
rm -f sgabios/.depend
$(MAKE) -C ipxe/src veryclean
$(MAKE) -C SLOF clean
+ rm -rf u-boot/build.e500
next prev parent reply other threads:[~2014-05-16 13:43 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-05-15 16:32 [Qemu-devel] [PATCH 0/9] PPC: e500: Use u-boot as firmware Alexander Graf
2014-05-15 16:32 ` [Qemu-devel] [PATCH 1/9] PPC: Make all e500 CPUs SVR aware Alexander Graf
2014-05-15 16:32 ` [Qemu-devel] [PATCH 2/9] PPC: Add definitions for GIVORs Alexander Graf
2014-05-15 16:32 ` [Qemu-devel] [PATCH 3/9] PPC: Fix SPR access control of L1CFG0 Alexander Graf
2014-05-15 16:32 ` [Qemu-devel] [PATCH 4/9] PPC: Add L1CFG1 SPR emulation Alexander Graf
2014-05-15 16:32 ` [Qemu-devel] [PATCH 5/9] PPC: Properly emulate L1CSR0 and L1CSR1 Alexander Graf
2014-05-15 16:32 ` [Qemu-devel] [PATCH 6/9] PPC: Add dcbtls emulation Alexander Graf
2014-05-15 16:32 ` [Qemu-devel] [PATCH 7/9] PPC: e500: Expose kernel load address in dt Alexander Graf
2014-05-15 16:32 ` [Qemu-devel] [PATCH 8/9] PPC: Add u-boot firmware for e500 Alexander Graf
2014-05-16 6:06 ` Gerd Hoffmann
2014-05-16 11:09 ` Alexander Graf
2014-05-16 13:11 ` Gerd Hoffmann
2014-05-16 13:43 ` Alexander Graf [this message]
2014-05-16 14:08 ` Gerd Hoffmann
2014-05-16 14:21 ` Alexander Graf
2014-05-15 16:32 ` [Qemu-devel] [PATCH 9/9] PPC: e500: Move to u-boot as firmware Alexander Graf
2014-05-15 16:38 ` [Qemu-devel] [Qemu-ppc] [PATCH 0/9] PPC: e500: Use " Alexander Graf
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=53761602.4060606@suse.de \
--to=agraf@suse.de \
--cc=kraxel@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=qemu-ppc@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).