Commit 6db07ce4 authored by Wambui Karuga's avatar Wambui Karuga Committed by Jani Nikula

drm/i915/debugfs: conversion to drm_device based logging macros.

Manual conversion of printk based logging macros to the struct
drm_device based logging macros in i915/i915_debugfs.c.
Also involves extracting the struct drm_i915_private device from various
intel types to use in the macros.

This does not convert various instances of the DRM_DEBUG macro due to
the lack of an analogous struct drm_device based logging macro.

References: https://lists.freedesktop.org/archives/dri-devel/2020-January/253381.htmlSigned-off-by: default avatarWambui Karuga <wambui.karugax@gmail.com>
Signed-off-by: default avatarJani Nikula <jani.nikula@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20200131093416.28431-3-wambui.karugax@gmail.com
parent 00376ccf
...@@ -2845,7 +2845,8 @@ static ssize_t i915_ipc_status_write(struct file *file, const char __user *ubuf, ...@@ -2845,7 +2845,8 @@ static ssize_t i915_ipc_status_write(struct file *file, const char __user *ubuf,
with_intel_runtime_pm(&dev_priv->runtime_pm, wakeref) { with_intel_runtime_pm(&dev_priv->runtime_pm, wakeref) {
if (!dev_priv->ipc_enabled && enable) if (!dev_priv->ipc_enabled && enable)
DRM_INFO("Enabling IPC: WM will be proper only after next commit\n"); drm_info(&dev_priv->drm,
"Enabling IPC: WM will be proper only after next commit\n");
dev_priv->wm.distrust_bios_wm = true; dev_priv->wm.distrust_bios_wm = true;
dev_priv->ipc_enabled = enable; dev_priv->ipc_enabled = enable;
intel_enable_ipc(dev_priv); intel_enable_ipc(dev_priv);
...@@ -3045,7 +3046,8 @@ static ssize_t i915_displayport_test_active_write(struct file *file, ...@@ -3045,7 +3046,8 @@ static ssize_t i915_displayport_test_active_write(struct file *file,
if (IS_ERR(input_buffer)) if (IS_ERR(input_buffer))
return PTR_ERR(input_buffer); return PTR_ERR(input_buffer);
DRM_DEBUG_DRIVER("Copied %d bytes from user\n", (unsigned int)len); drm_dbg(&to_i915(dev)->drm,
"Copied %d bytes from user\n", (unsigned int)len);
drm_connector_list_iter_begin(dev, &conn_iter); drm_connector_list_iter_begin(dev, &conn_iter);
drm_for_each_connector_iter(connector, &conn_iter) { drm_for_each_connector_iter(connector, &conn_iter) {
...@@ -3064,7 +3066,8 @@ static ssize_t i915_displayport_test_active_write(struct file *file, ...@@ -3064,7 +3066,8 @@ static ssize_t i915_displayport_test_active_write(struct file *file,
status = kstrtoint(input_buffer, 10, &val); status = kstrtoint(input_buffer, 10, &val);
if (status < 0) if (status < 0)
break; break;
DRM_DEBUG_DRIVER("Got %d for test active\n", val); drm_dbg(&to_i915(dev)->drm,
"Got %d for test active\n", val);
/* To prevent erroneous activation of the compliance /* To prevent erroneous activation of the compliance
* testing code, only accept an actual value of 1 here * testing code, only accept an actual value of 1 here
*/ */
...@@ -3633,7 +3636,8 @@ i915_cache_sharing_set(void *data, u64 val) ...@@ -3633,7 +3636,8 @@ i915_cache_sharing_set(void *data, u64 val)
if (val > 3) if (val > 3)
return -EINVAL; return -EINVAL;
DRM_DEBUG_DRIVER("Manually setting uncore sharing to %llu\n", val); drm_dbg(&dev_priv->drm,
"Manually setting uncore sharing to %llu\n", val);
with_intel_runtime_pm(&dev_priv->runtime_pm, wakeref) { with_intel_runtime_pm(&dev_priv->runtime_pm, wakeref) {
u32 snpcr; u32 snpcr;
...@@ -4433,19 +4437,20 @@ static ssize_t i915_dsc_fec_support_write(struct file *file, ...@@ -4433,19 +4437,20 @@ static ssize_t i915_dsc_fec_support_write(struct file *file,
struct drm_connector *connector = struct drm_connector *connector =
((struct seq_file *)file->private_data)->private; ((struct seq_file *)file->private_data)->private;
struct intel_encoder *encoder = intel_attached_encoder(to_intel_connector(connector)); struct intel_encoder *encoder = intel_attached_encoder(to_intel_connector(connector));
struct drm_i915_private *i915 = to_i915(encoder->base.dev);
struct intel_dp *intel_dp = enc_to_intel_dp(encoder); struct intel_dp *intel_dp = enc_to_intel_dp(encoder);
if (len == 0) if (len == 0)
return 0; return 0;
DRM_DEBUG_DRIVER("Copied %zu bytes from user to force DSC\n", drm_dbg(&i915->drm,
len); "Copied %zu bytes from user to force DSC\n", len);
ret = kstrtobool_from_user(ubuf, len, &dsc_enable); ret = kstrtobool_from_user(ubuf, len, &dsc_enable);
if (ret < 0) if (ret < 0)
return ret; return ret;
DRM_DEBUG_DRIVER("Got %s for DSC Enable\n", drm_dbg(&i915->drm, "Got %s for DSC Enable\n",
(dsc_enable) ? "true" : "false"); (dsc_enable) ? "true" : "false");
intel_dp->force_dsc_en = dsc_enable; intel_dp->force_dsc_en = dsc_enable;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment