drm: don't push static constants on stack for %*ph
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Fri, 2 Aug 2013 11:09:24 +0000 (14:09 +0300)
committerDave Airlie <airlied@redhat.com>
Wed, 7 Aug 2013 00:10:58 +0000 (10:10 +1000)
There is no need to pass constants via stack. The width may be explicitly
specified in the format.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Reviewed-by: David Herrmann <dh.herrmann@gmail.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
drivers/gpu/drm/nouveau/core/engine/disp/dport.c
drivers/gpu/drm/radeon/atombios_dp.c
drivers/gpu/drm/udl/udl_main.c

index 31cc8fe8e7f07ab26cc6ce90ee902a5a2887124a..054d9cff4f533596e0eb52ef4ad5497ce727ca82 100644 (file)
@@ -150,7 +150,7 @@ dp_link_train_update(struct dp_state *dp, u32 delay)
        if (ret)
                return ret;
 
-       DBG("status %*ph\n", 6, dp->stat);
+       DBG("status %6ph\n", dp->stat);
        return 0;
 }
 
index 064023bed480fb84eae64d505465601499ac4dfc..c239739736db5f047e495af913b8e7334e9cc2f7 100644 (file)
@@ -550,7 +550,7 @@ static bool radeon_dp_get_link_status(struct radeon_connector *radeon_connector,
                return false;
        }
 
-       DRM_DEBUG_KMS("link status %*ph\n", 6, link_status);
+       DRM_DEBUG_KMS("link status %6ph\n", link_status);
        return true;
 }
 
index 0ce2d7195256197f0b1bfe6edcb0485f0f74b1c6..f5ae57406f3447eacf3dbce309583ae303628824 100644 (file)
@@ -41,8 +41,8 @@ static int udl_parse_vendor_descriptor(struct drm_device *dev,
        total_len = usb_get_descriptor(usbdev, 0x5f, /* vendor specific */
                                    0, desc, MAX_VENDOR_DESCRIPTOR_SIZE);
        if (total_len > 5) {
-               DRM_INFO("vendor descriptor length:%x data:%*ph\n",
-                       total_len, 11, desc);
+               DRM_INFO("vendor descriptor length:%x data:%11ph\n",
+                       total_len, desc);
 
                if ((desc[0] != total_len) || /* descriptor length */
                    (desc[1] != 0x5f) ||   /* vendor descriptor type */