[PATCH v3] fpga: intel_sdm_mb: Add checking for INTEL_SIP_SMC_STATUS_BUSY (1)

[Chee, Tien Fong]

> -----Original Message-----
> From: Maniyam, Dinesh <dinesh.maniyam at intel.com>
> Sent: Tuesday, 31 May, 2022 3:51 PM
> To: u-boot at lists.denx.de
> Cc: Michal Simek <michal.simek at amd.com>; Chee, Tien Fong
> <tien.fong.chee at intel.com>; Hea, Kok Kiang <kok.kiang.hea at intel.com>;
> Gan, Yau Wai <yau.wai.gan at intel.com>; Kho, Sin Hui
> <sin.hui.kho at intel.com>; Lokanathan, Raaj <raaj.lokanathan at intel.com>;
> Maniyam, Dinesh <dinesh.maniyam at intel.com>
> Subject: [PATCH v3] fpga: intel_sdm_mb: Add checking for
> INTEL_SIP_SMC_STATUS_BUSY (1)
> 
> From: Dinesh Maniyam <dinesh.maniyam at intel.com>
> 
> Status busy means transfer is accepted but SDM does not have more freed
> buffer. It is not an error. Continue process the data if receive OK or BUSY
> status.
> 
> Signed-off-by: Dinesh Maniyam <dinesh.maniyam at intel.com>
> 
> ---
> 
> v2->v3
> -Copyright year updated and alignment adjusted.
> ---
>  drivers/fpga/intel_sdm_mb.c | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/fpga/intel_sdm_mb.c b/drivers/fpga/intel_sdm_mb.c
> index f5fd9a14c2..44ef7717d0 100644
> --- a/drivers/fpga/intel_sdm_mb.c
> +++ b/drivers/fpga/intel_sdm_mb.c
> @@ -1,6 +1,6 @@
>  // SPDX-License-Identifier: GPL-2.0+
>  /*
> - * Copyright (C) 2018 Intel Corporation <www.intel.com>
> + * Copyright (C) 2018-2022 Intel Corporation <www.intel.com>
>   */
> 
>  #include <common.h>
> @@ -70,7 +70,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;
> --
> 2.25.1

Reviewed-by: Tien Fong Chee <tien.fong.chee at intel.com>

Regards
Tien Fong