On Wed, Apr 15, 2020 at 08:19:09AM +0200, Michael Tretter wrote: > Hi Michael, > > On Tue, Apr 14, 2020 at 09:23:53PM +0200, Michael Graichen wrote: > > Am 14.04.20 um 12:53 schrieb Michael Tretter: > > > On Thu, Apr 09, 2020 at 02:44:00PM +0000, Michael Graichen wrote: > > > > This adds support to programm the PL part of the Zynq SoC, > > > > but only the non-secure way and no partial reconfiguration. It adds the > > > > 'zynq_fpga_manager' so we can use > > > > > > > > firmwareload -l > > > > firmwareload -t zynq-fpga-manager /mnt/mmc0.0/design_1_wrapper.bit > > > > > > > > to programm the PL. > > > > > > The patch copies a lot of code from the PL programming code for the ZynqMP. > > > The main differences are that the Zynq-7000 Bitstream header is 8 bytes > > > shorter than for the ZynqMP and that the Zynq does not go through the > > > firmware, but directly programs the PCAP. > > > > > > I think that we should extend the ZynqMP FPGA driver to be able to handle the > > > PL programming of the Zynq-7000 as well instead of adding a new driver for > > > that. > > > > > > Michael > > > > > > > yes, i have learned from drivers/firmware/zynqmp-fpga.c how to deal with the > > firmwareload command and how to validate the bitstream. > > In generally it would be better to only have one driver but since i have no > > access to a ZynqMP board i am kind of shy to touch these bits. > > I will happily test your patches on the ZynqMP:) I would really like to have > more users of the zynqmp-fpga driver. I dropped the patch for now in favor for the upcoming patches adding support for the existing driver ;) Sascha -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox