Skip to content

FPGA subsystem changes

Michal Simek u-boot 2 versions – New

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v2 Latest9 patches 0 repliesNew
  2. v16 patches 0 repliesSuperseded

Changes in v2 from the cover letter

- Fix compilation errors in Astro
- Exchange fpga_op and name in xilinx_desc because spartan2/3, virtex2
are not using names.
- Fix warning reported by Tom
board/spear/x600/fpga.c:177:2: warning: initialization from incompatible pointer type [enabled by default] board/spear/x600/fpga.c:177:2: warning: (near initialization for ?fpga[0].name?) [enabled by default]
- Add fix for apf27 and astro

Patches in v2 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/9fpga: spartan2: Avoid CamelCaseboard/esd/pmc440/fpga.c board/matrix_vision/mvsmr/fpga.c drivers/fpga/spartan2.c +3 more+46 -460 replies
2/9fpga: spartan3: Avoid CamelCaseboard/armadeus/apf27/fpga.c board/astro/mcf5373l/fpga.c board/balloon3/balloon3.c +8 more+57 -570 replies
3/9fpga: virtex2: Avoid CamelCaseboard/gen860t/fpga.c drivers/fpga/virtex2.c drivers/fpga/xilinx.c +2 more+44 -440 replies
4/9fpga: xilinx: Avoid CamelCase for in Xilinx_descboard/armadeus/apf27/fpga.c board/astro/mcf5373l/fpga.c board/balloon3/balloon3.c +16 more+79 -790 replies
5/9fpga: xilinx: Fix the rest of CamelCasesboard/astro/mcf5373l/fpga.c include/spartan2.h include/spartan3.h +2 more+99 -990 replies
6/9fpga: xilinx: Simplify load/dump/info function handlingboard/armadeus/apf27/fpga.c board/astro/mcf5373l/fpga.c drivers/fpga/spartan2.c +9 more+104 -2230 replies
7/9fpga: zynq: Remove sparse warningsdrivers/fpga/zynqpl.c+2 -20 replies
8/9fpga: zynq: Use helper functions for zynq dmadrivers/fpga/zynqpl.c+115 -860 replies
9/9fpga: zynq: Use helper function zynq_validate_bitstreamdrivers/fpga/zynqpl.c+25 -70 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v2 yet.

Read the v2 thread

Cover letter [U-Boot] [PATCH v2 0/9] FPGA subsystem changes

Hi,

Tom reported me one warning in spear board and I have run buildman for all boards which have xilinx_desc in board folder.

Below is the report for this patch series.

Thanks,
Michal

[u-boot]$ ./tools/buildman/buildman -b xnext/fpga apf27 zynq spear astro balloon3 pmc440 gen860t mvsmr mt_ventoux -se Summary of 10 commits for 35 boards (4 threads, 1 job per thread)
01: Merge branch 'next' of git://git.denx.de/u-boot-sh

      m68k: +   astro_mcf5373l

