On 05/02/08 14:20 -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). > > 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. Another patch would change buildrom to stop > patching Config.lb files. There is already a CBV2_CONFIG variable > that would work nicely for selecting the correct file. > > Comments? > > Signed-off-by: Myles Watson <[EMAIL PROTECTED]> Acked-by: Jordan Crouse <[EMAIL PROTECTED]>
Are you going to patch buildrom to remove the ROM name stuff too? If so, I ack it now if I don't see it until later. Jordan -- Jordan Crouse Systems Software Development Engineer Advanced Micro Devices, Inc. -- coreboot mailing list coreboot@coreboot.org http://www.coreboot.org/mailman/listinfo/coreboot
