drm: don't push static constants on stack for %*ph

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>
This commit is contained in:
Andy Shevchenko 2013-08-02 14:09:24 +03:00 committed by Dave Airlie
parent baa7094355
commit 08fcd72b14
3 changed files with 4 additions and 4 deletions

View 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;
}

View 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;
}

View 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 */