diff mbox series

drm/i915/debugfs: remove i915_dpcd file

Message ID 20200117150551.9836-1-jani.nikula@intel.com (mailing list archive)
State New, archived
Headers show
Series drm/i915/debugfs: remove i915_dpcd file | expand

Commit Message

Jani Nikula Jan. 17, 2020, 3:05 p.m. UTC 
We've long had the more generic /dev/drm_dp_auxN devices for the same
purpose. Drop the redundant and limited DPCD debugfs file.

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 drivers/gpu/drm/i915/i915_debugfs.c | 64 -----------------------------
 1 file changed, 64 deletions(-)

Comments

Ville Syrjälä Jan. 29, 2020, 11:57 a.m. UTC | #1 
On Fri, Jan 17, 2020 at 05:05:51PM +0200, Jani Nikula wrote:
> We've long had the more generic /dev/drm_dp_auxN devices for the same
> purpose. Drop the redundant and limited DPCD debugfs file.
> 
> Signed-off-by: Jani Nikula <jani.nikula@intel.com>

Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>

> ---
>  drivers/gpu/drm/i915/i915_debugfs.c | 64 -----------------------------
>  1 file changed, 64 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c
> index c05fa4205de8..e320b957b792 100644
> --- a/drivers/gpu/drm/i915/i915_debugfs.c
> +++ b/drivers/gpu/drm/i915/i915_debugfs.c
> @@ -4329,65 +4329,6 @@ int i915_debugfs_register(struct drm_i915_private *dev_priv)
>  					minor->debugfs_root, minor);
>  }
>  
> -struct dpcd_block {
> -	/* DPCD dump start address. */
> -	unsigned int offset;
> -	/* DPCD dump end address, inclusive. If unset, .size will be used. */
> -	unsigned int end;
> -	/* DPCD dump size. Used if .end is unset. If unset, defaults to 1. */
> -	size_t size;
> -	/* Only valid for eDP. */
> -	bool edp;
> -};
> -
> -static const struct dpcd_block i915_dpcd_debug[] = {
> -	{ .offset = DP_DPCD_REV, .size = DP_RECEIVER_CAP_SIZE },
> -	{ .offset = DP_PSR_SUPPORT, .end = DP_PSR_CAPS },
> -	{ .offset = DP_DOWNSTREAM_PORT_0, .size = 16 },
> -	{ .offset = DP_LINK_BW_SET, .end = DP_EDP_CONFIGURATION_SET },
> -	{ .offset = DP_SINK_COUNT, .end = DP_ADJUST_REQUEST_LANE2_3 },
> -	{ .offset = DP_SET_POWER },
> -	{ .offset = DP_EDP_DPCD_REV },
> -	{ .offset = DP_EDP_GENERAL_CAP_1, .end = DP_EDP_GENERAL_CAP_3 },
> -	{ .offset = DP_EDP_DISPLAY_CONTROL_REGISTER, .end = DP_EDP_BACKLIGHT_FREQ_CAP_MAX_LSB },
> -	{ .offset = DP_EDP_DBC_MINIMUM_BRIGHTNESS_SET, .end = DP_EDP_DBC_MAXIMUM_BRIGHTNESS_SET },
> -};
> -
> -static int i915_dpcd_show(struct seq_file *m, void *data)
> -{
> -	struct drm_connector *connector = m->private;
> -	struct intel_dp *intel_dp =
> -		enc_to_intel_dp(intel_attached_encoder(to_intel_connector(connector)));
> -	u8 buf[16];
> -	ssize_t err;
> -	int i;
> -
> -	if (connector->status != connector_status_connected)
> -		return -ENODEV;
> -
> -	for (i = 0; i < ARRAY_SIZE(i915_dpcd_debug); i++) {
> -		const struct dpcd_block *b = &i915_dpcd_debug[i];
> -		size_t size = b->end ? b->end - b->offset + 1 : (b->size ?: 1);
> -
> -		if (b->edp &&
> -		    connector->connector_type != DRM_MODE_CONNECTOR_eDP)
> -			continue;
> -
> -		/* low tech for now */
> -		if (WARN_ON(size > sizeof(buf)))
> -			continue;
> -
> -		err = drm_dp_dpcd_read(&intel_dp->aux, b->offset, buf, size);
> -		if (err < 0)
> -			seq_printf(m, "%04x: ERROR %d\n", b->offset, (int)err);
> -		else
> -			seq_printf(m, "%04x: %*ph\n", b->offset, (int)err, buf);
> -	}
> -
> -	return 0;
> -}
> -DEFINE_SHOW_ATTRIBUTE(i915_dpcd);
> -
>  static int i915_panel_show(struct seq_file *m, void *data)
>  {
>  	struct drm_connector *connector = m->private;
> @@ -4552,11 +4493,6 @@ int i915_debugfs_connector_add(struct drm_connector *connector)
>  	if (!root)
>  		return -ENODEV;
>  
> -	if (connector->connector_type == DRM_MODE_CONNECTOR_DisplayPort ||
> -	    connector->connector_type == DRM_MODE_CONNECTOR_eDP)
> -		debugfs_create_file("i915_dpcd", S_IRUGO, root,
> -				    connector, &i915_dpcd_fops);
> -
>  	if (connector->connector_type == DRM_MODE_CONNECTOR_eDP) {
>  		debugfs_create_file("i915_panel_timings", S_IRUGO, root,
>  				    connector, &i915_panel_fops);
> -- 
> 2.20.1
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Jani Nikula Jan. 29, 2020, 12:06 p.m. UTC | #2 
On Wed, 29 Jan 2020, Ville Syrjälä <ville.syrjala@linux.intel.com> wrote:
> On Fri, Jan 17, 2020 at 05:05:51PM +0200, Jani Nikula wrote:
>> We've long had the more generic /dev/drm_dp_auxN devices for the same
>> purpose. Drop the redundant and limited DPCD debugfs file.
>> 
>> Signed-off-by: Jani Nikula <jani.nikula@intel.com>
>
> Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>

Thanks, pushed to dinq.

BR,
Jani.

>
>> ---
>>  drivers/gpu/drm/i915/i915_debugfs.c | 64 -----------------------------
>>  1 file changed, 64 deletions(-)
>> 
>> diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c
>> index c05fa4205de8..e320b957b792 100644
>> --- a/drivers/gpu/drm/i915/i915_debugfs.c
>> +++ b/drivers/gpu/drm/i915/i915_debugfs.c
>> @@ -4329,65 +4329,6 @@ int i915_debugfs_register(struct drm_i915_private *dev_priv)
>>  					minor->debugfs_root, minor);
>>  }
>>  
>> -struct dpcd_block {
>> -	/* DPCD dump start address. */
>> -	unsigned int offset;
>> -	/* DPCD dump end address, inclusive. If unset, .size will be used. */
>> -	unsigned int end;
>> -	/* DPCD dump size. Used if .end is unset. If unset, defaults to 1. */
>> -	size_t size;
>> -	/* Only valid for eDP. */
>> -	bool edp;
>> -};
>> -
>> -static const struct dpcd_block i915_dpcd_debug[] = {
>> -	{ .offset = DP_DPCD_REV, .size = DP_RECEIVER_CAP_SIZE },
>> -	{ .offset = DP_PSR_SUPPORT, .end = DP_PSR_CAPS },
>> -	{ .offset = DP_DOWNSTREAM_PORT_0, .size = 16 },
>> -	{ .offset = DP_LINK_BW_SET, .end = DP_EDP_CONFIGURATION_SET },
>> -	{ .offset = DP_SINK_COUNT, .end = DP_ADJUST_REQUEST_LANE2_3 },
>> -	{ .offset = DP_SET_POWER },
>> -	{ .offset = DP_EDP_DPCD_REV },
>> -	{ .offset = DP_EDP_GENERAL_CAP_1, .end = DP_EDP_GENERAL_CAP_3 },
>> -	{ .offset = DP_EDP_DISPLAY_CONTROL_REGISTER, .end = DP_EDP_BACKLIGHT_FREQ_CAP_MAX_LSB },
>> -	{ .offset = DP_EDP_DBC_MINIMUM_BRIGHTNESS_SET, .end = DP_EDP_DBC_MAXIMUM_BRIGHTNESS_SET },
>> -};
>> -
>> -static int i915_dpcd_show(struct seq_file *m, void *data)
>> -{
>> -	struct drm_connector *connector = m->private;
>> -	struct intel_dp *intel_dp =
>> -		enc_to_intel_dp(intel_attached_encoder(to_intel_connector(connector)));
>> -	u8 buf[16];
>> -	ssize_t err;
>> -	int i;
>> -
>> -	if (connector->status != connector_status_connected)
>> -		return -ENODEV;
>> -
>> -	for (i = 0; i < ARRAY_SIZE(i915_dpcd_debug); i++) {
>> -		const struct dpcd_block *b = &i915_dpcd_debug[i];
>> -		size_t size = b->end ? b->end - b->offset + 1 : (b->size ?: 1);
>> -
>> -		if (b->edp &&
>> -		    connector->connector_type != DRM_MODE_CONNECTOR_eDP)
>> -			continue;
>> -
>> -		/* low tech for now */
>> -		if (WARN_ON(size > sizeof(buf)))
>> -			continue;
>> -
>> -		err = drm_dp_dpcd_read(&intel_dp->aux, b->offset, buf, size);
>> -		if (err < 0)
>> -			seq_printf(m, "%04x: ERROR %d\n", b->offset, (int)err);
>> -		else
>> -			seq_printf(m, "%04x: %*ph\n", b->offset, (int)err, buf);
>> -	}
>> -
>> -	return 0;
>> -}
>> -DEFINE_SHOW_ATTRIBUTE(i915_dpcd);
>> -
>>  static int i915_panel_show(struct seq_file *m, void *data)
>>  {
>>  	struct drm_connector *connector = m->private;
>> @@ -4552,11 +4493,6 @@ int i915_debugfs_connector_add(struct drm_connector *connector)
>>  	if (!root)
>>  		return -ENODEV;
>>  
>> -	if (connector->connector_type == DRM_MODE_CONNECTOR_DisplayPort ||
>> -	    connector->connector_type == DRM_MODE_CONNECTOR_eDP)
>> -		debugfs_create_file("i915_dpcd", S_IRUGO, root,
>> -				    connector, &i915_dpcd_fops);
>> -
>>  	if (connector->connector_type == DRM_MODE_CONNECTOR_eDP) {
>>  		debugfs_create_file("i915_panel_timings", S_IRUGO, root,
>>  				    connector, &i915_panel_fops);
>> -- 
>> 2.20.1
>> 
>> _______________________________________________
>> Intel-gfx mailing list
>> Intel-gfx@lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
diff mbox series

Patch

diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c
index c05fa4205de8..e320b957b792 100644
--- a/drivers/gpu/drm/i915/i915_debugfs.c
+++ b/drivers/gpu/drm/i915/i915_debugfs.c
@@ -4329,65 +4329,6 @@  int i915_debugfs_register(struct drm_i915_private *dev_priv)
 					minor->debugfs_root, minor);
 }
 
