| Message ID | 1517568320-15579-4-git-send-email-ramalingam.c@intel.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Fri, Feb 02, 2018 at 04:15:15PM +0530, Ramalingam C wrote: > When HDCP authentication is triggered on multiple connector, having > connector name and ID in debug message will be more informative. > > Signed-off-by: Ramalingam C <ramalingam.c@intel.com> > --- > drivers/gpu/drm/i915/intel_hdcp.c | 17 ++++++++++++----- > 1 file changed, 12 insertions(+), 5 deletions(-) > > diff --git a/drivers/gpu/drm/i915/intel_hdcp.c b/drivers/gpu/drm/i915/intel_hdcp.c > index 182a3c8a4e4a..b3f407cef8b0 100644 > --- a/drivers/gpu/drm/i915/intel_hdcp.c > +++ b/drivers/gpu/drm/i915/intel_hdcp.c > @@ -388,8 +388,10 @@ int intel_hdcp_auth_downstream(struct intel_connector *connector) > return -ENXIO; > } > > - DRM_DEBUG_KMS("HDCP is enabled (%d downstream devices)\n", > - num_downstream); > + DRM_DEBUG_KMS("[%s:%d] HDCP is enabled (%d downstream devices)\n", > + connector->base.name, > + connector->base.base.id, > + num_downstream); Your alignment is off, should be: DRM_DEBUG_KMS("[%s:%d] HDCP is enabled (%d downstream devices)\n", connector->base.name, connector->base.base.id, num_downstream); Same for the rest of the patch. With that fixed, Reviewed-by: Sean Paul <seanpaul@chromium.org> > return 0; > } > > @@ -515,7 +517,9 @@ static int intel_hdcp_auth(struct intel_connector *connector) > } > } > > - DRM_DEBUG_KMS("HDCP is enabled (no repeater present)\n"); > + DRM_DEBUG_KMS("[%s:%d] HDCP is enabled (no repeater present)\n", > + connector->base.name, > + connector->base.base.id); > return 0; > } > > @@ -541,7 +545,8 @@ static int _intel_hdcp_disable(struct intel_connector *connector) > return ret; > } > > - DRM_DEBUG_KMS("HDCP is disabled\n"); > + DRM_DEBUG_KMS("[%s:%d] HDCP is disabled\n", connector->base.name, > + connector->base.base.id); > return 0; > } > > @@ -743,7 +748,9 @@ int intel_hdcp_check_link(struct intel_connector *connector) > goto out; > } > > - DRM_DEBUG_KMS("HDCP link failed, retrying authentication\n"); > + DRM_DEBUG_KMS("[%s:%d] HDCP link failed, retrying authentication\n", > + connector->base.name, > + connector->base.base.id); > > ret = _intel_hdcp_disable(connector); > if (ret) { > -- > 2.7.4 >
diff --git a/drivers/gpu/drm/i915/intel_hdcp.c b/drivers/gpu/drm/i915/intel_hdcp.c index 182a3c8a4e4a..b3f407cef8b0 100644 --- a/drivers/gpu/drm/i915/intel_hdcp.c +++ b/drivers/gpu/drm/i915/intel_hdcp.c @@ -388,8 +388,10 @@ int intel_hdcp_auth_downstream(struct intel_connector *connector) return -ENXIO; } - DRM_DEBUG_KMS("HDCP is enabled (%d downstream devices)\n", - num_downstream); + DRM_DEBUG_KMS("[%s:%d] HDCP is enabled (%d downstream devices)\n", + connector->base.name, + connector->base.base.id, + num_downstream); return 0; } @@ -515,7 +517,9 @@ static int intel_hdcp_auth(struct intel_connector *connector) } } - DRM_DEBUG_KMS("HDCP is enabled (no repeater present)\n"); + DRM_DEBUG_KMS("[%s:%d] HDCP is enabled (no repeater present)\n", + connector->base.name, + connector->base.base.id); return 0; } @@ -541,7 +545,8 @@ static int _intel_hdcp_disable(struct intel_connector *connector) return ret; } - DRM_DEBUG_KMS("HDCP is disabled\n"); + DRM_DEBUG_KMS("[%s:%d] HDCP is disabled\n", connector->base.name, + connector->base.base.id); return 0; } @@ -743,7 +748,9 @@ int intel_hdcp_check_link(struct intel_connector *connector) goto out; } - DRM_DEBUG_KMS("HDCP link failed, retrying authentication\n"); + DRM_DEBUG_KMS("[%s:%d] HDCP link failed, retrying authentication\n", + connector->base.name, + connector->base.base.id); ret = _intel_hdcp_disable(connector); if (ret) {
When HDCP authentication is triggered on multiple connector, having connector name and ID in debug message will be more informative. Signed-off-by: Ramalingam C <ramalingam.c@intel.com> --- drivers/gpu/drm/i915/intel_hdcp.c | 17 ++++++++++++----- 1 file changed, 12 insertions(+), 5 deletions(-)