The patch number 10300 was added via Mauro Carvalho Chehab <mche...@redhat.com>
to http://linuxtv.org/hg/v4l-dvb master development tree.

Kernel patches in this development tree may be modified to be backward
compatible with older kernels. Compatibility modifications will be
removed before inclusion into the mainstream Kernel

If anyone has any objections, please let us know by sending a message to:
        Linux Media Mailing List <linux-me...@vger.kernel.org>

------

From: Mauro Carvalho Chehab  <mche...@redhat.com>
Reverting changeset 10298. It were committed with a wrong From: meta-tag


Signed-off-by: Mauro Carvalho Chehab <mche...@redhat.com>


---

 linux/drivers/media/video/dabusb.c                    |   76 +++-------
 linux/drivers/media/video/pvrusb2/pvrusb2-v4l2.c      |   12 -
 linux/drivers/media/video/s2255drv.c                  |   38 +----
 linux/drivers/media/video/usbvision/usbvision-core.c  |   44 ++---
 linux/drivers/media/video/usbvision/usbvision-i2c.c   |    6 
 linux/drivers/media/video/usbvision/usbvision-video.c |   36 +---
 6 files changed, 82 insertions(+), 130 deletions(-)

diff -r d0cfae040789 -r fee99c8e6568 linux/drivers/media/video/dabusb.c
--- a/linux/drivers/media/video/dabusb.c        Thu Jan 08 12:13:42 2009 +0000
+++ b/linux/drivers/media/video/dabusb.c        Fri Jan 23 00:29:03 2009 -0200
@@ -199,20 +199,17 @@ static void dabusb_iso_complete (struct 
                                        dst += len;
                                }
                                else
-                                       dev_err(&purb->dev->dev,
-                                               "dabusb_iso_complete: invalid 
len %d\n", len);
+                                       err("dabusb_iso_complete: invalid len 
%d", len);
                        }
                        else
                                dev_warn(&purb->dev->dev, "dabusb_iso_complete: 
corrupted packet status: %d\n", purb->iso_frame_desc[i].status);
                if (dst != purb->actual_length)
-                       dev_err(&purb->dev->dev,
-                               "dst!=purb->actual_length:%d!=%d\n",
-                                       dst, purb->actual_length);
+                       err("dst!=purb->actual_length:%d!=%d", dst, 
purb->actual_length);
        }
 
        if (atomic_dec_and_test (&s->pending_io) && !s->remove_pending && 
s->state != _stopped) {
                s->overruns++;
-               dev_err(&purb->dev->dev, "overrun (%d)\n", s->overruns);
+               err("overrun (%d)", s->overruns);
        }
        wake_up (&s->wait);
 }
