Received: by 2002:a05:6358:9144:b0:117:f937:c515 with SMTP id r4csp3759660rwr; Sun, 7 May 2023 19:37:58 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ5S0qNNw++l+PIdc6sHFZpeoJfs/cGAAMDsBmJruCVoxzdotCxL/ZOrqJEHffkajmfBO0bp X-Received: by 2002:a17:902:d510:b0:1ac:7ae7:3fdf with SMTP id b16-20020a170902d51000b001ac7ae73fdfmr1988873plg.41.1683513478213; Sun, 07 May 2023 19:37:58 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1683513478; cv=none; d=google.com; s=arc-20160816; b=v6jflCWC6pPg6GiIpb/Ux0N/g1aadLDUglc/0u605J5rmM5WwojxqJ1fB4fCxnLMA3 skw5YhLGNrtp7SztcImAypOuiLalnnlIKccDIgJfBHN2jjbruZcqQ1nEebUTHfpfDYC8 9uaiMQKTiHQRV1NKPDmSud4DkhLgpiFp92AlOdgXKwPFKYpM0clENg7nq86vW7rsDd1o Y+J2SkblGnvHDPOXWpSdkIYkUsLC/jdvL6Mox5Btqk3BXoIbcD/67k77xmzPxu29VQSv itFT4zuoquIb7Etaprul8j5gMT/0LXGjoAE0CBYU/Czda7Un5drBgjEXyiBRrbBIWPaP jROw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date:dkim-signature; bh=n6gP83Lrq5PGozgtGmrqja9E2jXJp9riw6MXJQR2MC0=; b=zWXzpHX17AFWghAlv8t1Ncg0bUfru6MiyObT99LtCQoiPmcuRRfcoCENyWh68F5BMK 5q1Aq6nYIUxIgQYTfGSWMKZaG3JCZOqQuvnNt1gdtxE9ykN6OoW8Lu5FnWqnWEjJ4raR S2LA5ISQhJqWLy2OyODaTtH5XrHzAmPZtAy8nqrXN00wSLbAB8kYTaNVyHIIEc6GHFfZ LEAQC8m6NpU+wSXFfP7uQl91h2ErvzR9ytFTUuM6pPtmLlZjMIuSBQOaenU9aDosdpRh 4I8bJwQIUnytP3ZkqR1nVXZFz4kyTnpO53I4AvhcKWxO9i5EZcOF/FcVuVUN9UBAn3jU w5rw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=SwpOfcoQ; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id i2-20020a1709026ac200b001a92672eb57si7043111plt.179.2023.05.07.19.37.42; Sun, 07 May 2023 19:37:58 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=SwpOfcoQ; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232095AbjEHCL7 (ORCPT + 99 others); Sun, 7 May 2023 22:11:59 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57626 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229744AbjEHCL6 (ORCPT ); Sun, 7 May 2023 22:11:58 -0400 Received: from dfw.source.kernel.org (dfw.source.kernel.org [IPv6:2604:1380:4641:c500::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 0D3441982; Sun, 7 May 2023 19:11:57 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 8B96861DF5; Mon, 8 May 2023 02:11:56 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 9F4EEC433EF; Mon, 8 May 2023 02:11:54 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1683511915; bh=W+BDWcTwbvNWQBCgt9uZZKh6zOCIi71k2crWd1L3W+k=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=SwpOfcoQzugMTe6nrRk10AeVJ9UQT/ItdgdISy7DTqoikrRz7Vrv19tbRp/olkgV8 UBkjm8j7t9GSZPAgHoCls3WCNgnbSQXRf3ZhOdMELvr3rXsYAfpcEYG5Br5ho9atNt Z55/nHxH0BlmftEffctJVa/X4ADi1yAzrawwLAglJRzFJ1EUHjOYOlcV4FxZiBgXjU 0q3LdRMHxvI7Yt17Ip2XdG0QclnuZjPEgGo93iPi4NuG6a3nBDbIl/QjhmTvpSv4fa RbizSWsueFK5ilHy2NvIztJrlGKzTk0eSEzrxhiJOKmAOBO7LUTWPxCnr8zjyMscRS 2i8vHZAd7BNVw== Date: Sun, 7 May 2023 19:15:36 -0700 From: Bjorn Andersson To: Leonard Lausen Cc: regressions@lists.linux.dev, Bjorn Andersson , Dmitry Baryshkov , Rob Clark , Abhinav Kumar , Stephen Boyd , Kuogee Hsieh , Johan Hovold , Sankeerth Billakanti , Sean Paul , David Airlie , Daniel Vetter , linux-arm-msm@vger.kernel.org, dri-devel@lists.freedesktop.org, freedreno@lists.freedesktop.org, linux-kernel@vger.kernel.org, Nikita Travkin Subject: Re: [PATCH] Revert "drm/msm/dp: Remove INIT_SETUP delay" Message-ID: <20230508021536.txtamifw2vkfncnx@ripper> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-Spam-Status: No, score=-4.4 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_MED, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, May 08, 2023 at 01:06:13AM +0000, Leonard Lausen wrote: > This reverts commit e17af1c9d861dc177e5b56009bd4f71ace688d97. > > Removing the delay of 100 units broke hot plug detection for USB-C displays on > qcom sc7180 lazor devices. Lazor uses mdss for hot plug detection and declares > dp_hot_plug_det in the dts. Other sc7180 based devices like aspire1 were not > affected by the regression, as they do not rely on mdss and dp_hot_plug_det for > hot plug detection. > > Signed-off-by: Leonard Lausen > Tested-by: Leonard Lausen # Trogdor (sc7180) > Suggested-by: Dmitry Baryshkov > > --- > drivers/gpu/drm/msm/dp/dp_display.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/msm/dp/dp_display.c b/drivers/gpu/drm/msm/dp/dp_display.c > index bde1a7ce442f..db9783ffd5cf 100644 > --- a/drivers/gpu/drm/msm/dp/dp_display.c > +++ b/drivers/gpu/drm/msm/dp/dp_display.c > @@ -1506,7 +1506,7 @@ void msm_dp_irq_postinstall(struct msm_dp *dp_display) > dp = container_of(dp_display, struct dp_display_private, dp_display); > > if (!dp_display->is_edp) > - dp_add_event(dp, EV_HPD_INIT_SETUP, 0, 0); > + dp_add_event(dp, EV_HPD_INIT_SETUP, 0, 100); When booting with the cable connected on my X13s, 100 is long enough for my display to time out and require me to disconnect and reconnect the cable again. Do we have any idea of why the reduction to 0 is causing an issue when using the internal HPD? Regards, Bjorn > } > > bool msm_dp_wide_bus_available(const struct msm_dp *dp_display) > -- > 2.30.2