From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 4673EC77B7F for ; Fri, 27 Jun 2025 13:04:23 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:MIME-Version: Content-Transfer-Encoding:Content-Type:References:In-Reply-To:Date:Cc:To:From :Subject:Message-ID:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=uyGlLJdsrpOs/6Btu8IbtzbV/xEAGIn+2sLR12eDSg8=; b=jDhaSoI4OWU108+mkB3+/0UVo0 z/9UeyfaSAJ/veSBYEiUKZdKBYYC04nxreDX4FeHaxsRHDFfgfGUN5rj/XFxoX1Jui+uz8mwzE2sD mdJPBKCUHOf3nU9cx0GzGUduHz12rv6YnmTJeijpEn4WfxBpsxW52zvdYDRE5E30xV/s9sWNB7mK8 9qZB7EyQp1Sm3WaM/RlzIDr5OKME5wvW2LdAJA4OmeN5MTiAIpCD0KpDedTEYOsRbl/Q+3WWc5qzF S58caYX51U3jeI/8qwK17jaKBlejQb8eNv7Z3+GWaIz6njycOYEjBgagjg4n5n8ylfupW9uHqWQxN iOwOakNQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1uV8kf-0000000Eiue-3W1V; Fri, 27 Jun 2025 13:04:21 +0000 Received: from sender4-pp-f112.zoho.com ([136.143.188.112]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1uV8Gy-0000000EePf-2Ck0; Fri, 27 Jun 2025 12:33:41 +0000 ARC-Seal: i=1; a=rsa-sha256; t=1751027616; cv=none; d=zohomail.com; s=zohoarc; b=n0u3gQ+u4hMzUNZXXQjCKP3SqP5F8ouXUwuImCJNtZvInphx8eik+K8fJ1kM5w1JKFkVUzujLOKqL3cuTG7xHlGhsz/I7JU0CBSXu2B+FL1OW9lvHpKZtqRjURGTfXIIekm5VWN1CdU4Vmlg5nSR1m55n7TILI84Am31IMmImrI= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1751027616; h=Content-Type:Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:MIME-Version:Message-ID:References:Subject:Subject:To:To:Message-Id:Reply-To; bh=uyGlLJdsrpOs/6Btu8IbtzbV/xEAGIn+2sLR12eDSg8=; b=kZGv/3cJp7bClypPYKjoB1yijMLBAUe6Q1JVfLu8BLHkKH4YKtr7UrUsJIl2HEcSrt2UhE78iDp/74e4BGs+fdg+SgkE0pw/MZH4RWIEf9gFciXkKqNxudlK6HwoT2Z4v63d32pWffOYg856JTbk7CX7utcjRHpnezz/4fHpIo8= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass header.i=collabora.com; spf=pass smtp.mailfrom=nfraprado@collabora.com; dmarc=pass header.from= DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; t=1751027616; s=zohomail; d=collabora.com; i=nfraprado@collabora.com; h=Message-ID:Subject:Subject:From:From:To:To:Cc:Cc:Date:Date:In-Reply-To:References:Content-Type:Content-Transfer-Encoding:MIME-Version:Message-Id:Reply-To; bh=uyGlLJdsrpOs/6Btu8IbtzbV/xEAGIn+2sLR12eDSg8=; b=E3dXl/WeONf/z8ONGbf/KQ2C2ljVFtrhJ3u5b5nmIVUvXWp11UosU5iqquCLAw0Q 8lxvbE/rT5Pzx7mFcdQtJheVcU6jouIzZ8A/XRcMlO73XZWRu738olHwDg3x0aYe/93 CEGiNT3V+UgH5iyVO/AOyHyfj9IqMxe8j+DwZZiA= Received: by mx.zohomail.com with SMTPS id 1751027612875820.9172933784699; Fri, 27 Jun 2025 05:33:32 -0700 (PDT) Message-ID: Subject: Re: [PATCH v4 1/1] drm/mediatek: Adjust bandwidth limit for DP From: =?ISO-8859-1?Q?N=EDcolas?= "F. R. A. Prado" To: CK Hu =?UTF-8?Q?=28=E8=83=A1=E4=BF=8A=E5=85=89=29?= , Project_Global_Chrome_Upstream_Group , "chunkuang.hu@kernel.org" , "simona@ffwll.ch" , AngeloGioacchino Del Regno , "airlied@gmail.com" , Mac Shen =?UTF-8?Q?=28=E6=B2=88=E4=BF=8A=29?= , "p.zabel@pengutronix.de" , "matthias.bgg@gmail.com" , Peng Liu =?UTF-8?Q?=28=E5=88=98=E9=B9=8F=29?= , LIANKUN YANG =?UTF-8?Q?=28=E6=9D=A8=E8=BF=9E=E5=9D=A4=29?= Cc: "dri-devel@lists.freedesktop.org" , "linux-mediatek@lists.infradead.org" , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" Date: Fri, 27 Jun 2025 08:33:30 -0400 In-Reply-To: <667a201e7893d6d4bf9ae0267a5afaf06faa5db8.camel@mediatek.com> References: <20250625095446.31726-1-liankun.yang@mediatek.com> <667a201e7893d6d4bf9ae0267a5afaf06faa5db8.camel@mediatek.com> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable User-Agent: Evolution 3.46.4-2 MIME-Version: 1.0 X-ZohoMailClient: External X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250627_053340_638462_EE85074B X-CRM114-Status: GOOD ( 34.06 ) X-BeenThere: linux-mediatek@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "Linux-mediatek" Errors-To: linux-mediatek-bounces+linux-mediatek=archiver.kernel.org@lists.infradead.org On Fri, 2025-06-27 at 01:32 +0000, CK Hu (=E8=83=A1=E4=BF=8A=E5=85=89) wrot= e: > On Wed, 2025-06-25 at 14:49 -0400, N=C3=ADcolas F. R. A. Prado wrote: > > External email : Please do not click links or open attachments > > until you have verified the sender or the content. > >=20 > >=20 > > On Wed, 2025-06-25 at 17:54 +0800, Liankun Yang wrote: > > > By adjusting the order of link training and relocating it to HPD, > > > link training can identify the usability of each lane in the > > > current > > > link. > > >=20 > > > It also supports handling signal instability and weakness due to > > > environmental issues, enabling the acquisition of a stable > > > bandwidth > > > for the current link. Subsequently, DP work can proceed based on > > > the actual maximum bandwidth. > > >=20 > > > It should training in the hpd event thread. > > > Check the mode with lane count and link rate of training. > > >=20 > > > If we're eDP and capabilities were already parsed we can skip > > > reading again because eDP panels aren't hotpluggable hence the > > > caps and training information won't ever change in a boot life > > >=20 > > > Therefore, bridge typec judgment is required for edp training in > > > atomic_enable function. > > >=20 > > > Signed-off-by: Liankun Yang > > > --- > > > Change in V4: > > > - Tested the internal eDP display on MT8195 Tomato and it is > > > fine. > > > Per suggestion from the previous thread: > > > https://urldefense.com/v3/__https://patchwork.kernel.org/project/linu= x-mediatek/patch/20250318140236.13650-2-liankun.yang@mediatek.com/__;!!CTRN= KA9wMg0ARbw!g0Ii2UaT9Wu1FktoGneG4gHV_zwIAwcqEihEA0W4XHKOW2vv7i5MLlekFB9AbbB= JvWKvbyaiQFkF-JpPcq0$ > >=20 > > Hi, > >=20 > > I tested this patch on MT8195 Tomato, on top of next-20250625. > > Indeed > > the internal eDP display is unaffected by this commit: it still > > works > > fine. > >=20 > > The external displays though not so much. I tested 3 different > > displays, using 2 different USBC-to-HDMI adapters, and in all cases > > the > > behavior was the same: > > - Before the patch, the image on the display is completely > > corrupted > > and unusable. The only discernible element on the display is the > > mouse > > cursor, which shows perfectly fine. Occasionally no image will be > > shown > > at all, but most of the times, the behavior is as described. > > - After the patch, nothing ever shows at all on the display. It is > > always black. > >=20 > > So while the external display support on Tomato is basically broken > > as > > of the latest next, this patch seems to regress the support even > > further. >=20 > Hi, Nicolas: >=20 > Have external display worked normally in previous version? > If no, I think this patch just let external display change from one > bad state to another bad state. > If yes, I think we should fix this problem first, and then apply this > patch to test. Hi, I do remember the external display working on Tomato before, but it's been a long time since I'd last tested. In any case, as I've described it is currently basically broken (just the mouse cursor renders fine), so indeed this patch is bringing the display from one bad state to another bad state. The bad state after the patch is even worse than before, since nothing even shows, so personally I think it'd make more sense to fix this issue before merging this patch, so we can actually make sure the patch works well, but I'll leave it for you to decide the best way forward. --=20 Thanks, N=C3=ADcolas