qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
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

  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).