staging/easycap: use %p for printing pointers
authorTomas Winkler <tomas.winkler@intel.com>
Tue, 15 Feb 2011 12:15:20 +0000 (14:15 +0200)
committerGreg Kroah-Hartman <gregkh@suse.de>
Fri, 18 Feb 2011 20:42:12 +0000 (12:42 -0800)
use %p instead of %X
drop casting of pointer to long long int

Cc: Mike Thomas <rmthomas@sciolus.org>
Signed-off-by: Tomas Winkler <tomas.winkler@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/easycap/easycap_main.c
drivers/staging/easycap/easycap_sound.c
drivers/staging/easycap/easycap_sound_oss.c

index fa7982e7a89fcb7598249af82224e0452e614930..e306357a2e5f1f22c577b41607729d93d4247620 100644 (file)
@@ -179,15 +179,14 @@ if (NULL == peasycap) {
        return -EFAULT;
 }
 if (memcmp(&peasycap->telltale[0], TELLTALE, strlen(TELLTALE))) {
-       SAY("ERROR: bad peasycap: 0x%08lX\n", (unsigned long int) peasycap);
+       SAY("ERROR: bad peasycap: %p\n", peasycap);
        return -EFAULT;
 }
 if (NULL == peasycap->pusb_device) {
        SAM("ERROR: peasycap->pusb_device is NULL\n");
        return -EFAULT;
 } else {
-       JOM(16, "0x%08lX=peasycap->pusb_device\n",
-                                       (long int)peasycap->pusb_device);
+       JOM(16, "peasycap->pusb_device=%p\n", peasycap->pusb_device);
 }
 file->private_data = peasycap;
 rc = wakeup_device(peasycap->pusb_device);
