USB: storage: use %*ph specifier to dump small buffers
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Fri, 5 Sep 2014 14:56:25 +0000 (17:56 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 24 Sep 2014 05:15:18 +0000 (22:15 -0700)
Instead of dereference each byte let's use %*ph specifier in the printk()
calls.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/storage/alauda.c
drivers/usb/storage/sddr09.c

index 6636a583da126bff3138d5a9d3a711224e5974d6..62c2d9daa7d6b0a949de72ae0dbbbc56ac1e148c 100644 (file)
@@ -415,14 +415,11 @@ static int alauda_init_media(struct us_data *us)
        if (alauda_get_media_signature(us, data) != USB_STOR_XFER_GOOD)
                return USB_STOR_TRANSPORT_ERROR;
 
-       usb_stor_dbg(us, "Media signature: %02X %02X %02X %02X\n",
-                    data[0], data[1], data[2], data[3]);
+       usb_stor_dbg(us, "Media signature: %4ph\n", data);
        media_info = alauda_card_find_id(data[1]);
        if (media_info == NULL) {
-               printk(KERN_WARNING
-                       "alauda_init_media: Unrecognised media signature: "
-                       "%02X %02X %02X %02X\n",
-                       data[0], data[1], data[2], data[3]);
+               pr_warn("alauda_init_media: Unrecognised media signature: %4ph\n",
+                       data);
                return USB_STOR_TRANSPORT_ERROR;
        }
 
@@ -513,7 +510,7 @@ static int alauda_check_status2(struct us_data *us)
        if (rc != USB_STOR_XFER_GOOD)
                return rc;
 
-       usb_stor_dbg(us, "%02X %02X %02X\n", data[0], data[1], data[2]);
+       usb_stor_dbg(us, "%3ph\n", data);
        if (data[0] & ALAUDA_STATUS_ERROR)
                return USB_STOR_XFER_ERROR;
 
index 38a4504ce450d535a2f71ab306ef73fe15a1a7cf..3847053d732cc6974bcfd6828043c6e79853cb4d 100644 (file)
@@ -1155,8 +1155,7 @@ sddr09_get_cardinfo(struct us_data *us, unsigned char flags) {
                return NULL;
        }
 
-       sprintf(blurbtxt, "sddr09: Found Flash card, ID = %02X %02X %02X %02X",
-               deviceID[0], deviceID[1], deviceID[2], deviceID[3]);
+       sprintf(blurbtxt, "sddr09: Found Flash card, ID = %4ph", deviceID);
 
        /* Byte 0 is the manufacturer */
        sprintf(blurbtxt + strlen(blurbtxt),