diff options
author | Peter Geis <pgwipeout@gmail.com> | 2022-09-02 14:45:42 -0400 |
---|---|---|
committer | Vinod Koul <vkoul@kernel.org> | 2022-09-04 20:53:40 +0530 |
commit | f340ed8664a55a467850ec1689996e63d9ee971a (patch) | |
tree | 49793a215bf43159f895f704fa7919e3f03155fe /virt | |
parent | 766ab0ded4c2cca84ac80c036ea38686ff43765b (diff) |
phy: rockchip-inno-usb2: Return zero after otg sync
The otg sync state patch reuses the ret variable, but fails to set it to
zero after use. This leads to a situation when the otg port is in
peripheral mode where the otg phy aborts halfway through setup. It also
fails to account for a failure to register the extcon notifier. Fix this
by using our own variable and skipping otg sync in case of failure.
Fixes: 8dc60f8da22f ("phy: rockchip-inno-usb2: Sync initial otg state")
Reported-by: Markus Reichl <m.reichl@fivetechno.de>
Reported-by: Michael Riesch <michael.riesch@wolfvision.net>
Signed-off-by: Peter Geis <pgwipeout@gmail.com>
Tested-by: Michael Riesch <michael.riesch@wolfvision.net>
Tested-by: Markus Reichl <m.reichl@fivetechno.de>
Reviewed-by: Samuel Holland <samuel@sholland.org>
Link: https://lore.kernel.org/r/20220902184543.1234835-1-pgwipeout@gmail.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
Diffstat (limited to 'virt')
0 files changed, 0 insertions, 0 deletions