OSDN Git Service

drm/radeon: use raw buffer printk specifier
authorDmitry Rozhkov <dmitry.rozhkov@linux.intel.com>
Thu, 21 Dec 2017 10:04:53 +0000 (12:04 +0200)
committerAlex Deucher <alexander.deucher@amd.com>
Mon, 19 Feb 2018 19:17:04 +0000 (14:17 -0500)
printk format strings accepting a single subsequent argument
are shorter thus easier to read.

Instead of having format strings accepting 3 different arguments
pointing to first 3 bytes of the same buffer rewrite the format
string to accept only one argument - the buffer - with "%3ph"
specifier.

Signed-off-by: Dmitry Rozhkov <dmitry.rozhkov@linux.intel.com>
Suggested-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/radeon/radeon_dp_mst.c

index 238e6eb..cd8a3ee 100644 (file)
@@ -718,7 +718,7 @@ radeon_dp_mst_check_status(struct radeon_connector *radeon_connector)
                                       DP_SINK_COUNT_ESI, esi, 8);
 go_again:
                if (dret == 8) {
-                       DRM_DEBUG_KMS("got esi %02x %02x %02x\n", esi[0], esi[1], esi[2]);
+                       DRM_DEBUG_KMS("got esi %3ph\n", esi);
                        ret = drm_dp_mst_hpd_irq(&radeon_connector->mst_mgr, esi, &handled);
 
                        if (handled) {
@@ -733,7 +733,7 @@ go_again:
                                dret = drm_dp_dpcd_read(&radeon_connector->ddc_bus->aux,
                                                        DP_SINK_COUNT_ESI, esi, 8);
                                if (dret == 8) {
-                                       DRM_DEBUG_KMS("got esi2 %02x %02x %02x\n", esi[0], esi[1], esi[2]);
+                                       DRM_DEBUG_KMS("got esi2 %3ph\n", esi);
                                        goto go_again;
                                }
                        } else