<div dir="ltr"><div dir="ltr"><div>Hi, Richard:</div><div><br></div><div>> I'd imagine the code is designed so you'd inherit uboot-config in the machine and this therefore could be made to work as is.</div><div><br></div><div>Yes, that probably is how the author intended it to be used, but unfortunately, that's not how it's being used in BSP layers, that fact is, in most BSP meta layers, they do not put it into INHERIT, only u-boot recipes are inheriting uboot-config.</div><div><br></div><div>I am on board there is no need to set IMAGE_FSTYPES in uboot-config.bbclass, maybe we should drop it from UBOOT_CONFIG and it would be like: 'UBOOT_CONFIG = "config,,binary"', there will be a empty item in the middle, so the BSP meta layers would work as usual.</div><div><br></div><div>Will send a V2 if you agree with this.</div><div><br></div><div>//Ming Liu</div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Richard Purdie <<a href="mailto:richard.purdie@linuxfoundation.org">richard.purdie@linuxfoundation.org</a>> 於 2019年5月16日 週四 上午10:23寫道：<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 Wed, 2019-05-15 at 21:22 +0200, <a href="mailto:liu.ming50@gmail.com" target="_blank">liu.ming50@gmail.com</a> wrote:<br>
> From: Ming Liu <<a href="mailto:liu.ming50@gmail.com" target="_blank">liu.ming50@gmail.com</a>><br>
> <br>
> The code dealing with IMAGE_FSTYPES in uboot-config.bbclass would not<br>
> take any effect since it's in a uboot particular class, but it's not<br>
> being inherited by any image recipes.<br>
> <br>
> Move the IMAGE_FSTYPES handling part to image.bbclass would make it<br>
> really work.<br>
> <br>
> Also drop some trivial debug code.<br>
> <br>
> Signed-off-by: Ming Liu <<a href="mailto:liu.ming50@gmail.com" target="_blank">liu.ming50@gmail.com</a>><br>
> ---<br>
> meta/classes/image.bbclass | 12 ++++++++++++<br>
> meta/classes/uboot-config.bbclass | 17 ++++-------------<br>
> meta/conf/documentation.conf | 2 +-<br>
> 3 files changed, 17 insertions(+), 14 deletions(-)<br>
<br>
This doesn't make sense, if we want to set IMAGE_FSTYPES, lets just set<br>
that.<br>
<br>
I'd imagine the code is designed so you'd inherit uboot-config in the<br>
machine and this therefore could be made to work as is.<br>
<br>
Moving the code to image.bbclass is the wrong thing to do though and<br>
I'd question whether we want UBOOT_CONFIG doing this at all...<br>
<br>
Cheers,<br>
<br>
Richard<br>
<br>
</blockquote></div>