This reverts commit 9e38935f.

Reason for revert: Nightly test issue.

Change-Id: I68af7380bd5aa3f9be8363b52b261b1cf310cd59

Update test version to support eRPC framework.

Signed-off-by: Summer Qin <summer.qin@arm.com>
Change-Id: I925c3088ded609e5a5ccffbf734fc07b21771c00

Add interface for eRPC transport in musca_s1 and an521.

Signed-off-by: Summer Qin <summer.qin@arm.com>
Change-Id: I1ac8b20472c5f71a5e166353c183068c07b19f48

Add cmake config for UART_TX_RX_CONTROL_ENABLED in NS side.

Change-Id: I09ce4281148c136391dca169433299ac61f19d03
Signed-off-by: Kevin Peng <kevin.peng@arm.com>

This patch enables callers of thrd_start() to provide a parameter that
will be passed to the thread entry function.
Previously this parameter was hard-coded to be NULL.
This is needed for further SPM modifications.

Change-Id: I5e0d5278867d26bc79b39a8594c3d6ef0eb1e561
Signed-off-by: Chris Brand <chris.brand@cypress.com>
Signed-off-by: Bohdan Hunko <Bohdan.Hunko@infineon.com>

Anton Komlev authored Feb 27, 2023 and Anton Komlev committed Mar 06, 2023

- Add Copyright placement guidance
- Clarify licencing information
- Remove a small typo

Signed-off-by: Anton Komlev <anton.komlev@arm.com>
Change-Id: I4c7cd0dd5607d6e60f6d7f1e87adefcc20006ba0

Define the type psa_key_attributes_t in the same way as Mbed TLS, as a
struct psa_key_attributes_s which has different definitions on the
client and server sides. This avoids needing to patch upstream
headers.

Signed-off-by: Gilles Peskine <Gilles.Peskine@arm.com>
Change-Id: I240349c725facca6e353e309cccfabda289adb46

This is also closer to the newly changed upstream code style in Mbed TLS.

Signed-off-by: Gilles Peskine <Gilles.Peskine@arm.com>
Change-Id: I8bed02a4a8a6b0fbfe18b96862c9e23979378154

The cmsis_includes_s target is linked into the other cmake  targets
which need the global compiler and linker options as well.

Change-Id: Id6d24b07500796f70284aac545b94c3643d046e9
Signed-off-by: Bence Balogh <bence.balogh@arm.com>

There are two compile definitions for PSA API IPC test
- TEST_PSA_API_IPC
- PSA_API_TEST_IPC

The first one is newly added, probably to align with the Kconfig
option.
Whilst they are two different things.
The old compile definition (the second one) should be used to avoid
duplication.

This patch addresses this issue.
Meanwhile align the crypto one with IPC as well.

Change-Id: I10a09f8a49a725de086f73c5f765a54152e4bb89
Signed-off-by: Kevin Peng <kevin.peng@arm.com>

Signed-off-by: Gilles Peskine <Gilles.Peskine@arm.com>
Change-Id: Ia62b977d5f1893bfe82989983b4887dad327d7d2

Remove redundant namespace 'PARTITION' to make it shorter.
And updated the flags bit assignment explanation comments for easier understanding.

Signed-off-by: Summer Qin <summer.qin@arm.com>
Change-Id: I218e08b945309017f07e2e595a3c684e00be4bdf

For unified referencing and clearly showcasing SPM required
memory symbols. Memory symbols can come from linker sections,
or be defined in sources with toolchain-specific attributes.

Signed-off-by: Ken Liu <Ken.Liu@arm.com>
Change-Id: I920329ad08c9515e370653a9c5d663f3970b7395

Change-Id: I86ff01413c4191ae87f9d8353a5aff72485a7117
Signed-off-by: David Hu <david.hu@arm.com>

Update configuration docs to align with changes which decouples option
configuration and validation in config header files.

Convert header file config system diagrams in svg format
It is difficult to modify existing figures in header_file_system.rst and
kconfig_header_file_system.rst.
Re-draw those figures in svg format and enable developers to more easily
update the diagram.

Change-Id: If88a6a07104ef54de127ccb9149c5ff4792b3cdd
Signed-off-by: David Hu <david.hu@arm.com>

Option settings and checks are mixed in config_<component>.h.
config_<component>.h is then included by other components outside the
current one.
Those checks often depends on build options managed in CMake/Kconfig
config. Many of these build options are transparent to other components.
For example, if a <component> specific build option is transparent to
NSPE, when NS is built separatedly from TF-M secure side, this build
option is missing in NS build and then the corresponding checks in
config_<component>.h will fail the build.

Decouple option settings and checks to solve the issue above.
- Add #ifndef/#endif pairs in config_base.h to collect all the base
  option settings.
- Let config_tfm.h include config_base.h. It becomes the common config
  header file to be included by all components.
