Skip to content

Commit

Permalink
net: hns3: clean up a type mismatch warning
Browse files Browse the repository at this point in the history
[ Upstream commit e79c0e3 ]

abs() returns signed long, which could not convert the type
as unsigned, and it may cause a mismatch type warning from
static tools. To fix it, this patch uses an variable to save
the abs()'s result and does a explicit conversion.

Signed-off-by: Guojia Liao <liaoguojia@huawei.com>
Signed-off-by: Guangbin Huang <huangguangbin2@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Sasha Levin <sashal@kernel.org>
  • Loading branch information
Guojia Liao authored and gregkh committed Sep 18, 2021
1 parent fb1ee02 commit 2c304c6
Showing 1 changed file with 8 additions and 1 deletion.
9 changes: 8 additions & 1 deletion drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_mbx.c
Expand Up @@ -10,7 +10,14 @@

static u16 hclge_errno_to_resp(int errno)
{
return abs(errno);
int resp = abs(errno);

/* The status for pf to vf msg cmd is u16, constrainted by HW.
* We need to keep the same type with it.
* The intput errno is the stander error code, it's safely to
* use a u16 to store the abs(errno).
*/
return (u16)resp;
}

/* hclge_gen_resp_to_vf: used to generate a synchronous response to VF when PF
Expand Down

0 comments on commit 2c304c6

Please sign in to comment.