[LinuxBIOS] [BULDROM] Further categorize boards by vendor

[Uwe Hermann]

On Wed, Oct 24, 2007 at 05:59:12PM -0600, Jordan Crouse wrote:
> Buildrom is getting to successful for its own good.. :)  Categorize the 
> boards by vendor to make them easier to find.

Acked-by: Uwe Hermann <uwe at hermann-uwe.de>

Just some small cosmetics before you commit:


> Signed-off-by: Jordan Crouse <jordan.crouse at amd.com>
> Index: buildrom-devel/config/platforms/Config.in
> ===================================================================
> --- buildrom-devel.orig/config/platforms/Config.in	2007-10-24 17:23:52.000000000 -0600
> +++ buildrom-devel/config/platforms/Config.in	2007-10-24 17:38:37.000000000 -0600
> @@ -1,6 +1,31 @@
>  menu "Platform Configuration"
>  
>  choice
> +	prompt "Vendor for the platform target"
> +	default VENDOR_AMD
> +	help
> +	  Select a vendor category for the target you want to build for

Missing full stop.


> +
> +config VENDOR_AMD
> +	bool "AMD"
> +
> +config VENDOR_ARTEC
> +	bool "Artec Group"
> +
> +config VENDOR_DIGITALLOGIC
> +	bool "Digital Logic"
> +
> +config VENDOR_GIGABYTE
> +	bool "Gigabyte"

GIGABYTE, as per website.


> +
> +config VENDOR_PCENGINES
> +	bool "PC Engines"
> +
> +config VENDOR_TYAN
> +	bool "Tyan"
> +endchoice
> +
> +choice
>  	prompt "Platform target for the ROM"
>  	default PLATFORM_NORWICH
>  	help
> @@ -8,38 +33,47 @@
>  
>  config PLATFORM_NORWICH
>         bool "AMD Geode LX 'Norwich'"
> +       depends VENDOR_AMD
>    gg     select PLATFORM
>  
>  config PLATFORM_DBE61
>         bool "Artec Group dbe61"

DBE61, I think.


> +       depends VENDOR_ARTEC
>         select PLATFORM
>  
>  config PLATFORM_ALIX1C
>         bool "PC Engines ALIX1.C"
> +       depends VENDOR_PCENGINES
>         select PLATFORM
>  
>  config PLATFORM_MSM800SEV
>         bool "Digital Logic msm800sev"

"Advanced Digital Logic MSM800SEV", as per website.


> +       depends VENDOR_DIGITALLOGIC
>         select PLATFORM
>  
>  config PLATFORM_DB800
>  	bool "AMD DB800"
> +	depends VENDOR_AMD
>  	select PLATFORM
>  
>  config PLATFORM_M57SLI
>         bool "Gigabyte M57SLI"

GIGABYTE GA-M57SLI-S4


> +       depends VENDOR_GIGABYTE
>         select PLATFORM
>  
>  config PLATFORM_TYAN_S2882
>         bool "Tyan S2882"
> +       depends VENDOR_TYAN
>         select PLATFORM
>  
>  config PLATFORM_TYAN_S2891
>         bool "Tyan S2891"
> +       depends VENDOR_TYAN
>         select PLATFORM
>  
>  config PLATFORM_SERENGETI_CHEETAH
>         bool "AMD Serengeti_Cheetah"

Debatable, but I'd prefer "Serengeti Cheetah" or
"Serengeti-Cheetah", whichever is closer to the "official" product name.


> +       depends VENDOR_AMD
>         select PLATFORM
>  endchoice
>  endmenu


Uwe.
-- 
http://www.hermann-uwe.de  | http://www.holsham-traders.de
http://www.crazy-hacks.org | http://www.unmaintained-free-software.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20071025/ca201777/attachment.sig>