Re: [PATCH] dtc: also check <yaml.h> for libyaml

From: Rob Herring
Date: Mon Jun 08 2020 - 15:09:42 EST


On Mon, Jun 8, 2020 at 2:42 AM Jiping Ma <jiping.ma2@xxxxxxxxxxxxx> wrote:
>
> yamltree.c includes <yaml.h>, If /usr/include/yaml.h does not exist,
> it fails to build.

Does this patch fix your issue?:

https://lore.kernel.org/linux-devicetree/20200505100319.741454-1-masahiroy@xxxxxxxxxx/


> Signed-off-by: Jiping Ma <jiping.ma2@xxxxxxxxxxxxx>
> ---
> scripts/dtc/Makefile | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/scripts/dtc/Makefile b/scripts/dtc/Makefile
> index b5a5b1c..b49dfea 100644
> --- a/scripts/dtc/Makefile
> +++ b/scripts/dtc/Makefile
> @@ -18,9 +18,13 @@ $(error dtc needs libyaml for DT schema validation support. \
> endif
> HOST_EXTRACFLAGS += -DNO_YAML
> else
> +ifeq ($(wildcard /usr/include/yaml.h),)
> +HOST_EXTRACFLAGS += -DNO_YAML
> +else
> dtc-objs += yamltree.o
> HOSTLDLIBS_dtc := $(shell pkg-config yaml-0.1 --libs)
> endif
> +endif
>
> # Generated files need one more search path to include headers in source tree
> HOSTCFLAGS_dtc-lexer.lex.o := -I $(srctree)/$(src)
> --
> 1.9.1
>