linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: Tom Rini <trini@kernel.crashing.org>
To: Olaf Hering <olh@suse.de>
Cc: Andrew Morton <akpm@osdl.org>, linuxppc-dev@lists.linuxppc.org
Subject: Re: [PATCH] fix missing option in binutils version check
Date: Mon, 12 Jul 2004 08:11:19 -0700	[thread overview]
Message-ID: <20040712151119.GL28002@smtp.west.cox.net> (raw)
In-Reply-To: <20040712090618.GB32413@suse.de>


On Mon, Jul 12, 2004 at 11:06:18AM +0200, Olaf Hering wrote:
>  On Thu, Jul 08, Tom Rini wrote:
>
> > OK.  After giving this a whirl (after a NAK from Matt, privately), it looks
> > like some versions of binutils don't make proper use of -many (in 2.14 it
> > will override ppc_cpu, and PPC_OPCODE_ANY doesn't look to be used).  So
> > that brings us to the following patch.  Olaf, can you please verify this
> > still works for you?  Paul, is this OK with you?
>
> This is not enough.
> as needs -many (or at least -maltivec) unconditionally for binutils 2.15
> gcc needs also -maltivec because -mppc seems to disable -maltivec again.

Hmm.  So with binutils 2.14, we need to do -many,-mppc as -many will
zero out all previous flags.  On binutils 2.15, -mppc always resets
the cpu_flags, thus clearing out -many.  -mppc64bridge, -me500 and
-m405/etc all do the same.  So far we end up being lucky, imho, that
don't trip over this on !CONFIG_6xx.  So while I'm not exactly happy
we have to do this, the following is the least ugly fix for this:

Signed-off-by: Tom Rini <trini@kernel.crashing.org>

--- 1.55/arch/ppc/Makefile	2004-07-05 03:27:10 -07:00
+++ edited//home/trini/work/kernel/pristine/linux-2.6/arch/ppc/Makefile	2004-07-12 08:10:10 -07:00
@@ -32,6 +32,7 @@
 endif

 cflags-$(CONFIG_4xx)		+= -Wa,-m405
+cflags-$(CONFIG_6xx)		+= -Wa,-maltivec
 cflags-$(CONFIG_E500)		+= -Wa,-me500
 cflags-$(CONFIG_PPC64BRIDGE)	+= -Wa,-mppc64bridge


--
Tom Rini
http://gate.crashing.org/~trini/

** Sent via the linuxppc-dev mail list. See http://lists.linuxppc.org/
** This list is shutting down 7/24/2004.

  reply	other threads:[~2004-07-12 15:11 UTC|newest]

Thread overview: 49+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-06-05  7:43 [PATCH] fix typo in binutils version check Olaf Hering
2004-06-05 10:40 ` Christian Kujau
2004-06-05 11:11   ` Olaf Hering
2004-06-05 13:11     ` Christian Kujau
2004-06-07 15:57     ` Tom Rini
2004-06-08 11:25 ` [PATCH] fix missing option " Olaf Hering
2004-06-10  0:16   ` Tom Rini
2004-06-14  9:15     ` Olaf Hering
2004-06-14 16:23       ` Tom Rini
2004-06-14 17:38         ` Olaf Hering
2004-06-14 18:07           ` Olaf Hering
2004-06-14 20:55             ` Tom Rini
2004-06-14 21:19               ` Olaf Hering
2004-06-15 16:12                 ` Tom Rini
2004-06-15 17:25                   ` Olaf Hering
2004-06-15 17:46                     ` Tom Rini
2004-07-03 22:29                       ` Olaf Hering
2004-07-04  1:30                         ` Stef Simoens
2004-07-04  2:41                           ` Tom Rini
2004-07-04  8:32                             ` Geert Uytterhoeven
2004-07-04 16:33                               ` Hollis Blanchard
2004-07-25  6:07                                 ` Alan Modra
2004-07-26 19:36                                   ` Tom Rini
2004-07-26 23:03                                     ` Alan Modra
2004-07-26 23:07                                       ` Tom Rini
2004-07-04 18:32                               ` Tom Rini
2004-07-04  7:23                         ` Olaf Hering
2004-07-05 18:32                           ` Tom Rini
2004-07-09  1:34                             ` Tom Rini
2004-07-09  1:43                               ` Tom Rini
2004-07-12  9:06                               ` Olaf Hering
2004-07-12 15:11                                 ` Tom Rini [this message]
2004-07-12 18:03                                   ` Tom Rini
2004-07-13 13:49                                     ` Olaf Hering
2004-07-13 14:01                                       ` Tom Rini
2004-07-13 14:02                                         ` Olaf Hering
2004-07-13 14:20                                           ` Tom Rini
2004-07-13 15:18                                             ` Olaf Hering
2004-07-13 15:26                                               ` Tom Rini
2004-07-13 15:29                                                 ` Olaf Hering
2004-07-13 19:45                                                   ` Tom Rini
2004-07-14 23:23                                                   ` Tom Rini
2004-07-15  7:54                                                     ` Geert Uytterhoeven
2004-07-15 12:50                                                     ` Olaf Hering
2004-07-15 14:15                                                       ` Tom Rini
2004-07-15 14:24                                                         ` Olaf Hering
2004-07-15 14:23                                                       ` Tom Rini
2004-07-05 18:18                         ` Tom Rini
  -- strict thread matches above, loose matches on Subject: below --
2004-06-10 16:19 Christian

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=20040712151119.GL28002@smtp.west.cox.net \
    --to=trini@kernel.crashing.org \
    --cc=akpm@osdl.org \
    --cc=linuxppc-dev@lists.linuxppc.org \
    --cc=olh@suse.de \
    /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).