<div dir="ltr"><div class="gmail_extra"><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div dir="ltr"><br></div></div></div></div>
<br><div class="gmail_quote">On Wed, Aug 23, 2017 at 8:20 PM, Bruce Ashfield <span dir="ltr">&lt;<a href="mailto:bruce.ashfield@windriver.com" target="_blank">bruce.ashfield@windriver.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">commit 44aea7b873 [kernel-yocto: ensure that only valid BSPs are built]<br>
introduced a new check to ensure that a valid BSP definition was found,<br>
rather than building something that &#39;closely&#39; matched the current<br>
MACHINE.<br>
<br>
This check breaks valid configurations which do not have a bsp<br>
definition but are otherwise completely configured machines.<br>
<br>
To allow both elements to co-exist (and not add warnings or errors<br>
to otherwise valid builds), we first check to see if an empty bsp<br>
definition was found, but then check to see if a defconfig was<br>
provided.<br>
<br>
If a defconfig has been provided, that is a sign that the board<br>
configuration is complete and we should continue the build without<br>
otherwise bothering the user.<br>
<br>
Tested on meta-raspberrypi and linux-yocto*<br>
<br>
Signed-off-by: Bruce Ashfield &lt;<a href="mailto:bruce.ashfield@windriver.com">bruce.ashfield@windriver.com</a>&gt;<br>
---<br>
<br>
</span>v2: fixed the badly placed -q<br>
<span class=""><br>
 meta/classes/kernel-yocto.<wbr>bbclass | 7 +++++--<br>
 1 file changed, 5 insertions(+), 2 deletions(-)<br>
<br>
diff --git a/meta/classes/kernel-yocto.<wbr>bbclass b/meta/classes/kernel-yocto.<wbr>bbclass<br>
</span>index 3c6df92131bc..20b4942320d6 100644<br>
<span class="">--- a/meta/classes/kernel-yocto.<wbr>bbclass<br>
+++ b/meta/classes/kernel-yocto.<wbr>bbclass<br>
@@ -143,8 +143,11 @@ do_kernel_metadata() {<br>
<br>
        # expand kernel features into their full path equivalents<br>
        bsp_definition=$(spp ${includes} --find -DKMACHINE=${KMACHINE} -DKTYPE=${LINUX_KERNEL_TYPE})<br>
-       if [ $? -ne 0 ] || [ -z &quot;${bsp_definition}&quot; ]; then<br>
-               bbfatal_log &quot;Could not locate BSP definiton for ${KMACHINE}/${LINUX_KERNEL_<wbr>TYPE}.&quot;<br>
+       if [ -z &quot;$bsp_definition&quot; ]; then<br>
</span>+               echo &quot;$sccs&quot; | grep -q defconfig<br>
<div class="HOEnZb"><div class="h5">+               if [ $? -ne 0 ]; then<br>
+                       bberror &quot;Could not locate BSP definition for ${KMACHINE}/${LINUX_KERNEL_<wbr>TYPE} and no defconfig was provided&quot;<br>
+               fi<br>
        fi<br>
        meta_dir=$(kgit --meta)<br><br></div></div></blockquote><div><br></div><div>Looks good.  </div></div></div></div>