From 2bf6a70eddc8842643f1209a1e9bcecf76ff7a3d Mon Sep 17 00:00:00 2001 From: Richard Burton Date: Mon, 19 Jun 2023 14:51:36 +0100 Subject: Small fixes based on feedback * Typo fix in documentation * Fix bug with build_default.py and passing in --make-verbose * Added additional requirement for third-party dependencies Signed-off-by: Richard Burton Change-Id: Icdac7d4583c4fe3000fe9bc784979b2d1cf70909 --- build_default.py | 2 +- docs/sections/building.md | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/build_default.py b/build_default.py index 1badb02..1d562f9 100755 --- a/build_default.py +++ b/build_default.py @@ -137,7 +137,7 @@ def run( make_command = f"{cmake_path} --build {build_dir} -j{make_jobs}" if make_verbose: - make_command += "--verbose" + make_command += " --verbose" logging.info(f"\n\n\n{make_command}\n\n\n") state = subprocess.run( make_command, shell=True, stdout=logpipe, stderr=subprocess.STDOUT diff --git a/docs/sections/building.md b/docs/sections/building.md index 7af3fdb..84ce271 100644 --- a/docs/sections/building.md +++ b/docs/sections/building.md @@ -155,6 +155,7 @@ Before proceeding, it is *essential* to ensure that the following prerequisites - xxd - unzip - Python Pillow + - curl > **Note:** Due to the fast paced nature of development, this list might not be exhaustive. Please refer to Tensorflow Lite Micro documentation for more info. @@ -198,7 +199,7 @@ The build parameters are: These parameters are optional and are only valid for Arm® *Cortex®-M* CPU targeted configurations. The default values points to the `dependencies/cmsis`, `dependencies/cmsis-dsp` and `dependencies/cmsis-nn` git submodules. Repositories are hosted here: [CMSIS-5](https://github.com/ARM-software/CMSIS_5.git), - [CMSIS-DPS](https://github.com/ARM-software/CMSIS-DSP) and [CMSIS-NN](https://github.com/ARM-software/CMSIS-NN.git). + [CMSIS-DSP](https://github.com/ARM-software/CMSIS-DSP) and [CMSIS-NN](https://github.com/ARM-software/CMSIS-NN.git). - `ETHOS_U_NPU_ENABLED`: Sets whether the use of *Ethos-U* NPU is available for the deployment target. By default, this is set and therefore application is built with *Ethos-U* NPU supported. -- cgit v1.2.1