On Tue, Apr 18, 2017 at 6:58 PM, Marek Vasut <ma...@denx.de> wrote: > On 04/18/2017 05:12 AM, Ley Foon Tan wrote: >> On Mon, Apr 17, 2017 at 6:39 PM, Marek Vasut <ma...@denx.de> wrote: >>> On 04/17/2017 05:05 AM, Ley Foon Tan wrote: >>>> On Fri, Apr 14, 2017 at 6:25 PM, Marek Vasut <ma...@denx.de> wrote: >>>>> On 04/13/2017 07:41 PM, Ley Foon Tan wrote: >>>>>> Add config and defconfig for the Arria10 and update socfpga_common.h. >>>>>> >>>>>> Signed-off-by: Tien Fong Chee <tien.fong.c...@intel.com> >>>>>> Signed-off-by: Ley Foon Tan <ley.foon....@intel.com> >>>>> >>>>> [...] >>>>> >>>>>> @@ -298,7 +306,10 @@ unsigned int cm_get_qspi_controller_clk_hz(void); >>>>>> */ >>>>>> #define CONFIG_SPL_FRAMEWORK >>>>>> #define CONFIG_SPL_TEXT_BASE CONFIG_SYS_INIT_RAM_ADDR >>>>>> -#define CONFIG_SPL_MAX_SIZE (64 * 1024) >>>>>> +#define CONFIG_SPL_MAX_SIZE CONFIG_SYS_INIT_RAM_SIZE >>>>>> +#if defined(CONFIG_TARGET_SOCFPGA_ARRIA10) >>>>>> +#define CONFIG_SPL_BOARD_INIT >>>>> >>>>> This should be in Kconfig and selected by arch/arm/mach-socfpga/Kconfig >>>>> entry for A10 >>>> We can't use Kconfig method for CONFIG_SPL_BOARD_INIT, it is not a >>>> Kconfig parameter now. >>> >>> OK. You can turn it into one, although that is out of the scope of this >>> patchset. >> Okay, I will add SPL_BOARD_INIT to arch/arm/mach-socfpga/Kconfig and >> select it for Arria 10. > > Do it in a separate patch please . Also, there's a script for turning > symbols into Kconfig symbols in u-boot. I think we don't need script if just add Kconfig to arch/arm/mach-socfpga/Kconfig, unless you want me add to common/spl/Kconfig and move this option to kconfig for all architectures.
Regards Ley Foon _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
