Commit 926b005c authored by Jani Nikula's avatar Jani Nikula

drm/i915: split out display debugfs to a separate file

The i915_debugfs.c has grown more than a little unwieldy. Split out the
display related debugfs code to a file of its own under display/,
initialized with a separate call. No functional changes.

v2:
- Also moved i915_frontbuffer_tracking, i915_gem_framebuffer,
  i915_power_domain_info, i915_dmc_info, i915_ipc_status (Ville)

Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
Reviewed-by: default avatarChris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: default avatarJani Nikula <jani.nikula@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20200211161451.6867-2-jani.nikula@intel.com
parent 06d3ff6e
...@@ -68,6 +68,7 @@ i915-$(CONFIG_COMPAT) += i915_ioc32.o ...@@ -68,6 +68,7 @@ i915-$(CONFIG_COMPAT) += i915_ioc32.o
i915-$(CONFIG_DEBUG_FS) += \ i915-$(CONFIG_DEBUG_FS) += \
i915_debugfs.o \ i915_debugfs.o \
i915_debugfs_params.o \ i915_debugfs_params.o \
display/intel_display_debugfs.o \
display/intel_pipe_crc.o display/intel_pipe_crc.o
i915-$(CONFIG_PERF_EVENTS) += i915_pmu.o i915-$(CONFIG_PERF_EVENTS) += i915_pmu.o
......
This diff is collapsed.
/* SPDX-License-Identifier: MIT */
/*
* Copyright © 2020 Intel Corporation
*/
#ifndef __INTEL_DISPLAY_DEBUGFS_H__
#define __INTEL_DISPLAY_DEBUGFS_H__
struct drm_connector;
struct drm_i915_private;
#ifdef CONFIG_DEBUG_FS
int intel_display_debugfs_register(struct drm_i915_private *i915);
int intel_connector_debugfs_add(struct drm_connector *connector);
#else
static inline int intel_display_debugfs_register(struct drm_i915_private *i915) { return 0; }
static inline int intel_connector_debugfs_add(struct drm_connector *connector) { return 0; }
#endif
#endif /* __INTEL_DISPLAY_DEBUGFS_H__ */
...@@ -49,6 +49,7 @@ ...@@ -49,6 +49,7 @@
#include "intel_audio.h" #include "intel_audio.h"
#include "intel_connector.h" #include "intel_connector.h"
#include "intel_ddi.h" #include "intel_ddi.h"
#include "intel_display_debugfs.h"
#include "intel_display_types.h" #include "intel_display_types.h"
#include "intel_dp.h" #include "intel_dp.h"
#include "intel_dp_link_training.h" #include "intel_dp_link_training.h"
...@@ -5851,7 +5852,7 @@ intel_dp_connector_register(struct drm_connector *connector) ...@@ -5851,7 +5852,7 @@ intel_dp_connector_register(struct drm_connector *connector)
if (ret) if (ret)
return ret; return ret;
i915_debugfs_connector_add(connector); intel_connector_debugfs_add(connector);
DRM_DEBUG_KMS("registering %s bus for %s\n", DRM_DEBUG_KMS("registering %s bus for %s\n",
intel_dp->aux.name, connector->kdev->kobj.name); intel_dp->aux.name, connector->kdev->kobj.name);
......
...@@ -45,6 +45,7 @@ ...@@ -45,6 +45,7 @@
#include "intel_audio.h" #include "intel_audio.h"
#include "intel_connector.h" #include "intel_connector.h"
#include "intel_ddi.h" #include "intel_ddi.h"
#include "intel_display_debugfs.h"
#include "intel_display_types.h" #include "intel_display_types.h"
#include "intel_dp.h" #include "intel_dp.h"
#include "intel_dpio_phy.h" #include "intel_dpio_phy.h"
...@@ -2818,7 +2819,7 @@ intel_hdmi_connector_register(struct drm_connector *connector) ...@@ -2818,7 +2819,7 @@ intel_hdmi_connector_register(struct drm_connector *connector)
if (ret) if (ret)
return ret; return ret;
i915_debugfs_connector_add(connector); intel_connector_debugfs_add(connector);
intel_hdmi_create_i2c_symlink(connector); intel_hdmi_create_i2c_symlink(connector);
......
This diff is collapsed.
...@@ -6,15 +6,17 @@ ...@@ -6,15 +6,17 @@
#ifndef __I915_DEBUGFS_H__ #ifndef __I915_DEBUGFS_H__
#define __I915_DEBUGFS_H__ #define __I915_DEBUGFS_H__
struct drm_i915_private;
struct drm_connector; struct drm_connector;
struct drm_i915_gem_object;
struct drm_i915_private;
struct seq_file;
#ifdef CONFIG_DEBUG_FS #ifdef CONFIG_DEBUG_FS
int i915_debugfs_register(struct drm_i915_private *dev_priv); int i915_debugfs_register(struct drm_i915_private *dev_priv);
int i915_debugfs_connector_add(struct drm_connector *connector); void i915_debugfs_describe_obj(struct seq_file *m, struct drm_i915_gem_object *obj);
#else #else
static inline int i915_debugfs_register(struct drm_i915_private *dev_priv) { return 0; } static inline int i915_debugfs_register(struct drm_i915_private *dev_priv) { return 0; }
static inline int i915_debugfs_connector_add(struct drm_connector *connector) { return 0; } static inline void i915_debugfs_describe_obj(struct seq_file *m, struct drm_i915_gem_object *obj) {}
#endif #endif
#endif /* __I915_DEBUGFS_H__ */ #endif /* __I915_DEBUGFS_H__ */
...@@ -51,6 +51,7 @@ ...@@ -51,6 +51,7 @@
#include "display/intel_bw.h" #include "display/intel_bw.h"
#include "display/intel_cdclk.h" #include "display/intel_cdclk.h"
#include "display/intel_csr.h" #include "display/intel_csr.h"
#include "display/intel_display_debugfs.h"
#include "display/intel_display_types.h" #include "display/intel_display_types.h"
#include "display/intel_dp.h" #include "display/intel_dp.h"
#include "display/intel_fbdev.h" #include "display/intel_fbdev.h"
...@@ -1320,6 +1321,7 @@ static void i915_driver_register(struct drm_i915_private *dev_priv) ...@@ -1320,6 +1321,7 @@ static void i915_driver_register(struct drm_i915_private *dev_priv)
/* Reveal our presence to userspace */ /* Reveal our presence to userspace */
if (drm_dev_register(dev, 0) == 0) { if (drm_dev_register(dev, 0) == 0) {
i915_debugfs_register(dev_priv); i915_debugfs_register(dev_priv);
intel_display_debugfs_register(dev_priv);
i915_setup_sysfs(dev_priv); i915_setup_sysfs(dev_priv);
/* Depends on sysfs having been initialized */ /* Depends on sysfs having been initialized */
......
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