staging: line6: drop variax model_data field
authorStefan Hajnoczi <stefanha@gmail.com>
Thu, 22 Nov 2012 19:49:22 +0000 (20:49 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 27 Nov 2012 00:13:50 +0000 (16:13 -0800)
The sysfs attrs to access the model dump have been removed so it's safe
to drop the model_data field.  The next step will be to simplify the
startup process since we no longer need to fetch this data via dump
requests, but that will be done in a later patch.

Signed-off-by: Stefan Hajnoczi <stefanha@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/line6/variax.c
drivers/staging/line6/variax.h

index 0adba32c766726f5ed758ac13b73fb267b8b20aa..0262d7def73c5360591179cb7d2a9193106d3d5a 100644 (file)
@@ -70,21 +70,6 @@ static void variax_startup2(unsigned long data);
 static void variax_startup4(unsigned long data);
 static void variax_startup5(unsigned long data);
 
-/*
-       Decode data transmitted by workbench.
-*/
-static void variax_decode(const unsigned char *raw_data, unsigned char *data,
-                         int raw_size)
-{
-       for (; raw_size > 0; raw_size -= 6) {
-               data[2] = raw_data[0] | (raw_data[1] << 4);
-               data[1] = raw_data[2] | (raw_data[3] << 4);
-               data[0] = raw_data[4] | (raw_data[5] << 4);
-               raw_data += 6;
-               data += 3;
-       }
-}
-
 static void variax_activate_async(struct usb_line6_variax *variax, int a)
 {
        variax->buffer_activate[VARIAX_OFFSET_ACTIVATE] = a;
@@ -214,18 +199,6 @@ void line6_variax_process_message(struct usb_line6_variax *variax)
                                        break;
 
                                case VARIAX_DUMP_PASS2:
-                                       /* model name is transmitted twice, so skip it here: */
-                                       variax_decode(buf +
-                                                     VARIAX_MODEL_HEADER_LENGTH,
-                                                     (unsigned char *)
-                                                     &variax->
-                                                     model_data.control +
-                                                     sizeof(variax->model_data.
-                                                            control)
-                                                     / 2,
-                                                     sizeof(variax->model_data.
-                                                            control)
-                                                     / 2 * 2);
                                        line6_dump_request_async
                                            (&variax->dumpreq, &variax->line6,
                                             2, VARIAX_DUMP_PASS3);
index f3d95fade026a8429c793a2c8dffba3dbe5990be..13d5680d2e59a783f257a5ce3c74cda95bb6c274 100644 (file)
@@ -44,21 +44,6 @@ enum {
        VARIAX_DUMP_PASS3
 };
 
-/**
-       Binary Variax model dump
-*/
-struct variax_model {
-       /**
-               Header information (including program name).
-       */
-       unsigned char name[18];
-
-       /**
-               Model parameters.
-       */
-       unsigned char control[78 * 2];
-};
-
 struct usb_line6_variax {
        /**
                Generic Line6 USB data.
@@ -77,11 +62,6 @@ struct usb_line6_variax {
        */
        unsigned char *buffer_activate;
 
-       /**
-               Current model settings.
-       */
-       struct variax_model model_data;
-
        /**
                Handler for device initializaton.
        */