On 05/12/2017 05:33 AM, Chee, Tien Fong wrote:
On Kha, 2017-05-11 at 14:04 +0200, Marek Vasut wrote:
On 05/11/2017 11:25 AM, tien.fong.chee@intel.com wrote:
From: Tien Fong Chee tien.fong.chee@intel.com
Move arch/arm FPGA driver header to include/intel_socfpga which would be shared between arch platform drivers and FPGA drivers.
Doesn't this patchset only add support for arria10 FPGA ? If so, this patch is pointless in the context of this patchset.
Arria10 FPGA header include fpga_manager.h as common header.There are other headers depend on above common header. So that means they are all related to each others. Hence this patch is needed to ensure all related restructure properly before adding Arria10 FPGA driver.
But that stuff can still be in arch/arm/mach-socfpga/include , why do you move stuff into include/intel_fpga ?
Signed-off-by: Tien Fong Chee tien.fong.chee@intel.com
arch/arm/mach-socfpga/reset_manager_arria10.c | 2 +- arch/arm/mach-socfpga/reset_manager_gen5.c | 5 +- arch/arm/mach-socfpga/system_manager_gen5.c | 2 +- drivers/ddr/altera/sdram.c | 5 +- drivers/fpga/socfpga.c | 5 +- drivers/fpga/socfpga_gen5.c | 2 +- include/intel_socfpga/fpga_manager.h | 37 +++++++++++++++ include/intel_socfpga/fpga_manager_gen5.h | 68 +++++++++++++++++++++++++++ 8 files changed, 116 insertions(+), 10 deletions(-) create mode 100644 include/intel_socfpga/fpga_manager.h create mode 100644 include/intel_socfpga/fpga_manager_gen5.h
[...]