From patchwork Thu Feb 25 21:15:06 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Rob Clark X-Patchwork-Id: 8426731 Return-Path: X-Original-To: patchwork-dri-devel@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork2.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.136]) by patchwork2.web.kernel.org (Postfix) with ESMTP id B2B9AC0553 for ; Thu, 25 Feb 2016 21:15:21 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 6397620394 for ; Thu, 25 Feb 2016 21:15:17 +0000 (UTC) Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) by mail.kernel.org (Postfix) with ESMTP id 7ABF62038F for ; Thu, 25 Feb 2016 21:15:16 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id E778A6E6B6; Thu, 25 Feb 2016 21:15:15 +0000 (UTC) X-Original-To: dri-devel@lists.freedesktop.org Delivered-To: dri-devel@lists.freedesktop.org Received: from mail-qg0-x235.google.com (mail-qg0-x235.google.com [IPv6:2607:f8b0:400d:c04::235]) by gabe.freedesktop.org (Postfix) with ESMTPS id 53D886E6B5 for ; Thu, 25 Feb 2016 21:15:12 +0000 (UTC) Received: by mail-qg0-x235.google.com with SMTP id b67so50655134qgb.1 for ; Thu, 25 Feb 2016 13:15:12 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=/zOatekSaKDs5B6l7pJNGVFgWvzPoqsimTpcg/pQSYg=; b=IGq/fwS6ziFXZ1SE4hkKsgommRiJ0sbS0q3Ltiu2+9ih64oPoV5QRs/p3dT8hObmFx Jitrva3n5ngqvVVi0iYo20a5ly8/HxB6PoAfGhd8YCt6D30+GSOrO1TbHyN/UeycJZol iox7aM7E89YbGledo6Z3b4xmAwRcst9EgO32k1s2vQ8ki8hPmRf7yIYAmELO0hDrUWlP WJ7r9PDKRUkbIUXQ6B+Ntc9ts+wR7ceTGIz2Hxr7dmBWknY+iczt5jjuAEabUQjmerPA IAA7H6SQvrj9U+LOL4mJinF/U+Oe+HwLsAVd1Rb0lJnDvN6gMmVCooDROghrLtKuXzSx dnCQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=/zOatekSaKDs5B6l7pJNGVFgWvzPoqsimTpcg/pQSYg=; b=bP94JsNGj3vvHuYL020G5t+z7/HBwFb3EYzJfQ0QEMKhOTnfETiLu2FKseyA7+9DAB X9zaXLxUO4kRhhHbqrpeWqscCMaIas2eolQb2+THC0aWQZ4IU9CPTAcNarlPZGUztAxk KKRcVD9CwcoTqwuW+X/01xTFjX8EjHcR578gQTRYhMnuAajs6hrz9zFLpy+Ua80/N0xC u9ml+/P51sBiRqAbDtelpLaWsoxcRVCoiDPDYSzTTS7vOqvrIBPR99fdF+lMVf3ikioI xd/E1ZZ/whjT1MX1XsriBPZgn1qxgA2u4XpIFzy4u4a4O+Zk/126fbI1V95SUvay5J+f zUQQ== X-Gm-Message-State: AG10YOT121ei6F3v1NJJ0rQvRNFBaiMB4Ry71wTp/Lf8ybjrfHYM7TUn/6uXqNYWk6jj3g== X-Received: by 10.140.38.104 with SMTP id s95mr20942120qgs.7.1456434911622; Thu, 25 Feb 2016 13:15:11 -0800 (PST) Received: from localhost (nat-pool-bos-t.redhat.com. [66.187.233.206]) by smtp.gmail.com with ESMTPSA id d71sm4011995qhd.18.2016.02.25.13.15.11 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 25 Feb 2016 13:15:11 -0800 (PST) From: Rob Clark To: dri-devel@lists.freedesktop.org Subject: [PATCH 2/2] drm/dp: add DPCD/AUX logging Date: Thu, 25 Feb 2016 16:15:06 -0500 Message-Id: <1456434906-29600-2-git-send-email-robdclark@gmail.com> X-Mailer: git-send-email 2.5.0 In-Reply-To: <1456434906-29600-1-git-send-email-robdclark@gmail.com> References: <1456434906-29600-1-git-send-email-robdclark@gmail.com> X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" X-Spam-Status: No, score=-1.8 required=5.0 tests=BAYES_00, DKIM_ADSP_CUSTOM_MED, DKIM_SIGNED, FREEMAIL_FROM, UNPARSEABLE_RELAY autolearn=no version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP Add a new drm_debug bit for turning on DPCD logging, to aid debugging with troublesome monitors. v2: don't try to hexdump the universe if driver returns -errno, and change the "too many retries" traces to DRM_ERROR() v3: rename to more generic "AUX" instead of DP specific DPCD, add DP_AUX_I2C_WRITE_STATUS_UPDATE Signed-off-by: Rob Clark Acked-by: Imre Deak --- drivers/gpu/drm/drm_dp_helper.c | 62 +++++++++++++++++++++++++++++++++-------- include/drm/drmP.h | 8 +++++- 2 files changed, 58 insertions(+), 12 deletions(-) diff --git a/drivers/gpu/drm/drm_dp_helper.c b/drivers/gpu/drm/drm_dp_helper.c index df64ed1..3ef35fd 100644 --- a/drivers/gpu/drm/drm_dp_helper.c +++ b/drivers/gpu/drm/drm_dp_helper.c @@ -160,6 +160,45 @@ int drm_dp_bw_code_to_link_rate(u8 link_bw) } EXPORT_SYMBOL(drm_dp_bw_code_to_link_rate); +static ssize_t aux_transfer(struct drm_dp_aux *aux, struct drm_dp_aux_msg *msg) +{ + ssize_t ret; + + DRM_DEBUG_AUX("%s: req=0x%02x, address=0x%05x, size=%zu\n", aux->name, + msg->request, msg->address, msg->size); + + if (unlikely(drm_debug & DRM_UT_AUX)) { + switch (msg->request & ~DP_AUX_I2C_MOT) { + case DP_AUX_NATIVE_WRITE: + case DP_AUX_I2C_WRITE: + case DP_AUX_I2C_WRITE_STATUS_UPDATE: + print_hex_dump(KERN_DEBUG, "DPCD: ", DUMP_PREFIX_OFFSET, + 16, 1, msg->buffer, msg->size, false); + break; + default: + break; + } + } + + ret = aux->transfer(aux, msg); + + DRM_DEBUG_AUX("%s: reply=0x%02x, size=%zd\n", aux->name, msg->reply, ret); + + if (unlikely(drm_debug & DRM_UT_AUX) && (ret > 0)) { + switch (msg->request & ~DP_AUX_I2C_MOT) { + case DP_AUX_NATIVE_READ: + case DP_AUX_I2C_READ: + print_hex_dump(KERN_DEBUG, "DPCD: ", DUMP_PREFIX_OFFSET, + 16, 1, msg->buffer, ret, false); + break; + default: + break; + } + } + + return ret; +} + #define AUX_RETRY_INTERVAL 500 /* us */ /** @@ -197,7 +236,7 @@ static int drm_dp_dpcd_access(struct drm_dp_aux *aux, u8 request, */ for (retry = 0; retry < 32; retry++) { - err = aux->transfer(aux, &msg); + err = aux_transfer(aux, &msg); if (err < 0) { if (err == -EBUSY) continue; @@ -205,7 +244,6 @@ static int drm_dp_dpcd_access(struct drm_dp_aux *aux, u8 request, goto unlock; } - switch (msg.reply & DP_AUX_NATIVE_REPLY_MASK) { case DP_AUX_NATIVE_REPLY_ACK: if (err < size) @@ -213,16 +251,18 @@ static int drm_dp_dpcd_access(struct drm_dp_aux *aux, u8 request, goto unlock; case DP_AUX_NATIVE_REPLY_NACK: + DRM_DEBUG_AUX("native nack (result=%d, size=%zu)\n", err, msg.size); err = -EIO; goto unlock; case DP_AUX_NATIVE_REPLY_DEFER: + DRM_DEBUG_AUX("native defer\n"); usleep_range(AUX_RETRY_INTERVAL, AUX_RETRY_INTERVAL + 100); break; } } - DRM_DEBUG_KMS("too many retries, giving up\n"); + DRM_ERROR("DPCD: too many retries, giving up!\n"); err = -EIO; unlock: @@ -549,12 +589,12 @@ static int drm_dp_i2c_do_msg(struct drm_dp_aux *aux, struct drm_dp_aux_msg *msg) int max_retries = max(7, drm_dp_i2c_retry_count(msg, dp_aux_i2c_speed_khz)); for (retry = 0, defer_i2c = 0; retry < (max_retries + defer_i2c); retry++) { - ret = aux->transfer(aux, msg); + ret = aux_transfer(aux, msg); if (ret < 0) { if (ret == -EBUSY) continue; - DRM_DEBUG_KMS("transaction failed: %d\n", ret); + DRM_DEBUG_AUX("transaction failed: %d\n", ret); return ret; } @@ -568,11 +608,11 @@ static int drm_dp_i2c_do_msg(struct drm_dp_aux *aux, struct drm_dp_aux_msg *msg) break; case DP_AUX_NATIVE_REPLY_NACK: - DRM_DEBUG_KMS("native nack (result=%d, size=%zu)\n", ret, msg->size); + DRM_DEBUG_AUX("native nack (result=%d, size=%zu)\n", ret, msg->size); return -EREMOTEIO; case DP_AUX_NATIVE_REPLY_DEFER: - DRM_DEBUG_KMS("native defer\n"); + DRM_DEBUG_AUX("native defer\n"); /* * We could check for I2C bit rate capabilities and if * available adjust this interval. We could also be @@ -601,12 +641,12 @@ static int drm_dp_i2c_do_msg(struct drm_dp_aux *aux, struct drm_dp_aux_msg *msg) return ret; case DP_AUX_I2C_REPLY_NACK: - DRM_DEBUG_KMS("I2C nack (result=%d, size=%zu\n", ret, msg->size); + DRM_DEBUG_AUX("I2C nack (result=%d, size=%zu)\n", ret, msg->size); aux->i2c_nack_count++; return -EREMOTEIO; case DP_AUX_I2C_REPLY_DEFER: - DRM_DEBUG_KMS("I2C defer\n"); + DRM_DEBUG_AUX("I2C defer\n"); /* DP Compliance Test 4.2.2.5 Requirement: * Must have at least 7 retries for I2C defers on the * transaction to pass this test @@ -625,7 +665,7 @@ static int drm_dp_i2c_do_msg(struct drm_dp_aux *aux, struct drm_dp_aux_msg *msg) } } - DRM_DEBUG_KMS("too many retries, giving up\n"); + DRM_ERROR("I2C: too many retries, giving up\n"); return -EREMOTEIO; } @@ -653,7 +693,7 @@ static int drm_dp_i2c_drain_msg(struct drm_dp_aux *aux, struct drm_dp_aux_msg *o return err == 0 ? -EPROTO : err; if (err < msg.size && err < ret) { - DRM_DEBUG_KMS("Partial I2C reply: requested %zu bytes got %d bytes\n", + DRM_DEBUG_AUX("Partial I2C reply: requested %zu bytes got %d bytes\n", msg.size, err); ret = err; } diff --git a/include/drm/drmP.h b/include/drm/drmP.h index 3c8422c..cc524b5 100644 --- a/include/drm/drmP.h +++ b/include/drm/drmP.h @@ -117,7 +117,7 @@ struct dma_buf_attachment; * drm.debug=0x2 will enable DRIVER messages * drm.debug=0x3 will enable CORE and DRIVER messages * ... - * drm.debug=0x3f will enable all messages + * drm.debug=0x7f will enable all messages * * An interesting feature is that it's possible to enable verbose logging at * run-time by echoing the debug value in its sysfs node: @@ -129,6 +129,7 @@ struct dma_buf_attachment; #define DRM_UT_PRIME 0x08 #define DRM_UT_ATOMIC 0x10 #define DRM_UT_VBL 0x20 +#define DRM_UT_AUX 0x40 extern __printf(2, 3) void drm_ut_debug_printk(const char *function_name, @@ -226,6 +227,11 @@ void drm_err(const char *format, ...); if (unlikely(drm_debug & DRM_UT_VBL)) \ drm_ut_debug_printk(__func__, fmt, ##args); \ } while (0) +#define DRM_DEBUG_AUX(fmt, args...) \ + do { \ + if (unlikely(drm_debug & DRM_UT_AUX)) \ + drm_ut_debug_printk(__func__, fmt, ##args); \ + } while (0) /*@}*/