From: Ramalingam C <ramalingam.c@intel.com>
To: Jani Nikula <jani.nikula@linux.intel.com>
Cc: intel-gfx <intel-gfx@lists.freedesktop.org>,
dri-devel <dri-devel@lists.freedesktop.org>
Subject: Re: [PATCH 5/5] drm/i915/hdcp: conversion to struct drm_device based logging macros.
Date: Wed, 12 Feb 2020 15:55:14 +0530 [thread overview]
Message-ID: <20200212102514.GB2939@intel.com> (raw)
In-Reply-To: <87o8u4untn.fsf@intel.com>
On 2020-02-12 at 12:01:08 +0200, Jani Nikula wrote:
> On Wed, 12 Feb 2020, Ramalingam C <ramalingam.c@intel.com> wrote:
> > Converts remaining instances of the printk based logging macros in
> > i915/display/intel_hdcp.c with the struct drm_device based macros
> > manually.
> >
> > This is continuation of commit 65833c463886 ("drm/i915/hdcp: conversion
> > to struct drm_device based logging macros.")
> >
> > Signed-off-by: Ramalingam C <ramalingam.c@intel.com>
> > cc: Jani Nikula <jani.nikula@linux.intel.com>
> > ---
> > drivers/gpu/drm/i915/display/intel_hdcp.c | 107 ++++++++++++----------
> > 1 file changed, 60 insertions(+), 47 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/i915/display/intel_hdcp.c b/drivers/gpu/drm/i915/display/intel_hdcp.c
> > index 11f204668a5c..3361796b59ca 100644
> > --- a/drivers/gpu/drm/i915/display/intel_hdcp.c
> > +++ b/drivers/gpu/drm/i915/display/intel_hdcp.c
> > @@ -43,6 +43,7 @@ static
> > int intel_hdcp_read_valid_bksv(struct intel_digital_port *intel_dig_port,
> > const struct intel_hdcp_shim *shim, u8 *bksv)
> > {
> > + struct drm_device *drm = intel_dig_port->base.base.dev;
>
> Please prefer adding struct drm_i915_private *i915 local variable
> instead of struct drm_device, and use it as &i915->drm.
>
> I know you're only adding this to get the pointer for logging, but in
> any given place we're much more likely to need i915 in the future than
> drm_device pointer.
>
> In some places you do add dev_priv and use it like that; please name the
> variable i915.
Sure. Whereever i am adding the variable, I will name it as i915. not
renaming the existing dev_priv.
Thanks,
-Ram
>
> BR,
> Jani.
>
>
> > int ret, i, tries = 2;
> >
> > /* HDCP spec states that we must retry the bksv if it is invalid */
> > @@ -54,7 +55,7 @@ int intel_hdcp_read_valid_bksv(struct intel_digital_port *intel_dig_port,
> > break;
> > }
> > if (i == tries) {
> > - DRM_DEBUG_KMS("Bksv is invalid\n");
> > + drm_dbg_kms(drm, "Bksv is invalid\n");
> > return -ENODEV;
> > }
> >
> > @@ -485,8 +486,8 @@ int intel_hdcp_validate_v_prime(struct intel_connector *connector,
> > return ret;
> > sha_idx += sizeof(sha_text);
> > } else {
> > - DRM_DEBUG_KMS("Invalid number of leftovers %d\n",
> > - sha_leftovers);
> > + drm_dbg_kms(&dev_priv->drm, "Invalid number of leftovers %d\n",
> > + sha_leftovers);
> > return -EINVAL;
> > }
> >
> > @@ -514,11 +515,11 @@ int intel_hdcp_validate_v_prime(struct intel_connector *connector,
> > rep_ctl | HDCP_SHA1_COMPLETE_HASH);
> > if (intel_de_wait_for_set(dev_priv, HDCP_REP_CTL,
> > HDCP_SHA1_COMPLETE, 1)) {
> > - DRM_ERROR("Timed out waiting for SHA1 complete\n");
> > + drm_err(&dev_priv->drm, "Timed out waiting for SHA1 complete\n");
> > return -ETIMEDOUT;
> > }
> > if (!(intel_de_read(dev_priv, HDCP_REP_CTL) & HDCP_SHA1_V_MATCH)) {
> > - DRM_DEBUG_KMS("SHA-1 mismatch, HDCP failed\n");
> > + drm_dbg_kms(&dev_priv->drm, "SHA-1 mismatch, HDCP failed\n");
> > return -ENXIO;
> > }
> >
> > @@ -537,7 +538,8 @@ int intel_hdcp_auth_downstream(struct intel_connector *connector)
> >
> > ret = intel_hdcp_poll_ksv_fifo(intel_dig_port, shim);
> > if (ret) {
> > - DRM_DEBUG_KMS("KSV list failed to become ready (%d)\n", ret);
> > + drm_dbg_kms(&dev_priv->drm,
> > + "KSV list failed to become ready (%d)\n", ret);
> > return ret;
> > }
> >
> > @@ -547,7 +549,7 @@ int intel_hdcp_auth_downstream(struct intel_connector *connector)
> >
> > if (DRM_HDCP_MAX_DEVICE_EXCEEDED(bstatus[0]) ||
> > DRM_HDCP_MAX_CASCADE_EXCEEDED(bstatus[1])) {
> > - DRM_DEBUG_KMS("Max Topology Limit Exceeded\n");
> > + drm_dbg_kms(&dev_priv->drm, "Max Topology Limit Exceeded\n");
> > return -EPERM;
> > }
> >
> > @@ -560,13 +562,14 @@ int intel_hdcp_auth_downstream(struct intel_connector *connector)
> > */
> > num_downstream = DRM_HDCP_NUM_DOWNSTREAM(bstatus[0]);
> > if (num_downstream == 0) {
> > - DRM_DEBUG_KMS("Repeater with zero downstream devices\n");
> > + drm_dbg_kms(&dev_priv->drm,
> > + "Repeater with zero downstream devices\n");
> > return -EINVAL;
> > }
> >
> > ksv_fifo = kcalloc(DRM_HDCP_KSV_LEN, num_downstream, GFP_KERNEL);
> > if (!ksv_fifo) {
> > - DRM_DEBUG_KMS("Out of mem: ksv_fifo\n");
> > + drm_dbg_kms(&dev_priv->drm, "Out of mem: ksv_fifo\n");
> > return -ENOMEM;
> > }
> >
> > @@ -576,7 +579,7 @@ int intel_hdcp_auth_downstream(struct intel_connector *connector)
> >
> > if (drm_hdcp_check_ksvs_revoked(&dev_priv->drm, ksv_fifo,
> > num_downstream)) {
> > - DRM_ERROR("Revoked Ksv(s) in ksv_fifo\n");
> > + drm_err(&dev_priv->drm, "Revoked Ksv(s) in ksv_fifo\n");
> > ret = -EPERM;
> > goto err;
> > }
> > @@ -594,12 +597,13 @@ int intel_hdcp_auth_downstream(struct intel_connector *connector)
> > }
> >
> > if (i == tries) {
> > - DRM_DEBUG_KMS("V Prime validation failed.(%d)\n", ret);
> > + drm_dbg_kms(&dev_priv->drm,
> > + "V Prime validation failed.(%d)\n", ret);
> > goto err;
> > }
> >
> > - DRM_DEBUG_KMS("HDCP is enabled (%d downstream devices)\n",
> > - num_downstream);
> > + drm_dbg_kms(&dev_priv->drm, "HDCP is enabled (%d downstream devices)\n",
> > + num_downstream);
> > ret = 0;
> > err:
> > kfree(ksv_fifo);
> > @@ -642,7 +646,8 @@ static int intel_hdcp_auth(struct intel_connector *connector)
> > if (ret)
> > return ret;
> > if (!hdcp_capable) {
> > - DRM_DEBUG_KMS("Panel is not HDCP capable\n");
> > + drm_dbg_kms(&dev_priv->drm,
> > + "Panel is not HDCP capable\n");
> > return -EINVAL;
> > }
> > }
> > @@ -659,7 +664,7 @@ static int intel_hdcp_auth(struct intel_connector *connector)
> > if (intel_de_wait_for_set(dev_priv,
> > HDCP_STATUS(dev_priv, cpu_transcoder, port),
> > HDCP_STATUS_AN_READY, 1)) {
> > - DRM_ERROR("Timed out waiting for An\n");
> > + drm_err(&dev_priv->drm, "Timed out waiting for An\n");
> > return -ETIMEDOUT;
> > }
> >
> > @@ -680,7 +685,7 @@ static int intel_hdcp_auth(struct intel_connector *connector)
> > return ret;
> >
> > if (drm_hdcp_check_ksvs_revoked(&dev_priv->drm, bksv.shim, 1)) {
> > - DRM_ERROR("BKSV is revoked\n");
> > + drm_err(&dev_priv->drm, "BKSV is revoked\n");
> > return -EPERM;
> > }
> >
> > @@ -706,7 +711,7 @@ static int intel_hdcp_auth(struct intel_connector *connector)
> > /* Wait for R0 ready */
> > if (wait_for(intel_de_read(dev_priv, HDCP_STATUS(dev_priv, cpu_transcoder, port)) &
> > (HDCP_STATUS_R0_READY | HDCP_STATUS_ENC), 1)) {
> > - DRM_ERROR("Timed out waiting for R0 ready\n");
> > + drm_err(&dev_priv->drm, "Timed out waiting for R0 ready\n");
> > return -ETIMEDOUT;
> > }
> >
> > @@ -743,8 +748,10 @@ static int intel_hdcp_auth(struct intel_connector *connector)
> > }
> >
> > if (i == tries) {
> > - DRM_DEBUG_KMS("Timed out waiting for Ri prime match (%x)\n",
> > - intel_de_read(dev_priv, HDCP_STATUS(dev_priv, cpu_transcoder, port)));
> > + drm_dbg_kms(&dev_priv->drm,
> > + "Timed out waiting for Ri prime match (%x)\n",
> > + intel_de_read(dev_priv, HDCP_STATUS(dev_priv,
> > + cpu_transcoder, port)));
> > return -ETIMEDOUT;
> > }
> >
> > @@ -753,7 +760,7 @@ static int intel_hdcp_auth(struct intel_connector *connector)
> > HDCP_STATUS(dev_priv, cpu_transcoder, port),
> > HDCP_STATUS_ENC,
> > ENCRYPT_STATUS_CHANGE_TIMEOUT_MS)) {
> > - DRM_ERROR("Timed out waiting for encryption\n");
> > + drm_err(&dev_priv->drm, "Timed out waiting for encryption\n");
> > return -ETIMEDOUT;
> > }
> >
> > @@ -765,7 +772,7 @@ static int intel_hdcp_auth(struct intel_connector *connector)
> > if (repeater_present)
> > return intel_hdcp_auth_downstream(connector);
> >
> > - DRM_DEBUG_KMS("HDCP is enabled (no repeater present)\n");
> > + drm_dbg_kms(&dev_priv->drm, "HDCP is enabled (no repeater present)\n");
> > return 0;
> > }
> >
> > @@ -1270,7 +1277,7 @@ static int hdcp2_authentication_key_exchange(struct intel_connector *connector)
> > return ret;
> >
> > if (msgs.send_cert.rx_caps[0] != HDCP_2_2_RX_CAPS_VERSION_VAL) {
> > - DRM_DEBUG_KMS("cert.rx_caps dont claim HDCP2.2\n");
> > + drm_dbg_kms(&dev_priv->drm, "cert.rx_caps dont claim HDCP2.2\n");
> > return -EINVAL;
> > }
> >
> > @@ -1279,7 +1286,7 @@ static int hdcp2_authentication_key_exchange(struct intel_connector *connector)
> > if (drm_hdcp_check_ksvs_revoked(&dev_priv->drm,
> > msgs.send_cert.cert_rx.receiver_id,
> > 1)) {
> > - DRM_ERROR("Receiver ID is revoked\n");
> > + drm_err(&dev_priv->drm, "Receiver ID is revoked\n");
> > return -EPERM;
> > }
> >
> > @@ -1446,7 +1453,7 @@ int hdcp2_authenticate_repeater_topology(struct intel_connector *connector)
> >
> > if (HDCP_2_2_MAX_CASCADE_EXCEEDED(rx_info[1]) ||
> > HDCP_2_2_MAX_DEVS_EXCEEDED(rx_info[1])) {
> > - DRM_DEBUG_KMS("Topology Max Size Exceeded\n");
> > + drm_dbg_kms(&dev_priv->drm, "Topology Max Size Exceeded\n");
> > return -EINVAL;
> > }
> >
> > @@ -1456,7 +1463,7 @@ int hdcp2_authenticate_repeater_topology(struct intel_connector *connector)
> >
> > if (seq_num_v < hdcp->seq_num_v) {
> > /* Roll over of the seq_num_v from repeater. Reauthenticate. */
> > - DRM_DEBUG_KMS("Seq_num_v roll over.\n");
> > + drm_dbg_kms(&dev_priv->drm, "Seq_num_v roll over.\n");
> > return -EINVAL;
> > }
> >
> > @@ -1465,7 +1472,7 @@ int hdcp2_authenticate_repeater_topology(struct intel_connector *connector)
> > if (drm_hdcp_check_ksvs_revoked(&dev_priv->drm,
> > msgs.recvid_list.receiver_ids,
> > device_cnt)) {
> > - DRM_ERROR("Revoked receiver ID(s) is in list\n");
> > + drm_err(&dev_priv->drm, "Revoked receiver ID(s) is in list\n");
> > return -EPERM;
> > }
> >
> > @@ -1487,25 +1494,27 @@ int hdcp2_authenticate_repeater_topology(struct intel_connector *connector)
> > static int hdcp2_authenticate_sink(struct intel_connector *connector)
> > {
> > struct intel_digital_port *intel_dig_port = intel_attached_dig_port(connector);
> > + struct drm_i915_private *dev_priv = to_i915(connector->base.dev);
> > struct intel_hdcp *hdcp = &connector->hdcp;
> > const struct intel_hdcp_shim *shim = hdcp->shim;
> > int ret;
> >
> > ret = hdcp2_authentication_key_exchange(connector);
> > if (ret < 0) {
> > - DRM_DEBUG_KMS("AKE Failed. Err : %d\n", ret);
> > + drm_dbg_kms(&dev_priv->drm, "AKE Failed. Err : %d\n", ret);
> > return ret;
> > }
> >
> > ret = hdcp2_locality_check(connector);
> > if (ret < 0) {
> > - DRM_DEBUG_KMS("Locality Check failed. Err : %d\n", ret);
> > + drm_dbg_kms(&dev_priv->drm,
> > + "Locality Check failed. Err : %d\n", ret);
> > return ret;
> > }
> >
> > ret = hdcp2_session_key_exchange(connector);
> > if (ret < 0) {
> > - DRM_DEBUG_KMS("SKE Failed. Err : %d\n", ret);
> > + drm_dbg_kms(&dev_priv->drm, "SKE Failed. Err : %d\n", ret);
> > return ret;
> > }
> >
> > @@ -1520,7 +1529,8 @@ static int hdcp2_authenticate_sink(struct intel_connector *connector)
> > if (hdcp->is_repeater) {
> > ret = hdcp2_authenticate_repeater_topology(connector);
> > if (ret < 0) {
> > - DRM_DEBUG_KMS("Repeater Auth Failed. Err: %d\n", ret);
> > + drm_dbg_kms(&dev_priv->drm,
> > + "Repeater Auth Failed. Err: %d\n", ret);
> > return ret;
> > }
> > }
> > @@ -1651,10 +1661,10 @@ static int hdcp2_authenticate_and_encrypt(struct intel_connector *connector)
> > }
> >
> > /* Clearing the mei hdcp session */
> > - DRM_DEBUG_KMS("HDCP2.2 Auth %d of %d Failed.(%d)\n",
> > - i + 1, tries, ret);
> > + drm_dbg_kms(drm, "HDCP2.2 Auth %d of %d Failed.(%d)\n",
> > + i + 1, tries, ret);
> > if (hdcp2_deauthenticate_port(connector) < 0)
> > - DRM_DEBUG_KMS("Port deauth failed.\n");
> > + drm_dbg_kms(drm, "Port deauth failed.\n");
> > }
> >
> > if (!ret) {
> > @@ -1665,9 +1675,9 @@ static int hdcp2_authenticate_and_encrypt(struct intel_connector *connector)
> > msleep(HDCP_2_2_DELAY_BEFORE_ENCRYPTION_EN);
> > ret = hdcp2_enable_encryption(connector);
> > if (ret < 0) {
> > - DRM_DEBUG_KMS("Encryption Enable Failed.(%d)\n", ret);
> > + drm_dbg_kms(drm, "Encryption Enable Failed.(%d)\n", ret);
> > if (hdcp2_deauthenticate_port(connector) < 0)
> > - DRM_DEBUG_KMS("Port deauth failed.\n");
> > + drm_dbg_kms(drm, "Port deauth failed.\n");
> > }
> > }
> >
> > @@ -1677,22 +1687,23 @@ static int hdcp2_authenticate_and_encrypt(struct intel_connector *connector)
> > static int _intel_hdcp2_enable(struct intel_connector *connector)
> > {
> > struct intel_hdcp *hdcp = &connector->hdcp;
> > + struct drm_device *drm = connector->base.dev;
> > int ret;
> >
> > - DRM_DEBUG_KMS("[%s:%d] HDCP2.2 is being enabled. Type: %d\n",
> > - connector->base.name, connector->base.base.id,
> > - hdcp->content_type);
> > + drm_dbg_kms(drm, "[%s:%d] HDCP2.2 is being enabled. Type: %d\n",
> > + connector->base.name, connector->base.base.id,
> > + hdcp->content_type);
> >
> > ret = hdcp2_authenticate_and_encrypt(connector);
> > if (ret) {
> > - DRM_DEBUG_KMS("HDCP2 Type%d Enabling Failed. (%d)\n",
> > - hdcp->content_type, ret);
> > + drm_dbg_kms(drm, "HDCP2 Type%d Enabling Failed. (%d)\n",
> > + hdcp->content_type, ret);
> > return ret;
> > }
> >
> > - DRM_DEBUG_KMS("[%s:%d] HDCP2.2 is enabled. Type %d\n",
> > - connector->base.name, connector->base.base.id,
> > - hdcp->content_type);
> > + drm_dbg_kms(drm, "[%s:%d] HDCP2.2 is enabled. Type %d\n",
> > + connector->base.name, connector->base.base.id,
> > + hdcp->content_type);
> >
> > hdcp->hdcp2_encrypted = true;
> > return 0;
> > @@ -1700,15 +1711,16 @@ static int _intel_hdcp2_enable(struct intel_connector *connector)
> >
> > static int _intel_hdcp2_disable(struct intel_connector *connector)
> > {
> > + struct drm_device *drm = connector->base.dev;
> > int ret;
> >
> > - DRM_DEBUG_KMS("[%s:%d] HDCP2.2 is being Disabled\n",
> > - connector->base.name, connector->base.base.id);
> > + drm_dbg_kms(drm, "[%s:%d] HDCP2.2 is being Disabled\n",
> > + connector->base.name, connector->base.base.id);
> >
> > ret = hdcp2_disable_encryption(connector);
> >
> > if (hdcp2_deauthenticate_port(connector) < 0)
> > - DRM_DEBUG_KMS("Port deauth failed.\n");
> > + drm_dbg_kms(drm, "Port deauth failed.\n");
> >
> > connector->hdcp.hdcp2_encrypted = false;
> >
> > @@ -1951,11 +1963,12 @@ static void intel_hdcp2_init(struct intel_connector *connector,
> > const struct intel_hdcp_shim *shim)
> > {
> > struct intel_hdcp *hdcp = &connector->hdcp;
> > + struct drm_device *drm = connector->base.dev;
> > int ret;
> >
> > ret = initialize_hdcp_port_data(connector, shim);
> > if (ret) {
> > - DRM_DEBUG_KMS("Mei hdcp data init failed\n");
> > + drm_dbg_kms(drm, "Mei hdcp data init failed\n");
> > return;
> > }
>
> --
> Jani Nikula, Intel Open Source Graphics Center
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
WARNING: multiple messages have this Message-ID (diff)
From: Ramalingam C <ramalingam.c@intel.com>
To: Jani Nikula <jani.nikula@linux.intel.com>
Cc: intel-gfx <intel-gfx@lists.freedesktop.org>,
dri-devel <dri-devel@lists.freedesktop.org>
Subject: Re: [Intel-gfx] [PATCH 5/5] drm/i915/hdcp: conversion to struct drm_device based logging macros.
Date: Wed, 12 Feb 2020 15:55:14 +0530 [thread overview]
Message-ID: <20200212102514.GB2939@intel.com> (raw)
In-Reply-To: <87o8u4untn.fsf@intel.com>
On 2020-02-12 at 12:01:08 +0200, Jani Nikula wrote:
> On Wed, 12 Feb 2020, Ramalingam C <ramalingam.c@intel.com> wrote:
> > Converts remaining instances of the printk based logging macros in
> > i915/display/intel_hdcp.c with the struct drm_device based macros
> > manually.
> >
> > This is continuation of commit 65833c463886 ("drm/i915/hdcp: conversion
> > to struct drm_device based logging macros.")
> >
> > Signed-off-by: Ramalingam C <ramalingam.c@intel.com>
> > cc: Jani Nikula <jani.nikula@linux.intel.com>
> > ---
> > drivers/gpu/drm/i915/display/intel_hdcp.c | 107 ++++++++++++----------
> > 1 file changed, 60 insertions(+), 47 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/i915/display/intel_hdcp.c b/drivers/gpu/drm/i915/display/intel_hdcp.c
> > index 11f204668a5c..3361796b59ca 100644
> > --- a/drivers/gpu/drm/i915/display/intel_hdcp.c
> > +++ b/drivers/gpu/drm/i915/display/intel_hdcp.c
> > @@ -43,6 +43,7 @@ static
> > int intel_hdcp_read_valid_bksv(struct intel_digital_port *intel_dig_port,
> > const struct intel_hdcp_shim *shim, u8 *bksv)
> > {
> > + struct drm_device *drm = intel_dig_port->base.base.dev;
>
> Please prefer adding struct drm_i915_private *i915 local variable
> instead of struct drm_device, and use it as &i915->drm.
>
> I know you're only adding this to get the pointer for logging, but in
> any given place we're much more likely to need i915 in the future than
> drm_device pointer.
>
> In some places you do add dev_priv and use it like that; please name the
> variable i915.
Sure. Whereever i am adding the variable, I will name it as i915. not
renaming the existing dev_priv.
Thanks,
-Ram
>
> BR,
> Jani.
>
>
> > int ret, i, tries = 2;
> >
> > /* HDCP spec states that we must retry the bksv if it is invalid */
> > @@ -54,7 +55,7 @@ int intel_hdcp_read_valid_bksv(struct intel_digital_port *intel_dig_port,
> > break;
> > }
> > if (i == tries) {
> > - DRM_DEBUG_KMS("Bksv is invalid\n");
> > + drm_dbg_kms(drm, "Bksv is invalid\n");
> > return -ENODEV;
> > }
> >
> > @@ -485,8 +486,8 @@ int intel_hdcp_validate_v_prime(struct intel_connector *connector,
> > return ret;
> > sha_idx += sizeof(sha_text);
> > } else {
> > - DRM_DEBUG_KMS("Invalid number of leftovers %d\n",
> > - sha_leftovers);
> > + drm_dbg_kms(&dev_priv->drm, "Invalid number of leftovers %d\n",
> > + sha_leftovers);
> > return -EINVAL;
> > }
> >
> > @@ -514,11 +515,11 @@ int intel_hdcp_validate_v_prime(struct intel_connector *connector,
> > rep_ctl | HDCP_SHA1_COMPLETE_HASH);
> > if (intel_de_wait_for_set(dev_priv, HDCP_REP_CTL,
> > HDCP_SHA1_COMPLETE, 1)) {
> > - DRM_ERROR("Timed out waiting for SHA1 complete\n");
> > + drm_err(&dev_priv->drm, "Timed out waiting for SHA1 complete\n");
> > return -ETIMEDOUT;
> > }
> > if (!(intel_de_read(dev_priv, HDCP_REP_CTL) & HDCP_SHA1_V_MATCH)) {
> > - DRM_DEBUG_KMS("SHA-1 mismatch, HDCP failed\n");
> > + drm_dbg_kms(&dev_priv->drm, "SHA-1 mismatch, HDCP failed\n");
> > return -ENXIO;
> > }
> >
> > @@ -537,7 +538,8 @@ int intel_hdcp_auth_downstream(struct intel_connector *connector)
> >
> > ret = intel_hdcp_poll_ksv_fifo(intel_dig_port, shim);
> > if (ret) {
> > - DRM_DEBUG_KMS("KSV list failed to become ready (%d)\n", ret);
> > + drm_dbg_kms(&dev_priv->drm,
> > + "KSV list failed to become ready (%d)\n", ret);
> > return ret;
> > }
> >
> > @@ -547,7 +549,7 @@ int intel_hdcp_auth_downstream(struct intel_connector *connector)
> >
> > if (DRM_HDCP_MAX_DEVICE_EXCEEDED(bstatus[0]) ||
> > DRM_HDCP_MAX_CASCADE_EXCEEDED(bstatus[1])) {
> > - DRM_DEBUG_KMS("Max Topology Limit Exceeded\n");
> > + drm_dbg_kms(&dev_priv->drm, "Max Topology Limit Exceeded\n");
> > return -EPERM;
> > }
> >
> > @@ -560,13 +562,14 @@ int intel_hdcp_auth_downstream(struct intel_connector *connector)
> > */
> > num_downstream = DRM_HDCP_NUM_DOWNSTREAM(bstatus[0]);
> > if (num_downstream == 0) {
> > - DRM_DEBUG_KMS("Repeater with zero downstream devices\n");
> > + drm_dbg_kms(&dev_priv->drm,
> > + "Repeater with zero downstream devices\n");
> > return -EINVAL;
> > }
> >
> > ksv_fifo = kcalloc(DRM_HDCP_KSV_LEN, num_downstream, GFP_KERNEL);
> > if (!ksv_fifo) {
> > - DRM_DEBUG_KMS("Out of mem: ksv_fifo\n");
> > + drm_dbg_kms(&dev_priv->drm, "Out of mem: ksv_fifo\n");
> > return -ENOMEM;
> > }
> >
> > @@ -576,7 +579,7 @@ int intel_hdcp_auth_downstream(struct intel_connector *connector)
> >
> > if (drm_hdcp_check_ksvs_revoked(&dev_priv->drm, ksv_fifo,
> > num_downstream)) {
> > - DRM_ERROR("Revoked Ksv(s) in ksv_fifo\n");
> > + drm_err(&dev_priv->drm, "Revoked Ksv(s) in ksv_fifo\n");
> > ret = -EPERM;
> > goto err;
> > }
> > @@ -594,12 +597,13 @@ int intel_hdcp_auth_downstream(struct intel_connector *connector)
> > }
> >
> > if (i == tries) {
> > - DRM_DEBUG_KMS("V Prime validation failed.(%d)\n", ret);
> > + drm_dbg_kms(&dev_priv->drm,
> > + "V Prime validation failed.(%d)\n", ret);
> > goto err;
> > }
> >
> > - DRM_DEBUG_KMS("HDCP is enabled (%d downstream devices)\n",
> > - num_downstream);
> > + drm_dbg_kms(&dev_priv->drm, "HDCP is enabled (%d downstream devices)\n",
> > + num_downstream);
> > ret = 0;
> > err:
> > kfree(ksv_fifo);
> > @@ -642,7 +646,8 @@ static int intel_hdcp_auth(struct intel_connector *connector)
> > if (ret)
> > return ret;
> > if (!hdcp_capable) {
> > - DRM_DEBUG_KMS("Panel is not HDCP capable\n");
> > + drm_dbg_kms(&dev_priv->drm,
> > + "Panel is not HDCP capable\n");
> > return -EINVAL;
> > }
> > }
> > @@ -659,7 +664,7 @@ static int intel_hdcp_auth(struct intel_connector *connector)
> > if (intel_de_wait_for_set(dev_priv,
> > HDCP_STATUS(dev_priv, cpu_transcoder, port),
> > HDCP_STATUS_AN_READY, 1)) {
> > - DRM_ERROR("Timed out waiting for An\n");
> > + drm_err(&dev_priv->drm, "Timed out waiting for An\n");
> > return -ETIMEDOUT;
> > }
> >
> > @@ -680,7 +685,7 @@ static int intel_hdcp_auth(struct intel_connector *connector)
> > return ret;
> >
> > if (drm_hdcp_check_ksvs_revoked(&dev_priv->drm, bksv.shim, 1)) {
> > - DRM_ERROR("BKSV is revoked\n");
> > + drm_err(&dev_priv->drm, "BKSV is revoked\n");
> > return -EPERM;
> > }
> >
> > @@ -706,7 +711,7 @@ static int intel_hdcp_auth(struct intel_connector *connector)
> > /* Wait for R0 ready */
> > if (wait_for(intel_de_read(dev_priv, HDCP_STATUS(dev_priv, cpu_transcoder, port)) &
> > (HDCP_STATUS_R0_READY | HDCP_STATUS_ENC), 1)) {
> > - DRM_ERROR("Timed out waiting for R0 ready\n");
> > + drm_err(&dev_priv->drm, "Timed out waiting for R0 ready\n");
> > return -ETIMEDOUT;
> > }
> >
> > @@ -743,8 +748,10 @@ static int intel_hdcp_auth(struct intel_connector *connector)
> > }
> >
> > if (i == tries) {
> > - DRM_DEBUG_KMS("Timed out waiting for Ri prime match (%x)\n",
> > - intel_de_read(dev_priv, HDCP_STATUS(dev_priv, cpu_transcoder, port)));
> > + drm_dbg_kms(&dev_priv->drm,
> > + "Timed out waiting for Ri prime match (%x)\n",
> > + intel_de_read(dev_priv, HDCP_STATUS(dev_priv,
> > + cpu_transcoder, port)));
> > return -ETIMEDOUT;
> > }
> >
> > @@ -753,7 +760,7 @@ static int intel_hdcp_auth(struct intel_connector *connector)
> > HDCP_STATUS(dev_priv, cpu_transcoder, port),
> > HDCP_STATUS_ENC,
> > ENCRYPT_STATUS_CHANGE_TIMEOUT_MS)) {
> > - DRM_ERROR("Timed out waiting for encryption\n");
> > + drm_err(&dev_priv->drm, "Timed out waiting for encryption\n");
> > return -ETIMEDOUT;
> > }
> >
> > @@ -765,7 +772,7 @@ static int intel_hdcp_auth(struct intel_connector *connector)
> > if (repeater_present)
> > return intel_hdcp_auth_downstream(connector);
> >
> > - DRM_DEBUG_KMS("HDCP is enabled (no repeater present)\n");
> > + drm_dbg_kms(&dev_priv->drm, "HDCP is enabled (no repeater present)\n");
> > return 0;
> > }
> >
> > @@ -1270,7 +1277,7 @@ static int hdcp2_authentication_key_exchange(struct intel_connector *connector)
> > return ret;
> >
> > if (msgs.send_cert.rx_caps[0] != HDCP_2_2_RX_CAPS_VERSION_VAL) {
> > - DRM_DEBUG_KMS("cert.rx_caps dont claim HDCP2.2\n");
> > + drm_dbg_kms(&dev_priv->drm, "cert.rx_caps dont claim HDCP2.2\n");
> > return -EINVAL;
> > }
> >
> > @@ -1279,7 +1286,7 @@ static int hdcp2_authentication_key_exchange(struct intel_connector *connector)
> > if (drm_hdcp_check_ksvs_revoked(&dev_priv->drm,
> > msgs.send_cert.cert_rx.receiver_id,
> > 1)) {
> > - DRM_ERROR("Receiver ID is revoked\n");
> > + drm_err(&dev_priv->drm, "Receiver ID is revoked\n");
> > return -EPERM;
> > }
> >
> > @@ -1446,7 +1453,7 @@ int hdcp2_authenticate_repeater_topology(struct intel_connector *connector)
> >
> > if (HDCP_2_2_MAX_CASCADE_EXCEEDED(rx_info[1]) ||
> > HDCP_2_2_MAX_DEVS_EXCEEDED(rx_info[1])) {
> > - DRM_DEBUG_KMS("Topology Max Size Exceeded\n");
> > + drm_dbg_kms(&dev_priv->drm, "Topology Max Size Exceeded\n");
> > return -EINVAL;
> > }
> >
> > @@ -1456,7 +1463,7 @@ int hdcp2_authenticate_repeater_topology(struct intel_connector *connector)
> >
> > if (seq_num_v < hdcp->seq_num_v) {
> > /* Roll over of the seq_num_v from repeater. Reauthenticate. */
> > - DRM_DEBUG_KMS("Seq_num_v roll over.\n");
> > + drm_dbg_kms(&dev_priv->drm, "Seq_num_v roll over.\n");
> > return -EINVAL;
> > }
> >
> > @@ -1465,7 +1472,7 @@ int hdcp2_authenticate_repeater_topology(struct intel_connector *connector)
> > if (drm_hdcp_check_ksvs_revoked(&dev_priv->drm,
> > msgs.recvid_list.receiver_ids,
> > device_cnt)) {
> > - DRM_ERROR("Revoked receiver ID(s) is in list\n");
> > + drm_err(&dev_priv->drm, "Revoked receiver ID(s) is in list\n");
> > return -EPERM;
> > }
> >
> > @@ -1487,25 +1494,27 @@ int hdcp2_authenticate_repeater_topology(struct intel_connector *connector)
> > static int hdcp2_authenticate_sink(struct intel_connector *connector)
> > {
> > struct intel_digital_port *intel_dig_port = intel_attached_dig_port(connector);
> > + struct drm_i915_private *dev_priv = to_i915(connector->base.dev);
> > struct intel_hdcp *hdcp = &connector->hdcp;
> > const struct intel_hdcp_shim *shim = hdcp->shim;
> > int ret;
> >
> > ret = hdcp2_authentication_key_exchange(connector);
> > if (ret < 0) {
> > - DRM_DEBUG_KMS("AKE Failed. Err : %d\n", ret);
> > + drm_dbg_kms(&dev_priv->drm, "AKE Failed. Err : %d\n", ret);
> > return ret;
> > }
> >
> > ret = hdcp2_locality_check(connector);
> > if (ret < 0) {
> > - DRM_DEBUG_KMS("Locality Check failed. Err : %d\n", ret);
> > + drm_dbg_kms(&dev_priv->drm,
> > + "Locality Check failed. Err : %d\n", ret);
> > return ret;
> > }
> >
> > ret = hdcp2_session_key_exchange(connector);
> > if (ret < 0) {
> > - DRM_DEBUG_KMS("SKE Failed. Err : %d\n", ret);
> > + drm_dbg_kms(&dev_priv->drm, "SKE Failed. Err : %d\n", ret);
> > return ret;
> > }
> >
> > @@ -1520,7 +1529,8 @@ static int hdcp2_authenticate_sink(struct intel_connector *connector)
> > if (hdcp->is_repeater) {
> > ret = hdcp2_authenticate_repeater_topology(connector);
> > if (ret < 0) {
> > - DRM_DEBUG_KMS("Repeater Auth Failed. Err: %d\n", ret);
> > + drm_dbg_kms(&dev_priv->drm,
> > + "Repeater Auth Failed. Err: %d\n", ret);
> > return ret;
> > }
> > }
> > @@ -1651,10 +1661,10 @@ static int hdcp2_authenticate_and_encrypt(struct intel_connector *connector)
> > }
> >
> > /* Clearing the mei hdcp session */
> > - DRM_DEBUG_KMS("HDCP2.2 Auth %d of %d Failed.(%d)\n",
> > - i + 1, tries, ret);
> > + drm_dbg_kms(drm, "HDCP2.2 Auth %d of %d Failed.(%d)\n",
> > + i + 1, tries, ret);
> > if (hdcp2_deauthenticate_port(connector) < 0)
> > - DRM_DEBUG_KMS("Port deauth failed.\n");
> > + drm_dbg_kms(drm, "Port deauth failed.\n");
> > }
> >
> > if (!ret) {
> > @@ -1665,9 +1675,9 @@ static int hdcp2_authenticate_and_encrypt(struct intel_connector *connector)
> > msleep(HDCP_2_2_DELAY_BEFORE_ENCRYPTION_EN);
> > ret = hdcp2_enable_encryption(connector);
> > if (ret < 0) {
> > - DRM_DEBUG_KMS("Encryption Enable Failed.(%d)\n", ret);
> > + drm_dbg_kms(drm, "Encryption Enable Failed.(%d)\n", ret);
> > if (hdcp2_deauthenticate_port(connector) < 0)
> > - DRM_DEBUG_KMS("Port deauth failed.\n");
> > + drm_dbg_kms(drm, "Port deauth failed.\n");
> > }
> > }
> >
> > @@ -1677,22 +1687,23 @@ static int hdcp2_authenticate_and_encrypt(struct intel_connector *connector)
> > static int _intel_hdcp2_enable(struct intel_connector *connector)
> > {
> > struct intel_hdcp *hdcp = &connector->hdcp;
> > + struct drm_device *drm = connector->base.dev;
> > int ret;
> >
> > - DRM_DEBUG_KMS("[%s:%d] HDCP2.2 is being enabled. Type: %d\n",
> > - connector->base.name, connector->base.base.id,
> > - hdcp->content_type);
> > + drm_dbg_kms(drm, "[%s:%d] HDCP2.2 is being enabled. Type: %d\n",
> > + connector->base.name, connector->base.base.id,
> > + hdcp->content_type);
> >
> > ret = hdcp2_authenticate_and_encrypt(connector);
> > if (ret) {
> > - DRM_DEBUG_KMS("HDCP2 Type%d Enabling Failed. (%d)\n",
> > - hdcp->content_type, ret);
> > + drm_dbg_kms(drm, "HDCP2 Type%d Enabling Failed. (%d)\n",
> > + hdcp->content_type, ret);
> > return ret;
> > }
> >
> > - DRM_DEBUG_KMS("[%s:%d] HDCP2.2 is enabled. Type %d\n",
> > - connector->base.name, connector->base.base.id,
> > - hdcp->content_type);
> > + drm_dbg_kms(drm, "[%s:%d] HDCP2.2 is enabled. Type %d\n",
> > + connector->base.name, connector->base.base.id,
> > + hdcp->content_type);
> >
> > hdcp->hdcp2_encrypted = true;
> > return 0;
> > @@ -1700,15 +1711,16 @@ static int _intel_hdcp2_enable(struct intel_connector *connector)
> >
> > static int _intel_hdcp2_disable(struct intel_connector *connector)
> > {
> > + struct drm_device *drm = connector->base.dev;
> > int ret;
> >
> > - DRM_DEBUG_KMS("[%s:%d] HDCP2.2 is being Disabled\n",
> > - connector->base.name, connector->base.base.id);
> > + drm_dbg_kms(drm, "[%s:%d] HDCP2.2 is being Disabled\n",
> > + connector->base.name, connector->base.base.id);
> >
> > ret = hdcp2_disable_encryption(connector);
> >
> > if (hdcp2_deauthenticate_port(connector) < 0)
> > - DRM_DEBUG_KMS("Port deauth failed.\n");
> > + drm_dbg_kms(drm, "Port deauth failed.\n");
> >
> > connector->hdcp.hdcp2_encrypted = false;
> >
> > @@ -1951,11 +1963,12 @@ static void intel_hdcp2_init(struct intel_connector *connector,
> > const struct intel_hdcp_shim *shim)
> > {
> > struct intel_hdcp *hdcp = &connector->hdcp;
> > + struct drm_device *drm = connector->base.dev;
> > int ret;
> >
> > ret = initialize_hdcp_port_data(connector, shim);
> > if (ret) {
> > - DRM_DEBUG_KMS("Mei hdcp data init failed\n");
> > + drm_dbg_kms(drm, "Mei hdcp data init failed\n");
> > return;
> > }
>
> --
> Jani Nikula, Intel Open Source Graphics Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
next prev parent reply other threads:[~2020-02-12 10:25 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-02-12 9:35 [PATCH 0/5] HDCP misc Ramalingam C
2020-02-12 9:35 ` [Intel-gfx] " Ramalingam C
2020-02-12 9:35 ` [PATCH 1/5] drm/hdcp: optimizing the srm handling Ramalingam C
2020-02-12 9:35 ` [Intel-gfx] " Ramalingam C
2020-02-12 9:35 ` [PATCH 2/5] drm/hdcp: fix DRM_HDCP_2_KSV_COUNT_2_LSBITS Ramalingam C
2020-02-12 9:35 ` [Intel-gfx] " Ramalingam C
2020-02-12 9:35 ` [PATCH 3/5] drm/i915: terminate reauth at stream management failure Ramalingam C
2020-02-12 9:35 ` [Intel-gfx] " Ramalingam C
2020-02-12 9:35 ` [PATCH 4/5] drm/i915: dont retry stream management at seq_num_m roll over Ramalingam C
2020-02-12 9:35 ` [Intel-gfx] " Ramalingam C
2020-02-12 9:35 ` [PATCH 5/5] drm/i915/hdcp: conversion to struct drm_device based logging macros Ramalingam C
2020-02-12 9:35 ` [Intel-gfx] " Ramalingam C
2020-02-12 10:01 ` Jani Nikula
2020-02-12 10:01 ` [Intel-gfx] " Jani Nikula
2020-02-12 10:25 ` Ramalingam C [this message]
2020-02-12 10:25 ` Ramalingam C
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200212102514.GB2939@intel.com \
--to=ramalingam.c@intel.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=intel-gfx@lists.freedesktop.org \
--cc=jani.nikula@linux.intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.