@@ -746,7 +745,7 @@ if (NULL == peasycap) {
        return -EFAULT;
 }
 if (memcmp(&peasycap->telltale[0], TELLTALE, strlen(TELLTALE))) {
-       SAY("ERROR: bad peasycap: 0x%08lX\n", (unsigned long int) peasycap);
+       SAY("ERROR: bad peasycap: %p\n", peasycap);
        return -EFAULT;
 }
 if (0 != kill_video_urbs(peasycap)) {
@@ -785,7 +784,7 @@ if (NULL == peasycap) {
        return -EFAULT;
 }
 if (memcmp(&peasycap->telltale[0], TELLTALE, strlen(TELLTALE))) {
-       SAY("ERROR: bad peasycap: 0x%08lX\n", (unsigned long int) peasycap);
+       SAY("ERROR: bad peasycap: %p\n", peasycap);
        return -EFAULT;
 }
 if (0 != kill_video_urbs(peasycap)) {
@@ -825,7 +824,7 @@ if (NULL == peasycap) {
        return;
 }
 if (memcmp(&peasycap->telltale[0], TELLTALE, strlen(TELLTALE))) {
-       SAY("ERROR: bad peasycap: 0x%08lX\n", (unsigned long int) peasycap);
+       SAY("ERROR: bad peasycap: %p\n", peasycap);
        return;
 }
 kd = isdongle(peasycap);
@@ -1041,7 +1040,7 @@ if (NULL == peasycap) {
        return -EFAULT;
 }
 if (memcmp(&peasycap->telltale[0], TELLTALE, strlen(TELLTALE))) {
-       SAY("ERROR: bad peasycap: 0x%08lX\n", (unsigned long int) peasycap);
+       SAY("ERROR: bad peasycap: %p\n", peasycap);
        return -EFAULT;
 }
 if (NULL == peasycap->pusb_device) {
@@ -1078,8 +1077,7 @@ if (0 <= kd && DONGLE_MANY > kd) {
                return -ERESTARTSYS;
        }
        if (memcmp(&peasycap->telltale[0], TELLTALE, strlen(TELLTALE))) {
-               SAY("ERROR: bad peasycap: 0x%08lX\n",
-                                               (unsigned long int) peasycap);
+               SAY("ERROR: bad peasycap: %p\n", peasycap);
                mutex_unlock(&easycapdc60_dongle[kd].mutex_video);
                return -ERESTARTSYS;
        }
@@ -2622,7 +2620,7 @@ if (NULL == peasycap) {
        return;
 }
 if (memcmp(&peasycap->telltale[0], TELLTALE, strlen(TELLTALE))) {
-       SAY("ERROR: bad peasycap: 0x%08lX\n", (unsigned long int) peasycap);
+       SAY("ERROR: bad peasycap: %p\n", peasycap);
        return;
 }
 peasycap->vma_many++;
@@ -2640,7 +2638,7 @@ if (NULL == peasycap) {
        return;
 }
 if (memcmp(&peasycap->telltale[0], TELLTALE, strlen(TELLTALE))) {
-       SAY("ERROR: bad peasycap: 0x%08lX\n", (unsigned long int) peasycap);
+       SAY("ERROR: bad peasycap: %p\n", peasycap);
        return;
 }
 peasycap->vma_many--;
@@ -2774,7 +2772,7 @@ if (NULL == peasycap) {
        return;
 }
 if (memcmp(&peasycap->telltale[0], TELLTALE, strlen(TELLTALE))) {
-       SAY("ERROR: bad peasycap: 0x%08lX\n", (unsigned long int) peasycap);
+       SAY("ERROR: bad peasycap: %p\n", peasycap);
        return;
 }
 if (peasycap->video_eof)
@@ -3280,15 +3278,7 @@ static int easycap_usb_probe(struct usb_interface *pusb_interface,
                        SAY("ERROR: Could not allocate peasycap\n");
                        return -ENOMEM;
                }
-               SAM("allocated 0x%08lX=peasycap\n", (unsigned long int) peasycap);
-/*vvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvv*/
-#ifdef EASYCAP_IS_VIDEODEV_CLIENT
-               SAM("where     0x%08lX=&peasycap->video_device\n",
-                               (unsigned long int) &peasycap->video_device);
-               SAM("and       0x%08lX=&peasycap->v4l2_device\n",
-                               (unsigned long int) &peasycap->v4l2_device);
-#endif /*EASYCAP_IS_VIDEODEV_CLIENT*/
-/*^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^*/
+               SAM("allocated %p=peasycap\n", peasycap);
 /*---------------------------------------------------------------------------*/
 /*
  *  PERFORM URGENT INTIALIZATIONS ...
@@ -4573,7 +4563,7 @@ static void easycap_usb_disconnect(struct usb_interface *pusb_interface)
 /*^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^*/
 /*---------------------------------------------------------------------------*/
        if (memcmp(&peasycap->telltale[0], TELLTALE, strlen(TELLTALE))) {
-               SAY("ERROR: bad peasycap: 0x%08lX\n", (unsigned long int) peasycap);
+               SAY("ERROR: bad peasycap: %p\n", peasycap);
                return;
        }
 /*---------------------------------------------------------------------------*/
index 626450a57140d0bcc2aeb9e48b1488d99c63f3c5..0d647c81ed5591bbb43182e3e512e7d6c5ac9439 100644 (file)
@@ -431,7 +431,7 @@ if (NULL == prt) {
        return -EFAULT;
 }
 if (NULL != prt->dma_area) {
-       JOT(8, "0x%08lX=prt->dma_area\n", (unsigned long int)prt->dma_area);
+       JOT(8, "prt->dma_area = %p\n", prt->dma_area);
        vfree(prt->dma_area);
        prt->dma_area = NULL;
 } else
index b18cd351837c4e15e21322ae831f45a285c30e30..e817b35718852ee187c516ee34379cd9632c6dc9 100644 (file)
@@ -351,7 +351,7 @@ if (memcmp(&peasycap->telltale[0], TELLTALE, strlen(TELLTALE))) {
 /*^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^*/
 /*---------------------------------------------------------------------------*/
 if (memcmp(&peasycap->telltale[0], TELLTALE, strlen(TELLTALE))) {
-       SAY("ERROR: bad peasycap: 0x%08lX\n", (unsigned long int) peasycap);
+       SAY("ERROR: bad peasycap: %p\n", peasycap);
        return -EFAULT;
 }
 /*---------------------------------------------------------------------------*/
@@ -377,7 +377,7 @@ if (NULL == peasycap) {
        return -EFAULT;
 }
 if (memcmp(&peasycap->telltale[0], TELLTALE, strlen(TELLTALE))) {
-       SAY("ERROR: bad peasycap: 0x%08lX\n", (unsigned long int) peasycap);
+       SAY("ERROR: bad peasycap: %p\n", peasycap);
        return -EFAULT;
 }
 if (0 != kill_audio_urbs(peasycap)) {
@@ -424,7 +424,7 @@ if (NULL == peasycap) {
        return -EFAULT;
 }
 if (memcmp(&peasycap->telltale[0], TELLTALE, strlen(TELLTALE))) {
-       SAY("ERROR: bad peasycap: 0x%08lX\n", (unsigned long int) peasycap);
+       SAY("ERROR: bad peasycap: %p\n", peasycap);
        return -EFAULT;
 }
 if (NULL == peasycap->pusb_device) {
@@ -460,8 +460,7 @@ if (0 <= kd && DONGLE_MANY > kd) {
                return -ERESTARTSYS;
        }
        if (memcmp(&peasycap->telltale[0], TELLTALE, strlen(TELLTALE))) {
-               SAY("ERROR: bad peasycap: 0x%08lX\n",
-                                               (unsigned long int) peasycap);
+               SAY("ERROR: bad peasycap: %p\n", peasycap);
                mutex_unlock(&easycapdc60_dongle[kd].mutex_audio);
                return -ERESTARTSYS;
        }