On 10/4/23 11:11, Patrice CHOTARD wrote:

On 9/27/23 14:44, Harald Seiler wrote:

...

Ensure that the RAM configuration line is only printed when CONFIG_SPL_DISPLAY_PRINT is set.

Signed-off-by: Harald Seiler hws@denx.de

drivers/ram/stm32mp1/stm32mp1_ram.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/ram/stm32mp1/stm32mp1_ram.c b/drivers/ram/stm32mp1/stm32mp1_ram.c index a6c19af9722..2808d07c3ae 100644 --- a/drivers/ram/stm32mp1/stm32mp1_ram.c +++ b/drivers/ram/stm32mp1/stm32mp1_ram.c @@ -126,7 +126,8 @@ static int stm32mp1_ddr_setup(struct udevice *dev) dev_dbg(dev, "no st,mem-name\n"); return -EINVAL; }

• printf("RAM: %s\n", config.info.name);

• if (CONFIG_IS_ENABLED(DISPLAY_PRINT))

• printf("RAM: %s\n", config.info.name);
  
  

  for (idx = 0; idx < ARRAY_SIZE(param); idx++) { ret = ofnode_read_u32_array(node, param[idx].name,

Reviewed-by: Patrice Chotard patrice.chotard@foss.st.com

Thanks Patrice

Apply on stm32/next

Thanks