Age | Commit message (Collapse) | Author |
|
In tools/ci.py we build rcar platform with aarch64-none-elf-gcc, but
we display it as 'ARM'.
Correct this by replacing ARM with GCC-AArch64.
Change-Id: I7c4de86f6cd4a2a007263703edee7a7c71dfa8f6
Signed-off-by: Nicola Mazzucato <nicola.mazzucato@arm.com>
|
|
This patch disables the Juno Debug module by default to free
up some memory. The option must be re-enabled when debugging.
Change-Id: Ife7f3e2c39c12167b05e9133bf956e04c3f25508
Signed-off-by: Jim Quigley <jim.quigley@arm.com>
|
|
The F_UART3 device used for logging purpose on SynQuacer platform
is not the "always-on device". The synquacer_rom module has a
responsibility to power on the peripherals. This causes unexpected
device access during I/O component initialization and system will
not boot.
Since I/O component aims to enable logging before the module runtime
is available, and current scp_romfw does not rely on logging feature
for debugging, this patch removes the scp_romfw logging feature.
This patch also removes the possible invalid access by uninitialized
pointer in f_uart3 module.
Change-Id: I47ba9d57925a09a79f64a037d30da6b7e30e1f84
Signed-off-by: Masahisa Kojima <masahisa.kojima@linaro.org>
|
|
With the use of CMSIS from Commit afd75a9 ("Refactor the architecture
layer"), ROM firmware is expected to set the vector table offset
register to the RAM firmware's vector table before jumping to the RAM
firmware. This patch sets the VTOR register to the RAM firmware's vector
table for synquacer platform.
Change-Id: I720fe8249b0d40a56028af0c26d00ef1aa2156d2
Signed-off-by: Masahisa Kojima <masahisa.kojima@linaro.org>
|
|
With the premise that a clock rate is 64 bits long, this commit
fixes the behaviour where the Juno HDLCD clock driver takes only
the lower 32 bits of the rate ignoring the remaining upper half.
With the assumption that only 32 bits are therefore required,
this commit includes a check whether the upper 32 bits are zero,
enforcing this requirement and returning FWK_E_PARAM if any
provided rate is bigger than 32 bits.
Change-Id: I0c72432a91d9e4a43c44b36e343a884058ebada9
Signed-off-by: Luca Vizzarro <Luca.Vizzarro@arm.com>
|
|
As per SCMIv2 specifications section 4.3.2.4, when querying the
power domain attributes, the attribute at bit 31 must be set
when the queried domain supports notifications. Because DEVICE
and DEVICE_DEBUG types of power domain domain support
notifications, these must set this bit when their attributes are
queried. This commit enforces this behaviour.
Change-Id: Ib307e61a86cdf0fcd63a91f982ce5bbcce39d013
Signed-off-by: Luca Vizzarro <Luca.Vizzarro@arm.com>
|
|
One of Juno's clocks is the I2S clock, which is a peripheral
unsupported by the Juno FVP model. This commit brings an
integration of Juno with the mock clock, so that this
can be mocked and tested while running under an FVP.
Change-Id: I726817562788842e190deb2bedfca99ae8503ef8
Signed-off-by: Luca Vizzarro <Luca.Vizzarro@arm.com>
|
|
The mock clock module allows to smoothly test the firmware while
using an emulated system, e.g. FVP. Some clock device drivers
require hardware features, such as I2C, in order to function
correctly. Under an emulated environment, these features cannot
be always reproduced, so the mock clock module allows to replace
these clock drivers for a smooth operation.
Change-Id: I022c56cdf0cadb062742f83d413fa9bdf4adc5aa
Signed-off-by: Luca Vizzarro <Luca.Vizzarro@arm.com>
|
|
Set CCIX speed to GEN3 to work with interposer C2C setup
In this use case set downstream/upstream port transmitter
preset value to 4.
Signed-off-by: Jagadeesh Ujja <jagadeesh.ujja@arm.com>
Change-Id: I635566574babbb7591b5bba97105f76f1f8e550a
|
|
This patch updates morello_system module such that SCC BOOT_GPR2 and
BOOT_GPR3 register values are used to set the AP core RVBAR register.
If both BOOT_GPR2 & BOOT_GPR3 registers are set to 0 then a default
Trusted SRAM based is used for RVBAR.
Change-Id: I37620a6733ee02a163dbdb98f51f0e4fc8e9bec7
Signed-off-by: Manoj Kumar <manoj.kumar3@arm.com>
|
|
This patch makes ROM code flexible in a way user can load FIP
capsule image in any memory that SCP can access and set the
SCC BOOT_GPR0 to that memory such that ROM code reads SCC
BOOT_GPR0 and uses that as FIP base if it has a non-zero value.
Change-Id: Iec4bb2fbc81ed48766680813d2dc0721e7c9ac86
Signed-off-by: Manoj Kumar <manoj.kumar3@arm.com>
|
|
This patch modifies the fip module such that the base address
used by the fip module becomes flexible and is passed by the
caller module (n1sdp_rom/morello_rom) in runtime.
This change also removes the config_fip.c file to avoid using
single fixed base address set at build time. These parameters
will be passed as arguments to get_entry function from the
caller module which now has the flexibility to dynamically
parse any memory address.
Change-Id: I1c94ca624df4f4557d76a2ba5dacaa743ed43a25
Signed-off-by: Manoj Kumar <manoj.kumar3@arm.com>
|
|
Disable Resource permission for juno in order to reclaim memory used
by this module and enable new features to be developed.
Change-Id: Ic95fabdb50e2224d6b3afa4aea8cefe2245871b8
Signed-off-by: Ahmed Gadallah <ahmed.gadallah@arm.com>
|
|
Errors fixed when "BS_FIRMWARE_HAS_RESOURCE_PERMISSIONS=no":
1) When an agent of type PSCI sends a "Base protocol: Protocol
attributes" command, the number of all protocols supported by the SCP
is returned instead of the number of protocols available for PSCI
agents only.
2) When an agent of type PSCI sends a "Base protocol: Discover list
protocols" command, all protocols supported by the SCP are returned
instead of the protocols available for PSCI agents only.
Signed-off-by: Ahmed Gadallah <ahmed.gadallah@arm.com>
Change-Id: Ib0a4930216d8da29c79c6724bd74cb51d5184b5b
|
|
This change fixes an undefined variable error in the get_value function.
Change-Id: Ib289094dcd945057505565e1179f6f4cf561de11
Signed-off-by: Leandro Belli <leandro.belli@arm.com>
|
|
The SCMI v2 specs defines that the RATE_SET command flags bits 31:4 are
reserved and must be zero. This commit fixes this behaviour by returning
INVALID_PARAMETERS as expected when any of these are set.
Change-Id: Ib217bbfa4aa52f1c9051906bb88db76e0422ec1c
Signed-off-by: Luca Vizzarro <Luca.Vizzarro@arm.com>
|
|
Whenever a function would return the status value FWK_E_PARAM, the
set_request_respond function, which is in charge of responding to
requests in certain cases, would respond with SCMI_GENERIC_ERROR. This
is an erratic behaviour, as there is a specific SCMI status code,
SCMI_INVALID_PARAMETERS, which is especially dedicated for parameters
error as in FWK_E_PARAM. This commit fixes this behaviour by adding
FWK_E_PARAM to the conditions.
Change-Id: Ib2b15294433cae90ede71ebf00f1b6ad868db914
Signed-off-by: Luca Vizzarro <Luca.Vizzarro@arm.com>
|
|
The configuration of the SCMI clock module for Juno is set to
hold a stopped state for all the configured clock elements.
Nonetheless, Juno actually starts all the clocks at startup,
causing a contradiction in the configuration. This commit fixes
this issue, by setting Juno's clock states to RUNNING at
startup.
Change-Id: Id315c909b06db40ea3b46bd242d05cafbeacbf33
Signed-off-by: Luca Vizzarro <Luca.Vizzarro@arm.com>
|
|
At startup all clocks may be running by default. Nonetheless, the
mod_scmi_clock_config_set_policy function, which is the API the
CLOCK_CONFIG_SET command refers to, sets its clocks state to be
stopped by default. This commit fixes this behaviour, by giving
the option per platform through its configuration to set the
state of the clocks at startup (running/stopped).
Change-Id: If39a2d710e4d778b10b649d07f05cfe17ad29e54
Signed-off-by: Luca Vizzarro <Luca.Vizzarro@arm.com>
|
|
The CLOCK_RATE_GET permissions are mistakenly set to
deny requests to clock ids 0 to 3 for OSPM. This is
only the case with CLOCK_RATE_SET. This commit fixes
this issue and enabling CLOCK_RATE_GET access to all
the clock ids.
Change-Id: I20abc234d49cbf30aa5c50713f9f03f339886c99
Signed-off-by: Luca Vizzarro <Luca.Vizzarro@arm.com>
|
|
When the rate_index parameter is set to be out of bounds
of the clock entries array, the return value should be
OUT_OF_RANGE according to the SCMIv2 spec. This commit
changes the return value from INVALID_PARAMETER to the
correct one.
Change-Id: I3f002823a3c9c710dd809953793901fbcb08fe4f
Signed-off-by: Luca Vizzarro <Luca.Vizzarro@arm.com>
|
|
SCMI v2 specifies that any reserved bits in command's attributes
must be zero. This commit enables compliance to CLOCK_CONFIG_SET
Change-Id: I5de91df28c2f12a26f0e10e797a1335c5be0c417
Signed-off-by: Luca Vizzarro <Luca.Vizzarro@arm.com>
|
|
As per section 4.8.1 of the SCMI v2 spec, the only valid reset id for
the Architectural reset state type is 0. All the other IDs available in
the 32-bit space are reserved for future use. This commit brings an
early exit condition that returns SCMI_INVALID_PARAMETERS when it
detects that any of those ids are used.
Change-Id: I81d7aee541e7577e58e242da6bebf14edd00b753
Signed-off-by: Luca Vizzarro <Luca.Vizzarro@arm.com>
|
|
As per section 4.8.2.6 of the SCMI v2 spec, the request notify_enable
parameter bits 31 to 1 must be set to zero. With this, this commit
enforces this requirement by returning SCMI_INVALID_PARAMETERS whenever
any of these bits are set.
Change-Id: Ic691254da83941d2179befc3e9209c9dbb7550ff
Signed-off-by: Luca Vizzarro <Luca.Vizzarro@arm.com>
|
|
As per section 4.1.4 of the SCMI v2 spec, the correct error code to
return whenever an invalid command is requested is SCMI_NOT_FOUND. The
original behaviour of the reset domain protocol was to return
SCMI_NOT_SUPPORTED. This commit fixes
this mistake.
Change-Id: Iee9eacf1fd71352be88ed0299d155f33842cd56c
Signed-off-by: Luca Vizzarro <Luca.Vizzarro@arm.com>
|
|
As per section 4.8.2.5 of the SCMI v2 spec, the request flags
bits 31 to 3 must be set to zero. With this, this commit
enforces this requirement by returning SCMI_INVALID_PARAMETERS whenever
any of these bits are set.
Change-Id: Ic5c327a78e6864fd759864b9b2c780aac2d021d6
Signed-off-by: Luca Vizzarro <Luca.Vizzarro@arm.com>
|
|
The protocol version for the reset domain protocol should be 0x10000 as
specified in section 4.8.2.1 of th SCMI v2 spec. This was erroneously
set to 0x20000 instead.
Change-Id: I14a5e8ba26a520200bb6303bbd891c790e81f3cc
Signed-off-by: Luca Vizzarro <Luca.Vizzarro@arm.com>
|
|
The POWER_STATE_NOTIFY command erroneously denies access to the PSCI
agent when there is no explicit declaration in the SCMI v2 spec that its
access is indeed denied. This commit removes this condition, therefore
enabling the PSCI aent to access this command.
Change-Id: Ie356def6b313091f3ba113bd806be21adeac9d09
Signed-off-by: Luca Vizzarro <Luca.Vizzarro@arm.com>
|
|
The SCMI power domain protocol does not support notifications
for all the the domains. Specifically, only DEVICE type ones do.
The code originally correctly picked this up and returned with an error,
with the exception that this error was SCMI_DENIED. As per section 4.1.4
of the SCMI v2 spec, this error is dedicated to permissions-related
problems. As this is not the case, this
commit replaces this error with a more suitable SCMI_NOT_SUPPORTED.
Change-Id: I537391bcce57a7849dae2ed26924ca1ad0b1016b
Signed-off-by: Luca Vizzarro <Luca.Vizzarro@arm.com>
|
|
When SCMI notifications are supported a timer is required to handle
graceful request. If this feature is not used timer_id must be
set to FWK_ID_NONE.
This patch also adds check for FWK_ID_NONE to prevent a binding
with no configuration set.
Change-Id: I25f982c2e8d686ac80dd0ebc2d9e64eac1522f3c
Signed-off-by: Leandro Belli <leandro.belli@arm.com>
|
|
Change-Id: Ie9d6dff2a443b5231803d35e6b584879e70e91b9
Signed-off-by: Nicola Mazzucato <nicola.mazzucato@arm.com>
|
|
Change-Id: Ifb167c12f4140f8e50220af93a7852af01af5904
Signed-off-by: Nicola Mazzucato <nicola.mazzucato@arm.com>
|
|
A previous commit [1] removed the status error when the margin is out of
margin. In fact this error is required to be returned so the HAL and its logic
to handle the error can take place.
Restore the error.
[1] 7b3bf3a2fc55d4a8f1a11ea416d791652f89e5da
Change-Id: Ieeaaaf23eaea85d8bfc26f0d74281f6c6114aed3
Signed-off-by: Nicola Mazzucato <nicola.mazzucato@arm.com>
|
|
This patch adds the release notes for v2.7.0. The change log lists the
major new features in the release.
Change-Id: Ia6ed47dcecf7936fd83fed8eb3acdc1ac1f5f76b
Signed-off-by: Jim Quigley <jim.quigley@arm.com>
|
|
This patch adds the rcar platform to the CI tests.
Change-Id: I3a83f6386ad1fcde2df07fe58e249a662b8b1760
Signed-off-by: Jim Quigley <jim.quigley@arm.com>
|
|
This patch adds support for the rcar platform running on FreeRTOS.
FreeRTOS source code is temporarily included into rcar product directory
until armv8-a support is added to CMSIS-FreeRTOS repository.
Change-Id: Ia828f903d52df236922fe7f6f548bce06ee131cc
Signed-off-by: Tsutomu Muroya <tsutomu.muroya.jy@bp.renesas.com>
Signed-off-by: Nicolas Royer <nroyer@baylibre.com>
|
|
Change-Id: I4ba1332eeea2fd19d2ae60281e0894a1c262ea58
Signed-off-by: Tsutomu Muroya <tsutomu.muroya.jy@bp.renesas.com>
Signed-off-by: Nicolas Royer <nroyer@baylibre.com>
|
|
The armclang toolchain does not support S-Records.
Change-Id: I9de19e9c857e7437782b959d583ce1fdb8188652
|
|
This patch adds the maintainers for the rcar platform.
Change-Id: I794005a6cd2148fc2431029ca67c53d63e7302c1
Signed-off-by: Nicolas Royer <nroyer@baylibre.com>
|
|
Change-Id: I4267a31991d36fd7b63b13a7cdb16ab28dda1c2d
Signed-off-by: Tsutomu Muroya <tsutomu.muroya.jy@bp.renesas.com>
Signed-off-by: Nicolas Royer <nroyer@baylibre.com>
|
|
Change-Id: I8ec83b7f1c9c58c2937a990e8043a6aa7a0f4588
Signed-off-by: Tsutomu Muroya <tsutomu.muroya.jy@bp.renesas.com>
Signed-off-by: Nicolas Royer <nroyer@baylibre.com>
|
|
Change-Id: I3c1d31d82a6bdd8ad490ff1f0e609d93ebcd5da0
Signed-off-by: Tsutomu Muroya <tsutomu.muroya.jy@bp.renesas.com>
Signed-off-by: Nicolas Royer <nroyer@baylibre.com>
|
|
Change-Id: I59ba3eb358fcfc9b76e0b7daa7621abfaeec5028
Signed-off-by: Tsutomu Muroya <tsutomu.muroya.jy@bp.renesas.com>
Signed-off-by: Chikara Asou <chikara.aso.uj@bp.renesas.com>
Signed-off-by: Nicolas Royer <nroyer@baylibre.com>
|
|
Change-Id: I44796fe0ea4c66e90d77c6cd93843914a7978f3b
Signed-off-by: Tsutomu Muroya <tsutomu.muroya.jy@bp.renesas.com>
Signed-off-by: Nicolas Royer <nroyer@baylibre.com>
|
|
Change-Id: Id50d856cdc888badf5dd658496d6d69d2ed3c06f
Signed-off-by: Tsutomu Muroya <tsutomu.muroya.jy@bp.renesas.com>
Signed-off-by: Nicolas Royer <nroyer@baylibre.com>
|
|
Change-Id: I69b575fbeaeb33fc1b76618d3d516c26e6c9a793
Signed-off-by: Tsutomu Muroya <tsutomu.muroya.jy@bp.renesas.com>
Signed-off-by: Nicolas Royer <nroyer@baylibre.com>
|
|
Change-Id: Iaec89b1b8d61222d836c02bc31313b3fc98a55f6
Signed-off-by: Tsutomu Muroya <tsutomu.muroya.jy@bp.renesas.com>
Signed-off-by: Nicolas Royer <nroyer@baylibre.com>
|
|
Change-Id: I979fecb8b71fb7a1f962574d8bc6f439e7bfb68d
Signed-off-by: Tsutomu Muroya <tsutomu.muroya.jy@bp.renesas.com>
Signed-off-by: Nicolas Royer <nroyer@baylibre.com>
|
|
Change-Id: I5c9e5c41af22725d70bf57fd84b83138b07d61f7
Signed-off-by: Tsutomu Muroya <tsutomu.muroya.jy@bp.renesas.com>
Signed-off-by: Nicolas Royer <nroyer@baylibre.com>
|
|
Change-Id: Id87daed4990c2af79f608f74438d6062915d74f1
Signed-off-by: Tsutomu Muroya <tsutomu.muroya.jy@bp.renesas.com>
Signed-off-by: Nicolas Royer <nroyer@baylibre.com>
|