- Only keep invalid config check in config_<component>.h. Renamed
  config_<component>.h as config_<component>_check.h.
  Users shall explicitly set config values in PROJECT_CONFIG_HEADER_FILE
  rather than relying on components to implicitly set default values.
- config_<component>_check.h is only included by <component> for check
  only.

Change-Id: I8e447a41dee515be53f8c7458f4ada3a965ad566
Signed-off-by: David Hu <david.hu@arm.com>

rt_main is an unnecessary layer that can be removed. Instead partition
entry point function can be called directly.
This simplifies the code and enables passing a parameter to the thread
using register r0. Previously passing parameter using r0 was impossible
because rt_main used r0.

Change-Id: I5e0d5278867d26bc79b39a8594c3d6ef0eb1e56d
Signed-off-by: Bohdan Hunko <Bohdan.Hunko@infineon.com>
Signed-off-by: Chris Brand <chris.brand@cypress.com>

Signed-off-by: Sherry Zhang <sherry.zhang2@arm.com>
Change-Id: I88cbac33905545784039ec62c2663402ca382c28

Create the corstone1000 flash image with the gpt layout via a script
The script uses dd command to create the disk image
and uses the sgdisk tool to write the gpt layout
dd is used again to write binaries to the disk image

Signed-off-by: Mohamed Omar Asaker <mohamed.omarasaker@arm.com>
Change-Id: I38c34f27c1bad1d450ca54b4fffd24a66ccd4746

Complete the simplification of the flash layout.
The flash layout contains only the static definitions
that describe the static layout and the boundries of the dynamic
regions.

The dynamic regions addresses are known by the GPT parser.

Signed-off-by: Mohamed Omar Asaker <mohamed.omarasaker@arm.com>
Change-Id: I2bbbd2ff64962b906ec723ffcd082ac1cc92d769

Adabt BL2 to use GPT parser find tfm and fip partitions, and then
extract info to populate MCUBOOT flashmap.

Side changes required:
Borrow 2k of BL2 code memory to Data memory (during linking)
i.e. Increase BL2_DATA_GAP_SIZE and decrease SE_BL2_PARTITION_SIZE

Signed-off-by: Mohamed Omar Asaker <mohamed.omarasaker@arm.com>
Change-Id: I62bdabaf13510d4a63d95643831ec5f48c9ca906

The commit prepares BL1 to adapt to new GPT-based flash layout.

BL1 does not incorporate a GPT parser and still uses a static
configuration to understand the flash.

The flash_layout.h is also modified/marked in a way to start
the process of its simplification.

Signed-off-by: Satish Kumar <satish.kumar01@arm.com>
Signed-off-by: Mohamed Omar Asaker <mohamed.omarasaker@arm.com>
Change-Id: Ie33803dff9f2729e45ee3b72f694a9929a4395d6

Removes the Strata flash driver from RSS and uses the Arm common one
instead.

Signed-off-by: Jamie Fox <jamie.fox@arm.com>
Change-Id: I30269df3d08bb96b9fc95140549bdb9b360fbbab

Removes the RSS CMSDK UART driver and uses the common Arm one instead.
Also propagates a small fix from the RSS CMSDK UART driver to the Arm
one.

Signed-off-by: Jamie Fox <jamie.fox@arm.com>
Change-Id: Ifc4fc28aeeed35d47ee4acc1201fdaa742eef6f1

Replaces RSS MPC drivers with the common Arm implementations.

Signed-off-by: Jamie Fox <jamie.fox@arm.com>
Change-Id: I339f7b8bf6d26ff8f7fb1733f3727f95eb31879f

The Kconfig tool now supports loading config files and merging them
into one. The merging is basically done by the later loaded config files
overriding the first loaded config files with dependencies respected.

With this feature, the Kconfig system will be aware of the configs such
as platform settings and profile settings rather than having default
values for all configurations.

The environment variables are for the Kconfig files which refer to env.
variables, for example the PLATFORM_PATH.
Now the PLATFORM_PATH is no longer passed as one of the arguments, it's
done via the --env argument now. Of course, any other env. variables can
be appended at will.

Change-Id: Ib39e13b29def382a9546cd7680c62a4f4d321f99
Signed-off-by: Kevin Peng <kevin.peng@arm.com>

The Kconfig system now has a separated routine rather than integrated
into the CMake Config system.
It is now easy to adjust the config steps for the Kconfig system without
impacting the CMake system.

Change-Id: I41e088a571448ab74ab9e669d73a623bc9af5e51
Signed-off-by: Kevin Peng <kevin.peng@arm.com>

CMake config and Kconfig system will be separated later.
Moving out some common codes from set_config.cmake

- Add pre_config.cmake for common codes
  - Move out platform preload.cmake to pre_config.cmake
  - Move out Library Model deprecation message to pre_config.cmake
  - Move out CMAKE_BUILD_TYPE setup to pre_config.cmake
  - Error checks (Library Model, TFM_SYSTEM_DSP and CMAKE_GENERATOR) are
    gathered together into pre_config.cmake
