Commit 1305f2b2 authored by Wei Yongjun's avatar Wei Yongjun Committed by Greg Kroah-Hartman

greybus: es2: fix error return code in ap_probe()

Fix to return a negative error code from the es2_arpc_in_enable() error
handling case instead of 0, as done elsewhere in this function.

Fixes: 9d9d3777 ("greybus: es2: Add a new bulk in endpoint for APBridgeA RPC")
Signed-off-by: default avatarWei Yongjun <weiyongjun1@huawei.com>
Acked-by: default avatarViresh Kumar <viresh.kumar@linaro.org>
Reviewed-by: default avatarJohan Hovold <johan@kernel.org>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 25633d1f
...@@ -1548,7 +1548,8 @@ static int ap_probe(struct usb_interface *interface, ...@@ -1548,7 +1548,8 @@ static int ap_probe(struct usb_interface *interface,
INIT_LIST_HEAD(&es2->arpcs); INIT_LIST_HEAD(&es2->arpcs);
spin_lock_init(&es2->arpc_lock); spin_lock_init(&es2->arpc_lock);
if (es2_arpc_in_enable(es2)) retval = es2_arpc_in_enable(es2);
if (retval)
goto error; goto error;
retval = gb_hd_add(hd); retval = gb_hd_add(hd);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment