[U-Boot] [PATCH v3 0/4] Add Intel Arria 10 SoC FPGA driver

[Marek Vasut]

On 05/17/2017 07:12 AM, Chee, Tien Fong wrote:
> On Isn, 2017-05-15 at 18:52 +0800, tien.fong.chee at intel.com wrote:
>> From: Tien Fong Chee <tien.fong.chee at 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@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@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?

I'd like A-B/R-B from Dinh/Ley first.

-- 
Best regards,
Marek Vasut