Commit 0b161d0a authored by Viresh Kumar's avatar Viresh Kumar Committed by Greg Kroah-Hartman

greybus: es1: create svc connection early enough

The svc connection needs to be ready before creating the URBs, otherwise
the svc version request might come in before the AP was ready to parse
them.
Signed-off-by: default avatarViresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@google.com>
parent d45b1b86
......@@ -563,6 +563,12 @@ static int ap_probe(struct usb_interface *interface,
return PTR_ERR(hd);
}
/* Initialize AP's greybus interface */
if (!gb_ap_svc_connection_create(hd)) {
retval = -EINVAL;
goto error;
}
es1 = hd_to_es1(hd);
es1->hd = hd;
es1->usb_intf = interface;
......@@ -632,12 +638,6 @@ static int ap_probe(struct usb_interface *interface,
es1->cport_out_urb_busy[i] = false; /* just to be anal */
}
/* Initialize AP's greybus interface */
if (!gb_ap_svc_connection_create(hd)) {
retval = -EINVAL;
goto error;
}
apb1_log_enable_dentry = debugfs_create_file("apb1_log_enable",
(S_IWUSR | S_IRUGO),
gb_debugfs_get(), es1,
......
......@@ -663,6 +663,12 @@ static int ap_probe(struct usb_interface *interface,
return PTR_ERR(hd);
}
/* Initialize AP's greybus interface */
if (!gb_ap_svc_connection_create(hd)) {
retval = -EINVAL;
goto error;
}
es1 = hd_to_es1(hd);
es1->hd = hd;
es1->usb_intf = interface;
......@@ -736,12 +742,6 @@ static int ap_probe(struct usb_interface *interface,
es1->cport_out_urb_busy[i] = false; /* just to be anal */
}
/* Initialize AP's greybus interface */
if (!gb_ap_svc_connection_create(hd)) {
retval = -EINVAL;
goto error;
}
apb1_log_enable_dentry = debugfs_create_file("apb1_log_enable",
(S_IWUSR | S_IRUGO),
gb_debugfs_get(), es1,
......
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