From patchwork Fri Nov 27 09:23:15 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Dmitry Baryshkov X-Patchwork-Id: 333606 Delivered-To: patch@linaro.org Received: by 2002:a17:906:4755:0:0:0:0 with SMTP id j21csp844214ejs; Fri, 27 Nov 2020 01:23:22 -0800 (PST) X-Google-Smtp-Source: ABdhPJzygVlSUfJGLdS7jxXNqDtMMGQdheVXR0JOYE6yNJKF1OPKEIsmlLzjbCQQC2Jgvx3Ibtf8 X-Received: by 2002:a17:906:851:: with SMTP id f17mr5632301ejd.392.1606469002283; Fri, 27 Nov 2020 01:23:22 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1606469002; cv=none; d=google.com; s=arc-20160816; b=MbWBVQtek2iiqMvqfLffjdrx9VUUEmvkePl45kL0irvflzeRbu5PcGxaarBoxHbDFE 1JKv8PsYi7nsHkZR7r0OY6hTsOS7U8OJ4M5lT9+fEPuV70r83xJbULBRn9GoUxLzjo1F QyEI12hjGDAaKwgrafVTrQm9ftnK6qFfbdchei5WgmzCgLxRuX5XjkJ2hL62dIr4k6vw 4hBLIHLYmSv7QmMY88o8ou3hW07PN0H0TGRNpSY1gzDOQQU2iTeOPYqb+GYjnOyVMcfP B3otykGnAk0tQVgc+nketZZ+tYLzWy/Ks+u6R74/2E3FsklnY7qW9wLLGQsVUo/cWD95 THSQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from:dkim-signature; bh=dDVGerivDhcovGvlrg4RdGN4PhZ+oBrQSqPNlOoN/KE=; b=PeAUc7Na7JU36QXe8xelbAxVLC72utBruN9Gx0yC3joc+Jpqv40P8UyZkGkuQ0wJcf 7bR4nUSmcILEizQ05pgJ+y8QYLR3u6B9BdRxm+AhkB1hg8MB/TWPwrdumEE/NZjZpzwM EfLUiRC5H5WgxG5guLBZy4QiL2/mUwA0RTuQ+zAqsbgfon4J1x70Uqy9zjjbRBTxYdzu SQe/5LC72lN9CUtymm9kmIFPdLA67BYZVenSfwnZsMweLkIo3Bvos6O/0tYbDKQhTWwm Y+ujq5ur6vkB9nuOv5JUK7R9Be1jeznnuLAmZY23fmk/366ydIiDFKaQY+d1pjVAqtwX KXXA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=xXo1mpme; spf=pass (google.com: domain of linux-arm-msm-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-arm-msm-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id k11si4511558ejb.369.2020.11.27.01.23.22; Fri, 27 Nov 2020 01:23:22 -0800 (PST) Received-SPF: pass (google.com: domain of linux-arm-msm-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=xXo1mpme; spf=pass (google.com: domain of linux-arm-msm-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-arm-msm-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726178AbgK0JXV (ORCPT + 15 others); Fri, 27 Nov 2020 04:23:21 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:42396 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726014AbgK0JXU (ORCPT ); Fri, 27 Nov 2020 04:23:20 -0500 Received: from mail-lj1-x241.google.com (mail-lj1-x241.google.com [IPv6:2a00:1450:4864:20::241]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 59189C0613D1 for ; Fri, 27 Nov 2020 01:23:20 -0800 (PST) Received: by mail-lj1-x241.google.com with SMTP id f18so5172301ljg.9 for ; Fri, 27 Nov 2020 01:23:20 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=dDVGerivDhcovGvlrg4RdGN4PhZ+oBrQSqPNlOoN/KE=; b=xXo1mpme3FQkxz7YOniDXUXLh1hheRztkvuFD0BgV002eFkuLs9/dcgDiWWCkpLt1h e1g5XSCd/9I2I221sERnEP5Fl3x2MhneduRxljWEjh4cMBhD/La7eSfE+qUnugOnZmvQ CTYXTVxuM/C5YrnndrLDTMSY/njX9IEXElhdv3mUHj9yJCDG845sHyPDxgeFGjR2Kt6B fYw7wfn1t5zKg1WL1I18oLC5Aq1lLeCDWszonc6zgKx8SDoPTC+FPtYEisuLI5e4jASN 3pFbXg6iE0k75/7eStyKP54QMo2n+JxTlPeaRbJPAhsys/nCn/ARQB+lI/K9/Yx9xU8m 29GQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=dDVGerivDhcovGvlrg4RdGN4PhZ+oBrQSqPNlOoN/KE=; b=o5xOyxLQ+U94YU+3JCC8wFyMIlqtD8TNjqSCoXccB8y6yz7CUfb7UJ6dRabX8yfa1G w0XCfQAv0SUjaLorQGIUMOyCG2bkLzynC+QTHLfACn/TqghwRJmQYDW+9wL+yxd2Konw K7VFx4QskkmmvZvb9ked5Kq0X7a+jXi9zAhbCIb5jXuIbHDqMZlImG0hotOqczYG6I04 DxOGtLShEHH2rdvz6BU+tyn2crG7jnmiatU6no5+uhwzQMH0nKybXJDusi1TdBsoJI1G c7nd6z914bnIrZJ/gKH72xOHTnAs/ovs3GvhdD3M/r8NU7gsRlIjzJyH9QjmtoW+inb1 8NLg== X-Gm-Message-State: AOAM533BaNUmlGRzaNa+N3enHXqk5vJ8Z9oHsZ+ZMGJnkP/kfoL56MUF EhNKLKCTz7LPX8N8jRGkpd1Z3A== X-Received: by 2002:a2e:998e:: with SMTP id w14mr3176710lji.100.1606468998738; Fri, 27 Nov 2020 01:23:18 -0800 (PST) Received: from eriador.lumag.spb.ru ([188.162.64.79]) by smtp.gmail.com with ESMTPSA id a6sm615442lfi.107.2020.11.27.01.23.16 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 27 Nov 2020 01:23:17 -0800 (PST) From: Dmitry Baryshkov To: dri-devel@lists.freedesktop.org, linux-arm-msm@vger.kernel.org Cc: Daniel Vetter , David Airlie , Andrzej Hajda , Neil Armstrong , Laurent Pinchart , Jonas Karlman , Jernej Skrabec , Vinod Koul , Sam Ravnborg , Manivannan Sadhasivam Subject: [PATCH 1/2] drm/bridge/lontium-lt9611uxc: fix waiting for EDID to become available Date: Fri, 27 Nov 2020 12:23:15 +0300 Message-Id: <20201127092316.122246-1-dmitry.baryshkov@linaro.org> X-Mailer: git-send-email 2.29.2 MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: linux-arm-msm@vger.kernel.org - Call wake_up() when EDID ready event is received to wake wait_event_interruptible_timeout() - Increase waiting timeout, reading EDID can take longer than 100ms, so let's be on a safe side. - Return NULL pointer from get_edid() callback rather than ERR_PTR() pointer, as DRM code does NULL checks rather than IS_ERR(). Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/bridge/lontium-lt9611uxc.c | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) -- 2.29.2 diff --git a/drivers/gpu/drm/bridge/lontium-lt9611uxc.c b/drivers/gpu/drm/bridge/lontium-lt9611uxc.c index 0c98d27f84ac..b708700e182d 100644 --- a/drivers/gpu/drm/bridge/lontium-lt9611uxc.c +++ b/drivers/gpu/drm/bridge/lontium-lt9611uxc.c @@ -145,8 +145,10 @@ static irqreturn_t lt9611uxc_irq_thread_handler(int irq, void *dev_id) lt9611uxc_unlock(lt9611uxc); - if (irq_status & BIT(0)) + if (irq_status & BIT(0)) { lt9611uxc->edid_read = !!(hpd_status & BIT(0)); + wake_up_all(<9611uxc->wq); + } if (irq_status & BIT(1)) { if (lt9611uxc->connector.dev) @@ -465,7 +467,7 @@ static enum drm_connector_status lt9611uxc_bridge_detect(struct drm_bridge *brid static int lt9611uxc_wait_for_edid(struct lt9611uxc *lt9611uxc) { return wait_event_interruptible_timeout(lt9611uxc->wq, lt9611uxc->edid_read, - msecs_to_jiffies(100)); + msecs_to_jiffies(500)); } static int lt9611uxc_get_edid_block(void *data, u8 *buf, unsigned int block, size_t len) @@ -503,7 +505,10 @@ static struct edid *lt9611uxc_bridge_get_edid(struct drm_bridge *bridge, ret = lt9611uxc_wait_for_edid(lt9611uxc); if (ret < 0) { dev_err(lt9611uxc->dev, "wait for EDID failed: %d\n", ret); - return ERR_PTR(ret); + return NULL; + } else if (ret == 0) { + dev_err(lt9611uxc->dev, "wait for EDID timeout\n"); + return NULL; } return drm_do_get_edid(connector, lt9611uxc_get_edid_block, lt9611uxc); From patchwork Fri Nov 27 09:23:16 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Dmitry Baryshkov X-Patchwork-Id: 333607 Delivered-To: patch@linaro.org Received: by 2002:a17:906:4755:0:0:0:0 with SMTP id j21csp844226ejs; Fri, 27 Nov 2020 01:23:23 -0800 (PST) X-Google-Smtp-Source: ABdhPJzmbn+rlPV7QrkR8xKfUqGKlwDb7PLugvrp4qR1KoSZ89pjwaNrbfnZ1tLY5p/6blk21IiL X-Received: by 2002:a17:906:f88e:: with SMTP id lg14mr6706473ejb.417.1606469003767; Fri, 27 Nov 2020 01:23:23 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1606469003; cv=none; d=google.com; s=arc-20160816; b=ik3zImxythFF99LAHwTXl8Hsr0FBGCc3SHCAQKzb1z1W0KR5MnIJlDXYM35SLJQrVI vy6p7ZF2vbZvEuayykGXcSqCC4V2hvPDDdHDgdXvFJet6wfpCyhO9Rt4gBksCx6pQrzC 2Fpl2Iz4MNiVk0ol6NdbhycCX7Ep2ccx8OCMR5RqVFbLurj1v1vPlu9gbfMPYbdi1tJK Wb3wEpuW56SaIDMqAx1M0Ox6xPl46oZHq+TjxKdp7iiGEbOSF+9jFu+v/JrYnhkT47lJ ango7vEdTnZEPUXp3VRNjYQU9roUtD9Me+jKXSEGBBCua8dUplcKE+qDbbQL4WtgDBZy nJrA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=Jap4CSSphbNhkQ0riEHLjy5ZHv6p/5lP6LbqE0EEZnE=; b=L9oBQ4i2LIiGETLWB18AX6tnYxUvlVw2gYm/tNYJ2gNn0NGPTr4tSj2+gPc4BRe5mG GSLpD+vnUdJecCeGJhP0plsCz1WR3Mx2UosH/Pean8QleOBYt6GPRDx64IMpeRTnXRyN xWfnpzq2YLs/zOJoiYC1ND1UWkCdJdgQqI0b68y+HneGd9Lxkzh+ACWohNFBaIh8mV8/ phIRbN9xPNQI2uzmUyqU2v99d5kRQLQbsE8SBd40tvQt6rS5kWYWW5pq9al+CGbZt6Ao aW2s8c8e6xFN9MQgkoIf1n3WXVrheOaCoZ1VNIWjFRbqAgHS0+3JfY0v9+AAKwSdi1z2 Dvsw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=VgzMNMCY; spf=pass (google.com: domain of linux-arm-msm-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-arm-msm-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id k11si4511558ejb.369.2020.11.27.01.23.23; Fri, 27 Nov 2020 01:23:23 -0800 (PST) Received-SPF: pass (google.com: domain of linux-arm-msm-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=VgzMNMCY; spf=pass (google.com: domain of linux-arm-msm-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-arm-msm-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726190AbgK0JXX (ORCPT + 15 others); Fri, 27 Nov 2020 04:23:23 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:42402 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726014AbgK0JXW (ORCPT ); Fri, 27 Nov 2020 04:23:22 -0500 Received: from mail-lj1-x243.google.com (mail-lj1-x243.google.com [IPv6:2a00:1450:4864:20::243]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 0E1F4C0613D1 for ; Fri, 27 Nov 2020 01:23:22 -0800 (PST) Received: by mail-lj1-x243.google.com with SMTP id y7so5168381lji.8 for ; Fri, 27 Nov 2020 01:23:21 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=Jap4CSSphbNhkQ0riEHLjy5ZHv6p/5lP6LbqE0EEZnE=; b=VgzMNMCYncvkjKZ7PGOsRpkTT1+54DEF/4ixFBc7ESvn9jYTi7c8ZUHHu5FiGgEoWG Ju+IiUrvVm93SUL/SLg+4KbI0MtS0zzN/fgX1TykpiB/DDk9Z4jjhab7BtieElUEWoOI md2ZVWBEr+F1b5UG/spnhu0pZdvj9ViMXlDSqqSev0usIQMKFqYbHlh/iTdZz1WW47my +VhfNNQgYyldQneoWQkGnPVqfR2bP7CmBPnRzQsrDNdR3Zdx+GW5xAiBgHXIe+/lN3Mu jNKohrj7OWNtuYNt8JU/ptDz1itNFH1ojC24cgiRSkqUyzxILtH/tCfMbp4GpzdHoQbb aA1w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=Jap4CSSphbNhkQ0riEHLjy5ZHv6p/5lP6LbqE0EEZnE=; b=kwwDF/n51eEiSgj6IHXxCO2eI/e2BEj/6hBLwxXKfQDIpULqv4/zUYhGC5GPSwM9mg 6X5Ebs6sw28ltiAgEE7P4et5QrQOK8tHK8MpJSj8h+1Av7ODKiJIwJUfosaZ3npl7TFb NoEzqn1X427F1bB2mGMPT0rB3lcjD3mfXjeEFQHGoOeiTBi2wONZkY37phCPFCGWlBiw 1xxaILRY+nTBULQioKhKmW59XiEuufID6l2qEcSjRvr4FRh6ZKquADgVySr+4fj27EIh 9aSzBe0GTe5TXmUiNYhLO8J4BelCybI8zXD59xdfX5QNHaMJ+h/o+nVFuB+DsrhEJ+6b BiAQ== X-Gm-Message-State: AOAM532zGNzvo//Nx5TCskNVsk1sxlCXLioWa9VukTUfHKq09ugdKYLt y9yGhKIj2LV3n1vNw0lUuxrLew== X-Received: by 2002:a2e:7a18:: with SMTP id v24mr2999342ljc.224.1606469000477; Fri, 27 Nov 2020 01:23:20 -0800 (PST) Received: from eriador.lumag.spb.ru ([188.162.64.79]) by smtp.gmail.com with ESMTPSA id a6sm615442lfi.107.2020.11.27.01.23.18 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 27 Nov 2020 01:23:19 -0800 (PST) From: Dmitry Baryshkov To: dri-devel@lists.freedesktop.org, linux-arm-msm@vger.kernel.org Cc: Daniel Vetter , David Airlie , Andrzej Hajda , Neil Armstrong , Laurent Pinchart , Jonas Karlman , Jernej Skrabec , Vinod Koul , Sam Ravnborg , Manivannan Sadhasivam Subject: [PATCH 2/2] drm/bridge/lontium-lt9611uxc: move HPD notification out of IRQ handler Date: Fri, 27 Nov 2020 12:23:16 +0300 Message-Id: <20201127092316.122246-2-dmitry.baryshkov@linaro.org> X-Mailer: git-send-email 2.29.2 In-Reply-To: <20201127092316.122246-1-dmitry.baryshkov@linaro.org> References: <20201127092316.122246-1-dmitry.baryshkov@linaro.org> MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: linux-arm-msm@vger.kernel.org drm hotplug handling code (drm_client_dev_hotplug()) can wait on mutex, thus delaying further lt9611uxc IRQ events processing. It was observed occasionally during bootups, when drm_client_modeset_probe() was waiting for EDID ready event, which was delayed because IRQ handler was stuck trying to deliver hotplug event. Move hotplug notifications from IRQ handler to separate work to be able to process IRQ events without delays. Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/bridge/lontium-lt9611uxc.c | 30 +++++++++++++++++----- 1 file changed, 24 insertions(+), 6 deletions(-) -- 2.29.2 Reviewed-by: Bjorn Andersson diff --git a/drivers/gpu/drm/bridge/lontium-lt9611uxc.c b/drivers/gpu/drm/bridge/lontium-lt9611uxc.c index b708700e182d..88630bc2921f 100644 --- a/drivers/gpu/drm/bridge/lontium-lt9611uxc.c +++ b/drivers/gpu/drm/bridge/lontium-lt9611uxc.c @@ -14,6 +14,7 @@ #include #include #include +#include #include @@ -36,6 +37,7 @@ struct lt9611uxc { struct mutex ocm_lock; struct wait_queue_head wq; + struct work_struct work; struct device_node *dsi0_node; struct device_node *dsi1_node; @@ -52,6 +54,7 @@ struct lt9611uxc { bool hpd_supported; bool edid_read; + bool hdmi_connected; uint8_t fw_version; }; @@ -151,15 +154,26 @@ static irqreturn_t lt9611uxc_irq_thread_handler(int irq, void *dev_id) } if (irq_status & BIT(1)) { - if (lt9611uxc->connector.dev) - drm_kms_helper_hotplug_event(lt9611uxc->connector.dev); - else - drm_bridge_hpd_notify(<9611uxc->bridge, !!(hpd_status & BIT(1))); + lt9611uxc->hdmi_connected = !!(hpd_status & BIT(1)); + schedule_work(<9611uxc->work); } return IRQ_HANDLED; } +void lt9611uxc_hpd_work(struct work_struct *work) +{ + struct lt9611uxc *lt9611uxc = container_of(work, struct lt9611uxc, work); + + if (lt9611uxc->connector.dev) + drm_kms_helper_hotplug_event(lt9611uxc->connector.dev); + else + drm_bridge_hpd_notify(<9611uxc->bridge, + lt9611uxc->hdmi_connected ? + connector_status_connected : + connector_status_disconnected); +} + static void lt9611uxc_reset(struct lt9611uxc *lt9611uxc) { gpiod_set_value_cansleep(lt9611uxc->reset_gpio, 1); @@ -447,7 +461,7 @@ static enum drm_connector_status lt9611uxc_bridge_detect(struct drm_bridge *brid struct lt9611uxc *lt9611uxc = bridge_to_lt9611uxc(bridge); unsigned int reg_val = 0; int ret; - int connected = 1; + bool connected = true; if (lt9611uxc->hpd_supported) { lt9611uxc_lock(lt9611uxc); @@ -457,8 +471,9 @@ static enum drm_connector_status lt9611uxc_bridge_detect(struct drm_bridge *brid if (ret) dev_err(lt9611uxc->dev, "failed to read hpd status: %d\n", ret); else - connected = reg_val & BIT(1); + connected = !!(reg_val & BIT(1)); } + lt9611uxc->hdmi_connected = connected; return connected ? connector_status_connected : connector_status_disconnected; @@ -931,6 +946,8 @@ static int lt9611uxc_probe(struct i2c_client *client, lt9611uxc->fw_version = ret; init_waitqueue_head(<9611uxc->wq); + INIT_WORK(<9611uxc->work, lt9611uxc_hpd_work); + ret = devm_request_threaded_irq(dev, client->irq, NULL, lt9611uxc_irq_thread_handler, IRQF_ONESHOT, "lt9611uxc", lt9611uxc); @@ -967,6 +984,7 @@ static int lt9611uxc_remove(struct i2c_client *client) struct lt9611uxc *lt9611uxc = i2c_get_clientdata(client); disable_irq(client->irq); + flush_scheduled_work(); lt9611uxc_audio_exit(lt9611uxc); drm_bridge_remove(<9611uxc->bridge);