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 v3 0/9] FPGA cleanup + zynq support
Date: Fri,  3 May 2013 09:14:07 +0200	[thread overview]
Message-ID: <cover.1367565248.git.michal.simek@xilinx.com> (raw)

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

-------------- 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/20130503/cb986704/attachment.pgp>

             reply	other threads:[~2013-05-03  7:14 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-05-03  7:14 Michal Simek [this message]
2013-05-03  7:14 ` [U-Boot] [PATCH v3 1/9] fpga: Clean coding style Michal Simek
2013-05-03  7:14 ` [U-Boot] [PATCH v3 2/9] fpga: Fix debug message compilation error Michal Simek
2013-05-03  7:14 ` [U-Boot] [PATCH v3 3/9] cmd: fpga: Clean coding style Michal Simek
2013-05-03  7:14 ` [U-Boot] [PATCH v3 4/9] cmd: fpga: Move fpga_loadbitstream to fpga.c Michal Simek
2013-05-03  7:14 ` [U-Boot] [PATCH v3 5/9] fpga: Change the first parameter in fpga_loadbitstream Michal Simek
2013-05-03  7:14 ` [U-Boot] [PATCH v3 6/9] cmd: fpga: Do not include net.h Michal Simek
2013-05-03  7:14 ` [U-Boot] [PATCH v3 7/9] fpga: zynq: Add support for loading bitstream Michal Simek
2013-05-03  7:14 ` [U-Boot] [PATCH v3 8/9] fpga: Check device name against bitstream name Michal Simek
2013-05-03  7:14 ` [U-Boot] [PATCH v3 9/9] fpga: Remove all CONFIG_SYS_* fpga related options Michal Simek
2013-05-03 19:50 ` [U-Boot] [PATCH v3 0/9] FPGA cleanup + zynq support Tom Rini

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.1367565248.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