Re: [PATCH] drivers: fpga: intel_sdm_mb: Check SIP SMC status in send_bitstream()

[Michal Simek <michal.simek@amd.com>]

On 7/1/25 06:43, Naresh Kumar Ravulapalli wrote:
> While sending bitstream via SIP SMC, busy status received does not
> correspond to error, instead it means transfer is accepted but SDM
> doesn't have any more free buffer space. Hence, data transmission
> is continued when busy status is received.
> 
> Signed-off-by: Naresh Kumar Ravulapalli <nareshkumar.ravulapalli@altera.com>
> ---
>   drivers/fpga/intel_sdm_mb.c | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/fpga/intel_sdm_mb.c b/drivers/fpga/intel_sdm_mb.c
> index a2f3b160a73..5f4aae47d6d 100644
> --- a/drivers/fpga/intel_sdm_mb.c
> +++ b/drivers/fpga/intel_sdm_mb.c
> @@ -687,7 +687,8 @@ static int send_bitstream(const void *rbf_data, size_t rbf_size)
>   			debug("wr_ret = %d, rbf_data = %p, buf_size = %08lx\n",
>   			      wr_ret, rbf_data, buf_size);
>   
> -			if (wr_ret)
> +			if (wr_ret != INTEL_SIP_SMC_STATUS_OK &&
> +			    wr_ret != INTEL_SIP_SMC_STATUS_BUSY)
>   				continue;
>   
>   			rbf_size -= buf_size;

Applied.
M