On Tue, 2 Jul 2024 at 21:27, Raymond Mao <raymond....@linaro.org> wrote: > > Add the mbedtls include directories into the build system. > > Signed-off-by: Raymond Mao <raymond....@linaro.org> > --- > Changes in v2 > - None. > Changes in v3 > - Remove changes for PLATFORM_CPPFLAGS. > Changes in v4 > - Fix errors when building without "O=". > - Minor fix of the include directories. > > Makefile | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/Makefile b/Makefile > index 07d7947c8af..fd855dbd5c9 100644 > --- a/Makefile > +++ b/Makefile > @@ -829,6 +829,12 @@ KBUILD_HOSTCFLAGS += $(if $(CONFIG_TOOLS_DEBUG),-g) > UBOOTINCLUDE := \ > -Iinclude \ > $(if $(KBUILD_SRC), -I$(srctree)/include) \ > + $(if $(CONFIG_MBEDTLS_LIB), \ > + "-DMBEDTLS_CONFIG_FILE=\"mbedtls_def_config.h\"" \
don't you need a full path to the config file? > + -I$(srctree)/lib/mbedtls \ > + -I$(srctree)/lib/mbedtls/port \ > + -I$(srctree)/lib/mbedtls/external/mbedtls \ > + -I$(srctree)/lib/mbedtls/external/mbedtls/include) \ Most of these paths are explicitly added in patch #3. Why don't we just keep them there? Thanks /Ilias > $(if $(CONFIG_$(SPL_)SYS_THUMB_BUILD), \ > $(if $(CONFIG_HAS_THUMB2), \ > $(if $(CONFIG_CPU_V7M), \ > -- > 2.25.1 >