-struct dpcd_block {
-	/* DPCD dump start address. */
-	unsigned int offset;
-	/* DPCD dump end address, inclusive. If unset, .size will be used. */
-	unsigned int end;
-	/* DPCD dump size. Used if .end is unset. If unset, defaults to 1. */
-	size_t size;
-	/* Only valid for eDP. */
-	bool edp;
-};
-
-static const struct dpcd_block i915_dpcd_debug[] = {
-	{ .offset = DP_DPCD_REV, .size = DP_RECEIVER_CAP_SIZE },
-	{ .offset = DP_PSR_SUPPORT, .end = DP_PSR_CAPS },
-	{ .offset = DP_DOWNSTREAM_PORT_0, .size = 16 },
-	{ .offset = DP_LINK_BW_SET, .end = DP_EDP_CONFIGURATION_SET },
-	{ .offset = DP_SINK_COUNT, .end = DP_ADJUST_REQUEST_LANE2_3 },
-	{ .offset = DP_SET_POWER },
-	{ .offset = DP_EDP_DPCD_REV },
-	{ .offset = DP_EDP_GENERAL_CAP_1, .end = DP_EDP_GENERAL_CAP_3 },
-	{ .offset = DP_EDP_DISPLAY_CONTROL_REGISTER, .end = DP_EDP_BACKLIGHT_FREQ_CAP_MAX_LSB },
-	{ .offset = DP_EDP_DBC_MINIMUM_BRIGHTNESS_SET, .end = DP_EDP_DBC_MAXIMUM_BRIGHTNESS_SET },
-};
-
-static int i915_dpcd_show(struct seq_file *m, void *data)
-{
-	struct drm_connector *connector = m->private;
-	struct intel_dp *intel_dp =
-		enc_to_intel_dp(intel_attached_encoder(to_intel_connector(connector)));
-	u8 buf[16];
-	ssize_t err;
-	int i;
-
-	if (connector->status != connector_status_connected)
-		return -ENODEV;
-
-	for (i = 0; i < ARRAY_SIZE(i915_dpcd_debug); i++) {
-		const struct dpcd_block *b = &i915_dpcd_debug[i];
-		size_t size = b->end ? b->end - b->offset + 1 : (b->size ?: 1);
-
-		if (b->edp &&
-		    connector->connector_type != DRM_MODE_CONNECTOR_eDP)
-			continue;
-
-		/* low tech for now */
-		if (WARN_ON(size > sizeof(buf)))
-			continue;
-
-		err = drm_dp_dpcd_read(&intel_dp->aux, b->offset, buf, size);
-		if (err < 0)
-			seq_printf(m, "%04x: ERROR %d\n", b->offset, (int)err);
-		else
-			seq_printf(m, "%04x: %*ph\n", b->offset, (int)err, buf);
-	}
-
-	return 0;
-}
-DEFINE_SHOW_ATTRIBUTE(i915_dpcd);
-
 static int i915_panel_show(struct seq_file *m, void *data)
 {
 	struct drm_connector *connector = m->private;
@@ -4552,11 +4493,6 @@  int i915_debugfs_connector_add(struct drm_connector *connector)
 	if (!root)
 		return -ENODEV;
 
-	if (connector->connector_type == DRM_MODE_CONNECTOR_DisplayPort ||
-	    connector->connector_type == DRM_MODE_CONNECTOR_eDP)
-		debugfs_create_file("i915_dpcd", S_IRUGO, root,
-				    connector, &i915_dpcd_fops);
-
 	if (connector->connector_type == DRM_MODE_CONNECTOR_eDP) {
 		debugfs_create_file("i915_panel_timings", S_IRUGO, root,
 				    connector, &i915_panel_fops);