From: Tim Bird <tim.bird@am.sony.com>
To: Ralf Baechle <ralf@linux-mips.org>
Cc: Tom Rini <trini@kernel.crashing.org>,
Thiemo Seufer <ths@networkno.de>,
"Bird, Tim" <Tim.Bird@am.sony.com>,
linux-mips@linux-mips.org
Subject: Re: [PATCH] fix mips/Makefile to support CROSS_COMPILE from envir onment var
Date: Mon, 08 May 2006 11:35:31 -0700 [thread overview]
Message-ID: <445F8F73.6030808@am.sony.com> (raw)
In-Reply-To: <20060504230647.GA3465@linux-mips.org>
Ralf Baechle wrote:
> On Thu, May 04, 2006 at 02:04:49PM -0700, Tom Rini wrote:
>
>> Let me ask a stupid question. With all of the ways to otherwise do a
>> cross compile, why a config option on MIPS? ARM*/SH*, which are at
>> least as likely to not be native-compiled, don't do that. Just
>> something I've always wondered, really.
>
> Having such information in an environment variable is imho terribly
> inelegant, having to pass it on the command line for each make
> invocation
> is terrible abuse for the fingertips so I went for this option which
> makes
> the makefile pick the right prefix.
And a great solution it would be, if it actually picked the
right prefix. :-) I understand that having the cross-compiler prefix
specified outside the build system has drawbacks. When I first got
into embedded Linux (many years ago), I was struck by the inelegance
of this also. However, hardcoding the prefixes in the Makefile is, IMHO,
not such a great idea either.
It really doesn't bother me if the "standard" prefixes are encoded in
a Makefile, but it would be nice to be able to select a non-standard
prefix the same way as for other Linux architectures.
I understand I can do that by turning off the "I'm cross-compiling"
option, but that seems counterintuitive given the current wording
of that option (not-withstanding the not-yet-mainlined help text).
My patch allows to keep the current behaviour, but also supports
use of the environment variable. I don't see any downside to it,
unless you are actively trying to discourage the use of the
environment variable. In my own experience, use of the environment
variable, while inelegant for manual use, has certain benefits
for automated use.
Regards,
-- Tim
=============================
Tim Bird
Architecture Group Chair, CE Linux Forum
Senior Staff Engineer, Sony Electronics
=============================
next prev parent reply other threads:[~2006-05-08 18:35 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-05-04 19:35 [PATCH] fix mips/Makefile to support CROSS_COMPILE from environment var Tim Bird
2006-05-04 20:55 ` Thiemo Seufer
2006-05-04 21:04 ` Tom Rini
2006-05-04 21:25 ` Kevin D. Kissell
2006-05-04 21:25 ` Kevin D. Kissell
2006-05-04 21:32 ` Kevin D. Kissell
2006-05-04 21:32 ` Kevin D. Kissell
2006-05-04 21:44 ` Tom Rini
2006-05-04 23:06 ` Ralf Baechle
2006-05-04 23:14 ` Tom Rini
2006-05-04 23:34 ` Tom Rini
2006-05-05 7:45 ` Geert Uytterhoeven
2006-05-05 18:45 ` Wolfgang Denk
2006-05-08 18:35 ` Tim Bird [this message]
2006-05-08 18:56 ` [PATCH] fix mips/Makefile to support CROSS_COMPILE from envir onment var Dan Malek
2006-05-04 21:17 ` [PATCH] fix mips/Makefile to support CROSS_COMPILE from environment var Tim Bird
2006-05-04 22:35 ` Thiemo Seufer
2006-05-08 18:22 ` Tim Bird
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=445F8F73.6030808@am.sony.com \
--to=tim.bird@am.sony.com \
--cc=linux-mips@linux-mips.org \
--cc=ralf@linux-mips.org \
--cc=ths@networkno.de \
--cc=trini@kernel.crashing.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.