From mboxrd@z Thu Jan  1 00:00:00 1970
Return-path: <ptxdist-bounces@pengutronix.de>
Received: from mediacenter.hi.pengutronix.de ([2001:6f8:1178:2::65])
	by metis.ext.pengutronix.de with esmtp (Exim 4.72)
	(envelope-from <mol@pengutronix.de>) id 1SC5cz-0004Ji-U3
	for ptxdist@pengutronix.de; Mon, 26 Mar 2012 10:50:13 +0200
Received: from mol by mediacenter.hi.pengutronix.de with local (Exim 4.72)
	(envelope-from <mol@pengutronix.de>) id 1SC5cz-0006xE-Sa
	for ptxdist@pengutronix.de; Mon, 26 Mar 2012 10:50:13 +0200
Date: Mon, 26 Mar 2012 10:50:13 +0200
From: Michael Olbrich <m.olbrich@pengutronix.de>
Message-ID: <20120326085013.GG27119@pengutronix.de>
References: <1332191692-20833-1-git-send-email-linux@bohmer.net>
	<20120323075331.GB20731@pengutronix.de>
	<CANqCtQKJxFtSN0NGKSxLLLP16wVREkccKJniqOiE1yr_vcTKpw@mail.gmail.com>
MIME-Version: 1.0
Content-Disposition: inline
In-Reply-To: <CANqCtQKJxFtSN0NGKSxLLLP16wVREkccKJniqOiE1yr_vcTKpw@mail.gmail.com>
Subject: Re: [ptxdist] [PATCH 1/9 v2] [x86_64] Add x86_64 architecture
 strings.
Reply-To: ptxdist@pengutronix.de
List-Id: PTXdist Development Mailing List <ptxdist.pengutronix.de>
List-Unsubscribe: <http://metis.pengutronix.de/cgi-bin/mailman/options/ptxdist>, 
	<mailto:ptxdist-request@pengutronix.de?subject=unsubscribe>
List-Archive: <http://metis.pengutronix.de/pipermail/ptxdist>
List-Post: <mailto:ptxdist@pengutronix.de>
List-Help: <mailto:ptxdist-request@pengutronix.de?subject=help>
List-Subscribe: <http://metis.pengutronix.de/cgi-bin/mailman/listinfo/ptxdist>, 
	<mailto:ptxdist-request@pengutronix.de?subject=subscribe>
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
Sender: ptxdist-bounces@pengutronix.de
Errors-To: ptxdist-bounces@pengutronix.de
To: ptxdist@pengutronix.de

On Sat, Mar 24, 2012 at 07:29:01PM +0100, Remy Bohmer wrote:
> 2012/3/23 Michael Olbrich <m.olbrich@pengutronix.de>:
> > I've seen the whole series, but I'm rather busy, and this requires a bit
> > more thinking. So it may take a while before we get everything in.
> >
> > On Mon, Mar 19, 2012 at 10:14:44PM +0100, Remy Bohmer wrote:
> >> The ARCH_X86_64 architecture is added as subtype of ARCH_X86 since it
> >> basically is the same architecture, and a mix of both architectures
> >> will be required to allow multilib support in a later commit.
> >>
> >> Signed-off-by: Remy Bohmer <linux@bohmer.net>
> >> ---
> >> =A0v2: no changes since v1
> >>
> >> =A0platforms/architecture.in | =A0 15 +++++++++++++--
> >> =A01 files changed, 13 insertions(+), 2 deletions(-)
> >>
> >> diff --git a/platforms/architecture.in b/platforms/architecture.in
> >> index ed36e8a..f00108e 100644
> >> --- a/platforms/architecture.in
> >> +++ b/platforms/architecture.in
> >> @@ -181,6 +181,16 @@ choice
> >>
> >> =A0endchoice
> >>
> >> +if ARCH_X86
> >> +#
> >> +# 32/64 bits platform
> >> +#
> >> +config ARCH_X86_64
> >> + =A0 =A0 bool "64 bit architecture"
> >> + =A0 =A0 help
> >> + =A0 =A0 =A0 Select this option if you build for a 64 bit architectur=
e.
> >> +
> >> +endif
> >
> > Have you considered adding ARCH_X86_32 as well (in a switch)? I think s=
ome
> > if our current "ifdef ARCH_X86" are really for 32bit only, so an option
> > would probably be useful.
> =

> I considered that but thought to look it like this: Assume the
> platform is 32 bit, unless specified otherwise.
> But maybe I have to look it at it more carefully. I drop it from the
> series, and will repost this patch later. First, finalize the rest of
> the series first.

You also need to do something with the "x86 optimization" choice. None of
the current choices support 64bit but there is hardware that could run with
either.

Michael

-- =

Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

-- =

ptxdist mailing list
ptxdist@pengutronix.de