# This is a BitKeeper generated patch for the following project:
# Project Name: Linux kernel tree
# This patch format is intended for GNU patch command version 2.5 or higher.
# This patch includes the following deltas:
#                  ChangeSet    1.626   -> 1.627  
#       drivers/usb/misc/tiglusb.c      1.4     -> 1.5    
#       drivers/usb/misc/emi26.c        1.3     -> 1.4    
#
# The following is the BitKeeper ChangeSet Log
# --------------------------------------------
# 02/07/07      [EMAIL PROTECTED]   1.627
# [PATCH] USB: printk janitorial fixes
# 
# Part 2, printk fixups for a the tiglusb and emi26 drivers.
# --------------------------------------------
#
diff -Nru a/drivers/usb/misc/emi26.c b/drivers/usb/misc/emi26.c
--- a/drivers/usb/misc/emi26.c  Sun Jul  7 12:35:58 2002
+++ b/drivers/usb/misc/emi26.c  Sun Jul  7 12:35:58 2002
@@ -54,7 +54,7 @@
        unsigned char *buffer =  kmalloc (length, GFP_KERNEL);
 
        if (!buffer) {
-               printk(KERN_ERR "emi26: kmalloc(%d) failed.", length);
+               err("emi26: kmalloc(%d) failed.", length);
                return -ENOMEM;
        }
        memcpy (buffer, data, length);
@@ -69,11 +69,11 @@
 static int emi26_set_reset (struct usb_device *dev, unsigned char reset_bit)
 {
        int response;
-       printk(KERN_INFO "%s - %d", __FUNCTION__, reset_bit);
+       info("%s - %d", __FUNCTION__, reset_bit);
        /* printk(KERN_DEBUG "%s - %d", __FUNCTION__, reset_bit); */
        response = emi26_writememory (dev, CPUCS_REG, &reset_bit, 1, 0xa0);
        if (response < 0) {
-               printk(KERN_ERR "emi26: set_reset (%d) failed", reset_bit);
+               err("emi26: set_reset (%d) failed", reset_bit);
        }
        return response;
 }
@@ -89,7 +89,7 @@
        /* Assert reset (stop the CPU in the EMI) */
        err = emi26_set_reset(dev,1);
        if (err < 0) {
-               printk(KERN_ERR "%s - error loading firmware: error = %d", 
__FUNCTION__, err);
+               err( "%s - error loading firmware: error = %d", __FUNCTION__, err);
                return err;
        }
 
@@ -97,7 +97,7 @@
        for (i=0; g_Loader[i].type == 0; i++) {
                err = emi26_writememory(dev, g_Loader[i].address, g_Loader[i].data, 
g_Loader[i].length, ANCHOR_LOAD_INTERNAL);
                if (err < 0) {
-                       printk(KERN_ERR "%s - error loading firmware: error = %d", 
__FUNCTION__, err);
+                       err("%s - error loading firmware: error = %d", __FUNCTION__, 
+err);
                        return err;
                }
        }
@@ -120,7 +120,7 @@
                }
                err = emi26_writememory(dev, addr, buf, i, ANCHOR_LOAD_FPGA);
                if (err < 0) {
-                       printk(KERN_ERR "%s - error loading firmware: error = %d", 
__FUNCTION__, err);
+                       err("%s - error loading firmware: error = %d", __FUNCTION__, 
+err);
                        return err;
                }
        } while (i > 0);
@@ -128,7 +128,7 @@
        /* Assert reset (stop the CPU in the EMI) */
        err = emi26_set_reset(dev,1);
        if (err < 0) {
-               printk(KERN_ERR "%s - error loading firmware: error = %d", 
__FUNCTION__, err);
+               err("%s - error loading firmware: error = %d", __FUNCTION__, err);
                return err;
        }
 
@@ -136,7 +136,7 @@
        for (i=0; g_Loader[i].type == 0; i++) {
                err = emi26_writememory(dev, g_Loader[i].address, g_Loader[i].data, 
g_Loader[i].length, ANCHOR_LOAD_INTERNAL);
                if (err < 0) {
-                       printk(KERN_ERR "%s - error loading firmware: error = %d", 
__FUNCTION__, err);
+                       err("%s - error loading firmware: error = %d", __FUNCTION__, 
+err);
                        return err;
                }
        }
