Age | Commit message (Collapse) | Author |
|
The Travis build keeps timing out apparently due to a reduced download
speed compared with some time ago. It is not rare for the build to fail
in the initial "repo sync" phase with the error message:
No output has been received in the last 10m0s, this potentially
indicates a stalled build or something wrong with the build itself.
Allowing more verbosity from "repo sync" could possibly avoid this
particular error, but then the second issue is that total build time
reaches 50 minutes very frequently which is the limit for Open Source
projects.
Since we have IBART to run xtest on HiKey, we can loose the test on
QEMU. Therefore, this commit keeps only the code rule checks in the
Travis script (checkpatch.pl and pycodestyle).
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
|
|
We want to enforce PEP 8 -- Style Guide for Python Code [1]. Therefore,
use the pycodestyle tool in the Travis CI script.
Link: [1] https://www.python.org/dev/peps/pep-0008/
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
|
|
Print ccache and disk space info before and after the build to help
monitor and troubleshoot issues.
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
|
|
The 64-bit toolchain is not needed to build and run the regression tests
for QEMU. Download only the 32-bit toolchain to save disk space and time.
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
|
|
The optee_os cloned using "repo" is not used, because we link to the
directory checked out by the CI process instead. Delete it to save about
160 MiB of disk space on the build server.
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
|
|
Save some more space on the build server.
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
|
|
Since build.git has been upgraded to GCC 8.2 [1], we are running out of
disk space when running the Travis CI script. This is because the GCC 8.2
binaries are much larger than 6.2:
218M gcc-arm-8.2-2018.08-x86_64-aarch64-linux-gnu.tar.xz
213M gcc-arm-8.2-2018.08-x86_64-arm-linux-gnueabihf.tar.xz
94M gcc-linaro-6.2.1-2016.11-x86_64_aarch64-linux-gnu.tar.xz
88M gcc-linaro-6.2.1-2016.11-x86_64_arm-linux-gnueabihf.tar.xz
This commit makes some room by deleting the packages once they have been
extracted.
Link: [1] https://github.com/OP-TEE/build/commit/53119b77e9f2
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
|
|
The QEMU test environment has recently been upgraded to use U-Boot
("qemu.mk: replace bios with u-boot" [1] and "default.xml: replace
bios_qemu_tz_arm with u-boot" [2]), which requires GCC 6 or newer.
Fixes the following CI build error:
*** Your GCC is older than 6.0 and is not supported
make[1]: *** [checkgcc6] Error 1
make: *** [u-boot] Error 2
Link: [1] https://github.com/OP-TEE/build/commit/248f5733bf29
Link: [2] https://github.com/OP-TEE/manifest/commit/2067ab3f9bbe
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
|
|
Having -Werror turned on by default can be annoying, because not everyone
uses the same compiler, and different compiler versions have different
warnings.
Therefore, provide CFG_WERROR to turn it on instead. Enable CFG_WERROR in
the CI scripts because we still don't want warnings with the officially
supported compilers.
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
|
|
Now that the Travis container-based environment is running Ubuntu Trusty
(14.04), some downloads and build steps are not needed anymore. Use the
tools that come with the distribution instead.
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
|
|
The default ccache directory used by buildroot is ~/.buildroot-ccache [1],
which is outside the paths saved by the Travis caching mechanism [2]. Fix
that by adding BR2_CCACHE_DIR=~/.ccache to the make command.
Link: [1] https://buildroot.org/downloads/manual/manual.html#ccache
Link: [2] https://docs.travis-ci.com/user/caching/#ccache-cache
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
|
|
The legacy (GCC 4.9) toolchain has been deprecated in build.git [1]. As a
result, 'make toolchain COMPILE_LEGACY=y' cannot be used anymore to
download the 4.9 compiler, and the following error happens during the
Travis CI build:
CC /home/travis/optee_repo/build/../out/bios-qemu/bios/entry.o
gcc: warning: '-mcpu=' is deprecated; use '-mtune=' or '-march=' instead
bios/entry.S:1:0: error: bad value (cortex-a15) for -mtune= switch
/*
^
Fix the issue by downloading the toolchain manually and setting
AARCH32_CROSS_COMPILE. buildroot still needs COMPILE_LEGACY=y
to use the proper configuration file [2], and common.mk still needs
LEGACY_AARCH32_CROSS_COMPILE when COMPILE_LEGACY=y so set it too.
Link: [1] https://github.com/OP-TEE/build/pull/263
Link: [2] https://github.com/OP-TEE/build/blob/master/br-ext/configs/toolchain-aarch32-legacy#L3
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
|
|
Now that build files support GCC 4.9 [1] and "make check" supports the
buildroot environment [2], revert commit 793884e19284
("ci: .travis.yml: clone stable version (3.0.0)") and set
COMPILE_LEGACY=y to instruct the build system to download and use the
legacy compiler (gcc-linaro-4.9-2015.02-3-x86_64_arm-linux-gnueabihf).
[1] commit 92582b57dd45 ("Support legacy compiler")
https://github.com/OP-TEE/build/commit/92582b57dd45
[2] commit 4763adc5b75a ("qemu-check.exp: fix expected prompts")
https://github.com/OP-TEE/build/commit/4763adc5b75a
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
|
|
Due to recent changes in manifest.git and build.git for buildroot
support, it is not possible to build with GCC4.9 anymore. One has
to use the toolchain installed by "make toolchains".
Since we want to keep GCC4.9 for now [1], let's revert to an older,
stable version of the source tree, except for the optee_* projects
which we keep at the tip of their master branch.
[1] Commit 148ea708383e ("ci: .travis.yml: use GCC 4.9")
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
|
|
Build with GCC 4.9 in order to detect compile errors specific to that
version of the compiler, such as:
core/tee/tadb.c:593:15: error: missing braces around initializer [-Werror=missing-braces]
const struct tadb_entry null_entry = { 0 };
^
core/tee/tadb.c:593:15: error: (near initialization for ‘null_entry.prop’) [-Werror=missing-braces]
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
Acked-by: Victor Chong <victor.chong@linaro.org>
Acked-by: Joakim Bech <joakim.bech@linaro.org>
|
|
This is the first of two commits to split the CI work in two: Travis
should run checkpatch and the QEMU regression (xtest), while Shippable
should run the platform builds. By doing so, we better utilize the
resources and hopefully minimize build timeouts.
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
|
|
Only test 64bit mode with default configuration
1. Build command
make PLATFORM=marvell-armada3700
2. Pass xtest
Signed-off-by: wangwen <wangwen@marvell.comi>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Kevin Peng <kevinp@marvell.com>
|
|
Added support for armv8 platform flavour.
- PLATFORM = ls-ls1012ardb
Signed-off-by: Pankaj Gupta <pankaj.gupta@nxp.com>
Reviewed-by: Sumit Garg <sumit.garg@nxp.com>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
|
|
Only tested 64-bit mode with default configurations:
1. build command
make PLATFORM=marvell-armada7080 CFG_ARM64_core=y
2. Passed xtest
Signed-off-by: Kevin Peng <kevinp@marvell.com>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
Acked-by: Etienne Carriere <etienne.carriere@linaro.org>
|
|
Added support for armv8 platform flavours as follows:
- PLATFORM = ls-ls1043ardb
- PLATFORM = ls-ls1046ardb
Signed-off-by: Sumit Garg <sumit.garg@nxp.com>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
|
|
Add basic support to get op-tee to run on SAMA5D2-XULT board.
The SoC is based on single core ARM Cortex-A5 and supports:
ARM TrustZone with support for configuring memory/peripherals as secure
Secure RTC
Secure boot
On-the-fly encryption/decryption of DDR bus
Tamper protection
Link: http://www.atmel.com/Images/Atmel-11267-32-bit-Cortex-A5-Microcontroller-SAMA5D2_Datasheet.pdf
Signed-off-by: Akshay Bhat <akshay.bhat@timesys.com>
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
|
|
Travis are currently migrating Linux distributions from Ubuntu Precise
(12.04) to Trusty (14.04). It seems that $HOME/bin already exists in
the new images, which causes an error as we do mkdir $HOME/bin.
Add a -p so that the directory creation doesn't fail when bin
exists. This allows the script to run on both Precise and Trusty.
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Igor Opaniuk <igor.opaniuk@linaro.org>
|
|
Signed-off-by: Joseph Chen <chenjh@rock-chips.com>
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
|
|
The HiKey 960 development platform is based around the Huawei Kirin 960
octa-core ARM big.LITTLE processor with four ARM Cortex-A73 and four
Cortex-A53 cores with 3GB of LPDDR4 SDRAM memory, 32GB of UFS 2.0 flash
storage, and the latest generation Mali G71 MP8 graphics processor.
See https://www.96boards.org/product/hikey960 for more details.
Signed-off-by: Victor Chong <victor.chong@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
|
|
Add i.MX7D support.
- Add register definition
- Add gpcv2 to powerup and powerdown cpu
- Introduce soc runtime detection, the final goal is to support i.MX
family using one image, but still far from it. Now using the runtime
detection, we could remove the CFG_MX[6,7][x] to simplify the code,
such as in imx psci cpu on/off using one function to support 6Q/7D
without CFG_[X].
- Add PSCI cpu/off/affinity
The scripts to build 7dsdb image.
make PLATFORM=imx-mx7dsabresd \
mkimage -A arm -O linux -C none -a 0xbdffffe4 -e 0xbe000000 \
-d out/arm-plat-imx/core/tee.bin uTee-7d
Signed-off-by: Peng Fan <peng.fan@nxp.com>
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
|
|
Add i.MX6 ULL EVK support.
i.MX6ULL is derivative from i.MX6UL, so reuse some code for i.MX6UL.
Signed-off-by: Peng Fan <peng.fan@nxp.com>
Acked-by: Etienne Carriere <etienne.carriere@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
|
|
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
|
|
Removes CFG_SMALL_PAGE_USER_TA and keep the code that was activated by
CFG_SMALL_PAGE_USER_TA=y. This means that CFG_SMALL_PAGE_USER_TA=n which
resulted in TA being mapped using 1 MiB or 2 MiB granularity is removed.
Tested-by: Jens Wiklander <jens.wiklander@linaro.org> (QEMU)
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
|
|
With recent developments in REE-FS SQL-FS has become redundant. This
patch removes SQL-FS.
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Tested-by: Jens Wiklander <jens.wiklander@linaro.org> (QEMU)
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
|
|
Fix "#elif with no expression" error when building with ti-am57xx flavor
Add am57xx to .travic.yml as supported platform
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Andrew F. Davis <afd@ti.com>
Signed-off-by: Igor Opaniuk <igor.opaniuk@linaro.org>
|
|
Signed-off-by: Andrew F. Davis <afd@ti.com>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
|
|
Signed-off-by: Mathieu Briand <mbriand@witekio.com>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Peng Fan <peng.fan@nxp.com>
|
|
Add testing build for vexpress-qemu_virt platform with the secure
data path feature enabled.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
|
|
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
|
|
Update readme and travis for mx6ulevk
Signed-off-by: Peng Fan <peng.fan@nxp.com>
Reviewed-by: Igor Opaniuk <igor.opaniuk@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
[Rebase on top of master]
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
|
|
Signed-off-by: Yan Yan <yan.yan@windriver.com>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
|
|
Enables Travis to perform a Coverity build and scan when a push
is made to the coverity_scan branch. In this case, the regular
build script is not necessary and skipped to save time.
The Coverity build is currently based on the default qemu
configuration.
Signed-off-by: Victor Chong <victor.chong@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
|
|
Acked-by: Joakim Bech <joakim.bech@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
|
|
Add auxilary wrapper for convenient check of commit/
changes in staging area/multiple commits with checkpatch.pl
Put common functions for .travis.yml and checkpatch.sh into a
separate file
Signed-off-by: Igor Opaniuk <igor.opaniuk@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
|
|
stm default plavor is 96board_c2.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
|
|
Workaround a checkpatch bug [1] by creating a non-empty file
const_structs.checkpatch. By adding the name of a struct that cannot
appear in valid code, we avoid the false warning (which occurs only
when no name is given) and this can't produce any unwanted side-effect.
[1] https://www.spinics.net/lists/kernel/threads.html#2364905
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
|
|
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
|
|
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
|
|
The checkpatch.pl script uses a file called const_structs.checkpatch to
check for correct usage of some kernel structures. We don't want those
checks, which are linux-specific, but we still need the file to be
present otherwise the tool will log a warning message [1].
[1] https://travis-ci.org/OP-TEE/optee_os/builds/167271505#L1720
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
|
|
Flavored 'b2260'.
Default no GDB boot, lock pl310, specific DDR size and UART instance.
Tested-by: Etienne Carriere <etienne.carriere@linaro.org> (b2120/b2260)
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
|
|
Signed-off-by: Yan Yan <yan.yan@windriver.com>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Peng Fan <peng.fan@nxp.com>
|
|
Signed-off-by: Joakim Bech <joakim.bech@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
|
|
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
|
|
Reduce verbosity of checkpatch.pl while still allowing warnings and
errors.
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
|
|
When running checkpatch.pl, show the abbreviated SHA1 and the commit
subject instead of the full SHA1 only.
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
|