Hello Eugen,
Thank you for applying the patch.
For this board we don't want to use SPL. I will send v2 patch to remove the custom prompt. Thanks.
Best regards, Mihai Sain
-----Original Message----- From: Eugen Hristev eugen.hristev@collabora.com Sent: Monday, July 24, 2023 11:13 AM To: Mihai Sain - M19926 Mihai.Sain@microchip.com; u-boot@lists.denx.de Cc: Cristian Birsan - M91496 Cristian.Birsan@microchip.com Subject: Re: [u-boot][PATCH] board: at91: sama5d29_curiosity: add initial support for sama5d29_curiosity
EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
Hello Mihai,
Thank you for the patch.
However, the at91 maintainer tree nowadays has a big problem, I cannot apply patches because the size of the SPL will overflow. There is a pending patch that fixes it, but it has not been tested. Here you can help, and test this integration branch (at least on sama5d2 ICP, but if you can test on multiple boards, it's even better) :
https://source.denx.de/u-boot/custodians/u-boot-at91/-/tree/testing?ref_type...
Once this is tested you can reply with a Tested-by: tag, and we can move along. Without testing the patch that fixes the size restraint (it's the CONFIG_LTO patch), the tree is stalled.
One small nitpick below, and I am looking forward for your reply,
Eugen
On 7/20/23 10:54, Mihai Sain wrote:
Add initial support for sama5d29_curiosity board.
Hardware: SoC: SAMA5D29 500 MHz DRAM: LPDDR2 512 MiB PMIC: MCP16502 Debug: UART0 Flash: QSPI NOR 8 MiB RGB LCD connector Mikrobus connectors x 2 SD-Card connectors x 2 USB 2.0 x 2
Signed-off-by: Mihai Sain mihai.sain@microchip.com
+CONFIG_SYS_PROMPT="[root@sama5d29 ~]$ "
Can you remove this. It looks like a Linux prompt and might be confusing for people
(and everywhere below)