At present we support specifying SPI flash devices to use in the device
tree and on the command line. Drop the second option, since it is a pain
to support nicely with driver model, and unnecessary.
Signed-off-by: NSimon Glass <sjg@chromium.org>

For debugging it is sometimes useful to write out data for inspection
using an external tool. Add a function which can write this data to a
given file.
Signed-off-by: NSimon Glass <sjg@chromium.org>

At present files are not truncated on writing. This is a useful feature.
Add support for this.
Signed-off-by: NSimon Glass <sjg@chromium.org>

At present to output a log message you need something like:

   log(UCLASS_SPI, LOCL_INFO, "message1");
   log(UCLASS_SPI, LOCL_INFO, "message2");

but many files use the same category throughout. Also it is helpful to
shorten the length of log names, providing helpers for common logging
levels. Add some macros so that it is possible to do:

   (top of file, before #includes)
   #define LOG_CATEGORY UCLASS_SPI

   (later in the file)
   log_info("message1");
   log_debug("message2");
   log_err("message3");
Signed-off-by: NSimon Glass <sjg@chromium.org>

This macro should have two parameters, not one. Fix it so that it
correctly resolves to _ret when logging is disabled.
Signed-off-by: NSimon Glass <sjg@chromium.org>

Of CONFIG_OF_TRANSLATE is enabled, this function still returns
untranslated bogus results. Add the missing translation.
Signed-off-by: NMarek Vasut <marek.vasut+renesas@gmail.com>
Cc: Simon Glass <sjg@chromium.org>
Cc: Tom Rini <trini@konsulko.com>
Reviewed-by: NSimon Glass <sjg@chromium.org>

The size should be decoded using of_n_size_cells(), make it so.
Signed-off-by: NMarek Vasut <marek.vasut+renesas@gmail.com>
Cc: Simon Glass <sjg@chromium.org>
Cc: Tom Rini <trini@konsulko.com>
Reviewed-by: NSimon Glass <sjg@chromium.org>

At present the tests run one after the other using a single CPU. This is
not very efficient. Bring in the concurrencytest module and run the tests
concurrently, using one process for each CPU by default. A -P option
allows this to be overridden, which is necessary for code-coverage to
function correctly.

This requires fixing a few tests which are currently not fully
independent.

At some point we might consider doing this across all pytests in U-Boot.
There is a pytest version that supports specifying the number of processes
to use, but it did not work for me.
Signed-off-by: NSimon Glass <sjg@chromium.org>

We only need to run driver-model tests with this config, since this is the
only thing that is different when CONFIG_OF_LIVE is not defined. Filter
out the other tests to same time.
Signed-off-by: NSimon Glass <sjg@chromium.org>

At present calling Uninit() always called ClearProgress() which outputs
a \r character as well as spaces to remove any progress information on the
line. This can mess up the normal output of binman and other tools. Fix
this by outputing this only when progress information has actually been
previous written.
Signed-off-by: NSimon Glass <sjg@chromium.org>

This variable is not documented or set up in the module. Fix this.
Signed-off-by: NSimon Glass <sjg@chromium.org>

There are a few test cases which print output. Suppress this so that tests
can run silently in the normal case.
Signed-off-by: NSimon Glass <sjg@chromium.org>

At present this test runs binman twice, which means that the temporary
files from the first run do not get cleaned up. Split this into two tests
to fix this problem.
Signed-off-by: NSimon Glass <sjg@chromium.org>

At present 'make check' leaves some temporary directories around. Part of
this is because we call tools.PrepareOutputDir() twice in some cases,
without calling tools.FinaliseOutputDir() in between.

Fix this.
Signed-off-by: NSimon Glass <sjg@chromium.org>

This module is often available in the sandbox_spl build created by
'make check'. Use this as a default path so that just typing 'binman -t'
(without setting PYTHONPATH) will generally run the tests.
Signed-off-by: NSimon Glass <sjg@chromium.org>

The 'result' variable counts the number of failures in running the tests.
Rename it to 'failures' to make this more obvious. Also tidy up a few
comments.
Signed-off-by: NSimon Glass <sjg@chromium.org>

At present the tests are run without any indication of what is running.
For the tests which start with a build this is pretty obvious, but for
tools it is not.

Add a name for each test we run, and print it before starting the test.
Signed-off-by: NSimon Glass <sjg@chromium.org>

Use 'export' to avoid repeating the path setup for each command.
Signed-off-by: NSimon Glass <sjg@chromium.org>

At present we use 'make tests' to run the tests. For many projects
'make check' is more common, so support that as well. Also add some help
to 'make help'.
Signed-off-by: NSimon Glass <sjg@chromium.org>

The filenames of the toolchains on kernel.org changes every now and then.
Fix it for the current change, and make the test use a regex so that it
has a better chance of passing with future changes too.
Signed-off-by: NSimon Glass <sjg@chromium.org>

At present the unicode filtering seems to get confused at times with
this error:

  UnicodeDecodeError: 'ascii' codec can't decode byte 0x80 in position
     32: ordinal not in range(128)

It seems to be due to self._nonprint being interpreted as UTF-8. Fix it
by using ordinals instead of characters, changing the string to set.
Signed-off-by: NSimon Glass <sjg@chromium.org>
Reviewed-by: NStephen Warren <swarren@nvidia.com>
Tested-by: NMichal Simek <michal.simek@xilinx.com>

With Python 2.7.15rc1, ConfigParser.SafeConfigParser has unfortunately
started returning unicode, for unknown reasons. Adjust the code to handle
this by converting everything to unicode. We cannot convert things to
ASCII since email addresses may be encoded with UTF-8.
Signed-off-by: NSimon Glass <sjg@chromium.org>

On my Ubuntu 18.04.1 machine two driver-model bus tests have started
failing recently. The problem appears to be that the DATA region of the
executable is protected. This does not seem correct, but perhaps there
is a reason.

To work around it, unprotect the regions in these tests before accessing
them.
Signed-off-by: NSimon Glass <sjg@chromium.org>

Enables cmd_avb and its dependencies need to run the AVB tests.
Reviewed-by: NSimon Glass <sjg@chromium.org>
Signed-off-by: NJens Wiklander <jens.wiklander@linaro.org>
[trini: Disable for sandbox_noblk]
Signed-off-by: NTom Rini <trini@konsulko.com>

Change get_sector_buf() to use map_sysmem() to get a pointer to the
CONFIG_FASTBOOT_BUF_ADDR in memory.
Signed-off-by: NJens Wiklander <jens.wiklander@linaro.org>
Reviewed-by: NSimon Glass <sjg@chromium.org>

Update the pymark.buildconfigspec to depend on 'cmd_mmc' in addition to
'cmd_avb' for those tests that needs more a more complete MMC
implementation or the "mmc" command.
Reviewed-by: NSimon Glass <sjg@chromium.org>
Signed-off-by: NJens Wiklander <jens.wiklander@linaro.org>

With CONFIG_OPTEE_TA_AVB use the trusted application AVB provided by
OP-TEE to manage rollback indexes and device-lock status.
Reviewed-by: NSimon Glass <sjg@chromium.org>
Signed-off-by: NJens Wiklander <jens.wiklander@linaro.org>

Tests the TEE uclass with a sandbox tee driver.
Reviewed-by: NSimon Glass <sjg@chromium.org>
Signed-off-by: NJens Wiklander <jens.wiklander@linaro.org>
[trini: initialize session to 0 in dm_test_tee]
Signed-off-by: NTom Rini <trini@konsulko.com>

Adds a sandbox tee driver which emulates a generic TEE with the OP-TEE
AVB TA.
Reviewed-by: NSimon Glass <sjg@chromium.org>
Signed-off-by: NJens Wiklander <jens.wiklander@linaro.org>
Reviewed-by: NSimon Glass <sjg@chromium.org>
[trini: Fix printf warnings in ta_avb_invoke_func, slots is uint]
Signed-off-by: NTom Rini <trini@konsulko.com>

Reviewed-by: NSimon Glass <sjg@chromium.org>
Signed-off-by: NJens Wiklander <jens.wiklander@linaro.org>

Adds a sandbox_tee node to enable the sandbox tee driver in all the
sandbox dts files.
Reviewed-by: NSimon Glass <sjg@chromium.org>
Signed-off-by: NJens Wiklander <jens.wiklander@linaro.org>

Adds configuration option OPTEE_TA_AVB and a header file describing the
interface to the Android Verified Boot 2.0 (AVB) trusted application
provided by OP-TEE.
Tested-by: NIgor Opaniuk <igor.opaniuk@linaro.org>
Reviewed-by: NIgor Opaniuk <igor.opaniuk@linaro.org>
Signed-off-by: NJens Wiklander <jens.wiklander@linaro.org>
Reviewed-by: NSimon Glass <sjg@chromium.org>

Adds support in optee supplicant to route signed (MACed) RPMB frames
from OP-TEE Secure OS to MMC and vice versa to manipulate the RPMB
partition.
Tested-by: NIgor Opaniuk <igor.opaniuk@linaro.org>
Signed-off-by: NJens Wiklander <jens.wiklander@linaro.org>
Reviewed-by: NSimon Glass <sjg@chromium.org>

Sync with 14e21cb8f811 ("arm64: dt: hikey: Add optee node"
from Linux kernel.
Reviewed-by: NSimon Glass <sjg@chromium.org>
Signed-off-by: NJens Wiklander <jens.wiklander@linaro.org>

Reviewed-by: NSimon Glass <sjg@chromium.org>
Signed-off-by: NJens Wiklander <jens.wiklander@linaro.org>

Adds a OP-TEE driver.

* Targets ARM and ARM64
* Supports using any U-Boot memory as shared memory
* Probes OP-TEE version using SMCs
* Uses OPTEE message protocol version 2 to communicate with secure world
Reviewed-by: NSimon Glass <sjg@chromium.org>
Tested-by: NIgor Opaniuk <igor.opaniuk@linaro.org>
Signed-off-by: NJens Wiklander <jens.wiklander@linaro.org>

Sync with c8bfafb15944 ("dt/bindings: add bindings for optee")
from Linux kernel.

Introduces linaro prefix and adds bindings for ARM TrustZone based OP-TEE
implementation.
Reviewed-by: NSimon Glass <sjg@chromium.org>
Signed-off-by: NJens Wiklander <jens.wiklander@linaro.org>

Adds a uclass to interface with a TEE (Trusted Execution Environment).

A TEE driver is a driver that interfaces with a trusted OS running in
some secure environment, for example, TrustZone on ARM cpus, or a
separate secure co-processor etc.

The TEE subsystem can serve a TEE driver for a Global Platform compliant
TEE, but it's not limited to only Global Platform TEEs.

The over all design is based on the TEE subsystem in the Linux kernel,
tailored for U-Boot.
Reviewed-by: NSimon Glass <sjg@chromium.org>
Tested-by: NIgor Opaniuk <igor.opaniuk@linaro.org>
Signed-off-by: NJens Wiklander <jens.wiklander@linaro.org>

Adds mmc_rpmb_route_frames() to route RPMB data frames from/to an
external entity.
Tested-by: NIgor Opaniuk <igor.opaniuk@linaro.org>
Signed-off-by: NJens Wiklander <jens.wiklander@linaro.org>
Reviewed-by: NSimon Glass <sjg@chromium.org>

Reviewed-by: NSimon Glass <sjg@chromium.org>
Signed-off-by: NJens Wiklander <jens.wiklander@linaro.org>