On Isn, 2017-06-05 at 11:32 +0800, Chee, Tien Fong wrote: > > > > > > On Sab, 2017-06-03 at 13:27 +0200, Marek Vasut wrote: > > > > On 05/29/2017 06:00 AM, tien.fong.c...@intel.com wrote: > > > > > > > > > From: Tien Fong Chee <tien.fong.c...@intel.com> > > > > > > This patch is for enabling the FPGA driver support on SPL. > > > > > > Signed-off-by: Tien Fong Chee <tien.fong.c...@intel.com> > > > --- > > > include/configs/socfpga_common.h | 1 + > > > 1 file changed, 1 insertion(+) > > > > > > diff --git a/include/configs/socfpga_common.h > > > b/include/configs/socfpga_common.h > > > index da7e4ad..e74e7eb 100644 > > > --- a/include/configs/socfpga_common.h > > > +++ b/include/configs/socfpga_common.h > > > @@ -110,6 +110,7 @@ > > > #define CONFIG_FPGA > > > #define CONFIG_FPGA_ALTERA > > > #define CONFIG_FPGA_SOCFPGA > > > +#define CONFIG_SPL_FPGA_SUPPORT > > This should be Kconfig symbol selected in board config. Why is it > > not > > Kconfig symbol ? > > Ahha....ijust recalled there is a very important reason i did this because CONFIG_CMD_FPGA which is wrapping all these FPGA related CONFIG is not part of kconfig. During the build, kconfig and Macro #define are in two different phase/order, so we might having build issue during kconfig build(1st phase) when some Macro #define which is 2nd phase are required.
> I have a patch that is specific for converting all FPGA related Macro > config by tool created which will be submited after this patch set. > Let me know if you prefer kconfig symbol. > > > > > > #define CONFIG_FPGA_COUNT 1 > > > #endif > > > #endif > > > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot