drivers: bcm_hwrng: add HWRNG driver

low level driver for Broadcom random number generator IP.

Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
Signed-off-by: Bharat Kumar Reddy Gooty <bharat.g

drivers: bcm_hwrng: add HWRNG driver

low level driver for Broadcom random number generator IP.

Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
Signed-off-by: Bharat Kumar Reddy Gooty <bharat.gooty@broadcom.com>
Signed-off-by: Sandeep Tripathy <sandeep.tripathy@broadcom.com>

show more ...

plat-rpi3: Use generic memory layout

plat-rpi3 have quite standard memory layout, so there is no sense
to maintain separate configuration if it possible to use generic
one.

Signed-off-by: Ying-Chun

plat-rpi3: Use generic memory layout

plat-rpi3 have quite standard memory layout, so there is no sense
to maintain separate configuration if it possible to use generic
one.

Signed-off-by: Ying-Chun Liu (PaulLiu) <paulliu@debian.org>
Acked-by: Etienne Carriere <etienne.carriere@linaro.org>
Acked-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>

show more ...

stm32mp1: shres: set GPIO secure hardening

Set secure hardening for the GPIOZ pins according to their
peripheral registration.

Signed-off-by: Etienne Carriere <etienne.carriere@st.com>
Acked-by: Je

stm32mp1: shres: set GPIO secure hardening

Set secure hardening for the GPIOZ pins according to their
peripheral registration.

Signed-off-by: Etienne Carriere <etienne.carriere@st.com>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

stm32mp1: shres: configure ETZPC protection

With this change, platform configures the ETZPC firewall
according to shared peripheral being assigned to either the
secure or the non-secure world.

Sign

stm32mp1: shres: configure ETZPC protection

With this change, platform configures the ETZPC firewall
according to shared peripheral being assigned to either the
secure or the non-secure world.

Signed-off-by: Etienne Carriere <etienne.carriere@st.com>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

stm32mp1: shres: check RCC secure hardening

This change add a platform consistency test between shared
resource registering and SoC RCC hardening.

When secure resources are registered, RCC secure h

stm32mp1: shres: check RCC secure hardening

This change add a platform consistency test between shared
resource registering and SoC RCC hardening.

When secure resources are registered, RCC secure hardening
must be enabled unless what secure world cannot guaranty
the resource reliability.

Signed-off-by: Etienne Carriere <etienne.carriere@st.com>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

stm32mp1: shres: secure clock parents

Add API function stm32mp_register_clock_parents_secure().
The function registers as secure the parent clock(s) of the
target clock reference. This API is used b

stm32mp1: shres: secure clock parents

Add API function stm32mp_register_clock_parents_secure().
The function registers as secure the parent clock(s) of the
target clock reference. This API is used by shared_resources.c
when a clock is registered as secure so that its dependencies
are also registered as secure.

Signed-off-by: Etienne Carriere <etienne.carriere@st.com>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

stm32mp1: shres: registering shared resources

This change implements a driver for the stm32mp1 resources that may
be assigned to either secure or non-secure worlds upon the platform
configuration.

stm32mp1: shres: registering shared resources

This change implements a driver for the stm32mp1 resources that may
be assigned to either secure or non-secure worlds upon the platform
configuration.

Other drivers shall register their resources (when applicable) using
the API functions stm32mp_register_{secure|non_secure}_periph*():
- stm32mp_register_*_periph() registers a resource from its
platform ID.
- stm32mp_register_*_periph_iomem() registers a resource from its
IOMEM base address.
- stm32mp_register_*_periph_gpio() registers a resource from its
GPIO reference, bank and position.

Shared resource driver exports some APIs to query a resource
registration state, stm32mp_periph_is_*(),
stm32mp_gpio_bank_is_*(), stm32mp_clock_is_*().

The driver saves the peripheral assignation. The API does not
allow peripherals to change state at runtime. Moverover, to
prevent testing a resource status before it is registered,
the first query on a resource state locks further registering.
Later attempt to register a peripheral will panic the core.

Resources are either secure on non-secure but clock that maybe
shared in which case it will be assigned to the secure world but
a platform service will allow non-secure to access the resource
(i.e. enable/disable the clock). Note such service is out of the
scope of this change, yet this explains API stm32mp_clock_is_shared().

