[LinuxBIOS] [patch 5/6] Add platform support for the db800 "salsa".

[Peter Stuge]

On Thu, Jun 14, 2007 at 11:16:38AM -0600, jordan.crouse at amd.com wrote:
> +++ buildrom/config/platforms/db800.conf	2007-06-14 11:09:03.000000000 -0600
> @@ -0,0 +1,39 @@
> +# Support for the AMD Geode 'norwich' platform

Is this for the norwich or the db800?


> +KERNEL_MK=$(PACKAGE_DIR)/kernel/norwich-kernel.mk
> +LINUXBIOS_MK=$(PACKAGE_DIR)/linuxbios/norwich-linuxbios.mk

..

> +LINUXBIOS_VENDOR=amd
> +LINUXBIOS_BOARD=db800

If there is reuse going on please write a few words about that and
why it is possible. (Along the lines of "The norwich code tries to be
really generic so it can be reused a lot" maybe?)


> +++ buildrom/config/platforms/platforms.conf	2007-06-14 11:08:54.000000000 -0600
> @@ -9,7 +9,7 @@
>  PLATFORM-$(CONFIG_PLATFORM_NORWICH) = norwich.conf
>  PLATFORM-$(CONFIG_PLATFORM_OLPC) = olpc.conf
>  PLATFORM-$(CONFIG_PLATFORM_MSM800SEV) = msm800sev.conf
> -
> +PLATFORM-$(CONFIG_PLATFORM_DB800) = db800.conf

Ah so there is already norwich support. Please mention why it can be
reused.


> +++ buildrom/packages/filo/conf/db800-Config	2007-06-14 11:08:54.000000000 -0600

Perhaps this could reuse some norwich stuff too?


//Peter