Re: [PATCH 35/43] drm/i915: Enable HDCP version that is best capable

Chris Wilson
2018-02-14T15:00:41Z

Quoting Ramalingam C (2018-02-14 14:51:10)
>
>
> On Wednesday 14 February 2018 08:12 PM, Chris Wilson wrote:
> > Quoting Ramalingam C (2018-02-14 14:13:50)
> >> @@ -1437,7 +1525,7 @@ static int _intel_hdcp2_disable(struct intel_hdcp *hdcp)
> >> {
> >> int ret;
> >>
> >> - DRM_ERROR("[%s:%d] HDCP2.2 is being Disabled\n",
> >> + DRM_INFO("[%s:%d] HDCP2.2 is being Disabled\n",
> >> hdcp->connector->base.name, hdcp->connector->base.base.id);
> > Will there be a third patch making this DEBUG_KMS?
> > -Chris
> Thanks Chris!
>
> Will address these comments. No, I feel its could to have DRM_INFO for
> state transition logs.
> In practical HDCP logs will not be that frequent. If needed I will
> change them into DEBUG in upcoming versions :)

Treat everything above DEBUG as communicating with the user. For the
user, this information is already available via the connector property.
-Chris