Re: [PATCH] kbuild: determine base DTB by suffix

From: Nicolas Schier
Date: Mon Dec 04 2023 - 06:18:05 EST


On Sun, Dec 03, 2023 at 05:05:48PM +0900, Masahiro Yamada wrote:
> When using the -dtbs syntax, you need to ensure to list the base first,
> as follows:
>
> foo-dtbs := foo_base.dtb foo_overlay1.dtbo foo_overlay2.dtbo
> dtb-y := foo.dtb
>
> You cannot do this arrangement:
>
> foo-dtbs := foo_overlay1.dtbo foo_overlay2.dtbo foo_base.dtb
>
> This restriction comes from $(firstword ...), but it is unneeded to
> rely on the order in the -dtbs syntax.
>
> Instead, you can simply determine the base by suffix because the
> base (*.dtb) and overlays (*.dtbo) use different suffixes.
>
> Signed-off-by: Masahiro Yamada <masahiroy@xxxxxxxxxx>
> ---

Thanks, looks good to me.

Reviewed-by: Nicolas Schier <nicolas@xxxxxxxxx>