Configure SCMI_RESET_DOMAIN, RESET_DOMAIN, OPTEE_RESET modules to control
OP-TEE resets through the SCMI protocol.

Signed-off-by: Valentin Caron <valentin.caron@foss.st.com>

Configure SCMI_CLOCK, CLOCK, OPTEE_CLOCK modules to control OP-TEE clocks
through the SCMI protocol.

Signed-off-by: Valentin Caron <valentin.caron@foss.st.com>

Add the support of SCP-FW on STM32MP2x.
Configure OPTEE_MBX and MSG_SMT modules to receive SCMI requests.

This patch is a skeleton, protocols will be added in next patches.

Signed-off-by: Valentin Caron <valentin.caron@foss.st.com>

This function is called by OP-TEE before arch_main_init() to configure
dynamically SCP-Fw resources.

OP-TEE parse "scmi" device-tree node to fill the scpfw_config structure
with SCMI resources before calling scmi_configure(). In "scmi" node,
all SCMI resources are described, sorted by protocol, and for each agent.

scpfw_config structure is defined in OP-TEE:
core/lib/scmi-server/include/scmi_agent_configuration.h

Signed-off-by: Valentin Caron <valentin.caron@foss.st.com>

Solve compilation issues get up by gcc by adding void in
function/declaration without any argument:
- error: function declaration isn’t a prototype
- error: old-style function definition

Signed-off-by: Valentin Caron <valentin.caron@foss.st.com>

Promoting a variable of the type uint8_t to uint64_t by casting
would first promote it to a signed integer, before converting it
to uint64_t. This has been fixed by making a separate uint64_t
variable to put the information in.

Signed-off-by: James Chesterman <James.Chesterman@arm.com>

Updated bitmask calculation functions to use UINT64_C(0x1) for shifts, ensuring
64-bit integer operations and preventing overflow issues when shifting large
values.

Signed-off-by: Wei-Chia Su <Wei-Chia.Su@arm.com>

James Chesterman authored May 02, 2025 and Leandro Belli committed May 02, 2025

Add TC3 to list of products to build for both the check_build.py
script and the build-test.yml for the CI.

Signed-off-by: James Chesterman <James.Chesterman@arm.com>

Mahmoud Elsabbagh authored Apr 24, 2025 and Leandro Belli committed May 02, 2025

Remove Vagrant file and all its references for documentation
and docker.

Signed-off-by: Mahmoud Elsabbagh <mahmoud.elsabbagh@arm.com>

Mahmoud Elsabbagh authored Apr 09, 2025 and Leandro Belli committed May 02, 2025

This will fix a potential null dereference in the SCMI module,
to ensure protocol_api is not dereferenced without a check.

Signed-off-by: Mahmoud Elsabbagh <mahmoud.elsabbagh@arm.com>

Fix logically dead code error, where ctx is just
assigned and then checked if its equal NULL, so the if
statement is dead code as ctx is never equall NULL.

Signed-off-by: Mahmoud Elsabbagh <mahmoud.elsabbagh@arm.com>

Attempting to perform a left shift of more than 31 bits causes
undefined behaviour. Add a cast to make the bit shift input 64
bits.

Signed-off-by: James Chesterman <James.Chesterman@arm.com>

There was an entry for power capping in the top level of the index.
This has been fixed so that, like power budget distributor, this
information is in the 'Detailed Description' section of the power
capping entry in the modules folder.

Signed-off-by: James Chesterman <James.Chesterman@arm.com>

A redundant element from scmi perf fast channel context is removed.

Signed-off-by: Md Golam Tanvir Zim <mdgolamtanvir.zim@arm.com>
Change-Id: Ie22283a009e51c757ebf018e3636cb38533242de

A left-over function from previous implementation of fast channel
is removed which is neither built nor needed.

Signed-off-by: Md Golam Tanvir Zim <mdgolamtanvir.zim@arm.com>
Change-Id: Ia02f8d06c9a97e6e377673f0da491863922fc208

There is a piece of code in the above function which will never be
executed and nor required has been removed in this commit.