- Add post_config.cmake for common codes
  - Move out the tfm_config setup from set_config.cmake to
    post_config.cmake.
  - Move out tfm_build_log_config.cmake inclusion to post_config.cmake
  - Move test repo fetch and config are separated and move to
    post_config.cmake

Change-Id: Idf39766d4f7c3e3933443c7eef31b9e0479064ea
Signed-off-by: Kevin Peng <kevin.peng@arm.com>

ER_EFLASH_DRIVER_RO section name is replaced by ER_CODE_SRAM.
So, the related definitions need to be updated too.

Signed-off-by: Summer Qin <summer.qin@arm.com>
Change-Id: I7f899595bdfe89e6f6bad09c655feb3db22f81ae

Removes the RSS timer, counter and MPU native drivers and uses the
equivalent ones in the common Arm drivers directory instead.

Signed-off-by: Jamie Fox <jamie.fox@arm.com>
Change-Id: I74940d22bfa7efb5065d058e3cb44056f78c27c4

The previous PSA API redirecting is statically performed in header
files, based on C preprocessors exported by the configuration system.
The C preprocessors make the configuration maintenance harder, and
users could not find the exact PSA API name in debug symbols, which
is confusing.

Now the redirecting turns into a runtime behavior. SPRTL calls
corresponded ABI for PSA API. The ABI is set in the Secure Partition
local storage by SPM when booting, based on Secure Partition types.
Check 'prv_process_metadata' and 'psa_api_ipc.c' for details.

Co-authored-by: Ken Liu <ken.liu@arm.com>
Signed-off-by: Sherry Zhang <sherry.zhang2@arm.com>
Change-Id: Ifc974941ddb6653389d9deca9f18f754b8f25b27

Adding GPT parser
Side changes required:
Includes the implementation of the `plat_get_image_source` function
in the platform.c file.

The GPT parser requires the function. Given the image id, it should
return handle to the IO device contains the image and image
specification that allows IO storage access to the image.

Signed-off-by: Mohamed Omar Asaker <mohamed.omarasaker@arm.com>
Change-Id: I3ff32c83fbc589c993f3f069985c0bae9efdad0a

THRD_EXPECTING_SCHEDULE() calls thrd_next() to decide whether
scheduling is required and when it is required, PendSV will
call thrd_next() again in ipc_schedule(). This duplicated
call of thrd_next() can be avoided by returning a status of
STATUS_NEED_SCHDULABLE from backend in these locations:

 - SVC_Handler
 - SPM cross API
 - Interrupt handler

Signed-off-by: Jianliang Shen <jianliang.shen@arm.com>
Change-Id: I3f1fccbb045938624089e856b5a98c3365c76fcc

Since Driver_GFC100_EFlash.c file uses definitions from device_definition.c.
It is better to split the gfc100 eflash defition from device definition to
mitigate the library link nesting issue.

Signed-off-by: Summer Qin <summer.qin@arm.com>
Change-Id: I7a39b43c2a7c7ce7505d4671a16714b41dd98dda

Remove duplicated definitions of dual-core memory access check flags.

Signed-off-by: David Hu <david.hu@arm.com>
Change-Id: Id834292d604c120833e2e10a352ca382e7051d2b

Update multi-core memory access check document according to the latest
implementation.

Signed-off-by: David Hu <david.hu@arm.com>
Change-Id: I901d80bf2a665509596b8c245bbeb0ff70bd5f14

Using phrase 'local storage' to replace 'scratch' and 'metadata'
and rename the file to remove the redundant namespace.

Signed-off-by: Ken Liu <Ken.Liu@arm.com>
Change-Id: I84bc9762e0da0aefee83039623cded05761f1d7d

platform_log defines log messages macros to be used by the platform code
It allows defining the module name to be added at the beginning of the log
message.
Based on build type PLAT_LOG_LEVEL is defined.
In case of Debug/RelWithDebInfo PLAT_LOG_LEVEL is defined to Debug level
else it is defined to OFF.

usage in source file:
...
INFO("msg");
ERROR("msg");
WARN("msg");
VERBOSE("msg");
DEBUG("msg");
...

Signed-off-by: Mohamed Omar Asaker <mohamed.omarasaker@arm.com>
Change-Id: Id730d3de18dcf2bbf37ed974a9403235fcb79f4c

'connection_t' for internal data structure, and 'handle' for
exposing to API callers.

- Basically a straight 'search and replace' is applied.
- Removed doxygen comments for internal APIs.

Signed-off-by: Ken Liu <Ken.Liu@arm.com>
Change-Id: If0282c3effb77518d970d5896dc8073665094bdd

David Hazi authored Nov 28, 2022 and Anton Komlev committed Feb 24, 2023

Signed-off-by: Dávid Házi <david.hazi@arm.com>
Co-authored-by: Mark Horvath <mark.horvath@arm.com>
Change-Id: If2235999552bf5de51a22f30c8e80ae2ca6c4d04