On Wed, Nov 26, 2008 at 2:19 PM, Peter Stuge <[EMAIL PROTECTED]> wrote: > ron minnich wrote: >> +++ arch/x86/Makefile (working copy) >> @@ -48,6 +48,13 @@ >> COMPRESSFLAG := -C nrv2b >> endif >> >> +# all depends on coreboot.rom and BLOBS. >> +# BLOBS may be empty. They are things such as smm that are searched for at >> runtime. >> +# They depend on the coreboot rom existing so that they can be placed there. >> +# The name of the blob is determined by the various blob rules. There is so >> much possible >> +# variation in them that it is not really possible to put a standard rule >> in this file. >> +rom:: $(obj)/coreboot.rom $(BLOBS) >> + >> $(obj)/coreboot.rom $(obj)/coreboot.map: $(obj)/coreboot.bootblock >> $(obj)/util/lar/lar lzma nrv2b $(obj)/coreboot.initram >> $(obj)/coreboot.stage2 $(obj)/option_table >> $(Q)printf " LAR $(subst $(shell pwd)/,,$(@))\n" >> $(Q)rm -f $(obj)/coreboot.rom >> @@ -306,4 +313,4 @@ >> $(Q)cp cscope.proj $(obj)/mainboard/$(MAINBOARDDIR)/kscope >> $(Q)sh util/mkdep $@ $(INITCFLAGS) "--" $(ALLSRC) >> # $(Q)sort -u -o $@ $@ >> - >> +.PHONY: rom > > I don't like to add PHONY targets, nor special purpose toplevel > dependencies. Could the boards that need them simply depend on > smm.elf?
we have a bunch already. I don't see a big deal with rom as the target. I can take the PHONY out however. > > I remember discussing blobs being sort-of stage3/4. Do we call them > that instead? > no, that makes no sense. Blobs are all over. VSA blob is in stage1. SMM is installed in stage2 phase 6. They are not classifiable. ron -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot

