Hi Michal, On 29 November 2017 at 03:00, Michal Simek <michal.si...@xilinx.com> wrote: > 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?
No other issues. Regards Simon _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
