[U-Boot] [PATCH] fpga: allow programming fpga from FIT image for all FPGA drivers

Michal Simek michal.simek at xilinx.com
Wed Nov 29 10:00:01 UTC 2017


On 29.11.2017 06:20, Goldschmidt Simon wrote:
> On 28.11.2017 14:46, Michal Simek wrote:
>> On 28.11.2017 10:08, Goldschmidt Simon wrote:
>>> Simon Goldschmidt wrote:
>>>> Hi Simon,
>>>>
>>>> Simon Glass wrote:
>>>>> I see that, although it is adding to the fpga header so presumably
>>>>> making it harder for someone to move this over.
>>>>
>>>> Yes, I'm not happy with changing the header and even xilinx C file to
>>>> add functionality for altera. However, this is due to the fact that a
>>>> core file still depends on an dogs implementation, which I removed.
>>>
>>> This should have meant "on an fpga implementation". No dogs or offences here.
>>> My mobile phone does not know the word "fpga", obviously.
>>>
>>>>
>>>>> Does anyone on cc know the plan fr conversion of FPGA to driver model?
>>>>> It does not look too tricky from a quick look at the header file.
>>>>
>>>> It does not look tricky, indeed. However, we should try to match this
>>>> to the Linux implementation regarding what's needed in the device
>>>> tree. And I just don't know that part, yet ;-)
>>
>> I have not a problem to enable others fpgas to use this functionality.
>> And transition should be done for all these drivers when someone has a time to do
>> it.
> 
> Great, thanks! So via which tree would this be included?

Origin code was applied via my tree. It means it shouldn't be an issue
to do that again.
Anyway I have tested it on zynqmp and this feature is still working
without any issue when this patch is applied.

sjg: Do you see any other issue with this patch?

Thanks,
Michal


More information about the U-Boot mailing list