[LinuxBIOS] [PATCH] buildrom: Consistency fixes

[Uwe Hermann]

See patch.

Uwe.

[Paul Menzel]

Hi Uwe,

I spotted a small typo.

Am Samstag, den 27.10.2007, 16:39 +0200 schrieb Uwe Hermann:

Index: config/platforms/Config.in
===================================================================
--- config/platforms/Config.in  (Revision 50)
+++ config/platforms/Config.in  (Arbeitskopie)
@@ -4,21 +4,21 @@
        prompt "Vendor for the platform target"
        default VENDOR_AMD
        help
-         Select the vendor for the plaform you are building for
+         Select the vendor for the plaform you are building for.

s/plaform/platform/

[...]

@@ -29,7 +29,7 @@
        prompt "Platform target for the ROM"
        default PLATFORM_NORWICH
        help
-         Select the platform you are building for
+         Select the platform you are building for.

What exactly changed here?

Viele Grüße

Paul

[Uwe Hermann]

On Sat, Oct 27, 2007 at 06:10:44PM +0200, Paul Menzel wrote:
> Index: config/platforms/Config.in
>
> --- config/platforms/Config.in  (Revision 50)
> +++ config/platforms/Config.in  (Arbeitskopie)
> @@ -4,21 +4,21 @@
>         prompt "Vendor for the platform target"
>         default VENDOR_AMD
>         help
>     Select the vendor for the plaform you are building for
>     Select the vendor for the plaform you are building for.
>
> s/plaform/platform/

Yep, true.
> [...]
>
> @@ -29,7 +29,7 @@
>         prompt "Platform target for the ROM"
>         default PLATFORM_NORWICH
>         help
>     Select the platform you are building for
>     Select the platform you are building for.
>
> What exactly changed here?

Same as above, missing full stop was added.

Uwe.

[Paul Menzel]

Am Samstag, den 27.10.2007, 18:29 +0200 schrieb Uwe Hermann:
> On Sat, Oct 27, 2007 at 06:10:44PM +0200, Paul Menzel wrote:
> > Index: config/platforms/Config.in
> >
> > --- config/platforms/Config.in  (Revision 50)
> > +++ config/platforms/Config.in  (Arbeitskopie)
> > @@ -4,21 +4,21 @@
> >         prompt "Vendor for the platform target"
> >         default VENDOR_AMD
> >         help
> >     Select the vendor for the plaform you are building for
> >     Select the vendor for the plaform you are building for.
> >
> > s/plaform/platform/
>
> Yep, true.
> > [...]
> >
> > @@ -29,7 +29,7 @@
> >         prompt "Platform target for the ROM"
> >         default PLATFORM_NORWICH
> >         help
> >     Select the platform you are building for
> >     Select the platform you are building for.
> >
> > What exactly changed here?
>
> Same as above, missing full stop was added.

That’s what it was. Thanks, for enlightening me.

Paul

[LinuxBIOS] buildrom: Consistency fixes

[Jordan Crouse]

On 27/10/07 16:39 +0200, Uwe Hermann wrote:
> See patch.
> Uwe.
>
> http://www.hermann-uwe.de  | http://www.holsham-traders.de
> http://www.crazy-hacks.org | http://www.unmaintained-free-software.org
> Small cosmetic and consistency fixes.
>
> Signed-off-by: Uwe Hermann

