Fpga code is pretty old and none has tried to clean it up. My attempt is related to new code I want to push to mainline which is add support for checking bitstream and if bitstream is valid for the selected device. For this I need to do cleanup code and move code from cmd_fpga.c to fpga.c in driver folder.
Zynq driver: Depends on previous zynq patches sent some days ago. Tested by: set fload tftp \${addr} fpga.bin\;fpga info 0\;fpga load 0 \${addr} \${filesize} set floadb tftp \${addr} download.bit\;fpga info 0\;fpga loadb 0 \${addr} \${filesize} set addr 10000000 run fload run floadb set addr 10000001 run fload run floadb set addr 10000002 run fload run floadb set addr 10000003 run fload run floadb Thanks for your comments, Michal Changes in v3: - Setup fpga_loadbitstream as weak function - New patch in this series - Rebase because of change in previous patch - fpga_validate should be global function - fix one printf function - change parameter name according to previous patch dev->devnum - New patch in this series Changes in v2: - Fix compilation warnings - Fix grammer in the commit message - Fix bugs reported by Tom Rini - Fix checkpatch warnings (fpga) - Fix comments (fpga) - Do not use CamelCase for XilinxZynq (fpga) - Move to fpga series and extend this driver - New patch in this series Michal Simek (9): fpga: Clean coding style fpga: Fix debug message compilation error cmd: fpga: Clean coding style cmd: fpga: Move fpga_loadbitstream to fpga.c fpga: Change the first parameter in fpga_loadbitstream cmd: fpga: Do not include net.h fpga: zynq: Add support for loading bitstream fpga: Check device name against bitstream name fpga: Remove all CONFIG_SYS_* fpga related options arch/arm/cpu/armv7/zynq/slcr.c | 35 +++ arch/arm/include/asm/arch-zynq/hardware.h | 10 +- arch/arm/include/asm/arch-zynq/sys_proto.h | 3 + board/xilinx/zynq/board.c | 37 +++ common/cmd_fpga.c | 254 +++++++-------------- drivers/fpga/Makefile | 1 + drivers/fpga/fpga.c | 225 +++++++++--------- drivers/fpga/xilinx.c | 146 ++++++++++++ drivers/fpga/zynqpl.c | 355 +++++++++++++++++++++++++++++ include/altera.h | 17 -- include/configs/M54455EVB.h | 2 +- include/configs/MERGERBOX.h | 2 +- include/configs/MVBC_P.h | 2 +- include/configs/MVBLM7.h | 2 +- include/configs/MVSMR.h | 2 +- include/configs/omap3_mvblx.h | 2 +- include/configs/zynq.h | 6 + include/fpga.h | 13 +- include/lattice.h | 3 - include/xilinx.h | 25 +- include/zynqpl.h | 59 +++++ 21 files changed, 848 insertions(+), 353 deletions(-) create mode 100644 drivers/fpga/zynqpl.c create mode 100644 include/zynqpl.h -- 1.8.2.1
pgp7I5fLp0Nfb.pgp
Description: PGP signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot