USB: ezusb: remove dependancy on usb_serial
authorRene Buergel <rene.buergel@sohard.de>
Thu, 13 Sep 2012 20:14:38 +0000 (22:14 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 14 Sep 2012 04:59:51 +0000 (21:59 -0700)
This patch removes the dependency on the usb_serial interface and names
some magic constants

Signed-off-by: René Bürgel <rene.buergel@sohard.de>
--
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/serial/ezusb.c
drivers/usb/serial/keyspan.c
drivers/usb/serial/keyspan_pda.c
drivers/usb/serial/whiteheat.c
include/linux/usb/serial.h

index 800e8eb600039b61b6b845af559fcf165bd78977..3048b52d2d39c4bff9a96ddcb944beeed98d71a9 100644 (file)
@@ -9,24 +9,24 @@
  */
 
 #include <linux/kernel.h>
-#include <linux/errno.h>
 #include <linux/init.h>
 #include <linux/slab.h>
-#include <linux/tty.h>
 #include <linux/module.h>
 #include <linux/usb.h>
-#include <linux/usb/serial.h>
 
 /* EZ-USB Control and Status Register.  Bit 0 controls 8051 reset */
 #define CPUCS_REG    0x7F92
 
-int ezusb_writememory(struct usb_serial *serial, int address,
+/* Command for writing to internal memory */
+#define WRITE_INT_RAM 0xA0
+
+int ezusb_writememory(struct usb_device *dev, int address,
                                unsigned char *data, int length, __u8 request)
 {
        int result;
        unsigned char *transfer_buffer;
 
-       if (!serial->dev) {
+       if (!dev) {
                printk(KERN_ERR "ezusb: %s - no physical device present, "
                       "failing.\n", __func__);
                return -ENODEV;
@@ -34,25 +34,25 @@ int ezusb_writememory(struct usb_serial *serial, int address,
 
        transfer_buffer = kmemdup(data, length, GFP_KERNEL);
        if (!transfer_buffer) {
-               dev_err(&serial->dev->dev, "%s - kmalloc(%d) failed.\n",
+               dev_err(&dev->dev, "%s - kmalloc(%d) failed.\n",
                                                        __func__, length);
                return -ENOMEM;
        }
-       result = usb_control_msg(serial->dev, usb_sndctrlpipe(serial->dev, 0),
-                    request, 0x40, address, 0, transfer_buffer, length, 3000);
+       result = usb_control_msg(dev, usb_sndctrlpipe(dev, 0), request,
+                                USB_DIR_OUT | USB_TYPE_VENDOR | USB_RECIP_DEVICE,
+                                address, 0, transfer_buffer, length, 3000);
+
        kfree(transfer_buffer);
        return result;
 }
 EXPORT_SYMBOL_GPL(ezusb_writememory);
 
-int ezusb_set_reset(struct usb_serial *serial, unsigned char reset_bit)
+int ezusb_set_reset(struct usb_device *dev, unsigned char reset_bit)
 {
-       int response;
-
-       response = ezusb_writememory(serial, CPUCS_REG, &reset_bit, 1, 0xa0);
+       int response = ezusb_writememory(dev, CPUCS_REG, &reset_bit, 1, WRITE_INT_RAM);
        if (response < 0)
-               dev_err(&serial->dev->dev, "%s- %d failed\n",
-                                               __func__, reset_bit);
+               dev_err(&dev->dev, "%s-%d failed: %d\n",
+                                               __func__, reset_bit, response);
        return response;
 }
 EXPORT_SYMBOL_GPL(ezusb_set_reset);
index af0b70eaf032f6617c36b596ce70776842a20d9f..f0d4f3fcf4f7dc372ee52f1adfa01ed50b755df4 100644 (file)
@@ -1241,12 +1241,12 @@ static int keyspan_fake_startup(struct usb_serial *serial)
        dbg("Uploading Keyspan %s firmware.", fw_name);
 
                /* download the firmware image */
-       response = ezusb_set_reset(serial, 1);
+       response = ezusb_set_reset(serial->dev, 1);
 
        record = (const struct ihex_binrec *)fw->data;
 
        while (record) {
-               response = ezusb_writememory(serial, be32_to_cpu(record->addr),
+               response = ezusb_writememory(serial->dev, be32_to_cpu(record->addr),
                                             (unsigned char *)record->data,
                                             be16_to_cpu(record->len), 0xa0);
                if (response < 0) {
@@ -1260,7 +1260,7 @@ static int keyspan_fake_startup(struct usb_serial *serial)
        release_firmware(fw);
                /* bring device out of reset. Renumeration will occur in a
                   moment and the new device will bind to the real driver */
-       response = ezusb_set_reset(serial, 0);
+       response = ezusb_set_reset(serial->dev, 0);
 
        /* we don't want this device to have a driver assigned to it. */
        return 1;
index a4ac3cfeffc4b84d6fa8ac41236bad5f2bf6e724..1290b6f0a050379bfd0a19989e495379ef16e2cc 100644 (file)
@@ -682,7 +682,7 @@ static int keyspan_pda_fake_startup(struct usb_serial *serial)
        const struct firmware *fw;
 
        /* download the firmware here ... */
-       response = ezusb_set_reset(serial, 1);
+       response = ezusb_set_reset(serial->dev, 1);
 
        if (0) { ; }
 #ifdef KEYSPAN
@@ -707,7 +707,7 @@ static int keyspan_pda_fake_startup(struct usb_serial *serial)
        record = (const struct ihex_binrec *)fw->data;
 
        while (record) {
-               response = ezusb_writememory(serial, be32_to_cpu(record->addr),
+               response = ezusb_writememory(serial->dev, be32_to_cpu(record->addr),
                                             (unsigned char *)record->data,
                                             be16_to_cpu(record->len), 0xa0);
                if (response < 0) {
@@ -722,7 +722,7 @@ static int keyspan_pda_fake_startup(struct usb_serial *serial)
        release_firmware(fw);
        /* bring device out of reset. Renumeration will occur in a moment
           and the new device will bind to the real driver */
-       response = ezusb_set_reset(serial, 0);
+       response = ezusb_set_reset(serial->dev, 0);
 
        /* we want this device to fail to have a driver assigned to it. */
        return 1;
index 473635e7f5dbdf8bb4c28c20102f12b438b31d77..fc72591711e4ea5349d4a759eb4cac44b0d97be7 100644 (file)
@@ -213,13 +213,13 @@ static int whiteheat_firmware_download(struct usb_serial *serial,
                goto out;
        }
        ret = 0;
-       response = ezusb_set_reset (serial, 1);
+       response = ezusb_set_reset(serial->dev, 1);
 
        record = (const struct ihex_binrec *)loader_fw->data;
        while (record) {
-               response = ezusb_writememory (serial, be32_to_cpu(record->addr),
-                                             (unsigned char *)record->data,
-                                             be16_to_cpu(record->len), 0xa0);
+               response = ezusb_writememory(serial->dev, be32_to_cpu(record->addr),
+                                            (unsigned char *)record->data,
+                                            be16_to_cpu(record->len), 0xa0);
                if (response < 0) {
                        dev_err(&serial->dev->dev, "%s - ezusb_writememory "
                                "failed for loader (%d %04X %p %d)\n",
@@ -230,15 +230,15 @@ static int whiteheat_firmware_download(struct usb_serial *serial,
                record = ihex_next_binrec(record);
        }
 
-       response = ezusb_set_reset(serial, 0);
+       response = ezusb_set_reset(serial->dev, 0);
 
        record = (const struct ihex_binrec *)firmware_fw->data;
        while (record && be32_to_cpu(record->addr) < 0x1b40)
                record = ihex_next_binrec(record);
        while (record) {
-               response = ezusb_writememory (serial, be32_to_cpu(record->addr),
-                                             (unsigned char *)record->data,
-                                             be16_to_cpu(record->len), 0xa3);
+               response = ezusb_writememory(serial->dev, be32_to_cpu(record->addr),
+                                            (unsigned char *)record->data,
+                                            be16_to_cpu(record->len), 0xa3);
                if (response < 0) {
                        dev_err(&serial->dev->dev, "%s - ezusb_writememory "
                                "failed for first firmware step "
@@ -250,13 +250,13 @@ static int whiteheat_firmware_download(struct usb_serial *serial,
                ++record;
        }
 
-       response = ezusb_set_reset(serial, 1);
+       response = ezusb_set_reset(serial->dev, 1);
 
        record = (const struct ihex_binrec *)firmware_fw->data;
        while (record && be32_to_cpu(record->addr) < 0x1b40) {
-               response = ezusb_writememory (serial, be32_to_cpu(record->addr),
-                                             (unsigned char *)record->data,
-                                             be16_to_cpu(record->len), 0xa0);
+               response = ezusb_writememory(serial->dev, be32_to_cpu(record->addr),
+                                            (unsigned char *)record->data,
+                                            be16_to_cpu(record->len), 0xa0);
                if (response < 0) {
                        dev_err(&serial->dev->dev, "%s - ezusb_writememory "
                                "failed for second firmware step "
@@ -268,7 +268,7 @@ static int whiteheat_firmware_download(struct usb_serial *serial,
                ++record;
        }
        ret = 0;
-       response = ezusb_set_reset (serial, 0);
+       response = ezusb_set_reset(serial->dev, 0);
  out:
        release_firmware(loader_fw);
        release_firmware(firmware_fw);
index 9e9a0a8991e4af7c105c9f738457a4e47a454fb9..6bed6db619b2449421c4817e4150ab7bfa72b793 100644 (file)
@@ -301,9 +301,9 @@ extern void usb_serial_port_softint(struct usb_serial_port *port);
 extern int usb_serial_suspend(struct usb_interface *intf, pm_message_t message);
 extern int usb_serial_resume(struct usb_interface *intf);
 
-extern int ezusb_writememory(struct usb_serial *serial, int address,
+extern int ezusb_writememory(struct usb_device *dev, int address,
                             unsigned char *data, int length, __u8 bRequest);
-extern int ezusb_set_reset(struct usb_serial *serial, unsigned char reset_bit);
+extern int ezusb_set_reset(struct usb_device *dev, unsigned char reset_bit);
 
 /* USB Serial console functions */
 #ifdef CONFIG_USB_SERIAL_CONSOLE