greybus: control: remove unused request handler
authorJohan Hovold <johan@hovoldconsulting.com>
Mon, 2 Nov 2015 10:56:59 +0000 (11:56 +0100)
committerGreg Kroah-Hartman <gregkh@google.com>
Mon, 2 Nov 2015 19:26:44 +0000 (11:26 -0800)
Remove request handler for control protocol that makes no sense as we do
not have any incoming control requests defined.

Signed-off-by: Johan Hovold <johan@hovoldconsulting.com>
Reviewed-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@google.com>
drivers/staging/greybus/control.c

index 630b5b6466673291a070a499b8c7aa3cdd17347e..ffc07a475845c14e6c37b9c30b440f2d7d7ac0a9 100644 (file)
@@ -60,40 +60,6 @@ int gb_control_disconnected_operation(struct gb_control *control, u16 cport_id)
                                 sizeof(request), NULL, 0);
 }
 
-static int gb_control_request_recv(u8 type, struct gb_operation *op)
-{
-       struct gb_connection *connection = op->connection;
-       struct gb_protocol_version_response *version;
-
-       switch (type) {
-       case GB_CONTROL_TYPE_PROBE_AP:
-               // TODO
-               // Send authenticated block of data, confirming this module is
-               // an AP.
-               break;
-       case GB_REQUEST_TYPE_PROTOCOL_VERSION:
-               if (!gb_operation_response_alloc(op, sizeof(*version),
-                                                GFP_KERNEL)) {
-                       dev_err(&connection->bundle->dev,
-                               "%s: error allocating response\n", __func__);
-                       return -ENOMEM;
-               }
-
-               version = op->response->payload;
-               version->major = GB_CONTROL_VERSION_MAJOR;
-               version->minor = GB_CONTROL_VERSION_MINOR;
-               break;
-       case GB_CONTROL_TYPE_CONNECTED:
-       case GB_CONTROL_TYPE_DISCONNECTED:
-               break;
-       default:
-               WARN_ON(1);
-               break;
-       }
-
-       return 0;
-}
-
 static int gb_control_connection_init(struct gb_connection *connection)
 {
        struct gb_control *control;
@@ -129,7 +95,6 @@ static struct gb_protocol control_protocol = {
        .minor                  = 1,
        .connection_init        = gb_control_connection_init,
        .connection_exit        = gb_control_connection_exit,
-       .request_recv           = gb_control_request_recv,
        .flags                  = GB_PROTOCOL_SKIP_CONTROL_CONNECTED |
                                  GB_PROTOCOL_SKIP_CONTROL_DISCONNECTED,
 };