[coreboot] Trying to add Kconfig support for M57SLI - build error

[ron minnich]

This is *almost* perfect. Questions and changes.

Index: src/cpu/amd/socket_AM2/Kconfig
===================================================================
--- src/cpu/amd/socket_AM2/Kconfig	(revision 0)
+++ src/cpu/amd/socket_AM2/Kconfig	(revision 0)
@@ -0,0 +1,30 @@
+config CPU_AMD_SOCKET_AM2
+	bool
+	default false
+
+config CPU_SOCKET_TYPE
+	hex
+	default 0x10
+	depends on CPU_AMD_SOCKET_AM2

It's a type 0x10?

+
+config K8_REV_F_SUPPORT
+	int	
+	default 1
+	depends on CPU_AMD_SOCKET_AM2

Socket AM2 supports REV F? I don't know, just asking.


+config CPU_SOCKET_TYPE
+	hex
+	default 0x11
+	depends on CPU_AMD_SOCKET_AM2

it's in here twice.


Index: src/cpu/amd/socket_AM2/Makefile.inc
===================================================================
--- src/cpu/amd/socket_AM2/Makefile.inc	(revision 0)
+++ src/cpu/amd/socket_AM2/Makefile.inc	(revision 0)
@@ -0,0 +1,14 @@
+obj-$(CONFIG_CPU_AMD_SOCKET_AM2) += socket_AM.o
+subdirs-$(CONFIG_CPU_AMD_SOCKET_AM2) += ../model_fxx
+subdirs-$(CONFIG_CPU_AMD_SOCKET_AM2) += ../dualcore
+subdirs-$(CONFIG_CPU_AMD_SOCKET_AM2) += ../mtrr
+subdirs-$(CONFIG_CPU_AMD_SOCKET_AM2) += ../../x86/tsc
+subdirs-$(CONFIG_CPU_AMD_SOCKET_AM2) += ../../x86/mtrr
+subdirs-$(CONFIG_CPU_AMD_SOCKET_AM2) += ../../x86/fpu
+subdirs-$(CONFIG_CPU_AMD_SOCKET_AM2) += ../../x86/mmx
+subdirs-$(CONFIG_CPU_AMD_SOCKET_AM2) += ../../x86/sse
+subdirs-$(CONFIG_CPU_AMD_SOCKET_AM2) += ../../x86/lapic
+subdirs-$(CONFIG_CPU_AMD_SOCKET_AM2) += ../../x86/cache
+subdirs-$(CONFIG_CPU_AMD_SOCKET_AM2) += ../../x86/mtrr
+subdirs-$(CONFIG_CPU_AMD_SOCKET_AM2) += ../../x86/pae
+subdirs-$(CONFIG_CPU_AMD_SOCKET_AM2) += ../../x86/smm

Just make these -y. This Makefile.inc is *conditionally* included from
the src/cpu/amd/Makefile.inc. There's no reason to make
them conditional in turn.

Let's go around one more time taking into account my comments. We are
almost there.

ron