All of lore.kernel.org
 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.