+ Alejandro
On Mon, Apr 07, 2025 at 09:11:10PM +0800, Wentao Liang wrote:
The function efx_devlink_info_board_cfg() calls the function devlink_info_serial_number_put(), but does not check its return value.
Return the error code if either the devlink_info_serial_number_put() or the efx_mcdi_get_board_cfg() fails.The control flow of the code is changed a little bit to simplify the code. The functionality of the code remain the same.
Fixes: 14743ddd2495 ("sfc: add devlink info support for ef100") Cc: stable@vger.kernel.org # v6.3+ Signed-off-by: Wentao Liang vulab@iscas.ac.cn
Hi Wentao,
In his review of v1 Edward Cree said:
"Looking at the rest of the file, all the calls to devlink_info_*_put() in this driver ignore the return value, not just this one. I think this may have been an intentional decision to only report errors in getting the info from FW, which seems reasonable to me. "If not, then all the calls need fixing, not just this one. CCing Alejandro, original author of this code, for his opinion.
I have CCed Aljandro on this email to see if he can help. And in any case, I think we need to come to a consensus on Ed's point before moving forwards with this patch.