Re: [RFC 05/13] ARM: dts: ti: add support for building .dtb files containing symbols

[Geert Uytterhoeven <geert@linux-m68k.org>]

Hi Tero,

On Tue, Apr 17, 2018 at 11:10 AM, Tero Kristo <t-kristo@ti.com> wrote:
> Symbols in the DTB files are needed so that overlays can be applied
> properly. Add support for building TI SoC specific .dtb files, the
> source is fetched from parent directory.
>
> Signed-off-by: Tero Kristo <t-kristo@ti.com>

Thanks for your patch!

> ---
>  arch/arm/boot/dts/ti/Makefile | 5 +++++
>  1 file changed, 5 insertions(+)
>
> diff --git a/arch/arm/boot/dts/ti/Makefile b/arch/arm/boot/dts/ti/Makefile
> index b26328b..7d2d411 100644
> --- a/arch/arm/boot/dts/ti/Makefile
> +++ b/arch/arm/boot/dts/ti/Makefile
> @@ -1 +1,6 @@
> +DTC_FLAGS += -@ -Wno-reg_format -Wno-avoid_default_addr_size

Why the -W options?

> +
> +$(obj)/%.dtb: $(src)/../%.dts FORCE

I guess you want a dependency on $(DTC), too?

> +       $(call if_changed_dep,dtc)
> +
>  clean-files    += *.dtbo

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds