From: Chee, Tien Fong <tien.fong.chee@intel.com>
Date: Wed, 17 May 2017 05:12:13 +0000
Subject: [U-Boot] [PATCH v3 0/4] Add Intel Arria 10 SoC FPGA driver
In-Reply-To: <1494845579-16110-1-git-send-email-tien.fong.chee@intel.com>
References: <1494845579-16110-1-git-send-email-tien.fong.chee@intel.com>
Message-ID: <1494997932.24462.1.camel@intel.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
To: u-boot@lists.denx.de

On Isn, 2017-05-15 at 18:52 +0800, tien.fong.chee at intel.com wrote:
> From: Tien Fong Chee <tien.fong.chee@intel.com>
> 
> This is the 3rd version of patchset to adds support for Intel Arria
> 10 SoC FPGA
> driver. This version mainly resolved comments from Marek in [v2].This
> series is
> working on top of u-boot-socfpga-next branch
> http://git.denx.de/?p=u-boot/u-boot-socfpga.git;a=shortlog;h=refs/hea
> ds/next.
> 
> [v2]: https://www.mail-archive.com/u-boot at lists.denx.de/msg248541.htm
> l
> 
> v2 -> v3 changes:
> -----------------
> - Changed the commit messages on patch 1.
> - Changed to size_t for size, moved timeout #define into c files,
> reverted deleted
>   line, and fixing copyright extending issue on patch 2.
> - No change on patch 3.
> - Reverted all fpga header files back to arch/arm/mach-socfpga/.
> - Changed unsigned int to u32, size_t for size, and fixing
> indentation on patch 4.
> 
> Patchset history
> ----------------
> [v1]: https://www.mail-archive.com/u-boot at lists.denx.de/msg247788.htm
> l
> 
> Tien Fong Chee (4):
>   arm: socfpga: Remove unused passing parameter of
> socfpga_bridges_reset
>   arm: socfpga: Restructure FPGA driver in the preparation to support
>     A10.
>   arm: socfpga: Move FPGA manager driver to FPGA driver
>   arm: socfpga: Add FPGA driver support for Arria 10
> 
>  arch/arm/mach-socfpga/Makefile                     |   1 -
>  arch/arm/mach-socfpga/fpga_manager.c               |  78 ----
>  arch/arm/mach-socfpga/include/mach/fpga_manager.h  |  70 +--
>  .../include/mach/fpga_manager_arria10.h            | 100 +++++
>  .../mach/{fpga_manager.h => fpga_manager_gen5.h}   |  69 ++-
>  .../include/mach/reset_manager_arria10.h           |   2 +-
>  arch/arm/mach-socfpga/reset_manager_arria10.c      |   4 +-
>  drivers/Makefile                                   |   1 +
>  drivers/fpga/Makefile                              |   2 +
>  drivers/fpga/socfpga.c                             | 241 +---------
>  drivers/fpga/socfpga_arria10.c                     | 487
> +++++++++++++++++++++
>  drivers/fpga/{socfpga.c => socfpga_gen5.c}         |  98 +++--
>  include/configs/socfpga_common.h                   |   3 +-
>  13 files changed, 693 insertions(+), 463 deletions(-)
>  delete mode 100644 arch/arm/mach-socfpga/fpga_manager.c
>  create mode 100644 arch/arm/mach-
> socfpga/include/mach/fpga_manager_arria10.h
>  copy arch/arm/mach-socfpga/include/mach/{fpga_manager.h =>
> fpga_manager_gen5.h} (57%)
>  create mode 100644 drivers/fpga/socfpga_arria10.c
>  copy drivers/fpga/{socfpga.c => socfpga_gen5.c} (85%)
> 
Hi Marek, this whole patchset looks okay?

Thanks.