@@ -144,7 +144,7 @@
        /* De-assert reset (let the CPU run) */
        err = emi26_set_reset(dev,0);
        if (err < 0) {
-               printk(KERN_ERR "%s - error loading firmware: error = %d", 
__FUNCTION__, err);
+               err("%s - error loading firmware: error = %d", __FUNCTION__, err);
                return err;
        }
 
@@ -153,7 +153,7 @@
                if (!INTERNAL_RAM(g_Firmware[i].address)) {
                        err = emi26_writememory(dev, g_Firmware[i].address, 
g_Firmware[i].data, g_Firmware[i].length, ANCHOR_LOAD_EXTERNAL);
                        if (err < 0) {
-                               printk(KERN_ERR "%s - error loading firmware: error = 
%d", __FUNCTION__, err);
+                               err("%s - error loading firmware: error = %d", 
+__FUNCTION__, err);
                                return err;
                        }
                }
@@ -162,7 +162,7 @@
        /* Assert reset (stop the CPU in the EMI) */
        err = emi26_set_reset(dev,1);
        if (err < 0) {
-               printk(KERN_ERR "%s - error loading firmware: error = %d", 
__FUNCTION__, err);
+               err("%s - error loading firmware: error = %d", __FUNCTION__, err);
                return err;
        }
 
@@ -170,7 +170,7 @@
                if (INTERNAL_RAM(g_Firmware[i].address)) {
                        err = emi26_writememory(dev, g_Firmware[i].address, 
g_Firmware[i].data, g_Firmware[i].length, ANCHOR_LOAD_INTERNAL);
                        if (err < 0) {
-                               printk(KERN_ERR "%s - error loading firmware: error = 
%d", __FUNCTION__, err);
+                               err("%s - error loading firmware: error = %d", 
+__FUNCTION__, err);
                                return err;
                        }
                }
@@ -179,7 +179,7 @@
        /* De-assert reset (let the CPU run) */
        err = emi26_set_reset(dev,0);
        if (err < 0) {
-               printk(KERN_ERR "%s - error loading firmware: error = %d", 
__FUNCTION__, err);
+               err("%s - error loading firmware: error = %d", __FUNCTION__, err);
                return err;
        }
 
@@ -197,7 +197,7 @@
 
 static void * emi26_probe(struct usb_device *dev, unsigned int if_num, const struct 
usb_device_id *id)
 {
-       printk(KERN_INFO "%s start", __FUNCTION__); 
+       info("%s start", __FUNCTION__); 
        
        if((dev->descriptor.idVendor == EMI26_VENDOR_ID) && (dev->descriptor.idProduct 
== EMI26_PRODUCT_ID)) {
                emi26_load_firmware(dev);
diff -Nru a/drivers/usb/misc/tiglusb.c b/drivers/usb/misc/tiglusb.c
--- a/drivers/usb/misc/tiglusb.c        Sun Jul  7 12:35:58 2002
+++ b/drivers/usb/misc/tiglusb.c        Sun Jul  7 12:35:58 2002
@@ -60,7 +60,7 @@
 static int clear_device (struct usb_device *dev)
 {
        if (usb_set_configuration (dev, dev->config[0].bConfigurationValue) < 0) {
-               printk ("tiglusb: clear_device failed\n");
+               err ("tiglusb: clear_device failed");
                return -1;
        }
 
@@ -74,13 +74,13 @@
 
        pipe = usb_sndbulkpipe (dev, 1);
        if (usb_clear_halt (dev, usb_pipeendpoint (pipe))) {
-               printk ("tiglusb: clear_pipe (r), request failed\n");
+               err("tiglusb: clear_pipe (r), request failed");
                return -1;
        }
 
        pipe = usb_sndbulkpipe (dev, 2);
        if (usb_clear_halt (dev, usb_pipeendpoint (pipe))) {
-               printk ("tiglusb: clear_pipe (w), request failed\n");
+               err ("tiglusb: clear_pipe (w), request failed");
                return -1;
        }
 
@@ -171,7 +171,7 @@
        if (result == -ETIMEDOUT) {     /* NAK */
                ret = result;
                if (!bytes_read) {
-                       printk ("quirk !\n");
+                       warn ("quirk !");
                }
                warn ("tiglusb_read, NAK received.");
                goto out;
@@ -233,7 +233,7 @@
        } else if (result == -EPIPE) {  /* STALL -- shouldn't happen */
                warn ("CLEAR_FEATURE request to remove STALL condition.");
                if (usb_clear_halt (s->dev, usb_pipeendpoint (pipe)))
-                       warn ("send_packet, request failed\n");
+                       warn ("send_packet, request failed");
                //clear_device(s->dev);
                ret = result;
                goto out;
@@ -273,12 +273,12 @@
                timeout = arg;  // timeout value in tenth of seconds
                break;
        case IOCTL_TIUSB_RESET_DEVICE:
-               printk (KERN_DEBUG "IOCTL_TIGLUSB_RESET_DEVICE\n");
+               dbg ("IOCTL_TIGLUSB_RESET_DEVICE");
                if (clear_device (s->dev))
                        ret = -EIO;
                break;
        case IOCTL_TIUSB_RESET_PIPES:
-               printk (KERN_DEBUG "IOCTL_TIGLUSB_RESET_PIPES\n");
+               dbg ("IOCTL_TIGLUSB_RESET_PIPES");
                if (clear_pipes (s->dev))
                        ret = -EIO;
                break;
@@ -326,8 +326,8 @@
        ptiglusb_t s;
        char name[8];
 
-       printk ("tiglusb: probing vendor id 0x%x, device id 0x%x ifnum:%d\n",
-               dev->descriptor.idVendor, dev->descriptor.idProduct, ifnum);
+       dbg ("tiglusb: probing vendor id 0x%x, device id 0x%x ifnum:%d",
+            dev->descriptor.idVendor, dev->descriptor.idProduct, ifnum);
 
        /* 
         * We don't handle multiple configurations. As of version 0x0103 of 
@@ -341,7 +341,7 @@
                return NULL;
 
        if (usb_set_configuration (dev, dev->config[0].bConfigurationValue) < 0) {
-               printk ("tiglusb_probe: set_configuration failed\n");
+               err ("tiglusb_probe: set_configuration failed");
                return NULL;
        }
 
@@ -358,7 +358,7 @@
        dbg ("bound to interface: %d", ifnum);
 
        sprintf (name, "%d", s->minor);
-       printk ("tiglusb: registering to devfs : major = %d, minor = %d, node = %s\n", 
TIUSB_MAJOR,
+       info ("tiglusb: registering to devfs : major = %d, minor = %d, node = %s", 
+TIUSB_MAJOR,
                (TIUSB_MINOR + s->minor), name);
        s->devfs =
            devfs_register (devfs_handle, name, DEVFS_FL_DEFAULT, TIUSB_MAJOR,
@@ -366,7 +366,7 @@
                            NULL);
 
        /* Display firmware version */
-       printk ("tiglusb: link cable version %i.%02x\n",
+       info ("tiglusb: link cable version %i.%02x",
                dev->descriptor.bcdDevice >> 8, dev->descriptor.bcdDevice & 0xff);
 
        return s;
@@ -377,7 +377,7 @@
        ptiglusb_t s = (ptiglusb_t) drv_context;
 
        if (!s || !s->dev)
-               printk ("bogus disconnect");
+               warn ("bogus disconnect");
 
        s->remove_pending = 1;
        wake_up (&s->wait);
@@ -396,7 +396,7 @@
        /* unregister device */
        devfs_unregister (s->devfs);
        s->devfs = NULL;
-       printk ("tiglusb: device disconnected\n");
+       info ("tiglusb: device disconnected");
 }
 
 static struct usb_device_id tiglusb_ids[] = {
@@ -452,7 +452,7 @@
 
        /* register device */
        if (devfs_register_chrdev (TIUSB_MAJOR, "tiglusb", &tiglusb_fops)) {
-               printk ("tiglusb: unable to get major %d\n", TIUSB_MAJOR);
+               err ("tiglusb: unable to get major %d", TIUSB_MAJOR);
                return -EIO;
        }
 


-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
We have stuff for geeks like you.
http://thinkgeek.com/sf
_______________________________________________
[EMAIL PROTECTED]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

Reply via email to