On Tue, Feb 05, 2008 at 02:20:06PM -0700, Myles Watson wrote:
This patch changes all rom names that aren't coreboot.rom in Config.lb files.
I think that since the directory specifies the architecture and the board, it is redundant information to name it something else, and it makes it more difficult to automate the build process (buildrom).
Great.
In buildrom we should just use Config-options.lb files instead of patching or keeping our own. It just adds more to maintain, with very little benefit. The correct place for Config.lb files is in the coreboot-v2 tree.
The next patch would add Config-lab.lb files for each architecture supported by buildrom.
That would work - but what about other payloads? Maybe we could use a naming scheme like
Config-buildrom-$(PAYLOAD).lb
and then make buildrom look for such a file, perhaps falling back to the generic Config.lb file if it does not exist?
Thanks, Ward.