<div>Hi,</div><div><br></div><div>Below is a trivial patch for filo. filo does not compile with the grub interface right out of the SVN:</div><div><br></div><div><div>coreboot/payloads/filo/build/main/grub/completions.o: In function `print_completions':</div>

<div>completions.c:(.text+0x2f2): undefined reference to `IS_PC_SLICE_TYPE_BSD'</div></div><div><br></div><div>This patch defines this macro to 0 (false), like what is done in fs/filesys.h, which fixes the problem. However, this may not be the right way to fix it. Please do check before applying.</div>

<div><br></div><div>---</div><div><br></div><div>I also have a remark about USB support in filo: The default configuration for filo enables support for USB, although the default configuration for libpayload does not. There's no harm done, since you have to configure both, but it appears more logical to have USB disabled in both config. Anyways, that's not really a problem :)</div>

<div><br></div><div>--- </div><div><br></div><div>Signed-off-by: Aurelien Guillaume <<a href="mailto:aurelien@iwi.me">aurelien@iwi.me</a>></div><div><br></div><div>Index: main/grub/completions.c</div><div>===================================================================</div>

<div>--- main/grub/completions.c<span class="Apple-tab-span" style="white-space:pre">   </span>(revision 138)</div><div>+++ main/grub/completions.c<span class="Apple-tab-span" style="white-space:pre">    </span>(working copy)</div>

<div>@@ -23,6 +23,7 @@</div><div> #include <config.h></div><div> #include <grub/shared.h></div><div> #define current_slice 0</div><div>+#define IS_PC_SLICE_TYPE_BSD(type) 0</div><div> </div><div> static int do_completion;</div>

<div> static int unique;</div><div><br></div>-- <br>Aurélien Guillaume<br><br>