public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Michal Simek <michal.simek@xilinx.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2 0/9] FPGA subsystem changes
Date: Wed,  7 May 2014 15:03:24 +0200	[thread overview]
Message-ID: <cover.1399467802.git.michal.simek@xilinx.com> (raw)

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>

             reply	other threads:[~2014-05-07 13:03 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-05-07 13:03 Michal Simek [this message]
2014-05-07 13:03 ` [U-Boot] [PATCH v2 1/9] fpga: spartan2: Avoid CamelCase Michal Simek
2014-05-07 13:03 ` [U-Boot] [PATCH v2 2/9] fpga: spartan3: " Michal Simek
2014-05-07 13:03 ` [U-Boot] [PATCH v2 3/9] fpga: virtex2: " Michal Simek
2014-05-07 13:03 ` [U-Boot] [PATCH v2 4/9] fpga: xilinx: Avoid CamelCase for in Xilinx_desc Michal Simek
2014-05-07 13:03 ` [U-Boot] [PATCH v2 5/9] fpga: xilinx: Fix the rest of CamelCases Michal Simek
2014-05-07 13:03 ` [U-Boot] [PATCH v2 6/9] fpga: xilinx: Simplify load/dump/info function handling Michal Simek
2014-05-07 13:03 ` [U-Boot] [PATCH v2 7/9] fpga: zynq: Remove sparse warnings Michal Simek
2014-05-07 13:03 ` [U-Boot] [PATCH v2 8/9] fpga: zynq: Use helper functions for zynq dma Michal Simek
2014-05-07 13:03 ` [U-Boot] [PATCH v2 9/9] fpga: zynq: Use helper function zynq_validate_bitstream Michal Simek

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=cover.1399467802.git.michal.simek@xilinx.com \
    --to=michal.simek@xilinx.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox