On 08/28/2018 09:11 AM, Dinh Nguyen wrote:

On 08/28/2018 08:50 AM, richard.gong@linux.intel.com wrote:

quoted

From: Richard Gong <redacted>

Enable fpga framework, Stratix 10 SoC FPGA manager and Stratix10
Service Layer

Please update your commit header be "arm64: defconfig: enable fpga and
service layer"

Ok, I will make update in the next submission.

quoted

Signed-off-by: Richard Gong <redacted>
Signed-off-by: Alan Tull <atull@kernel.org>
---
v2: this patch is added in patch set version 2
v3: no change
v4: s/CONFIG_INTEL_SERVICE/CONFIG_STRATIX10_SERVICE/
     add CONFIG_OF_FPGA_REGION=y
     s/Intel/Stratix10/ in subject line
v5: no change
v6: no change
v7: no change
v8: no change
---
  arch/arm64/configs/defconfig | 7 +++++++
  1 file changed, 7 insertions(+)

diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
index f67e8d5..2ad50a5 100644
--- a/arch/arm64/configs/defconfig
+++ b/arch/arm64/configs/defconfig

@@ -185,6 +185,7 @@ CONFIG_BLK_DEV_LOOP=y
  CONFIG_BLK_DEV_NBD=m
  CONFIG_VIRTIO_BLK=y
  CONFIG_BLK_DEV_NVME=m
+CONFIG_STRATIX10_SERVICE=y

Can this be a module?

No, Service layer driver need be compiled as part of kernel image.

Service layer driver provides services for service clients, it must be 
loaded & initialized before service clients (FPGA manager and remote 
status update drivers).

quoted

  CONFIG_SRAM=y
  CONFIG_EEPROM_AT25=m
  # CONFIG_SCSI_PROC_FS is not set

@@ -642,6 +643,12 @@ CONFIG_PHY_TEGRA_XUSB=y
  CONFIG_HISI_PMU=y
  CONFIG_QCOM_L2_PMU=y
  CONFIG_QCOM_L3_PMU=y
+CONFIG_MESON_EFUSE=m

What does this change have to do with this patch?

My mistake, I will fix in the next submission.

quoted

+CONFIG_FPGA=y
+CONFIG_FPGA_MGR_STRATIX10_SOC=y
+CONFIG_FPGA_REGION=y
+CONFIG_FPGA_BRIDGE=y
+CONFIG_OF_FPGA_REGION=y

Can these be modules?

CONFIG_FPGA_MGR_STRATIX10_SOC could be module, but the remaining 
components need be compiled as part of kernel image.

Dinh