| Message ID | 1535464245-11638-8-git-send-email-richard.gong@linux.intel.com (mailing list archive) |
|---|---|
| State | Superseded, archived |
| Headers | show |
| Series | Add Intel Stratix10 FPGA manager and service layer | expand |
On 08/28/2018 08:50 AM, richard.gong@linux.intel.com wrote: > From: Richard Gong <richard.gong@intel.com> > > 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" > Signed-off-by: Richard Gong <richard.gong@intel.com> > 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? > 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? > +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? Dinh
On 08/28/2018 09:11 AM, Dinh Nguyen wrote: > > > On 08/28/2018 08:50 AM, richard.gong@linux.intel.com wrote: >> From: Richard Gong <richard.gong@intel.com> >> >> 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. > > >> Signed-off-by: Richard Gong <richard.gong@intel.com> >> 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). > >> 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. > >> +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 >
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 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 +CONFIG_FPGA=y +CONFIG_FPGA_MGR_STRATIX10_SOC=y +CONFIG_FPGA_REGION=y +CONFIG_FPGA_BRIDGE=y +CONFIG_OF_FPGA_REGION=y CONFIG_QCOM_QFPROM=y CONFIG_ROCKCHIP_EFUSE=y CONFIG_UNIPHIER_EFUSE=y