Signed-off-by: Md Golam Tanvir Zim <mdgolamtanvir.zim@arm.com>
Change-Id: I6531b690c1b5163ab675cd6ef1c0b01156687cce

The create_async_request_set_rate() function creates an event with a
delayed response, returning FWK_PENDING to indicate that the event
was successfully created and the device context updated. Previously,
clock_set_rate() erroneously checked the return value using a
condition that always evaluated to true due to the use of the OR
(||) operator.

This commit fixes the condition so that the debug log is only
triggered if the return value is not FWK_PENDING, ensuring that the
function's behavior correctly reflects the expected return value and
does not produce misleading debug output.

Signed-off-by: Md Golam Tanvir Zim <mdgolamtanvir.zim@arm.com>
Change-Id: Iaa3e0bd2c3937cc3a13a7bb3b7df4a5b75c4d7ba

- Updated .gitlab/templates/linting.yml to use YAML block scalars (|-)
  for all check scripts to avoid introducing leading spaces on line breaks.
- Added commit_hash.strip() in check_style.py and check_copyright.py
  to prevent accidental whitespace in Git arguments.

Signed-off-by: Wei-Chia Su <Wei-Chia.Su@arm.com>

Shriram authored Apr 16, 2025 and Leandro Belli committed Apr 16, 2025

The AxNSE and AxPROT[1] attributes programmed for an ATU region are used
to determine the physical address space(PAS) of the transaction. Other
attributes such as AxCACHE[x] and AxPROT[x] must be set to 'passthrough'
so that the value at the input bus is transferred to the output bus as
is.

The default ATU_ENCODE_ATTRIBUTES macros incorrectly set the other bus
attributes to zero instead of passthrough. So, fix this by updating the
following macros:
- ATU_ENCODE_ATTRIBUTES_ROOT_PAS
- ATU_ENCODE_ATTRIBUTES_SECURE_PAS
- ATU_ENCODE_ATTRIBUTES_NON_SECURE_PAS
- ATU_ENCODE_ATTRIBUTES_REALM_PAS

Signed-off-by: Shriram <shriram.k@arm.com>
Change-Id: I310bd2315feba01c95999813d6d19bdce999f79d

Shriram authored Apr 16, 2025 and Leandro Belli committed Apr 16, 2025

In the ATU manage mode, the output bus attributes for an ATU region is
set in the ATUROBA register without clearing the existing configuration.
In certain scenarios, this may lead to incorrect output bus attributes
being configured for an ATU region when an ATU region with different
output bus attributes had been programmed previously in that register.

So, fix the ATU region output bus attributes programming in the
atu_add_region() function.

Signed-off-by: Shriram <shriram.k@arm.com>
Change-Id: I7627068712aacbfc7715436118c47d8fcbdcae11

The resulting highest level from a composite state was being cast back
and forth from an int to an unsigned int. This patch removes that and
maintains consistency as an unsigned int.

Signed-off-by: Katherine Vincent <katherine.vincent@arm.com>

Include unit tests to verify the correct of the decrementing
state level from composite state.

Signed-off-by: Katherine Vincent <katherine.vincent@arm.com>

The resulting level state from a composite state was being possibly
erroneously decremented. This has been addressed in this patch to only
decrement if not already zero.

Signed-off-by: Katherine Vincent <katherine.vincent@arm.com>

Wei-Chia Su authored Apr 03, 2025 and Leandro Belli committed Apr 16, 2025

Introduces a security scanning job to the templates pipeline that analyzes new
code changes for potential vulnerabilities. The scan results are output in JSON
format, converted into a structured payload, and used to post summary comments
on internal or external GitLab merge requests.

Signed-off-by: Wei-Chia Su <Wei-Chia.Su@arm.com>

Avoid using `origin/main` from the local mirror, which may be outdated.
Instead, fetch main from the public repo and compute `merge-base` from it,
ensuring consistency when comparing against `FETCH_HEAD` from the MR.

Signed-off-by: Leandro Belli <leandro.belli@arm.com>