Acked-by: Jordan Crouse
> Index: config/platforms/m57sli.conf
>
> --- config/platforms/m57sli.conf	(Revision 50)
> +++ config/platforms/m57sli.conf	(Arbeitskopie)
> @@ -1,4 +1,4 @@
> -# Support for the Gigabyte M57SLI-S4 board
> +# Support for the GIGABYTE GA-M57SLI-S4 board
>
> #### Platform configuration
> Index: config/platforms/Config.in
>
> --- config/platforms/Config.in	(Revision 50)
> +++ config/platforms/Config.in	(Arbeitskopie)
> @@ -4,21 +4,21 @@
>    prompt "Vendor for the platform target"
>    default VENDOR_AMD
>    help
>  Select the vendor for the plaform you are building for
>  Select the vendor for the plaform you are building for.
>
> config VENDOR_AMD
>    bool "AMD"
>
> -config VENDOR_ARTEC
> +config VENDOR_ARTEC_GROUP
>    bool "Artec Group"
>
> -config VENDOR_DIGITALLOGIC
> bool "Digital Logic"
>
> +config VENDOR_ADVANCED_DIGITAL_LOGIC
> bool "Advanced Digital Logic"
>
> config VENDOR_GIGABYTE
>    bool "GIGABYTE"
>
> -config VENDOR_PCENGINES
> +config VENDOR_PC_ENGINES
>    bool "PC Engines"
>
> config VENDOR_TYAN
> @@ -29,7 +29,7 @@
>    prompt "Platform target for the ROM"
>    default PLATFORM_NORWICH
>    help
>  Select the platform you are building for
>  Select the platform you are building for.
>
> config PLATFORM_NORWICH
>         bool "AMD Geode LX 'Norwich'"
> @@ -38,17 +38,17 @@
>
> config PLATFORM_DBE61
>         bool "Artec Group DBE61"
>   depends VENDOR_ARTEC
>   depends VENDOR_ARTEC_GROUP
>   select PLATFORM
>
> config PLATFORM_ALIX1C
>         bool "PC Engines ALIX1.C"
>   depends VENDOR_PCENGINES
>   depends VENDOR_PC_ENGINES
>   select PLATFORM
>
> config PLATFORM_MSM800SEV
>         bool "Advanced Digital Logic MSM800SEV"
>   depends VENDOR_DIGITALLOGIC
>   depends VENDOR_ADVANCED_DIGITAL_LOGIC
>   select PLATFORM
>
> config PLATFORM_DB800
> @@ -56,7 +56,7 @@
>    depends VENDOR_AMD
>    select PLATFORM
>
> -config PLATFORM_M57SLI
> +config PLATFORM_GA_M57SLI_S4
>         bool "GIGABYTE GA-M57SLI-S4"
>         depends VENDOR_GIGABYTE
>         select PLATFORM
> Index: packages/kernel/m57sli-kernel.mk
> ===================================================================
> --- packages/kernel/m57sli-kernel.mk	(Revision 50)
> +++ packages/kernel/m57sli-kernel.mk	(Arbeitskopie)
> @@ -1,4 +1,4 @@
> -# Build file for the Gigabyte M57SLI-S4 LAB kernel
> +# Build file for the GIGABYTE GA-M57SLI-S4 LAB kernel
>
> KERNEL_URL=http://kernel.org/pub/linux/kernel/v2.6/
>  KERNEL_SOURCE=linux-$(KERNEL_VERSION).tar.bz2
> Index: packages/kernel/alix1c-kernel.mk
> ===================================================================
> --- packages/kernel/alix1c-kernel.mk	(Revision 50)
> +++ packages/kernel/alix1c-kernel.mk	(Arbeitskopie)
> @@ -1,4 +1,4 @@
> -# Build file for the PCENGINES ALIX1.C LAB kernel
> +# Build file for the PC Engines ALIX1.C LAB kernel
>
> KERNEL_URL=http://kernel.org/pub/linux/kernel/v2.6/
>  KERNEL_SOURCE=linux-$(KERNEL_VERSION).tar.bz2
>

[LinuxBIOS] buildrom: Consistency fixes

[Uwe Hermann]

On Mon, Oct 29, 2007 at 10:04:28AM -0600, Jordan Crouse wrote:
> > Small cosmetic and consistency fixes.
> >
> > Signed-off-by: Uwe Hermann
>
> Acked-by: Jordan Crouse

Thanks, r51 (together with the typo noticed by Paul Menzel).

Uwe.