]> git.proxmox.com Git - mirror_ubuntu-jammy-kernel.git/commitdiff
greybus: kill old cport handlers
authorAlex Elder <elder@linaro.org>
Thu, 16 Oct 2014 11:35:37 +0000 (06:35 -0500)
committerGreg Kroah-Hartman <greg@kroah.com>
Fri, 17 Oct 2014 16:14:11 +0000 (18:14 +0200)
The original CPort message handlers are not needed.  All incoming
data is passed to handlers based on the protocol used over the
connection over which the data was transferred.  So get rid of the
old CPort handler code.

Signed-off-by: Alex Elder <elder@linaro.org>
Signed-off-by: Greg Kroah-Hartman <greg@kroah.com>
drivers/staging/greybus/gbuf.c

index 12c8c23e23ab9f6deea8443aba61fa4f88018a9d..348ee7c27a07e416391327230f4828c2001b49d2 100644 (file)
@@ -141,8 +141,6 @@ void gb_deregister_cport_complete(u16 cport_id)
 void greybus_cport_in(struct greybus_host_device *hd, u16 cport_id,
                        u8 *data, size_t length)
 {
-       struct gb_cport_handler *ch;
-       struct gbuf *gbuf;
        struct gb_connection *connection;
 
        connection = gb_hd_connection_find(hd, cport_id);
@@ -152,34 +150,6 @@ void greybus_cport_in(struct greybus_host_device *hd, u16 cport_id,
                return;
        }
        gb_connection_operation_recv(connection, data, length);
-
-       /* first check to see if we have a cport handler for this cport */
-       ch = &cport_handler[cport_id];
-       if (!ch->handler) {
-               /* Ugh, drop the data on the floor, after logging it... */
-               dev_err(hd->parent,
-                       "Received data for cport %d, but no handler!\n",
-                       cport_id);
-               return;
-       }
-
-       gbuf = greybus_alloc_gbuf(connection, ch->handler, length, false,
-                                       GFP_ATOMIC, ch->context);
-
-       if (!gbuf) {
-               /* Again, something bad went wrong, log it... */
-               pr_err("can't allocate gbuf???\n");
-               return;
-       }
-
-       /*
-        * FIXME:
-        * Very dumb copy data method for now, if this is slow (odds are it will
-        * be, we should move to a model where the hd "owns" all buffers, but we
-        * want something up and working first for now.
-        */
-       memcpy(gbuf->transfer_buffer, data, length);
-       gbuf->actual_length = length;
 }
 EXPORT_SYMBOL_GPL(greybus_cport_in);