Re: [CT-NG:patch 10/19] Add ARCH_CPU for m68k architecture

["Yann E. MORIN" <yann dot morin dot 1998 at anciens dot enib dot fr>]

Remy, Bart, All,

On Friday 09 April 2010 21:04:29 Remy Bohmer wrote:
> Add config option to configure the ARCH_CPU define for
> the m68k CPU's that we known
[--SNIP--]
>  config ARCH_m68k
> -    select ARCH_SUPPORTS_32
> -    select ARCH_DEFAULT_32
> -    select ARCH_DEFAULT_BE
> -    select ARCH_SUPPORT_CPU
> -     help
> -      The m68k architecture
> +	select ARCH_SUPPORT_CPU
> +	select ARCH_SUPPORTS_32
> +	select ARCH_DEFAULT_32
> +	select ARCH_DEFAULT_BE

Gratuitous re-ordering.
Space-damage: use spaces, not tabs.

> +	help
> +	  The Motorola 68k architecture (http://) or
> +	  The Freescale Coldfire architecture (http://)
> +
> +choice
> +	bool
> +	prompt "CPU"
[--SNIP--]

I don't want to maintain a list of possible values. It is a nightmare to
maintain, it breaks when a new version of gcc recognises a new value.

The user is responsible for properly setting this in the menuconfig.
Dropped.

(Yes, I know that alpha is doing it, but that's an error. And sh just sets
a variant to be used to to build the tuple.)

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