andrius
/
arm-trusted-firmware
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

fix(intel): fix fpga config write return mechanism 

This revert commit 279c8015fefcb544eb311b9052f417fc02ab84aa.
The previous change breaks this feature compatibility with Linux driver.
Hence, the fix for the earlier issue is going to be fixed in uboot instead.

Signed-off-by: Abdul Halim, Muhammad Hadi Asyrafi <muhammad.hadi.asyrafi.abdul.halim@intel.com>
Signed-off-by: Jit Loon Lim <jit.loon.lim@intel.com>
Signed-off-by: Sieu Mun Tang <sieu.mun.tang@intel.com>
Change-Id: I93220243bad65ed53322050d990544c7df4ce66b
This commit is contained in:
Abdul Halim, Muhammad Hadi Asyrafi 2020-11-05 18:00:03 +08:00 committed by Sieu Mun Tang
parent 984e236e0d
commit ef51b097bf
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
plat/intel/soc/common/socfpga_sip_svc.c
View File

@ -270,8 +270,9 @@ static uint32_t intel_fpga_config_write(uint64_t mem, uint64_t size)
intel_fpga_sdm_write_all();
if (!is_address_in_ddr_range(mem, size) ||
is_fpga_config_buffer_full())
is_fpga_config_buffer_full()) {
return INTEL_SIP_SMC_STATUS_REJECTED;
}
for (i = 0; i < FPGA_CONFIG_BUFFER_SIZE; i++) {
int j = (i + current_buffer) % FPGA_CONFIG_BUFFER_SIZE;
@ -288,8 +289,9 @@ static uint32_t intel_fpga_config_write(uint64_t mem, uint64_t size)
}
}
if (is_fpga_config_buffer_full())
if (is_fpga_config_buffer_full()) {
return INTEL_SIP_SMC_STATUS_BUSY;
}
return INTEL_SIP_SMC_STATUS_OK;
}