[coreboot] [PATCH] coreboot-v3: Default to qemu
Myles Watson
mylesgw at gmail.com
Tue Dec 2 19:48:17 CET 2008
On Tue, Dec 2, 2008 at 11:34 AM, Jordan Crouse <jordan at cosmicpenguin.net>wrote:
> Myles Watson wrote:
>
>> On Tue, Dec 2, 2008 at 9:45 AM, Jordan Crouse <jordan at cosmicpenguin.net
>> >wrote:
>>
>> If no .config exists, then default to qemu for defconfig targets. As a
>>> further extension, I think we should consider allowing the user to pass
>>> the
>>> desired mainboard via an environment variable:
>>>
>>> make MAINBOARD=amd/db800 defconfig
>>>
>>
>>
>> I think we already do that. Here's the snippet from buildrom's
>> packages/coreboot-v3/coreboot-v3.mk.
>>
>> @ make -C $(CBV3_SRC_DIR) defconfig \
>> MAINBOARDDIR="$(COREBOOT_VENDOR)/$(CBV3_BOARD)" \
>> > $(CBV3_CONFIG_LOG) 2>&1
>>
>> Am I misunderstanding what you want to do? Did we used to allow that and
>> it's broken since then?
>>
>
> No, it still works. I guess thats fine - MAINBOARDDIR is slightly
> unintuitive but we can deal.
I guess we could add a MAINBOARD that gets copied into MAINBOARDDIR, or
change all MAINBOARDDIR to MAINBOARD. I think that's a pretty trivial
change if it only has to change in the Makefile.
Your patch:
Acked-by: Myles Watson <mylesgw at gmail.com>
Thanks,
Myles
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20081202/f92fddf3/attachment.html>
More information about the coreboot
mailing list