Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 00c0923f authored by Wang Wensheng's avatar Wang Wensheng Committed by Greg Kroah-Hartman
Browse files

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

[ Upstream commit 4c7d9c69adadfc31892c7e8e134deb3546552106 ]

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>
Signed-off-by: default avatarSasha Levin <sashal@kernel.org>
parent 8e241e72

drivers/infiniband/hw/hfi1/firmware.c

+1 −0
Original line number Diff line number Diff line
@@ -1924,6 +1924,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 */