staging: rtl8188eu: replace open-coded print_hex_dump()
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Thu, 10 Dec 2015 15:43:54 +0000 (17:43 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 8 Feb 2016 03:00:46 +0000 (19:00 -0800)
No need to reinvent a wheel, i.e. print_hex_dump(). Replace an open-coded
variant by generic kernel helper.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8188eu/core/rtw_recv.c

index 110b8c0b6cd7b734de3abfb0cdd35b40912521ec..3f03999b4d22a2aa94c580aaa24a288e744a8caf 100644 (file)
@@ -1274,32 +1274,25 @@ static int validate_recv_frame(struct adapter *adapter,
        /* Dump rx packets */
        rtw_hal_get_def_var(adapter, HAL_DEF_DBG_DUMP_RXPKT, &(bDumpRxPkt));
        if (bDumpRxPkt == 1) {/* dump all rx packets */
-               int i;
-               DBG_88E("#############################\n");
-
-               for (i = 0; i < 64; i += 8)
-                       DBG_88E("%02X:%02X:%02X:%02X:%02X:%02X:%02X:%02X:\n", *(ptr+i),
-                               *(ptr+i+1), *(ptr+i+2), *(ptr+i+3), *(ptr+i+4), *(ptr+i+5), *(ptr+i+6), *(ptr+i+7));
-               DBG_88E("#############################\n");
+               if (_drv_err_ <= GlobalDebugLevel) {
+                       pr_info(DRIVER_PREFIX "#############################\n");
+                       print_hex_dump(KERN_INFO, DRIVER_PREFIX, DUMP_PREFIX_NONE,
+                                       16, 1, ptr, 64, false);
+                       pr_info(DRIVER_PREFIX "#############################\n");
+               }
        } else if (bDumpRxPkt == 2) {
-               if (type == WIFI_MGT_TYPE) {
-                       int i;
-                       DBG_88E("#############################\n");
-
-                       for (i = 0; i < 64; i += 8)
-                               DBG_88E("%02X:%02X:%02X:%02X:%02X:%02X:%02X:%02X:\n", *(ptr+i),
-                                       *(ptr+i+1), *(ptr+i+2), *(ptr+i+3), *(ptr+i+4), *(ptr+i+5), *(ptr+i+6), *(ptr+i+7));
-                       DBG_88E("#############################\n");
+               if ((_drv_err_ <= GlobalDebugLevel) && (type == WIFI_MGT_TYPE)) {
+                       pr_info(DRIVER_PREFIX "#############################\n");
+                       print_hex_dump(KERN_INFO, DRIVER_PREFIX, DUMP_PREFIX_NONE,
+                                       16, 1, ptr, 64, false);
+                       pr_info(DRIVER_PREFIX "#############################\n");
                }
        } else if (bDumpRxPkt == 3) {
-               if (type == WIFI_DATA_TYPE) {
-                       int i;
-                       DBG_88E("#############################\n");
-
-                       for (i = 0; i < 64; i += 8)
-                               DBG_88E("%02X:%02X:%02X:%02X:%02X:%02X:%02X:%02X:\n", *(ptr+i),
-                                       *(ptr+i+1), *(ptr+i+2), *(ptr+i+3), *(ptr+i+4), *(ptr+i+5), *(ptr+i+6), *(ptr+i+7));
-                       DBG_88E("#############################\n");
+               if ((_drv_err_ <= GlobalDebugLevel) && (type == WIFI_DATA_TYPE)) {
+                       pr_info(DRIVER_PREFIX "#############################\n");
+                       print_hex_dump(KERN_INFO, DRIVER_PREFIX, DUMP_PREFIX_NONE,
+                                       16, 1, ptr, 64, false);
+                       pr_info(DRIVER_PREFIX "#############################\n");
                }
        }
        switch (type) {