[U-Boot] [PATCH v3 05/12] arm: socfpga: fpgamgr: Disable FPGA Manager for Stratix 10

[Marek Vasut]

On 10/17/2016 05:30 PM, Chin Liang See wrote:
> On Sen, 2016-10-17 at 17:20 +0200, Marek Vasut wrote:
>> On 10/17/2016 05:14 PM, Chin Liang See wrote:
>>>
>>> On Sen, 2016-10-17 at 15:42 +0200, Marek Vasut wrote:
>>>>
>>>> On 10/17/2016 03:35 PM, See, Chin Liang wrote:
>>>>>
>>>>>
>>>>> On Min, 2016-10-16 at 17:34 +0200, Marek Vasut wrote:
>>>>>>
>>>>>>
>>>>>> On 10/13/2016 10:33 AM, Chin Liang See wrote:
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> Disable the FPGA Manager for Stratix 10 SoC as we are not
>>>>>>> using this for SOCVP
>>>>>> If it's not used on SoCVP, then shouldn't this be disabled
>>>>>> only
>>>>>> for
>>>>>> SoCVP instead of S10 ?
>>>>>>
>>>>> We will be enhancing this code to support the hardware /
>>>>> emulation
>>>>> in
>>>>> later phase. In another word, will switch the support from
>>>>> SOCVP to
>>>>> hardware once its available.
>>>>>
>>>> This is confusing, what would happen if someone tries to use old
>>>> u-
>>>> boot
>>>> on real hardware ?
>>> It won't work until added drivers for Clocks, Reset and DDR.
>>> Current
>>> state would be good for SOCVP only where not all hardware is
>>> simulated.
>> In that case, mark it as SoCVP to prevent confusion please.
>>
> 
> In this case, I can update the config to indicate this is S10 SOCVP.
> 
Yes please.


-- 
Best regards,
Marek Vasut