+/mnt/disk/xnext/fpga/.bm-work/00/arch/m68k/cpu/mcf532x/cpu_init.c: In function 'cpu_init_f':
+/mnt/disk/xnext/fpga/.bm-work/00/arch/m68k/cpu/mcf532x/cpu_init.c:211:10: warning: unused variable 'wdog' [-Wunused-variable] +/mnt/disk/xnext/fpga/.bm-work/00/arch/m68k/lib/bootm.c: In function 'do_bootm_linux':
+/mnt/disk/xnext/fpga/.bm-work/00/arch/m68k/lib/bootm.c:53:8: warning: unused variable 'rd_len' [-Wunused-variable] +/mnt/disk/xnext/fpga/.bm-work/00/arch/m68k/lib/bootm.c:99:12: warning: 'initrd_start' may be used uninitialized in this function [-Wuninitialized] +/mnt/disk/xnext/fpga/.bm-work/00/arch/m68k/lib/bootm.c:99:12: warning: 'initrd_end' may be used uninitialized in this function [-Wuninitialized] +/mnt/disk/xnext/fpga/.bm-work/00/arch/m68k/lib/bootm.c:99:12: warning: 'cmd_start' may be used uninitialized in this function [-Wuninitialized] +/mnt/disk/xnext/fpga/.bm-work/00/arch/m68k/lib/bootm.c:99:12: warning: 'cmd_end' may be used uninitialized in this function [-Wuninitialized] +/mnt/disk/xnext/fpga/.bm-work/00/board/astro/mcf5373l/mcf5373l.c: In function 'initdram':
+/mnt/disk/xnext/fpga/.bm-work/00/board/astro/mcf5373l/mcf5373l.c:83:5: warning: pointer targets in passing argument 1 of 'get_ram_size' differ in signedness [-Wpointer-sign] +/mnt/disk/xnext/fpga/.bm-work/00/include/common.h:470:6: note: expected 'long int *' but argument is of type 'long unsigned int *' +/mnt/disk/xnext/fpga/.bm-work/00/board/astro/mcf5373l/fpga.c:168:2: warning: initialization from incompatible pointer type [enabled by default] +/mnt/disk/xnext/fpga/.bm-work/00/board/astro/mcf5373l/fpga.c:168:2: warning: (near initialization for 'altera_fns.write') [enabled by default] +/mnt/disk/xnext/fpga/.bm-work/00/board/astro/mcf5373l/fpga.c: In function 'astro5373l_altera_load':
+/mnt/disk/xnext/fpga/.bm-work/00/board/astro/mcf5373l/fpga.c:196:20: warning: assignment from incompatible pointer type [enabled by default]
02: fpga: spartan2: Avoid CamelCase
03: fpga: spartan3: Avoid CamelCase
04: fpga: virtex2: Avoid CamelCase
05: fpga: xilinx: Avoid CamelCase for in Xilinx_desc
06: fpga: xilinx: Fix the rest of CamelCases
07: fpga: xilinx: Simplify load/dump/info function handling
08: fpga: zynq: Remove sparse warnings
09: fpga: zynq: Use helper functions for zynq dma
10: fpga: zynq: Use helper function zynq_validate_bitstream

Changes in v2:
- Fix compilation errors in Astro
- Exchange fpga_op and name in xilinx_desc because spartan2/3, virtex2
are not using names.
- Fix warning reported by Tom
board/spear/x600/fpga.c:177:2: warning: initialization from incompatible pointer type [enabled by default] board/spear/x600/fpga.c:177:2: warning: (near initialization for ?fpga[0].name?) [enabled by default]
- Add fix for apf27 and astro

Michal Simek (7):
fpga: spartan2: Avoid CamelCase
fpga: spartan3: Avoid CamelCase
fpga: virtex2: Avoid CamelCase
fpga: xilinx: Avoid CamelCase for in Xilinx_desc
fpga: xilinx: Fix the rest of CamelCases
fpga: xilinx: Simplify load/dump/info function handling
fpga: zynq: Remove sparse warnings

Siva Durga Prasad Paladugu (2):
fpga: zynq: Use helper functions for zynq dma
fpga: zynq: Use helper function zynq_validate_bitstream

 board/armadeus/apf27/fpga.c          |   7 +-
 board/astro/mcf5373l/fpga.c          |  47 +++----
 board/balloon3/balloon3.c            |   4 +-
 board/esd/pmc440/fpga.c              |   8 +-
 board/gen860t/fpga.c                 |   6 +-
 board/matrix_vision/mvsmr/fpga.c     |   6 +-
 board/spear/x600/fpga.c              |   4 +-
 board/teejet/mt_ventoux/mt_ventoux.c |   4 +-
 board/xilinx/zynq/board.c            |  14 +--
 drivers/fpga/spartan2.c              |  46 ++++---
 drivers/fpga/spartan3.c              |  46 ++++---
 drivers/fpga/virtex2.c               |  40 +++---
 drivers/fpga/xilinx.c                | 179 +++------------------------
 drivers/fpga/zynqpl.c                | 234 +++++++++++++++++++++--------------
 include/spartan2.h                   |  72 ++++++-----
 include/spartan3.h                   |  84 ++++++-------
 include/virtex2.h                    |  66 +++++-----
 include/xilinx.h                     |  65 +++++-----
 include/zynqpl.h                     |  16 ++-
 19 files changed, 438 insertions(+), 510 deletions(-)

--
1.8.2.3

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20140507/2f84e57a/attachment.pgp>