Commit 4c7d9c69 authored by Wang Wensheng's avatar Wang Wensheng Committed by Jason Gunthorpe
Browse files

IB/hfi1: Fix error return code in parse_platform_config() 

Fix to return a negative error code from the error handling case instead
of 0, as done elsewhere in this function.

Fixes: 77241056 ("IB/hfi1: add driver files")
Link: https://lore.kernel.org/r/20210408113140.103032-1-wangwensheng4@huawei.com


Reported-by: default avatarHulk Robot <hulkci@huawei.com>
Signed-off-by: default avatarWang Wensheng <wangwensheng4@huawei.com>
Signed-off-by: default avatarJason Gunthorpe <jgg@nvidia.com>
parent 10dd83db

drivers/infiniband/hw/hfi1/firmware.c

+1 −0
Original line number Diff line number Diff line
@@ -1916,6 +1916,7 @@ int parse_platform_config(struct hfi1_devdata *dd) 
			dd_dev_err(dd, "%s: Failed CRC check at offset %ld\n",

				   __func__, (ptr -

				   (u32 *)dd->platform_config.data));

			ret = -EINVAL;

			goto bail;

		}

		/* Jump the CRC DWORD */