<div dir="ltr"><div>On Thu, Mar 1, 2018 at 6:38 AM, Peter Stuge <span dir="ltr"><<a href="mailto:peter@stuge.se" target="_blank">peter@stuge.se</a>></span> wrote:<br></div><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Marshall Dawson wrote:<br>
>    1. You can skip this step in your instructions.  Instead, when running<br>
<span class="">>    make menuconfig, look in Chipset / ChromeOS and select "Build for<br>
>    ChromeOS".  This should cause Depthcharge to appear as a payload option.<br>
<br>
</span>I think this is a design issue in coreboot that ought to be fixed.<br>
<br>
Depthcharge can be useful on its own, it should not be artificially tied down.<span class="HOEnZb"><font color="#888888"><br></font></span></blockquote><div><br></div><div>Depthcharge is an external (to coreboot) project with the only goal to support booting Chrome OS. You shouldn't really expect it to do anything else unless you want to maintain a fork for that on your own. Any other use that may work now can break upstream (on <a href="http://chromium.googlesource.com">chromium.googlesource.com</a>) at any moment, and I think we just regularly sync that repository back into the <a href="http://coreboot.org">coreboot.org</a> mirror.</div><div><br></div><div>That said, CONFIG_CHROMEOS in coreboot doesn't really mean "this has to boot Chrome OS and nothing else"... it rather means "select code and features that are required for Chrome OS devices". Much of that "required for Chrome OS devices" is specifically "required to run depthcharge" (e.g. the lb_gpios in the coreboot table, or the vboot-capable FMAP), so disabling it just means depthcharge won't work right. If you want to build/run depthcharge for whatever reason (even to boot some other Linux distro as long as you package its kernel in a vblock), just leaving CONFIG_CHROMEOS enabled is probably the right thing. In many ways CONFIG_CHROMEOS in coreboot really just means CONFIG_SUPPORT_FOR_DEPTHCHARGE.</div><div><br></div><div><span style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:small;font-style:normal;font-variant-ligatures:normal;font-variant-caps:normal;font-weight:400;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">The Speedy issue seems to be a simple bug / missing feature in the payloads/external/depthcharge/Makefile. The configs in payload/libpayload/configs/ were originally just added as an FYI that people could manually copy in for make oldconfig. Then someone seems to have later written that depthcharge Makefile with the expectation that there'd always be a config for every board. That wasn't the intention and I don't think we want to go there, because often a config is shared across all boards in a family (e.g. configs/veyron for veyron_jerry and veyron_speedy, or configs/gru for kevin and bob). If we want to have that automatic selection from a Makefile, we should set up a simple translation table somewhere so it can find out which config belongs to which board. (Symlinks would also be an option, but I think some people don't like them in Git because of Windows. I don't think we should just duplicate the files because that always becomes a mess sooner or later.)</span></div><div><span style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:small;font-style:normal;font-variant-ligatures:normal;font-variant-caps:normal;font-weight:400;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline"><br></span></div><div>While we're at it, I think the depthcharge Makefile should also inject CONFIG_CHROMEOS=y in the libpayload config. That's how they're built for Chrome OS and that's the only way they're guaranteed to be compatible with depthcharge. (Again, it's not so much a "can only boot Chrome OS" option, more of a "select the right defaults for Chromebooks/depthcharge" option.)</div></div></div></div>