diff options
| author |  Colin Ian King <colin.i.king@gmail.com>
| 2021-12-11 18:00:54 +0000 |
|---|---|---|
| committer |  Vinod Koul <vkoul@kernel.org>
| 2021-12-14 14:45:07 +0530 |
| commit | b156117aed1b9d192efcb27f5b37f78cd21fa545 (patch) | |
| tree | 33ffd1cfd5de3826858d6a98875f4446f1f0c4db /drivers/phy/rockchip | |
| parent | phy: lan966x: Remove set_speed function (diff) | |
| download | linux-b156117aed1b9d192efcb27f5b37f78cd21fa545.tar.gz linux-b156117aed1b9d192efcb27f5b37f78cd21fa545.tar.bz2 linux-b156117aed1b9d192efcb27f5b37f78cd21fa545.zip | |
phy: rockchip-inno-usb2: remove redundant assignment to variable delay
Variable delay is being assigned to zero and the code falls through to
the next case in a switch statement that returns out of the function.
The variable is never read in this scenario and so the assignment is
redundant and can be removed.
Cleans up scan-build static analysis warning:
drivers/phy/rockchip/phy-rockchip-inno-usb2.c:753:3: warning: Value
stored to 'delay' is never read [deadcode.DeadStores]
Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
Link: https://lore.kernel.org/r/20211211180054.525368-1-colin.i.king@gmail.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
Diffstat (limited to 'drivers/phy/rockchip')
| -rw-r--r-- | drivers/phy/rockchip/phy-rockchip-inno-usb2.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/phy/rockchip/phy-rockchip-inno-usb2.c b/drivers/phy/rockchip/phy-rockchip-inno-usb2.c index 1938365abbb3..9f95b587e2c0 100644 --- a/drivers/phy/rockchip/phy-rockchip-inno-usb2.c +++ b/drivers/phy/rockchip/phy-rockchip-inno-usb2.c @@ -750,7 +750,6 @@ static void rockchip_chg_detect_work(struct work_struct *work) fallthrough; case USB_CHG_STATE_SECONDARY_DONE: rphy->chg_state = USB_CHG_STATE_DETECTED; - delay = 0; fallthrough; case USB_CHG_STATE_DETECTED: /* put the controller in normal mode */ |