From: Sudarsana Reddy Kalluru <skall...@marvell.com>
Date: Fri, 26 Jul 2019 08:52:14 -0700

> +int qed_mcp_nvm_set_cfg(struct qed_hwfn *p_hwfn, struct qed_ptt *p_ptt,
> +                     u16 option_id, u8 entity_id, u16 flags, u8 *p_buf,
> +                     u32 len)
> +{
> +     u32 mb_param = 0, resp, param;
> +     int rc;
 ...
> +     rc = qed_mcp_nvm_wr_cmd(p_hwfn, p_ptt,
> +                             DRV_MSG_CODE_SET_NVM_CFG_OPTION,
> +                             mb_param, &resp, &param, len, (u32 *)p_buf);
> +
> +     return rc;

'rc' is completely unnecessary, please just return the function result
directly.

Thank you.

Reply via email to