aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorWei Yongjun <weiyongjun1@huawei.com>2016-10-19 09:17:53 -0400
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2016-10-24 09:51:00 -0400
commit1305f2b2f52af5986f44dfbb1a6fe58ae875aa61 (patch)
treec608d044988da35c40123a7779d83a4108841ae8
parent25633d1f5dd7ea35c77aae12c039a80e46abec01 (diff)
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: 9d9d3777a9db ("greybus: es2: Add a new bulk in endpoint for APBridgeA RPC") Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com> Acked-by: Viresh Kumar <viresh.kumar@linaro.org> Reviewed-by: Johan Hovold <johan@kernel.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r--drivers/staging/greybus/es2.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/staging/greybus/es2.c b/drivers/staging/greybus/es2.c
index 071bb1cfd3ae..baab460eeaa3 100644
--- a/drivers/staging/greybus/es2.c
+++ b/drivers/staging/greybus/es2.c
@@ -1548,7 +1548,8 @@ static int ap_probe(struct usb_interface *interface,
1548 INIT_LIST_HEAD(&es2->arpcs); 1548 INIT_LIST_HEAD(&es2->arpcs);
1549 spin_lock_init(&es2->arpc_lock); 1549 spin_lock_init(&es2->arpc_lock);
1550 1550
1551 if (es2_arpc_in_enable(es2)) 1551 retval = es2_arpc_in_enable(es2);
1552 if (retval)
1552 goto error; 1553 goto error;
1553 1554
1554 retval = gb_hd_add(hd); 1555 retval = gb_hd_add(hd);