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 <[email protected]> Sent: Monday, July 24, 2023 11:13 AM To: Mihai Sain - M19926 <[email protected]>; [email protected] Cc: Cristian Birsan - M91496 <[email protected]> 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=heads 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 <[email protected]> > --- > +CONFIG_SYS_PROMPT="[root@sama5d29 ~]$ " Can you remove this. It looks like a Linux prompt and might be confusing for people (and everywhere below)
