[U-Boot] [PATCH 01/19] configs: Add FPGA loadfs config for Arria 10

[Marek Vasut <marex@denx.de>]

On 08/30/2017 07:59 AM, Chee, Tien Fong wrote:
> On Sel, 2017-08-29 at 13:51 +0200, Marek Vasut wrote:
>> On 08/29/2017 12:45 PM, tien.fong.chee at intel.com wrote:
>>>
>>> From: Tien Fong Chee <tien.fong.chee@intel.com>
>>>
>>> This config allow FPGA design loaded from FAT fs to FPGA manager.
>>>
>>> Signed-off-by: Tien Fong Chee <tien.fong.chee@intel.com>
>>> ---
>>>  configs/socfpga_arria10_defconfig |    1 +
>>>  1 files changed, 1 insertions(+), 0 deletions(-)
>>>
>>> diff --git a/configs/socfpga_arria10_defconfig
>>> b/configs/socfpga_arria10_defconfig
>>> index 53ab66f..d555743 100644
>>> --- a/configs/socfpga_arria10_defconfig
>>> +++ b/configs/socfpga_arria10_defconfig
>>> @@ -29,3 +29,4 @@ CONFIG_DWAPB_GPIO=y
>>>  CONFIG_DM_MMC=y
>>>  CONFIG_SYS_NS16550=y
>>>  CONFIG_USE_TINY_PRINTF=y
>>> +CONFIG_CMD_FPGA_LOADFS=y
>> You should enable stuff only after you add the necessary support bits
>> ...
>>
> I enabled it as early as possible, because i would like to use
> compiling for detecting any code errors while adding the neccesssary
> support bits. I can rearrange the patches sequence.
> 
Except you enable feature which cannot work, so this is wrong. If you
need this for debugging purposes, great, but in the series it should go
after the groundwork is laid.

-- 
Best regards,
Marek Vasut