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

Ramalingam C
2018-02-14T15:05:53Z

On Wednesday 14 February 2018 08:30 PM, Chris Wilson wrote:
> 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
In that case, I will convert them into DEBUG logs
Thanks,
--Ram