src/cpu/x86/smm/Makefile.inc:33: warning: overriding commands for target
`/home/myles/try/buildrom-devel/work/coreboot/svn/build/cpu/x86/smm/smm.o'
src/cpu/x86/smm/Makefile.inc:33: warning: ignoring old commands for target
`/home/myles/try/buildrom-devel/work/coreboot/svn/build/cpu/x86/smm/smm.o'
src/cpu/x86/smm/Makefile.inc:36: warning: overriding commands for target
`/home/myles/try/buildrom-devel/work/coreboot/svn/build/cpu/x86/smm/smm'
src/cpu/x86/smm/Makefile.inc:36: warning: ignoring old commands for target
`/home/myles/try/buildrom-devel/work/coreboot/svn/build/cpu/x86/smm/smm'
src/cpu/x86/smm/Makefile.inc:41: warning: overriding commands for target
`/home/myles/try/buildrom-devel/work/coreboot/svn/build/cpu/x86/smm/smm_bin.c'
src/cpu/x86/smm/Makefile.inc:41: warning: ignoring old commands for target
`/home/myles/try/buildrom-devel/work/coreboot/svn/build/cpu/x86/smm/smm_bin.c'

my...@orangutan:svn$ grep -Ir subdirs.*smm src/ | grep -v svn
src/cpu/x86/Makefile.inc:subdirs-y += smm
src/cpu/via/model_c7/Makefile.inc:subdirs-y += ../../x86/smm
src/cpu/via/model_c3/Makefile.inc:subdirs-y += ../../x86/smm
src/cpu/intel/ep80579/Makefile.inc:subdirs-y += ../../x86/smm

First it gets included with src/cpu/x86, then with every cpu model.  Can we
just remove those?

subdirs-y += amd
subdirs-y += intel
subdirs-y += via
subdirs-y += emulation
subdirs-y += x86

This is a little strange, though.  All the rest are brands.

To be consistent, I think we should remove subdirs-y += x86 from
src/cpu/Makefile.inc, and let every CPU include it.  Patch attached.

Signed-off-by: Myles Watson <[email protected]>

I'd be equally happy with every CPU (since we only have x86), just doing
subdir-y += ../../x86 so they didn't have to be repeated.

Thanks,
Myles
Index: svn/src/cpu/Makefile.inc
===================================================================
--- svn.orig/src/cpu/Makefile.inc
+++ svn/src/cpu/Makefile.inc
@@ -2,4 +2,3 @@ subdirs-y += amd
 subdirs-y += intel
 subdirs-y += via
 subdirs-y += emulation
-subdirs-y += x86
Index: svn/src/cpu/x86/Makefile.inc
===================================================================
--- svn.orig/src/cpu/x86/Makefile.inc
+++ /dev/null
@@ -1 +0,0 @@
-subdirs-y += smm
-- 
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to