[U-Boot] [PATCH v4 14/19] arm: socfpga: Add SPL support for Arria 10

Dinh Nguyen dinguyen at kernel.org
Mon Apr 10 22:05:45 UTC 2017



On 04/10/2017 03:43 PM, Dinh Nguyen wrote:
> 
> 
> On 04/05/2017 04:32 AM, Ley Foon Tan wrote:
>> Add SPL support for Arria 10.
>>
>> Signed-off-by: Tien Fong Chee <tien.fong.chee at intel.com>
>> Signed-off-by: Ley Foon Tan <ley.foon.tan at intel.com>
>> ---
>>  arch/arm/mach-socfpga/spl.c | 74 ++++++++++++++++++++++++++++++++++++++++++---
>>  1 file changed, 69 insertions(+), 5 deletions(-)
>>
>> diff --git a/arch/arm/mach-socfpga/spl.c b/arch/arm/mach-socfpga/spl.c
>> index 69c433c..e1e62c2 100644
>> --- a/arch/arm/mach-socfpga/spl.c
>> +++ b/arch/arm/mach-socfpga/spl.c
>> @@ -19,23 +19,32 @@
>>  #include <asm/arch/sdram.h>
>>  #include <asm/arch/scu.h>
>>  #include <asm/arch/nic301.h>
>> +#include <asm/sections.h>
>> +#include <fdtdec.h>
>> +#include <watchdog.h>
>> +#if defined(CONFIG_TARGET_SOCFPGA_ARRIA10)
>> +#include <asm/arch/pinmux.h>
>> +#endif
>>  
>>  DECLARE_GLOBAL_DATA_PTR;
>>  
>> +#if defined(CONFIG_TARGET_SOCFPGA_GEN5)
>>  static struct pl310_regs *const pl310 =
>>  	(struct pl310_regs *)CONFIG_SYS_PL310_BASE;
>>  static struct scu_registers *scu_regs =
>>  	(struct scu_registers *)SOCFPGA_MPUSCU_ADDRESS;
>>  static struct nic301_registers *nic301_regs =
>>  	(struct nic301_registers *)SOCFPGA_L3REGS_ADDRESS;
>> -static struct socfpga_system_manager *sysmgr_regs =
>> +#endif
>> +
>> +static const struct socfpga_system_manager *sysmgr_regs =
>>  	(struct socfpga_system_manager *)SOCFPGA_SYSMGR_ADDRESS;
>>  
>>  u32 spl_boot_device(void)
>>  {
>>  	const u32 bsel = readl(&sysmgr_regs->bootinfo);
>>  
>> -	switch (bsel & 0x7) {
>> +	switch (SYSMGR_GET_BOOTINFO_BSEL(bsel)) {
>>  	case 0x1:	/* FPGA (HPS2FPGA Bridge) */
>>  		return BOOT_DEVICE_RAM;
>>  	case 0x2:	/* NAND Flash (1.8V) */
>> @@ -68,6 +77,7 @@ u32 spl_boot_mode(const u32 boot_device)
>>  }
>>  #endif
>>  
>> +#if defined(CONFIG_TARGET_SOCFPGA_GEN5)
>>  static void socfpga_nic301_slave_ns(void)
>>  {
>>  	writel(0x1, &nic301_regs->lwhps2fpgaregs);
>> @@ -85,6 +95,7 @@ void board_init_f(ulong dummy)
>>  #endif
>>  	unsigned long sdram_size;
>>  	unsigned long reg;
>> +	int ret;
>>  
>>  	/*
>>  	 * First C code to run. Clear fake OCRAM ECC first as SBE
>> @@ -117,7 +128,11 @@ void board_init_f(ulong dummy)
>>  	/* Put everything into reset but L4WD0. */
>>  	socfpga_per_reset_all();
>>  	/* Put FPGA bridges into reset too. */
>> -	socfpga_bridges_reset(1);
>> +	ret = socfpga_bridges_reset(1);
>> +	if (ret) {
>> +		printf("socfpga_bridges_reset() failed: %d\n", ret);
>> +		hang();
>> +	}
>>  
>>  	socfpga_per_reset(SOCFPGA_RESET(SDR), 0);
>>  	socfpga_per_reset(SOCFPGA_RESET(UART0), 0);
>> @@ -150,7 +165,11 @@ void board_init_f(ulong dummy)
>>  
>>  	/* De-assert reset for peripherals and bridges based on handoff */
>>  	reset_deassert_peripherals_handoff();
>> -	socfpga_bridges_reset(0);
>> +	ret = socfpga_bridges_reset(0);
>> +	if (ret) {
>> +		printf("socfpga_bridges_reset() failed: %d\n", ret);
>> +		hang();
>> +	}
>>  
>>  	debug("Unfreezing/Thaw all I/O banks\n");
>>  	/* unfreeze / thaw all IO banks */
>> @@ -180,8 +199,53 @@ void board_init_f(ulong dummy)
>>  		hang();
>>  	}
>>  
>> -	socfpga_bridges_reset(1);
>> +	ret = socfpga_bridges_reset(1);
>> +	if (ret) {
>> +		printf("socfpga_bridges_reset() failed: %d\n", ret);
>> +		hang();
>> +	}
>>  
>>  	/* Configure simple malloc base pointer into RAM. */
>>  	gd->malloc_base = CONFIG_SYS_TEXT_BASE + (1024 * 1024);
>>  }
>> +#elif defined(CONFIG_TARGET_SOCFPGA_ARRIA10)
>> +#ifdef CONFIG_SPL_BOARD_INIT
>> +void spl_board_init(void)
> 
> Don't need to add another check for SPL_BOARD_INIT, it's already defined
> for CONFIG_TARGET_SOCFPGA_ARRIA10.
> 

Also, this patch is causing the DE0-NANO/Atlas board to no longer boot!

Dinh



More information about the U-Boot mailing list