<br><br><div class="gmail_quote">On Tue, Dec 2, 2008 at 11:34 AM, Jordan Crouse <span dir="ltr"><<a href="mailto:jordan@cosmicpenguin.net">jordan@cosmicpenguin.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div><div></div><div class="Wj3C7c">Myles Watson wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
On Tue, Dec 2, 2008 at 9:45 AM, Jordan Crouse <<a href="mailto:jordan@cosmicpenguin.net" target="_blank">jordan@cosmicpenguin.net</a>>wrote:<br>
<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
If no .config exists, then default to qemu for defconfig targets.  As a<br>
further extension, I think we should consider allowing the user to pass the<br>
desired mainboard via an environment variable:<br>
<br>
make MAINBOARD=amd/db800 defconfig<br>
</blockquote>
<br>
<br>
I think we already do that.  Here's the snippet from  buildrom's<br>
packages/coreboot-v3/<a href="http://coreboot-v3.mk" target="_blank">coreboot-v3.mk</a>.<br>
<br>
    @ make -C $(CBV3_SRC_DIR) defconfig \<br>
        MAINBOARDDIR="$(COREBOOT_VENDOR)/$(CBV3_BOARD)" \<br>
        > $(CBV3_CONFIG_LOG) 2>&1<br>
<br>
Am I misunderstanding what you want to do?  Did we used to allow that and<br>
it's broken since then?<br>
</blockquote>
<br></div></div>
No, it still works.  I guess thats fine - MAINBOARDDIR is slightly unintuitive but we can deal.<font color="#888888"></font></blockquote><div><br>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.<br>
<br>Your patch:<br>Acked-by: Myles Watson <<a href="mailto:mylesgw@gmail.com">mylesgw@gmail.com</a>> <br></div></div><br>Thanks,<br>Myles<br>