Re: [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

Just some small cosmetics before you commit:
> Signed-off-by: Jordan Crouse 
> 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.