Katherine Vincent authored Jan 28, 2025 and Leandro Belli committed Apr 16, 2025

This patch introduces unit tests for the newly added Power Capping
Requester module.

Signed-off-by: Katherine Vincent <katherine.vincent@arm.com>

Katherine Vincent authored Jan 15, 2025 and Leandro Belli committed Apr 16, 2025

This patch adds the new requester module for the Power Capping
protocol. It handles one command to set a power cap to a power
domain.

Signed-off-by: Katherine Vincent <katherine.vincent@arm.com>

Mahmoud Elsabbagh authored Apr 09, 2025 and Leandro Belli committed Apr 16, 2025

Fix bad bit shift error, ensure that the bit
shifting happens in 64-bit by casting to uint64_t
before shifting, so there is no chance for
sign-extension being done implicitly.

Signed-off-by: Mahmoud Elsabbagh <mahmoud.elsabbagh@arm.com>

Remove .devcontainer.json file.

Signed-off-by: Mahmoud Elsabbagh <mahmoud.elsabbagh@arm.com>

Girish Pathak authored Apr 02, 2025 and Leandro Belli committed Apr 15, 2025

Aligned doxygen tags used in new modules
with existing codebase conventions.

It also fixes comments that mentions wrong
return values

Signed-off-by: Girish Pathak <girish.pathak@arm.com>

Manoj Kumar Prabakaran authored Apr 09, 2024 and Abhishek Rawat committed Apr 09, 2025

This change fixes the compilation issue in SMCF module's
mgi_validate_header.

Signed-off-by: Manoj Kumar Prabakaran <manojkumar.prabakaran@arm.com>
Signed-off-by: Abhishek Rawat <abhishek.rawat@arm.com>

This patch adds input validation and bounds checking when accessing
child nodes and shared context tables. Specifically:
- Added fwk_assert() checks in cfgm_get_child_node() and
  mxp_get_child_node() to validate NULL pointers and ensure
  child_index is within bounds.
- Added runtime checks in cmn_cyprus_init_node_info() to ensure
  rnsam_entry and ldid values are within valid limits before
  accessing shared context tables.

Signed-off-by: Wen Ping Teh <wenping.teh@arm.com>

Moved get_child_count() to the internal header file,
cmn_cyprus_discovery_setup.h, to enable other CMN-Cyprus modules to
get the child count.

Signed-off-by: Wen Ping Teh <wenping.teh@arm.com>

This change added the necessary check for event_count and allow an early
failure if __fwk_init is called with 0 events as the event queues will
not be usable.

Signed-off-by: Abhishek Rawat <abhishek.rawat@arm.com>

This ensures that a division by zero does not occur.

Signed-off-by: James Chesterman <James.Chesterman@arm.com>

This patch refactors how the agent view of clock states are managed.
Previously, the clock states per agent were stored in a big array
under module context which would occupy unncessary memory not being
in use. In this change, the big array is broken down into smaller
arrays and put under agent context, occupying only necessary memory.

This patch also refactors and renames some clock state related
functions for improved efficiency and readability.

Change-Id: Iaf954165d9baaa570b91885b3367943468b95a6c
Signed-off-by: Md Golam Tanvir Zim <mdgolamtanvir.zim@arm.com>

Removed unnecessary mock functions and added reset functions to
ensure a clean test environment.

Change-Id: Ie56c58ed56ffe18ef100016720f66ba8ba46b1ea
Signed-off-by: Md Golam Tanvir Zim <mdgolamtanvir.zim@arm.com>

Update image reference in check-commit-msg job to use the
CONTAINER_REGISTRY_PATH variable. This allows the pipeline to pull the
image from a configurable location.

Signed-off-by: Wen Ping Teh <wenping.teh@arm.com>

Fix comments by making them get defined when the documentation is
being built.

Signed-off-by: James Chesterman <James.Chesterman@arm.com>

Fix comment by adding return type to the documentation

Signed-off-by: James Chesterman <James.Chesterman@arm.com>