From patchwork Thu Aug 20 18:08:08 2009 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: clean up intel Kconfig/Makefile.inc usage Date: Thu, 20 Aug 2009 18:08:08 -0000 From: ron minnich X-Patchwork-Id: 151 Message-Id: <13426df10908201108ydc617eetb5113f24366a3a92@mail.gmail.com> To: coreboot ron Acked-by: Patrick Georgi Change the intel cpu makefile.inc so that it fits the model better. - intel/Makefile.inc only mentions sockets - those sockets are conditionally included - makefile.inc in socket directories are almost all unconditionally included - Get rid of if where possible, use -$(CONFIG_VARIABLE) instead as per Kconfig standards in linux kernel See the Kconfig.tex documentation for questions. Signed-off-by: Ronald G. Minnich Index: src/cpu/intel/Makefile.inc =================================================================== --- src/cpu/intel/Makefile.inc (revision 4558) +++ src/cpu/intel/Makefile.inc (working copy) @@ -2,9 +2,8 @@ # Therefore: # ONLY include Makefile.inc from socket directories! -subdirs-y += speedstep -subdirs-y += socket_mFCPGA478 -subdirs-y += socket_PGA370 +subdirs-$(CONFIG_CPU_INTEL_SOCKET_MFCPGA478) += socket_mFCPGA478 +subdirs-$(CONFIG_CPU_INTEL_SOCKET_PGA370) += socket_PGA370 #socket_mPGA478 #socket_mPGA479M Index: src/cpu/intel/speedstep/Makefile.inc =================================================================== --- src/cpu/intel/speedstep/Makefile.inc (revision 4558) +++ src/cpu/intel/speedstep/Makefile.inc (working copy) @@ -1,5 +1 @@ -ifeq ($(CONFIG_HAVE_ACPI_TABLES), y) -ifeq ($(CONFIG_CPU_INTEL_SOCKET_MFCPGA478), y) - obj-y += acpi.o -endif -endif +obj-$(CONFIG_HAVE_ACPI_TABLES) += acpi.o Index: src/cpu/intel/socket_mFCPGA478/Makefile.inc =================================================================== --- src/cpu/intel/socket_mFCPGA478/Makefile.inc (revision 4558) +++ src/cpu/intel/socket_mFCPGA478/Makefile.inc (working copy) @@ -1,17 +1,17 @@ -ifeq ($(CONFIG_CPU_INTEL_SOCKET_MFCPGA478),y) - obj-y += socket_mFCPGA478.o - subdirs-y += ../model_69x - subdirs-y += ../model_6dx - subdirs-y += ../model_6ex - subdirs-y += ../model_6fx - subdirs-y += ../../x86/tsc - subdirs-y += ../../x86/mtrr - subdirs-y += ../../x86/fpu - subdirs-y += ../../x86/mmx - subdirs-y += ../../x86/sse - subdirs-y += ../../x86/lapic - subdirs-y += ../../x86/cache - subdirs-y += ../../x86/smm - subdirs-y += ../microcode - subdirs-y += ../hyperthreading -endif +obj-y += socket_mFCPGA478.o +subdirs-y += ../model_69x +subdirs-y += ../model_6dx +subdirs-y += ../model_6ex +subdirs-y += ../model_6fx +subdirs-y += ../../x86/tsc +subdirs-y += ../../x86/mtrr +subdirs-y += ../../x86/fpu +subdirs-y += ../../x86/mmx +subdirs-y += ../../x86/sse +subdirs-y += ../../x86/lapic +subdirs-y += ../../x86/cache +subdirs-y += ../../x86/smm +subdirs-y += ../microcode +subdirs-y += ../hyperthreading +subdirs-y += ../speedstep + Index: src/cpu/intel/socket_PGA370/Makefile.inc =================================================================== --- src/cpu/intel/socket_PGA370/Makefile.inc (revision 4558) +++ src/cpu/intel/socket_PGA370/Makefile.inc (working copy) @@ -18,17 +18,16 @@ ## Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA ## -ifeq ($(CONFIG_CPU_INTEL_SOCKET_PGA370),y) - obj-y += socket_PGA370.o - subdirs-y += ../model_6xx - subdirs-y += ../../x86/tsc - subdirs-y += ../../x86/mtrr - subdirs-y += ../../x86/fpu - subdirs-y += ../../x86/mmx - subdirs-y += ../../x86/sse - subdirs-y += ../../x86/lapic - subdirs-y += ../../x86/cache - subdirs-y += ../../x86/smm - subdirs-y += ../microcode -endif +obj-y += socket_PGA370.o +subdirs-y += ../model_6xx +subdirs-y += ../../x86/tsc +subdirs-y += ../../x86/mtrr +subdirs-y += ../../x86/fpu +subdirs-y += ../../x86/mmx +subdirs-y += ../../x86/sse +subdirs-y += ../../x86/lapic +subdirs-y += ../../x86/cache +subdirs-y += ../../x86/smm +subdirs-y += ../microcode +