Patchwork clean up intel Kconfig/Makefile.inc usage

login
register
about
Submitter ron minnich
Date 2009-08-20 18:08:08
Message ID <13426df10908201108ydc617eetb5113f24366a3a92@mail.gmail.com>
Download mbox | patch
Permalink /patch/151/
State Accepted
Headers show

Comments

ron minnich - 2009-08-20 18:08:08
ron
Patrick Georgi - 2009-08-21 09:43:02
Am 20.08.2009 20:08, schrieb ron minnich:
> ron
>    
Acked-by: Patrick Georgi <patrick.georgi@coresystems.de>
ron minnich - 2009-08-21 18:28:34
On Fri, Aug 21, 2009 at 2:43 AM, Patrick Georgi<patrick@georgi-clan.de> wrote:
> Am 20.08.2009 20:08, schrieb ron minnich:
>>
>> ron
>>
>
> Acked-by: Patrick Georgi <patrick.georgi@coresystems.de>
>

Committed revision 4561.

Patch

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 <rminnich@gmail.com>
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
 
+