Signed-off-by: Etienne Carriere <etienne.carriere@st.com>
Signed-off-by: Nicolas Le Bayon <nicolas.le.bayon@st.com>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

stm32_gpio: fix inline comment on GPIO bank verification

Local function ckeck_gpio_bank() panics if expected conditions
are not met. This change corrects inline comment that state
the function retur

stm32_gpio: fix inline comment on GPIO bank verification

Local function ckeck_gpio_bank() panics if expected conditions
are not met. This change corrects inline comment that state
the function returns a error code.

Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

stm32_i2c: handle pinctrl

Get pinctrl support from stm32_gpio.h into STM32 I2C driver.
When device tree content defines pins related to an I2C interface,
the I2C driver saves the pins configuration

stm32_i2c: handle pinctrl

Get pinctrl support from stm32_gpio.h into STM32 I2C driver.
When device tree content defines pins related to an I2C interface,
the I2C driver saves the pins configuration instances and set the
registered pins in the expected power mode at runtime.

Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

MAINTAINERS: update Broadcom reviewer list

Add Sandeep Tripathy <sandeep.tripathy@broadcom.com> as reviewer.
bcm-optee-feedback-list.pdl@broadcom.com is the mail distribution
list having currently

MAINTAINERS: update Broadcom reviewer list

Add Sandeep Tripathy <sandeep.tripathy@broadcom.com> as reviewer.
bcm-optee-feedback-list.pdl@broadcom.com is the mail distribution
list having currently active or interested tee developers within
Broadcom. Keep this entry as a list.

Signed-off-by: Sandeep Tripathy <sandeep.tripathy@broadcom.com>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>

show more ...

MAINTAINERS: fix Igor Opaniuk's email, make him a Poplar reviewer

Igor no longer works for Linaro and emails to igor.opaniuk@linaro.org
cannot be delivered. However, he still helps with the Poplar b

MAINTAINERS: fix Igor Opaniuk's email, make him a Poplar reviewer

Igor no longer works for Linaro and emails to igor.opaniuk@linaro.org
cannot be delivered. However, he still helps with the Poplar board, so
remove his old Linaro address from the general section and add him to
the Poplar section as a reviewer.

Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Acked-by: Igor Opaniuk <igor.opaniuk@gmail.com>

show more ...

stm32mp1: fix ordering in IOMEM mapping registering

Swap RCC_BASE and PWR_BASE mapping registering for a nice alpha
ordering of the mapping definitions.

Signed-off-by: Etienne Carriere <etienne.car

stm32mp1: fix ordering in IOMEM mapping registering

Swap RCC_BASE and PWR_BASE mapping registering for a nice alpha
ordering of the mapping definitions.

Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

stm32mp1: prefer vaddr_t to uintptr_t

Use vaddr_t and paddr_t instead of uintptr_t where applicable.

This change also simplifies some platform get-base-address functions
to use io_pa_or_va().

Sign

stm32mp1: prefer vaddr_t to uintptr_t

Use vaddr_t and paddr_t instead of uintptr_t where applicable.

This change also simplifies some platform get-base-address functions
to use io_pa_or_va().

Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

stm32mp1: make all local variables be initialized

Update platform to conform with OP-TEE directive about local variables
initialization.

Also rename variable labels excep into exceptions as more ex

stm32mp1: make all local variables be initialized

Update platform to conform with OP-TEE directive about local variables
initialization.

Also rename variable labels excep into exceptions as more explicit.

Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

stm32_uart: assert clock and register base address

Assert clock and UART base address were found from
DTB instead of panicking. This can help debugging.
Invalid clock or iomem base address already l

stm32_uart: assert clock and register base address

Assert clock and UART base address were found from
DTB instead of panicking. This can help debugging.
Invalid clock or iomem base address already lead to
core panic, an assertion here is far enough.

Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

stm32_uart: make all local variable be initialized

Update stm32_uart driver to conform with OP-TEE directive about local
variables initialization.

Signed-off-by: Etienne Carriere <etienne.carriere@

stm32_uart: make all local variable be initialized

Update stm32_uart driver to conform with OP-TEE directive about local
variables initialization.

Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

stm32_etzpc: make all local variables be initialized

Update stm32_etzpc driver to conform with OP-TEE directive about local
variables initialization.

Signed-off-by: Etienne Carriere <etienne.carrie

stm32_etzpc: make all local variables be initialized

