Hi Brian,
On 25/7/2026 7:57 pm, Sune Brian wrote:
[CAUTION: This email is from outside your organization. Unless you trust the
sender, do not click on links or open attachments as it may be a fraudulent
email attempting to steal your information and/or compromise your computer.]
Hi Alif,
I am not too good at checking the history of these patches.
I think the board.c is missing naming sync.
As for PL330 the original CMD is done by me so history-wise
I can trace more easily.
Is it possible to update this patch with the remaining board.c
and message the patch with the PL330 the current reviewing one?
Not sure how to due with the missing board.c
Thanks,
Brian
Sure, I will submit v2 with the changes only for board.c
Best regards,
Alif
On Sat, Jul 25, 2026 at 5:49 PM Yuslaimi, Alif Zakuan
<[email protected]> wrote:
Hi Brian,
On 24/7/2026 6:25 pm, Sune Brian wrote:
[CAUTION: This email is from outside your organization. Unless you trust the
sender, do not click on links or open attachments as it may be a fraudulent
email attempting to steal your information and/or compromise your computer.]
On Fri, Jul 24, 2026 at 12:09 PM Yuslaimi, Alif Zakuan
<[email protected]> wrote:
Hello Brian,
On 23/7/2026 4:59 pm, Sune Brian wrote:
[CAUTION: This email is from outside your organization. Unless you trust the
sender, do not click on links or open attachments as it may be a fraudulent
email attempting to steal your information and/or compromise your computer.]
HI Alif,
Just reminding, duplicated patch:
https://patchwork.ozlabs.org/project/uboot/patch/[email protected]/.
Which is under reviewed.
Thanks,
Brian
Hi Alif,
My original patch does grep the entire u-boot repository and update it.
However how T.F. redo that patch or any missing I did not double check
again.
Tom also noticed the new board's naming update is missing and updated
the naming as well.
After the patch applied, the board.c did not notice the missing renaming
but only the PL330.
I am okay with the old patch removal due to the title not properly formed.
I also suggest doing a full grep again to verify any missing.
Thanks,
Brian
After grepping the whole repository I found that only these two files
needed fixing.
Since this is already under Tom's radar, and your patch has already been
submitted and reviewed earlier, I think we can reject my patch here.
Thanks,
Alif
Thanks for the heads up. How about the fix for
arch/arm/mach-socfpga/board.c?
Best regards,
Alif
On Thu, Jul 23, 2026 at 10:43 AM <[email protected]> wrote:
From: Alif Zakuan Yuslaimi <[email protected]>
Commit 62f7a9460209 ("Replace TARGET namespace and cleanup properly")
replaces the TARGET namespace to ARCH for all SoCFPGA devices.
Some of the TARGET namespaces are still left out, update leftover TARGET
to ARCH following current SoCFPGA naming.
Fixes: 62f7a9460209 ("Replace TARGET namespace and cleanup properly")
Signed-off-by: Alif Zakuan Yuslaimi <[email protected]>
---
arch/arm/mach-socfpga/board.c | 4 ++--
cmd/Kconfig | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/arm/mach-socfpga/board.c b/arch/arm/mach-socfpga/board.c
index b202ca258bc..b662afc1ad7 100644
--- a/arch/arm/mach-socfpga/board.c
+++ b/arch/arm/mach-socfpga/board.c
@@ -210,8 +210,8 @@ void lmb_arch_add_memory(void)
}
#endif
-#if (defined(CONFIG_TARGET_SOCFPGA_ARRIA10) || \
- defined(CONFIG_TARGET_SOCFPGA_GEN5)) && defined(CONFIG_XPL_BUILD)
+#if (defined(CONFIG_ARCH_SOCFPGA_ARRIA10) || \
+ defined(CONFIG_ARCH_SOCFPGA_GEN5)) && defined(CONFIG_XPL_BUILD)
unsigned long board_spl_mmc_get_uboot_raw_sector(struct mmc *mmc,
unsigned long raw_sect)
{
diff --git a/cmd/Kconfig b/cmd/Kconfig
index 35ce42f8397..b724f4f8d49 100644
--- a/cmd/Kconfig
+++ b/cmd/Kconfig
@@ -1911,7 +1911,7 @@ menu "Shell scripting commands"
config CMD_C5_PL330_DMA
bool "Release Reset DMA Channels for PL330 Handshake"
- depends on TARGET_SOCFPGA_CYCLONE5
+ depends on ARCH_SOCFPGA_CYCLONE5
help
Provides access to Reset Manager Per2ModRst. Enables DMA
channels for ARM PrimeCell PL330 via reset release.
--
2.43.7
