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

Reply via email to