Hi Ilias,
On Wed, 3 Jul 2024 at 07:35, Ilias Apalodimas ilias.apalodimas@linaro.org wrote:
On Tue, 2 Jul 2024 at 21:27, Raymond Mao raymond.mao@linaro.org wrote:
Add the mbedtls include directories into the build system.
Signed-off-by: Raymond Mao raymond.mao@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?
We still need this for building other modules other than the MbedTLS
library itself. For example, when those files include the "hash-checksum.h".
Regards, Raymond