Message ID | 20200424090850.96778-1-zhengbin13@huawei.com |
---|---|
State | Accepted |
Delegated to: | David Miller |
Headers | show |
Series | [-next] net: phy: dp83867: Remove unneeded semicolon | expand |
On Fri, Apr 24, 2020 at 05:08:50PM +0800, Zheng Bin wrote: > Fixes coccicheck warning: > > drivers/net/phy/dp83867.c:368:2-3: Unneeded semicolon > drivers/net/phy/dp83867.c:403:2-3: Unneeded semicolon > > Reported-by: Hulk Robot <hulkci@huawei.com> > Signed-off-by: Zheng Bin <zhengbin13@huawei.com> Reviewed-by: Andrew Lunn <andrew@lunn.ch> Andrew
From: Zheng Bin <zhengbin13@huawei.com> Date: Fri, 24 Apr 2020 17:08:50 +0800 > Fixes coccicheck warning: > > drivers/net/phy/dp83867.c:368:2-3: Unneeded semicolon > drivers/net/phy/dp83867.c:403:2-3: Unneeded semicolon > > Reported-by: Hulk Robot <hulkci@huawei.com> > Signed-off-by: Zheng Bin <zhengbin13@huawei.com> Applied, thanks.
diff --git a/drivers/net/phy/dp83867.c b/drivers/net/phy/dp83867.c index b55e3c0403ed..4017ae1692d8 100644 --- a/drivers/net/phy/dp83867.c +++ b/drivers/net/phy/dp83867.c @@ -365,7 +365,7 @@ static int dp83867_get_downshift(struct phy_device *phydev, u8 *data) break; default: return -EINVAL; - }; + } *data = enable ? count : DOWNSHIFT_DEV_DISABLE; @@ -400,7 +400,7 @@ static int dp83867_set_downshift(struct phy_device *phydev, u8 cnt) phydev_err(phydev, "Downshift count must be 1, 2, 4 or 8\n"); return -EINVAL; - }; + } val = DP83867_DOWNSHIFT_EN; val |= FIELD_PREP(DP83867_DOWNSHIFT_ATTEMPT_MASK, count);
Fixes coccicheck warning: drivers/net/phy/dp83867.c:368:2-3: Unneeded semicolon drivers/net/phy/dp83867.c:403:2-3: Unneeded semicolon Reported-by: Hulk Robot <hulkci@huawei.com> Signed-off-by: Zheng Bin <zhengbin13@huawei.com> --- drivers/net/phy/dp83867.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) -- 2.26.0.106.g9fadedd