Update stm32_etzpc driver to conform with OP-TEE directive about local
variables initialization.

Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

stm32_etzpc: local validation functions may be unused

Local functions valid_decprot_id() valid_tzma_id() are used only
in debug configuration. This adds a __maybe_unused attribute to
prevent compil

stm32_etzpc: local validation functions may be unused

Local functions valid_decprot_id() valid_tzma_id() are used only
in debug configuration. This adds a __maybe_unused attribute to
prevent compiler from warning with a trace like:

core/drivers/stm32_etzpc.c:100:13: warning: ‘valid_tzma_id’ defined but not used [-Wunused-function]
static bool valid_tzma_id(unsigned int id)
^~~~~~~~~~~~~
core/drivers/stm32_etzpc.c:95:13: warning: ‘valid_decprot_id’ defined but not used [-Wunused-function]
static bool valid_decprot_id(unsigned int id)
^~~~~~~~~~~~~~~~

Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

stm32_gpio: rename excep into exceptions

Rename variable labels excep into exceptions as more explicit.

Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <

stm32_gpio: rename excep into exceptions

Rename variable labels excep into exceptions as more explicit.

Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

stm32_gpio: make all local variables be initialized

Update stm32_gpio driver to conform with OP-TEE directive about local
variables initialization.

Signed-off-by: Etienne Carriere <etienne.carriere

stm32_gpio: make all local variables be initialized

Update stm32_gpio driver to conform with OP-TEE directive about local
variables initialization.

Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

stm32mp1: embed stm32_rng driver

Platform embeds RNG driver and maps the RNG1 interface registers.

Signed-off-by: Etienne Carriere <etienne.carriere@st.com>
Acked-by: Jens Wiklander <jens.wiklander

stm32mp1: embed stm32_rng driver

Platform embeds RNG driver and maps the RNG1 interface registers.

Signed-off-by: Etienne Carriere <etienne.carriere@st.com>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

stm32_rng: driver for STM32 RNG

Drivers is embedded upon CFG_STM32_RNG=y.

The driver main API functions are:
- stm32_rng_read() to get a buffer of random bytes,
- stm32_rng_read_raw() to get a buff

stm32_rng: driver for STM32 RNG

Drivers is embedded upon CFG_STM32_RNG=y.

The driver main API functions are:
- stm32_rng_read() to get a buffer of random bytes,
- stm32_rng_read_raw() to get a buffer of random bytes assuming the
RNG hardware is ready, i.e clock enabled.

The device driver is initialized from DT resource when a secure DTB,
currently the embedded DTB, is found.

STM32 RNG driver assumes the platform supports at most RNG instance
in the secure world.

Signed-off-by: Etienne Carriere <etienne.carriere@st.com>
Signed-off-by: Lionel Debieve <lionel.debieve@st.com>
Signed-off-by: Nicolas Le Bayon <nicolas.le.bayon@st.com>
Signed-off-by: Yann Gautier <yann.gautier@st.com>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

stm32mp1: util for shared resources refcount

Implement {incr|decr}_shrefcnt(refcount, secure_flag) to provide
reference counting for secure, non secure and resources used both
from secure and non se

stm32mp1: util for shared resources refcount

Implement {incr|decr}_shrefcnt(refcount, secure_flag) to provide
reference counting for secure, non secure and resources used both
from secure and non secure world.

Functions {incr|decr}_refcnt(refcount) is a secure refcount only
reference counting.

Signed-off-by: Etienne Carriere <etienne.carriere@st.com>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

build: remove pedantic from build directives

This change removes -pedantic from the build directive from all
warning build level. The rational is that OP-TEE OS package no
longer supports being buil

build: remove pedantic from build directives

This change removes -pedantic from the build directive from all
warning build level. The rational is that OP-TEE OS package no
longer supports being built in pedantic mode allowing now use
of implementation sequence that are rejected in pedantic mode,
for example using = { } for initializing structured local variables
in a structure agnostic way which OP-TEE OS community as agreed to
use.

Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

ci: .shippable.yml: add build for mbedtls

Adds shippable build configuration for using mbedtls as crypto library.

Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Tested-by: Jerome Forissie

ci: .shippable.yml: add build for mbedtls

Adds shippable build configuration for using mbedtls as crypto library.

Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Tested-by: Jerome Forissier <jerome.forissier@linaro.org> (HiKey960, mbedtls, GP)
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...