@@ -233,14 +230,13 @@ static int dabusb_alloc_buffers (pdabusb
        while (transfer_len < (s->total_buffer_size << 10)) {
                b = kzalloc(sizeof (buff_t), GFP_KERNEL);
                if (!b) {
-                       dev_err(&s->usbdev->dev,
-                               "kzalloc(sizeof(buff_t))==NULL\n");
+                       err("kzalloc(sizeof(buff_t))==NULL");
                        goto err;
                }
                b->s = s;
                b->purb = usb_alloc_urb(packets, GFP_KERNEL);
                if (!b->purb) {
-                       dev_err(&s->usbdev->dev, "usb_alloc_urb == NULL\n");
+                       err("usb_alloc_urb == NULL");
                        kfree (b);
                        goto err;
                }
@@ -249,8 +245,7 @@ static int dabusb_alloc_buffers (pdabusb
                if (!b->purb->transfer_buffer) {
                        kfree (b->purb);
                        kfree (b);
-                       dev_err(&s->usbdev->dev,
-                               "kmalloc(%d)==NULL\n", transfer_buffer_length);
+                       err("kmalloc(%d)==NULL", transfer_buffer_length);
                        goto err;
                }
 
@@ -294,11 +289,10 @@ static int dabusb_bulk (pdabusb_t s, pbu
 
        ret=usb_bulk_msg(s->usbdev, pipe, pb->data, pb->size, &actual_length, 
100);
        if(ret<0) {
-               dev_err(&s->usbdev->dev,
-                       "usb_bulk_msg failed(%d)\n", ret);
+               err("dabusb: usb_bulk_msg failed(%d)",ret);
 
                if (usb_set_interface (s->usbdev, _DABUSB_IF, 1) < 0) {
-                       dev_err(&s->usbdev->dev, "set_interface failed\n");
+                       err("set_interface failed");
                        return -EINVAL;
                }
 
@@ -307,7 +301,7 @@ static int dabusb_bulk (pdabusb_t s, pbu
        if( ret == -EPIPE ) {
                dev_warn(&s->usbdev->dev, "CLEAR_FEATURE request to remove 
STALL condition.\n");
                if(usb_clear_halt(s->usbdev, usb_pipeendpoint(pipe)))
-                       dev_err(&s->usbdev->dev, "request failed\n");
+                       err("request failed");
        }
 
        pb->size = actual_length;
@@ -325,8 +319,7 @@ static int dabusb_writemem (pdabusb_t s,
        unsigned char *transfer_buffer =  kmalloc (len, GFP_KERNEL);
 
        if (!transfer_buffer) {
-               dev_err(&s->usbdev->dev,
-                       "dabusb_writemem: kmalloc(%d) failed.\n", len);
+               err("dabusb_writemem: kmalloc(%d) failed.", len);
                return -ENOMEM;
        }
 
@@ -359,8 +352,7 @@ static int dabusb_loadmem (pdabusb_t s, 
 #if LINUX_VERSION_CODE >= KERNEL_VERSION(2,6,27)
        ret = request_ihex_firmware(&fw, "dabusb/firmware.fw", &s->usbdev->dev);
        if (ret) {
-               dev_err(&s->usbdev->dev,
-                       "Failed to load \"dabusb/firmware.fw\": %d\n", ret);
+               err("Failed to load \"dabusb/firmware.fw\": %d\n", ret);
                goto out;
        }
 #endif
@@ -386,14 +378,11 @@ static int dabusb_loadmem (pdabusb_t s, 
 #endif
                if (ret < 0) {
 #if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,27)
-                       dev_err(&s->usbdev->dev,
-                               "dabusb_writemem failed (%d %04X %p %d)\n",
-                               ret, ptr->Address, ptr->Data, ptr->Length);
-#else
-                       dev_err(&s->usbdev->dev,
-                               "dabusb_writemem failed (%d %04X %p %d)\n",
-                               ret, be32_to_cpu(rec->addr),
-                               rec->data, be16_to_cpu(rec->len));
+                       err("dabusb_writemem failed (%d %04X %p %d)", ret, 
ptr->Address, ptr->Data, ptr->Length);
+#else
+                       err("dabusb_writemem failed (%d %04X %p %d)", ret,
+                           be32_to_cpu(rec->addr), rec->data,
+                           be16_to_cpu(rec->len));
 #endif
                        break;
                }
@@ -454,16 +443,14 @@ static int dabusb_fpga_download (pdabusb
        dbg("Enter dabusb_fpga_download (internal)");
 
        if (!b) {
-               dev_err(&s->usbdev->dev,
-                       "kmalloc(sizeof(bulk_transfer_t))==NULL\n");
+               err("kmalloc(sizeof(bulk_transfer_t))==NULL");
                return -ENOMEM;
        }
 
 #if LINUX_VERSION_CODE >= KERNEL_VERSION(2,6,27)
        ret = request_firmware(&fw, "dabusb/bitstream.bin", &s->usbdev->dev);
        if (ret) {
-               dev_err(&s->usbdev->dev,
-                       "Failed to load \"dabusb/bitstream.bin\": %d\n", ret);
+               err("Failed to load \"dabusb/bitstream.bin\": %d\n", ret);
                kfree(b);
                return ret;
        }
@@ -495,7 +482,7 @@ static int dabusb_fpga_download (pdabusb
 #endif
                ret = dabusb_bulk (s, b);
                if (ret < 0) {
-                       dev_err(&s->usbdev->dev, "dabusb_bulk failed.\n");
+                       err("dabusb_bulk failed.");
                        break;
                }
                mdelay (1);
@@ -550,11 +537,9 @@ static int dabusb_startrek (pdabusb_t s)
 
                        ret = usb_submit_urb (end->purb, GFP_KERNEL);
                        if (ret) {
-                               dev_err(&s->usbdev->dev,
-                                       "usb_submit_urb returned:%d\n", ret);
+                               err("usb_submit_urb returned:%d", ret);
                                if (dabusb_add_buf_tail (s, &s->free_buff_list, 
&s->rec_buff_list))
-                                       dev_err(&s->usbdev->dev,
-                                               "startrek: dabusb_add_buf_tail 
failed\n");
+                                       err("startrek: dabusb_add_buf_tail 
failed");
                                break;
                        }
                        else
@@ -597,8 +582,7 @@ static ssize_t dabusb_read (struct file 
 
                        spin_unlock_irqrestore(&s->lock, flags);
 
-                       dev_err(&s->usbdev->dev,
-                               "error: rec_buf_list is empty\n");
+                       err("error: rec_buf_list is empty");
                        goto err;
                }
 
@@ -627,8 +611,7 @@ static ssize_t dabusb_read (struct file 
 
                        if (list_empty (&s->rec_buff_list)) {
                                spin_unlock_irqrestore(&s->lock, flags);
-                               dev_err(&s->usbdev->dev,
-                                       "error: still no buffer available.\n");
+                               err("error: still no buffer available.");
                                goto err;
                        }
                        spin_unlock_irqrestore(&s->lock, flags);
@@ -649,7 +632,7 @@ static ssize_t dabusb_read (struct file 
                dbg("copy_to_user:%p %p %d",buf, purb->transfer_buffer + 
s->readptr, cnt);
 
                if (copy_to_user (buf, purb->transfer_buffer + s->readptr, 
cnt)) {
-                       dev_err(&s->usbdev->dev, "read: copy_to_user failed\n");
+                       err("read: copy_to_user failed");
                        if (!ret)
                                ret = -EFAULT;
                        goto err;
@@ -663,8 +646,7 @@ static ssize_t dabusb_read (struct file 
                if (s->readptr == purb->actual_length) {
                        // finished, take next buffer
                        if (dabusb_add_buf_tail (s, &s->free_buff_list, 
&s->rec_buff_list))
-                               dev_err(&s->usbdev->dev,
-                                       "read: dabusb_add_buf_tail failed\n");
+                               err("read: dabusb_add_buf_tail failed");
                        s->readptr = 0;
                }
        }
@@ -700,7 +682,7 @@ static int dabusb_open (struct inode *in
        }
        if (usb_set_interface (s->usbdev, _DABUSB_IF, 1) < 0) {
                mutex_unlock(&s->mutex);
-               dev_err(&s->usbdev->dev, "set_interface failed\n");
+               err("set_interface failed");
                return -EINVAL;
        }
        s->opened = 1;
@@ -725,7 +707,7 @@ static int dabusb_release (struct inode 
 
        if (!s->remove_pending) {
                if (usb_set_interface (s->usbdev, _DABUSB_IF, 0) < 0)
-                       dev_err(&s->usbdev->dev, "set_interface failed\n");
+                       err("set_interface failed");
        }
        else
                wake_up (&s->remove_ok);
@@ -845,7 +827,7 @@ static int dabusb_probe (struct usb_inte
        s->devnum = intf->minor;
 
        if (usb_reset_configuration (usbdev) < 0) {
-               dev_err(&intf->dev, "reset_configuration failed\n");
+               err("reset_configuration failed");
                goto reject;
        }
        if (le16_to_cpu(usbdev->descriptor.idProduct) == 0x2131) {
@@ -856,7 +838,7 @@ static int dabusb_probe (struct usb_inte
                dabusb_fpga_download (s, NULL);
 
                if (usb_set_interface (s->usbdev, _DABUSB_IF, 0) < 0) {
-                       dev_err(&intf->dev, "set_interface failed\n");
+                       err("set_interface failed");
                        goto reject;
                }
        }
diff -r d0cfae040789 -r fee99c8e6568 
linux/drivers/media/video/pvrusb2/pvrusb2-v4l2.c
--- a/linux/drivers/media/video/pvrusb2/pvrusb2-v4l2.c  Thu Jan 08 12:13:42 
2009 +0000
+++ b/linux/drivers/media/video/pvrusb2/pvrusb2-v4l2.c  Fri Jan 23 00:29:03 
2009 -0200
@@ -1275,9 +1275,8 @@ static void pvr2_v4l2_dev_init(struct pv
                dip->minor_type = pvr2_v4l_type_video;
                nr_ptr = video_nr;
                if (!dip->stream) {
-                       pr_err(KBUILD_MODNAME
-                               ": Failed to set up pvrusb2 v4l video dev"
-                               " due to missing stream instance\n");
+                       err("Failed to set up pvrusb2 v4l video dev"
+                           " due to missing stream instance");
                        return;
                }
                break;
@@ -1294,8 +1293,8 @@ static void pvr2_v4l2_dev_init(struct pv
                break;
        default:
                /* Bail out (this should be impossible) */
-               pr_err(KBUILD_MODNAME ": Failed to set up pvrusb2 v4l dev"
-                   " due to unrecognized config\n");
+               err("Failed to set up pvrusb2 v4l dev"
+                   " due to unrecognized config");
                return;
        }
 
@@ -1311,8 +1310,7 @@ static void pvr2_v4l2_dev_init(struct pv
                                   dip->v4l_type, mindevnum) < 0) &&
            (video_register_device(&dip->devbase,
                                   dip->v4l_type, -1) < 0)) {
-               pr_err(KBUILD_MODNAME
-                       ": Failed to register pvrusb2 v4l device\n");
+               err("Failed to register pvrusb2 v4l device");
        }
 
        printk(KERN_INFO "pvrusb2: registered device %s%u [%s]\n",
diff -r d0cfae040789 -r fee99c8e6568 linux/drivers/media/video/s2255drv.c
--- a/linux/drivers/media/video/s2255drv.c      Thu Jan 08 12:13:42 2009 +0000
+++ b/linux/drivers/media/video/s2255drv.c      Fri Jan 23 00:29:03 2009 -0200
@@ -337,18 +337,13 @@ static long s2255_vendor_req(struct s225
                             u16 index, u16 value, void *buf,
                             s32 buf_len, int bOut);
 
-/* dev_err macro with driver name */
-#define S2255_DRIVER_NAME "s2255"
-#define s2255_dev_err(dev, fmt, arg...)                                        
\
-               dev_err(dev, S2255_DRIVER_NAME " - " fmt, ##arg)
-
 #define dprintk(level, fmt, arg...)                                    \
        do {                                                            \
                if (*s2255_debug >= (level)) {                          \
-                       printk(KERN_DEBUG S2255_DRIVER_NAME             \
-                               ": " fmt, ##arg);                       \
+                       printk(KERN_DEBUG "s2255: " fmt, ##arg);        \
                }                                                       \
        } while (0)
+
 
 static struct usb_driver s2255_driver;
 
@@ -534,14 +529,14 @@ static void s2255_fwchunk_complete(struc
        int len;
        dprintk(100, "udev %p urb %p", udev, urb);
        if (urb->status) {
-               dev_err(&udev->dev, "URB failed with status %d\n", urb->status);
+               dev_err(&udev->dev, "URB failed with status %d", urb->status);
                atomic_set(&data->fw_state, S2255_FW_FAILED);
                /* wake up anything waiting for the firmware */
                wake_up(&data->wait_fw);
                return;
        }
        if (data->fw_urb == NULL) {
-               s2255_dev_err(&udev->dev, "disconnected\n");
+               dev_err(&udev->dev, "s2255 disconnected\n");
                atomic_set(&data->fw_state, S2255_FW_FAILED);
                /* wake up anything waiting for the firmware */
                wake_up(&data->wait_fw);
@@ -1284,7 +1279,7 @@ static int vidioc_streamon(struct file *
        }
 
        if (!res_get(dev, fh)) {
-               s2255_dev_err(&dev->udev->dev, "stream busy\n");
+               dev_err(&dev->udev->dev, "s2255: stream busy\n");
                return -EBUSY;
        }
 
@@ -1551,8 +1546,7 @@ static int s2255_open(struct file *file)
 
        switch (atomic_read(&dev->fw_data->fw_state)) {
        case S2255_FW_FAILED:
-               s2255_dev_err(&dev->udev->dev,
-                       "firmware load failed. retrying.\n");
+               err("2255 firmware load failed. retrying.\n");
                s2255_fwload_start(dev, 1);
                wait_event_timeout(dev->fw_data->wait_fw,
                                   ((atomic_read(&dev->fw_data->fw_state)
@@ -2180,8 +2174,7 @@ static int s2255_board_init(struct s2255
 
        printk(KERN_INFO "2255 usb firmware version %d \n", fw_ver);
        if (fw_ver < CUR_USB_FWVER)
-               dev_err(&dev->udev->dev,
-                       "usb firmware not up to date %d\n", fw_ver);
+               err("usb firmware not up to date %d\n", fw_ver);
 
        for (j = 0; j < MAX_CHANNELS; j++) {
                dev->b_acquire[j] = 0;
@@ -2236,13 +2229,13 @@ static void read_pipe_completion(struct 
        dprintk(100, "read pipe completion %p, status %d\n", purb,
                purb->status);
        if (pipe_info == NULL) {
-               dev_err(&purb->dev->dev, "no context!\n");
+               err("no context !");
                return;
        }
 
        dev = pipe_info->dev;
        if (dev == NULL) {
-               dev_err(&purb->dev->dev, "no context!\n");
+               err("no context !");
                return;
        }
        status = purb->status;
@@ -2294,7 +2287,7 @@ static int s2255_start_readpipe(struct s
                pipe_info->stream_urb = usb_alloc_urb(0, GFP_KERNEL);
                if (!pipe_info->stream_urb) {
                        dev_err(&dev->udev->dev,
-                               "ReadStream: Unable to alloc URB\n");
+                               "ReadStream: Unable to alloc URB");
                        return -ENOMEM;
                }
                /* transfer buffer allocated in board_init */
@@ -2399,7 +2392,7 @@ static void s2255_stop_readpipe(struct s
        int j;
 
        if (dev == NULL) {
-               s2255_dev_err(&dev->udev->dev, "invalid device\n");
+               err("s2255: invalid device");
                return;
        }
        dprintk(4, "stop read pipe\n");
@@ -2461,7 +2454,7 @@ static int s2255_probe(struct usb_interf
        /* allocate memory for our device state and initialize it to zero */
        dev = kzalloc(sizeof(struct s2255_dev), GFP_KERNEL);
        if (dev == NULL) {
-               s2255_dev_err(&interface->dev, "out of memory\n");
+               err("s2255: out of memory");
                goto error;
        }
 
@@ -2495,7 +2488,7 @@ static int s2255_probe(struct usb_interf
        }
 
        if (!dev->read_endpoint) {
-               dev_err(&interface->dev, "Could not find bulk-in endpoint\n");
+               dev_err(&interface->dev, "Could not find bulk-in endpoint");
                goto error;
        }
 
@@ -2591,7 +2584,7 @@ static void s2255_disconnect(struct usb_
 }
 
 static struct usb_driver s2255_driver = {
-       .name = S2255_DRIVER_NAME,
+       .name = "s2255",
        .probe = s2255_probe,
        .disconnect = s2255_disconnect,
        .id_table = s2255_table,
@@ -2605,8 +2598,7 @@ static int __init usb_s2255_init(void)
        result = usb_register(&s2255_driver);
 
        if (result)
-               pr_err(KBUILD_MODNAME
-                       ": usb_register failed. Error number %d\n", result);
+               err("usb_register failed. Error number %d", result);
 
        dprintk(2, "s2255_init: done\n");
        return result;
diff -r d0cfae040789 -r fee99c8e6568 
linux/drivers/media/video/usbvision/usbvision-core.c
--- a/linux/drivers/media/video/usbvision/usbvision-core.c      Thu Jan 08 
12:13:42 2009 +0000
+++ b/linux/drivers/media/video/usbvision/usbvision-core.c      Fri Jan 23 
00:29:03 2009 -0200
@@ -381,9 +381,8 @@ int usbvision_scratch_alloc(struct usb_u
        usbvision->scratch = vmalloc_32(scratch_buf_size);
        scratch_reset(usbvision);
        if(usbvision->scratch == NULL) {
-               dev_err(&usbvision->dev->dev,
-                       "%s: unable to allocate %d bytes for scratch\n",
-                               __func__, scratch_buf_size);
+               err("%s: unable to allocate %d bytes for scratch",
+                   __func__, scratch_buf_size);
                return -ENOMEM;
        }
        return 0;
@@ -492,9 +491,8 @@ int usbvision_decompress_alloc(struct us
        int IFB_size = MAX_FRAME_WIDTH * MAX_FRAME_HEIGHT * 3 / 2;
        usbvision->IntraFrameBuffer = vmalloc_32(IFB_size);
        if (usbvision->IntraFrameBuffer == NULL) {
-               dev_err(&usbvision->dev->dev,
-                       "%s: unable to allocate %d for compr. frame buffer\n",
-                               __func__, IFB_size);
+               err("%s: unable to allocate %d for compr. frame buffer",
+                   __func__, IFB_size);
                return -ENOMEM;
        }
        return 0;
@@ -1521,9 +1519,8 @@ static void usbvision_isocIrq(struct urb
        errCode = usb_submit_urb (urb, GFP_ATOMIC);
 
        if(errCode) {
-               dev_err(&usbvision->dev->dev,
-                       "%s: usb_submit_urb failed: error %d\n",
-                               __func__, errCode);
+               err("%s: usb_submit_urb failed: error %d",
+                   __func__, errCode);
        }
 
        return;
@@ -1554,8 +1551,7 @@ int usbvision_read_reg(struct usb_usbvis
                                0, (__u16) reg, buffer, 1, HZ);
 
        if (errCode < 0) {
-               dev_err(&usbvision->dev->dev,
-                       "%s: failed: error %d\n", __func__, errCode);
+               err("%s: failed: error %d", __func__, errCode);
                return errCode;
        }
        return buffer[0];
@@ -1583,8 +1579,7 @@ int usbvision_write_reg(struct usb_usbvi
                                USB_RECIP_ENDPOINT, 0, (__u16) reg, &value, 1, 
HZ);
 
        if (errCode < 0) {
-               dev_err(&usbvision->dev->dev,
-                       "%s: failed: error %d\n", __func__, errCode);
+               err("%s: failed: error %d", __func__, errCode);
        }
        return errCode;
 }
@@ -1864,8 +1859,7 @@ int usbvision_set_output(struct usb_usbv
                                 0, (__u16) USBVISION_LXSIZE_O, value, 4, HZ);
 
                if (errCode < 0) {
-                       dev_err(&usbvision->dev->dev,
-                               "%s failed: error %d\n", __func__, errCode);
+                       err("%s failed: error %d", __func__, errCode);
                        return errCode;
                }
                usbvision->curwidth = usbvision->stretch_width * UsbWidth;
@@ -2251,7 +2245,7 @@ static int usbvision_set_dram_settings(s
                             (__u16) USBVISION_DRM_PRM1, value, 8, HZ);
 
        if (rc < 0) {
-               dev_err(&usbvision->dev->dev, "%sERROR=%d\n", __func__, rc);
+               err("%sERROR=%d", __func__, rc);
                return rc;
        }
 
@@ -2459,9 +2453,8 @@ int usbvision_set_alternate(struct usb_u
                PDEBUG(DBG_FUNC,"setting alternate %d with wMaxPacketSize=%u", 
dev->ifaceAlt,dev->isocPacketSize);
                errCode = usb_set_interface(dev->dev, dev->iface, 
dev->ifaceAlt);
                if (errCode < 0) {
-                       dev_err(&dev->dev->dev,
-                               "cannot change alternate number to %d 
(error=%i)\n",
-                                       dev->ifaceAlt, errCode);
+                       err ("cannot change alternate number to %d (error=%i)",
+                                                       dev->ifaceAlt, errCode);
                        return errCode;
                }
        }
@@ -2512,8 +2505,7 @@ int usbvision_init_isoc(struct usb_usbvi
 
                urb = usb_alloc_urb(USBVISION_URB_FRAMES, GFP_KERNEL);
                if (urb == NULL) {
-                       dev_err(&usbvision->dev->dev,
-                               "%s: usb_alloc_urb() failed\n", __func__);
+                       err("%s: usb_alloc_urb() failed", __func__);
                        return -ENOMEM;
                }
                usbvision->sbuf[bufIdx].urb = urb;
@@ -2545,9 +2537,8 @@ int usbvision_init_isoc(struct usb_usbvi
                        errCode = usb_submit_urb(usbvision->sbuf[bufIdx].urb,
                                                 GFP_KERNEL);
                if (errCode) {
-                       dev_err(&usbvision->dev->dev,
-                               "%s: usb_submit_urb(%d) failed: error %d\n",
-                                       __func__, bufIdx, errCode);
+                       err("%s: usb_submit_urb(%d) failed: error %d",
+                           __func__, bufIdx, errCode);
                }
        }
 
@@ -2596,9 +2587,8 @@ void usbvision_stop_isoc(struct usb_usbv
                errCode = usb_set_interface(usbvision->dev, usbvision->iface,
                                            usbvision->ifaceAlt);
                if (errCode < 0) {
-                       dev_err(&usbvision->dev->dev,
-                               "%s: usb_set_interface() failed: error %d\n",
-                                       __func__, errCode);
+                       err("%s: usb_set_interface() failed: error %d",
+                           __func__, errCode);
                        usbvision->last_error = errCode;
                }
                regValue = (16-usbvision_read_reg(usbvision, 
USBVISION_ALTER_REG)) & 0x0F;
diff -r d0cfae040789 -r fee99c8e6568 
linux/drivers/media/video/usbvision/usbvision-i2c.c
--- a/linux/drivers/media/video/usbvision/usbvision-i2c.c       Thu Jan 08 
12:13:42 2009 +0000
+++ b/linux/drivers/media/video/usbvision/usbvision-i2c.c       Fri Jan 23 
00:29:03 2009 -0200
@@ -120,8 +120,7 @@ static inline int usb_find_address(struc
                /* try extended address code... */
                ret = try_write_address(i2c_adap, addr, retries);
                if (ret != 1) {
-                       dev_err(&i2c_adap->dev,
-                               "died at extended address code, while 
writing\n");
+                       err("died at extended address code, while writing");
                        return -EREMOTEIO;
                }
                add[0] = addr;
@@ -130,8 +129,7 @@ static inline int usb_find_address(struc
                        addr |= 0x01;
                        ret = try_read_address(i2c_adap, addr, retries);
                        if (ret != 1) {
-                               dev_err(&i2c_adap->dev,
-                                       "died at extended address code, while 
reading\n");
+                               err("died at extended address code, while 
reading");
                                return -EREMOTEIO;
                        }
                }
diff -r d0cfae040789 -r fee99c8e6568 
linux/drivers/media/video/usbvision/usbvision-video.c
--- a/linux/drivers/media/video/usbvision/usbvision-video.c     Thu Jan 08 
12:13:42 2009 +0000
+++ b/linux/drivers/media/video/usbvision/usbvision-video.c     Fri Jan 23 
00:29:03 2009 -0200
@@ -329,7 +329,7 @@ static void usbvision_create_sysfs(struc
                        return;
        } while (0);
 
-       dev_err(&vdev->dev, "%s error: %d\n", __func__, res);
+       err("%s error: %d\n", __func__, res);
 }
 
 static void usbvision_remove_sysfs(struct video_device *vdev)
@@ -487,9 +487,8 @@ static int vidioc_g_register (struct fil
        /* NT100x has a 8-bit register space */
        errCode = usbvision_read_reg(usbvision, reg->reg&0xff);
        if (errCode < 0) {
-               dev_err(&usbvision->vdev->dev,
-                       "%s: VIDIOC_DBG_G_REGISTER failed: error %d\n",
-                               __func__, errCode);
+               err("%s: VIDIOC_DBG_G_REGISTER failed: error %d",
+                   __func__, errCode);
                return errCode;
        }
        reg->val = errCode;
@@ -508,9 +507,8 @@ static int vidioc_s_register (struct fil
        /* NT100x has a 8-bit register space */
        errCode = usbvision_write_reg(usbvision, reg->reg&0xff, reg->val);
        if (errCode < 0) {
-               dev_err(&usbvision->vdev->dev,
-                       "%s: VIDIOC_DBG_S_REGISTER failed: error %d\n",
-                               __func__, errCode);
+               err("%s: VIDIOC_DBG_S_REGISTER failed: error %d",
+                   __func__, errCode);
                return errCode;
        }
        return 0;
@@ -1191,9 +1189,7 @@ static int usbvision_radio_open(struct f
        mutex_lock(&usbvision->lock);
 
        if (usbvision->user) {
-               dev_err(&usbvision->rdev->dev,
-                       "%s: Someone tried to open an already opened USBVision 
Radio!\n",
-                               __func__);
+               err("%s: Someone tried to open an already opened USBVision 
Radio!", __func__);
                errCode = -EBUSY;
        }
        else {
@@ -1417,8 +1413,7 @@ static struct video_device *usbvision_vd
        struct video_device *vdev;
 
        if (usb_dev == NULL) {
-               dev_err(&usbvision->dev->dev,
-                       "%s: usbvision->dev is not set\n", __func__);
+               err("%s: usbvision->dev is not set", __func__);
                return NULL;
        }
 
@@ -1529,9 +1524,7 @@ static int __devinit usbvision_register_
        return 0;
 
  err_exit:
-       dev_err(&usbvision->dev->dev,
-               "USBVision[%d]: video_register_device() failed\n",
-                       usbvision->nr);
+       err("USBVision[%d]: video_register_device() failed", usbvision->nr);
        usbvision_unregister_video(usbvision);
        return -1;
 }
@@ -1682,20 +1675,20 @@ static int __devinit usbvision_probe(str
        }
        endpoint = &interface->endpoint[1].desc;
        if (!usb_endpoint_xfer_isoc(endpoint)) {
-               dev_err(&intf->dev, "%s: interface %d. has non-ISO endpoint!\n",
+               err("%s: interface %d. has non-ISO endpoint!",
                    __func__, ifnum);
-               dev_err(&intf->dev, "%s: Endpoint attributes %d",
+               err("%s: Endpoint attributes %d",
                    __func__, endpoint->bmAttributes);
                return -ENODEV;
        }
        if (usb_endpoint_dir_out(endpoint)) {
-               dev_err(&intf->dev, "%s: interface %d. has ISO OUT endpoint!\n",
+               err("%s: interface %d. has ISO OUT endpoint!",
                    __func__, ifnum);
                return -ENODEV;
        }
 
        if ((usbvision = usbvision_alloc(dev)) == NULL) {
-               dev_err(&intf->dev, "%s: couldn't allocate USBVision struct\n", 
__func__);
+               err("%s: couldn't allocate USBVision struct", __func__);
                return -ENOMEM;
        }
 
@@ -1718,7 +1711,7 @@ static int __devinit usbvision_probe(str
        usbvision->alt_max_pkt_size = kmalloc(32*
                                              usbvision->num_alt,GFP_KERNEL);
        if (usbvision->alt_max_pkt_size == NULL) {
-               dev_err(&intf->dev, "usbvision: out of memory!\n");
+               err("usbvision: out of memory!\n");
                mutex_unlock(&usbvision->lock);
                return -ENOMEM;
        }
@@ -1779,8 +1772,7 @@ static void __devexit usbvision_disconne
        PDEBUG(DBG_PROBE, "");
 
        if (usbvision == NULL) {
-               dev_err(&usbvision->dev->dev,
-                       "%s: usb_get_intfdata() failed\n", __func__);
+               err("%s: usb_get_intfdata() failed", __func__);
                return;
        }
        usb_set_intfdata (intf, NULL);


---

Patch is available at: 
http://linuxtv.org/hg/v4l-dvb/rev/fee99c8e65682cc4045de73b9fe84f10ad3c48b7

_______________________________________________
linuxtv-commits mailing list
linuxtv-commits@linuxtv.org
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to