On Mon, May 26, 2025 at 06:33:28PM +0200, Marco Felsch wrote: > On 25-05-26, Michael Tretter wrote: > > The PBL has to pass a full description of the SDRAM to OP-TEE to allow > > OP-TEE to handle dynamic shared memory in the entire SDRAM. Thus, the > > PBL needs to read the full memory configuration. > > > > Signed-off-by: Michael Tretter <m.tret...@pengutronix.de> > > --- > > arch/arm/mach-rockchip/dmc.c | 37 ++++++++++++++++++++++++++++++++++--- > > include/mach/rockchip/dmc.h | 2 ++ > > 2 files changed, 36 insertions(+), 3 deletions(-) > > > > diff --git a/arch/arm/mach-rockchip/dmc.c b/arch/arm/mach-rockchip/dmc.c > > index > > 62a7ef8f1e38e989e84f08f6f4a6586be3e85532..260b8be9c7da9e8f83323822eab92a439ef4e2ef > > 100644 > > --- a/arch/arm/mach-rockchip/dmc.c > > +++ b/arch/arm/mach-rockchip/dmc.c > > @@ -171,11 +171,12 @@ resource_size_t rk3568_ram0_size(void) > > #define RK3588_PMUGRF_OS_REG4 0x210 > > #define RK3588_PMUGRF_OS_REG5 0x214 > > > > -resource_size_t rk3588_ram0_size(void) > > +size_t rk3588_ram_sizes(phys_addr_t *base, resource_size_t *size, size_t n) > > Nit: could be void since you don't check the return value.
The return value is used in a later patch. There the array size is passed in 'n' and the actually used array entries are returned from rk3588_ram_sizes(). Sascha -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
