[coreboot] patch: add support for blobs; example SMM blob included

Peter Stuge peter at stuge.se
Wed Nov 26 23:19:21 CET 2008


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?

I remember discussing blobs being sort-of stage3/4. Do we call them
that instead?


//Peter




More information about the coreboot mailing list