Re: [PATCH v2 4/5] arch: arm: mach-socfpga: Adjust a raw sectors for MMC loading of U-Boot from SPL

[Jan Kiszka]

On 03.12.25 21:32, Ralph Siemsen wrote:
> Hi Jan,
> 
> On Mon, Dec 1, 2025 at 2:33 AM Jan Kiszka <jan.kis...@siemens.com> wrote:
> 
>> diff --git a/arch/arm/mach-socfpga/board.c b/arch/arm/mach-socfpga/board.c
>> index 28554b7a109..0b04f2bcacf 100644
>> --- a/arch/arm/mach-socfpga/board.c
>> +++ b/arch/arm/mach-socfpga/board.c
>> @@ -209,3 +210,13 @@ void lmb_arch_add_memory(void)
>>         }
>>  }
>>  #endif
>> +
>> +#if (defined(TARGET_SOCFPGA_ARRIA10) || defined(TARGET_SOCFPGA_GEN5)) && \
>> +    defined(CONFIG_XPL_BUILD)
> 
> This seems to be missing two CONFIG_ prefixes:
> 
> #if (defined(CONFIG_TARGET_SOCFPGA_ARRIA10) ||
> defined(CONFIG_TARGET_SOCFPGA_GEN5)) &&
> 

Indeed. I was lacking a target to catch this. v3 is on the way.

Thanks,
Jan

-- 
Siemens AG, Foundational Technologies
Linux Expert Center