<div dir="ltr"><div>> I wonder if we need to make it warn about missing functionality if someone is using the old option though?<br></div><div><br></div>Agreed, will do that in V2.<div><br></div><div>//Ming Liu</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr"><<a href="mailto:richard.purdie@linuxfoundation.org">richard.purdie@linuxfoundation.org</a>> 於 2019年5月16日 週四 下午12:55寫道：<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Thu, 2019-05-16 at 10:56 +0200, Ming Liu wrote:<br>
> Hi, Richard:<br>
> <br>
> > I'd imagine the code is designed so you'd inherit uboot-config in<br>
> the machine and this therefore could be made to work as is.<br>
> <br>
> Yes, that probably is how the author intended it to be used, but<br>
> unfortunately, that's not how it's being used in BSP layers, that<br>
> fact is, in most BSP meta layers, they do not put it into INHERIT,<br>
> only u-boot recipes are inheriting uboot-config.<br>
> <br>
> I am on board there is no need to set IMAGE_FSTYPES in uboot-<br>
> config.bbclass, maybe we should drop it from UBOOT_CONFIG and it<br>
> would be like: 'UBOOT_CONFIG = "config,,binary"', there will be a<br>
> empty item in the middle, so the BSP meta layers would work as usual.<br>
> <br>
> Will send a V2 if you agree with this.<br>
<br>
I agree, I wonder if we need to make it warn about missing<br>
functionality if someone is using the old option though?<br>
<br>
Cheers,<br>
<br>
Richard<br>
<br>
</blockquote></div>