Commit eb765e4e authored by Alex Elder's avatar Alex Elder Committed by Greg Kroah-Hartman

greybus: core: don't set up endo until host device is initialized

Currently, the data structure representing an Endo is set up at the
time a host device gets created.  This is too early.

Once the control infrastructure is in place, there's no sense in
setting up the Endo utnil after we have heard from the SVC via a
probe operation on our control CPort.  And even then, there's
no real point until we've successfully authenticated with the SVC,
which will be indicated by the arrival of the Control protocol
"connected" operation request notifying us that our SVC CPort
is operational.

In addition to this logical argument, we also can't actually
receive any messages on the Control CPort until the host device
is set up and ready to receive messages.  At the point we're
currently setting up the Endo data structure, that has not yet
been done.

Define a new exported function greybus_endo_setup(), which will
be used (for now) as the entry point for setting up the Endo
data structure.  Arrange to call it in the host USB driver
probe method, *after* we are set up for handling messages.

Note: Once the control protocol has been implemented, this function
may no longer need to be exported.
Signed-off-by: default avatarAlex Elder <elder@linaro.org>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@google.com>
parent 344943d2
...@@ -177,9 +177,6 @@ struct greybus_host_device *greybus_create_hd(struct greybus_host_driver *driver ...@@ -177,9 +177,6 @@ struct greybus_host_device *greybus_create_hd(struct greybus_host_driver *driver
size_t buffer_size_max) size_t buffer_size_max)
{ {
struct greybus_host_device *hd; struct greybus_host_device *hd;
struct gb_endo *endo;
u16 endo_id = 0x4755; // FIXME - get endo "ID" from the SVC
u8 ap_intf_id = 0x01; // FIXME - get AP interface ID from the SVC
/* /*
* Validate that the driver implements all of the callbacks * Validate that the driver implements all of the callbacks
...@@ -213,16 +210,23 @@ struct greybus_host_device *greybus_create_hd(struct greybus_host_driver *driver ...@@ -213,16 +210,23 @@ struct greybus_host_device *greybus_create_hd(struct greybus_host_driver *driver
ida_init(&hd->cport_id_map); ida_init(&hd->cport_id_map);
hd->buffer_size_max = buffer_size_max; hd->buffer_size_max = buffer_size_max;
return hd;
}
EXPORT_SYMBOL_GPL(greybus_create_hd);
int greybus_endo_setup(struct greybus_host_device *hd, u16 endo_id,
u8 ap_intf_id)
{
struct gb_endo *endo;
endo = gb_endo_create(hd, endo_id, ap_intf_id); endo = gb_endo_create(hd, endo_id, ap_intf_id);
if (IS_ERR(endo)) { if (IS_ERR(endo))
greybus_remove_hd(hd); return PTR_ERR(endo);
return ERR_CAST(endo);
}
hd->endo = endo; hd->endo = endo;
return hd; return 0;
} }
EXPORT_SYMBOL_GPL(greybus_create_hd); EXPORT_SYMBOL_GPL(greybus_endo_setup);
void greybus_remove_hd(struct greybus_host_device *hd) void greybus_remove_hd(struct greybus_host_device *hd)
{ {
......
...@@ -551,6 +551,8 @@ static int ap_probe(struct usb_interface *interface, ...@@ -551,6 +551,8 @@ static int ap_probe(struct usb_interface *interface,
bool bulk_out_found = false; bool bulk_out_found = false;
int retval = -ENOMEM; int retval = -ENOMEM;
int i; int i;
u16 endo_id = 0x4755; // FIXME - get endo "ID" from the SVC
u8 ap_intf_id = 0x01; // FIXME - get endo "ID" from the SVC
u8 svc_interval = 0; u8 svc_interval = 0;
udev = usb_get_dev(interface_to_usbdev(interface)); udev = usb_get_dev(interface_to_usbdev(interface));
...@@ -659,6 +661,17 @@ static int ap_probe(struct usb_interface *interface, ...@@ -659,6 +661,17 @@ static int ap_probe(struct usb_interface *interface,
gb_debugfs_get(), es1, gb_debugfs_get(), es1,
&apb1_log_enable_fops); &apb1_log_enable_fops);
/*
* XXX Soon this will be initiated later, with a combination
* XXX of a Control protocol probe operation and a
* XXX subsequent Control protocol connected operation for
* XXX the SVC connection. At that point we know we're
* XXX properly connected to an Endo.
*/
retval = greybus_endo_setup(hd, endo_id, ap_intf_id);
if (retval)
goto error;
return 0; return 0;
error: error:
ap_disconnect(interface); ap_disconnect(interface);
......
...@@ -551,6 +551,8 @@ static int ap_probe(struct usb_interface *interface, ...@@ -551,6 +551,8 @@ static int ap_probe(struct usb_interface *interface,
bool bulk_out_found = false; bool bulk_out_found = false;
int retval = -ENOMEM; int retval = -ENOMEM;
int i; int i;
u16 endo_id = 0x4755; // FIXME - get endo "ID" from the SVC
u8 ap_intf_id = 0x01; // FIXME - get endo "ID" from the SVC
u8 svc_interval = 0; u8 svc_interval = 0;
udev = usb_get_dev(interface_to_usbdev(interface)); udev = usb_get_dev(interface_to_usbdev(interface));
...@@ -659,6 +661,17 @@ static int ap_probe(struct usb_interface *interface, ...@@ -659,6 +661,17 @@ static int ap_probe(struct usb_interface *interface,
gb_debugfs_get(), es1, gb_debugfs_get(), es1,
&apb1_log_enable_fops); &apb1_log_enable_fops);
/*
* XXX Soon this will be initiated later, with a combination
* XXX of a Control protocol probe operation and a
* XXX subsequent Control protocol connected operation for
* XXX the SVC connection. At that point we know we're
* XXX properly connected to an Endo.
*/
retval = greybus_endo_setup(hd, endo_id, ap_intf_id);
if (retval)
goto error;
return 0; return 0;
error: error:
ap_disconnect(interface); ap_disconnect(interface);
......
...@@ -108,6 +108,8 @@ struct greybus_host_device { ...@@ -108,6 +108,8 @@ struct greybus_host_device {
struct greybus_host_device *greybus_create_hd(struct greybus_host_driver *hd, struct greybus_host_device *greybus_create_hd(struct greybus_host_driver *hd,
struct device *parent, struct device *parent,
size_t buffer_size_max); size_t buffer_size_max);
int greybus_endo_setup(struct greybus_host_device *hd, u16 endo_id,
u8 ap_intf_id);
void greybus_remove_hd(struct greybus_host_device *hd); void greybus_remove_hd(struct greybus_host_device *hd);
struct greybus_driver { struct greybus_driver {
......
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