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, ¶m, len, (u32 *)p_buf); > + > + return rc; 'rc' is completely unnecessary, please just return the function result directly. Thank you.