| commit | 2ce1929c9c1ef7f33a4d3a5971986de189d1af28 | [log] [tgz] |
|---|---|---|
| author | Dan Carpenter <dan.carpenter@oracle.com> | Wed Sep 25 14:05:54 2019 +0300 |
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | Wed Jan 29 10:24:33 2020 +0100 |
| tree | 8021b740b8adad41f110754414ecc761f8932996 | |
| parent | 570082bdbc41df50ed49b5aa066902874f205b55 [diff] |
net: ethernet: stmmac: Fix signedness bug in ipq806x_gmac_of_parse()
[ Upstream commit 231042181dc9d6122c6faba64e99ccb25f13cc6c ]
The "gmac->phy_mode" variable is an enum and in this context GCC will
treat it as an unsigned int so the error handling will never be
triggered.
Fixes: b1c17215d718 ("stmmac: add ipq806x glue layer")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Sasha Levin <sashal@kernel.org>