Hi Stefan,
On 5/17/21 11:40 AM, Stefan Roese wrote:
Hi Michal,
On 17.05.21 11:37, Michal Simek wrote:
Hi,
On 5/17/21 11:04 AM, Stefan Roese wrote:
Added Tom to Cc.
On 17.05.21 10:01, Heinrich Schuchardt wrote:
On 17.05.21 08:39, Michal Simek wrote:
On 5/16/21 11:17 AM, Heinrich Schuchardt wrote:
Save some KiB when building xilinx_versal_virt_defconfig by disabling Unicode capitalization support. This avoids build failures when adding new features for the UEFI sub-system.
Signed-off-by: Heinrich Schuchardt xypron.glpk@gmx.de
It would be preferable to have a size check for the u-boot binary. This would avoid seeing problems only during execution.
configs/xilinx_versal_virt_defconfig | 1 + 1 file changed, 1 insertion(+)
diff --git a/configs/xilinx_versal_virt_defconfig b/configs/xilinx_versal_virt_defconfig index 707693713a..8bc2ff4a4c 100644 --- a/configs/xilinx_versal_virt_defconfig +++ b/configs/xilinx_versal_virt_defconfig @@ -111,3 +111,4 @@ CONFIG_USB_GADGET_PRODUCT_NUM=0x0300 CONFIG_USB_GADGET_DOWNLOAD=y CONFIG_USB_FUNCTION_THOR=y CONFIG_OF_LIBFDT_OVERLAY=y +# CONFIG_EFI_UNICODE_CAPITALIZATION is not set -- 2.30.2
What's the build issue? Any log? U-Boot is getting bigger and bigger and
Hello Michal,
this job failed: https://source.denx.de/u-boot/custodians/u-boot-efi/-/jobs/269614
After deactivating CONFIG_EFI_UNICODE_CAPITALIZATION I get success: https://source.denx.de/u-boot/custodians/u-boot-efi/-/jobs/269620
I had a similar, most likely related issue a few days ago:
https://dev.azure.com/sr0718/u-boot/_build/results?buildId=87&view=resul...
Tom git bi-sected this to this commit, which only adds a bit tiny bit of image size I assume:
"I bisected this down to: commit 368b3f6db4540f913c436e5287be8356bc9a2891 Author: Stefan Chulski stefanc@marvell.com Date: Mon May 3 08:08:44 2021 +0200
phy: introduce 1000BaseX and 2500BaseX modes
Signed-off-by: Stefan Chulski stefanc@marvell.com Signed-off-by: Stefan Roese sr@denx.de
And while I'm not sure off-hand how it does it, it's causing the breakage. Maybe something is overflowing now?"
I have rebase Heinrich patches on the top of HEAD + my queue and I can't see any issue https://source.denx.de/u-boot/custodians/u-boot-microblaze/-/tree/for-heinri...
It means likely what Stefan mentioned is likely correct.
The mentioned PHY patch was not pushed to mainline (yet). Please add it:
http://patchwork.ozlabs.org/project/uboot/patch/20210503060853.3743626-2-sr@...
and run the test again.
https://pastebin.com/pd9i8LEi I have applied this patch and I can't see any issue. Also pipeline passed without any issue. https://source.denx.de/u-boot/custodians/u-boot-microblaze/-/pipelines/7558
Can you apply this one and try on your end? http://patchwork.ozlabs.org/project/uboot/patch/7a7c5d657a968190e0676251dbd8...
it is in my queue and when u-boot is bigger then certain limit it can cause issue with early stack allocation.
Thanks, Michal