Commit d7ea30a5 authored by Johan Hovold's avatar Johan Hovold Committed by Greg Kroah-Hartman

greybus: hd: add optional cport enable and disable callbacks

Add optional cport enable and disable callbacks to the greybus host
drivers, that can be used to initialise and allocate/release resources
associated with a cport during connection setup/teardown (e.g. software
queues and hardware state).
Signed-off-by: default avatarJohan Hovold <johan@hovoldconsulting.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@google.com>
parent 2846d3ed
...@@ -284,6 +284,34 @@ gb_connection_create_range(struct greybus_host_device *hd, ...@@ -284,6 +284,34 @@ gb_connection_create_range(struct greybus_host_device *hd,
return NULL; return NULL;
} }
static int gb_connection_hd_cport_enable(struct gb_connection *connection)
{
struct greybus_host_device *hd = connection->hd;
int ret;
if (!hd->driver->cport_enable)
return 0;
ret = hd->driver->cport_enable(hd, connection->hd_cport_id);
if (ret) {
dev_err(&connection->dev,
"failed to enable host cport: %d\n", ret);
return ret;
}
return 0;
}
static void gb_connection_hd_cport_disable(struct gb_connection *connection)
{
struct greybus_host_device *hd = connection->hd;
if (!hd->driver->cport_disable)
return;
hd->driver->cport_disable(hd, connection->hd_cport_id);
}
struct gb_connection *gb_connection_create(struct gb_bundle *bundle, struct gb_connection *gb_connection_create(struct gb_bundle *bundle,
u16 cport_id, u8 protocol_id) u16 cport_id, u8 protocol_id)
{ {
...@@ -439,10 +467,14 @@ static int gb_connection_init(struct gb_connection *connection) ...@@ -439,10 +467,14 @@ static int gb_connection_init(struct gb_connection *connection)
struct gb_protocol *protocol = connection->protocol; struct gb_protocol *protocol = connection->protocol;
int ret; int ret;
ret = gb_connection_svc_connection_create(connection); ret = gb_connection_hd_cport_enable(connection);
if (ret) if (ret)
return ret; return ret;
ret = gb_connection_svc_connection_create(connection);
if (ret)
goto err_hd_cport_disable;
ret = gb_connection_control_connected(connection); ret = gb_connection_control_connected(connection);
if (ret) if (ret)
goto err_svc_destroy; goto err_svc_destroy;
...@@ -470,6 +502,8 @@ static int gb_connection_init(struct gb_connection *connection) ...@@ -470,6 +502,8 @@ static int gb_connection_init(struct gb_connection *connection)
gb_connection_control_disconnected(connection); gb_connection_control_disconnected(connection);
err_svc_destroy: err_svc_destroy:
gb_connection_svc_connection_destroy(connection); gb_connection_svc_connection_destroy(connection);
err_hd_cport_disable:
gb_connection_hd_cport_disable(connection);
return ret; return ret;
} }
...@@ -492,6 +526,7 @@ static void gb_connection_exit(struct gb_connection *connection) ...@@ -492,6 +526,7 @@ static void gb_connection_exit(struct gb_connection *connection)
connection->protocol->connection_exit(connection); connection->protocol->connection_exit(connection);
gb_connection_control_disconnected(connection); gb_connection_control_disconnected(connection);
gb_connection_svc_connection_destroy(connection); gb_connection_svc_connection_destroy(connection);
gb_connection_hd_cport_disable(connection);
} }
/* /*
......
...@@ -75,6 +75,8 @@ struct greybus_host_device; ...@@ -75,6 +75,8 @@ struct greybus_host_device;
struct greybus_host_driver { struct greybus_host_driver {
size_t hd_priv_size; size_t hd_priv_size;
int (*cport_enable)(struct greybus_host_device *hd, u16 cport_id);
int (*cport_disable)(struct greybus_host_device *hd, u16 cport_id);
void (*connection_create)(struct gb_connection *connection); void (*connection_create)(struct gb_connection *connection);
void (*connection_destroy)(struct gb_connection *connection); void (*connection_destroy)(struct gb_connection *connection);
int (*message_send)(struct greybus_host_device *hd, u16 dest_cport_id, int (*message_send)(struct greybus_host_device *hd, u16 dest_cport_id,
......
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