diff options
| author | Xu Yang <xu.yang_2@nxp.com> | 2022-12-15 13:54:09 +0800 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2023-01-17 17:05:02 +0100 |
| commit | f96c0384047257365371a8ac217107c0371586f1 (patch) | |
| tree | c14149c6393653589ab826d7c47223e04555936d /drivers/usb/isp1760 | |
| parent | 74622f0a81d0c2bcfc39f9192b788124e8c7f0af (diff) | |
usb: chipidea: core: fix possible constant 0 if use IS_ERR(ci->role_switch)
After successfully probed, ci->role_switch would only be NULL or a valid
pointer. IS_ERR(ci->role_switch) will always return 0. So no need to wrap
it with IS_ERR, otherwise the logic is wrong.
Fixes: e1b5d2bed67c ("usb: chipidea: core: handle usb role switch in a common way")
cc: <stable@vger.kernel.org>
Signed-off-by: Xu Yang <xu.yang_2@nxp.com>
Link: https://lore.kernel.org/r/20221215055409.3760523-1-xu.yang_2@nxp.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/usb/isp1760')
0 files changed, 0 insertions, 0 deletions