From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 25733 invoked by alias); 19 Sep 2012 20:26:45 -0000 Received: (qmail 25714 invoked by uid 22791); 19 Sep 2012 20:26:44 -0000 X-SWARE-Spam-Status: No, hits=-1.3 required=5.0 tests=AWL,BAYES_00,FREEMAIL_ENVFROM_END_DIGIT,FREEMAIL_FROM,KHOP_THREADED,RCVD_IN_DNSWL_NONE,URIBL_DBL_SPAM X-Spam-Check-By: sourceware.org Received: from smtp07.smtpout.orange.fr (HELO smtp.smtpout.orange.fr) (80.12.242.129) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Wed, 19 Sep 2012 20:26:30 +0000 Received: from treguer.localnet ([90.32.151.241]) by mwinf5d42 with ME id 18SU1k0075Cksaa038SULV; Wed, 19 Sep 2012 22:26:29 +0200 From: "Yann E. MORIN" To: crossgcc@sourceware.org Subject: Re: [PATCH] config/arch+scripts/build/arch: Add Microblaze =?iso-8859-1?q?architecture=09support?= Date: Wed, 19 Sep 2012 20:26:00 -0000 User-Agent: KMail/1.13.5 (Linux/3.5.4-treguer; KDE/4.4.5; x86_64; ; ) Cc: David Holsgrove References: <79a03dc829edb6188436.1347934614@localhost.localdomain> In-Reply-To: <79a03dc829edb6188436.1347934614@localhost.localdomain> MIME-Version: 1.0 Content-Type: Text/Plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Message-Id: <201209192226.27824.yann.morin.1998@free.fr> X-IsSubscribed: yes Mailing-List: contact crossgcc-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: crossgcc-owner@sourceware.org X-SW-Source: 2012-09/txt/msg00058.txt.bz2 David, All, On Tuesday 18 September 2012 04:16:54 David Holsgrove wrote: > # HG changeset patch > # User David Holsgrove > # Date 1347933988 -36000 > # Node ID 79a03dc829edb6188436a2c7cc33ead9e27827c4 > # Parent 2858a24a584642e263a920b4214c815c172ed547 > config/arch+scripts/build/arch: Add Microblaze architecture support Usually, we just specify the component familly, the component name, and the action being done. As microblaze is still experimental (at least in CT-NG!), saying so in the commmit log would be good. In this case, a subject like the following would be more obvious: arch/microblaze: add new architecture > Add Microblaze architecture support (depends on EXPERIMENTAL) > > Signed-off-by: "David Holsgrove" > > diff -r 2858a24a5846 -r 79a03dc829ed config/arch/microblaze.in > --- /dev/null Thu Jan 01 00:00:00 1970 +0000 > +++ b/config/arch/microblaze.in Tue Sep 18 12:06:28 2012 +1000 > @@ -0,0 +1,12 @@ > +# MicroBlaze specific config options > + > +## depends on EXPERIMENTAL > +## > +## select ARCH_SUPPORT_ARCH > +## select ARCH_FLOAT_SW > +## select ARCH_SUPPORTS_BOTH_ENDIAN > +## select ARCH_DEFAULT_BE > +## select ARCH_DEFAULT_HAS_MMU What archs usually do is: - if the MMU is always available (eg. x86): select ARCH_USE_MMU - if the MMU is either available or missing (eg. arm): select ARCH_SUPPORTS_BOTH_MMU select ARCH_DEFAULT_HAS_MMU (only if the MMU is 'default') - if the arch never has an MMU (eg. blackfin), select nothing AFAICS, microblaze is like ARM: use of the MMU is configurable. Also, please add to the help entry that upstream projects are not all up to full support for microblaze, and that support is being added. If there are out-of-tree public patches and/or alternate repositories, you may also want to point to them in the help entry, so that interested parties can use them until support is upstream. Othwerwise, looks good! Regards, Yann E. MORIN. -- .-----------------.--------------------.------------------.--------------------. | Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: | | +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ | | +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no | | http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. | '------------------------------^-------^------------------^--------------------' -- For unsubscribe information see http://sourceware.org/lists.html#faq