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 gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (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 69E40C5DF66 for ; Mon, 17 Aug 2026 16:20:03 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 7CD8310E261; Mon, 17 Aug 2026 16:20:02 +0000 (UTC) Authentication-Results: gabe.freedesktop.org; dkim=pass (1024-bit key; unprotected) header.d=collabora.com header.i=sebastian.reichel@collabora.com header.b="W2NS3n8H"; dkim-atps=neutral Received: from sender4-op-o11.zoho.com (sender4-op-o11.zoho.com [136.143.188.11]) by gabe.freedesktop.org (Postfix) with ESMTPS id A866210E261 for ; Mon, 17 Aug 2026 16:20:01 +0000 (UTC) ARC-Seal: i=1; a=rsa-sha256; t=1786983588; cv=none; d=zohomail.com; s=zohoarc; b=Xr8qyASLoDgQ2DjiqPrEPNFn9Up3WmtOy3y8eb3DEHDR65PciA/p0wJPP3OQ/o27Nwwp2tAVIDJr+IMA5+TBXFdXcJnUy2rSV0IbepDwbZz9Lq3lSINS/HRkfv9DlMsQRHDrtBB6sYHrc9ZvqiCH6pWR1XMGd2l4nCtaYlMuYyA= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1786983588; h=Content-Type:Cc:Cc:Date:Date:From:From:In-Reply-To:MIME-Version:Message-ID:Subject:Subject:To:To:Message-Id:Reply-To; bh=xzNR0kuhuJBFEV0f82GT0CErGpD2go5KMmOhRyLWgCI=; b=XXIIbKa2RLX/5PZfNWuexqcRzDnHHvw3u9tpLiY4CF6vAQicYvuhUkfBySZ3YY5MfY50qftNXBS3QslkmHQeECgHikjSgTvf8NdbeE7jWmhl46VMs/hDaA2Dv/Ms5yR6Mh5gBOBI1wuKr5wP65e7UJLxqNn1ifI3DAQXv6W9Oug= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass header.i=collabora.com; spf=pass smtp.mailfrom=sebastian.reichel@collabora.com; dmarc=pass header.from= DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; t=1786983588; s=zohomail; d=collabora.com; i=sebastian.reichel@collabora.com; h=Date:Date:From:From:To:To:Cc:Cc:Subject:Subject:Message-ID:MIME-Version:Content-Type:In-Reply-To:Message-Id:Reply-To; bh=xzNR0kuhuJBFEV0f82GT0CErGpD2go5KMmOhRyLWgCI=; b=W2NS3n8HZey3KIRtc8L/1VaoVfRELxoyzSIg0FUREnTIz3aD24iyMRBHdnl+bo9G vNde7evp7rssgxpHWcVT7FwPJri3RzgqELGWqiQM1DvQdjPPZmqoyLbfaFp6Y9TrFKW EpHLsTjzhxn5JSeroAMBWwhK0LePxsgOfMjPvzQg= Received: by mx.zohomail.com with SMTPS id 1786983584671747.0867418837129; Mon, 17 Aug 2026 09:19:44 -0700 (PDT) Received: by venus (Postfix, from userid 1000) id 401A7182033; Mon, 17 Aug 2026 18:19:37 +0200 (CEST) Date: Mon, 17 Aug 2026 18:19:37 +0200 From: Sebastian Reichel To: Andy Yan Cc: Sandy Huang , Heiko =?utf-8?Q?St=C3=BCbner?= , Andy Yan , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , Andrzej Hajda , Neil Armstrong , Robert Foss , Laurent Pinchart , Jonas Karlman , Jernej Skrabec , Rob Herring , Krzysztof Kozlowski , Conor Dooley , David Airlie , Simona Vetter , Dmitry Baryshkov , Luca Ceresoli , Philipp Zabel , Cristian Ciocaltea , Damon Ding , Dmitry Baryshkov , Alexey Charkov , dri-devel@lists.freedesktop.org, linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, kernel@collabora.com, linux-arm-kernel@lists.infradead.org, Sashiko Subject: Re: [PATCH v11 01/21] drm/bridge: synopsys: dw-dp: Register DP AUX on bridge attach Message-ID: References: <20260806-synopsys-dw-dp-improvements-v11-0-0d508505f383@collabora.com> <20260806-synopsys-dw-dp-improvements-v11-1-0d508505f383@collabora.com> <39274bba.18a3.1a00a939caf.Coremail.andyshrk@163.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="u5w64ha2nlvvjth4" Content-Disposition: inline In-Reply-To: <39274bba.18a3.1a00a939caf.Coremail.andyshrk@163.com> X-Zoho-Virus-Status: 1 X-Zoho-AV-Stamp: zmail-av-0.2.10.1.5.2/286.975.6 X-ZohoMailClient: External X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" --u5w64ha2nlvvjth4 Content-Type: text/plain; protected-headers=v1; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Subject: Re: [PATCH v11 01/21] drm/bridge: synopsys: dw-dp: Register DP AUX on bridge attach MIME-Version: 1.0 Hello Andy, On Sun, Aug 16, 2026 at 08:37:21PM +0800, Andy Yan wrote: > Hello Sebastian=EF=BC=8C >=20 > At 2026-08-07 01:02:07, "Sebastian Reichel" wrote: > >Unregister the DP AUX device at the right spot as documented in the > >drm_dp_aux_register() function description. This helps that it is > >only accessed when the DRM device is ready and the bridge is powered > >and initialized (further fixes are required for that). > > > >Fixes: 86eecc3a9c2e ("drm/bridge: synopsys: Add DW DPTX Controller suppo= rt library") > >Reported-by: Sashiko > >Signed-off-by: Sebastian Reichel > >--- > > drivers/gpu/drm/bridge/synopsys/dw-dp.c | 55 +++++++++++++++++++++-----= ------- > > 1 file changed, 35 insertions(+), 20 deletions(-) > > > >diff --git a/drivers/gpu/drm/bridge/synopsys/dw-dp.c b/drivers/gpu/drm/b= ridge/synopsys/dw-dp.c > >index 8b8529e1723c..60feb3d1e14b 100644 > >--- a/drivers/gpu/drm/bridge/synopsys/dw-dp.c > >+++ b/drivers/gpu/drm/bridge/synopsys/dw-dp.c > >@@ -1813,7 +1813,36 @@ static struct drm_bridge_state *dw_dp_bridge_atom= ic_duplicate_state(struct drm_b > > return &state->base; > > } > >=20 > >+static int dw_dp_bridge_attach(struct drm_bridge *bridge, > >+ struct drm_encoder *encoder, > >+ enum drm_bridge_attach_flags flags) > >+{ > >+ struct dw_dp *dp =3D bridge_to_dp(bridge); > >+ struct device *dev =3D dp->dev; > >+ int ret; > >+ > >+ dp->aux.dev =3D dev; > >+ dp->aux.drm_dev =3D encoder->dev; > >+ dp->aux.name =3D dev_name(dev); > >+ dp->aux.transfer =3D dw_dp_aux_transfer; > >+ > >+ ret =3D drm_dp_aux_register(&dp->aux); > >+ if (ret) > >+ dev_err(dev, "Aux register failed: %d\n", ret); > >+ > >+ return ret; > >+} > >+ > >+static void dw_dp_bridge_detach(struct drm_bridge *bridge) > >+{ > >+ struct dw_dp *dp =3D bridge_to_dp(bridge); > >+ > >+ drm_dp_aux_unregister(&dp->aux); > >+} > >+ > > static const struct drm_bridge_funcs dw_dp_bridge_funcs =3D { > >+ .attach =3D dw_dp_bridge_attach, > >+ .detach =3D dw_dp_bridge_detach, > > .atomic_duplicate_state =3D dw_dp_bridge_atomic_duplicate_state, > > .atomic_destroy_state =3D drm_atomic_helper_bridge_destroy_state, > > .atomic_create_state =3D drm_atomic_helper_bridge_create_state, > >@@ -2043,20 +2072,10 @@ struct dw_dp *dw_dp_bind(struct device *dev, str= uct drm_encoder *encoder, > > if (ret) > > return ERR_PTR(ret); > >=20 > >- dp->aux.dev =3D dev; > >- dp->aux.drm_dev =3D encoder->dev; > >- dp->aux.name =3D dev_name(dev); > >- dp->aux.transfer =3D dw_dp_aux_transfer; > >- ret =3D drm_dp_aux_register(&dp->aux); > >- if (ret) { > >- dev_err_probe(dev, ret, "Aux register failed\n"); > >- return ERR_PTR(ret); > >- } > >- > > ret =3D drm_bridge_attach(encoder, bridge, NULL, DRM_BRIDGE_ATTACH_NO_= CONNECTOR); > > if (ret) { > > dev_err_probe(dev, ret, "Failed to attach bridge\n"); > >- goto unregister_aux; > >+ return ERR_PTR(ret); > > } > >=20 > > dw_dp_init_hw(dp); > >@@ -2064,37 +2083,33 @@ struct dw_dp *dw_dp_bind(struct device *dev, str= uct drm_encoder *encoder, > > ret =3D phy_init(dp->phy); > > if (ret) { > > dev_err_probe(dev, ret, "phy init failed\n"); > >- goto unregister_aux; > >+ return ERR_PTR(ret); > > } > >=20 > > ret =3D devm_add_action_or_reset(dev, dw_dp_phy_exit, dp); > > if (ret) > >- goto unregister_aux; > >+ return ERR_PTR(ret); > >=20 > > dp->irq =3D platform_get_irq(pdev, 0); > > if (dp->irq < 0) { > > ret =3D dp->irq; > >- goto unregister_aux; > >+ return ERR_PTR(ret); > > } > >=20 > > ret =3D devm_request_threaded_irq(dev, dp->irq, NULL, dw_dp_irq, > > IRQF_ONESHOT, dev_name(dev), dp); > > if (ret) { > > dev_err_probe(dev, ret, "failed to request irq\n"); >=20 >=20 > After this patch[0] is applied, there will be a conflict here. >=20 > [0]https://lore.kernel.org/all/20260723131649.134127-6-panchuang@vivo.co= m/#t This got applied after I send this. I do rebase to latest drm-misc-next on every submission, so it will be fixed with the next version. Greetings, -- Sebastian >=20 >=20 > >- goto unregister_aux; > >+ return ERR_PTR(ret); > > } >=20 >=20 >=20 > >=20 > > return dp; > >- > >-unregister_aux: > >- drm_dp_aux_unregister(&dp->aux); > >- return ERR_PTR(ret); > > } > > EXPORT_SYMBOL_GPL(dw_dp_bind); > >=20 > > void dw_dp_unbind(struct dw_dp *dp) > > { > >- drm_dp_aux_unregister(&dp->aux); > >+ /* nothing to do */ > > } > > EXPORT_SYMBOL_GPL(dw_dp_unbind); > >=20 > > > >--=20 > >2.53.0 > > --u5w64ha2nlvvjth4 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCgAdFiEE72YNB0Y/i3JqeVQT2O7X88g7+poFAmqDNJUACgkQ2O7X88g7 +poGaxAAgeNiIBmqZCzTK0GPvq5iKwuyIr8+xL44ArVV7Lqm9EAor3E8wc7d7rZz /uXUPWw76k/rt+tba7ZwpwtjdXIYIc7uvq9x6HyCIxUeAjPemr5q4iLF/eK5o4XF flNISlAmFhzvmRL9doDKsZri1TclNHl+JzJUieSQEOlkZtFF6Bw/elJ2WUV7dFdS gs+t5Rig5fOpfdOgDyWWs3NBhUWe818KXfHtUMoX7FsHAGb8Al5p9WN3ARCc4rYp mRggpOE/EVgD+KMsUXSC/k3iVdNOKAHBwMbJ0burCj69G4iS+TaVtINHR0tzmivj 9B+1/9Kh29D+sLT3gVB4QjPvGnrtft1/xmB3Y9PLuUePZrpjGmSGmvrE9Me8FDsg J9hEBp+xl0LdcQDAj3ifbJNFqUl2f3VWF47snIkMahGty6WB2/zIgO/UIC2mbMB0 GjUHEqJxiPNrHBi/nIEx+XYpky7Xa2T9mRmWea+Md6gRybg/bdGlF0ji+QVxH8tu 5kURLI73p0w9h3Xn6SRREfZqMc2JmbaE+S++dJXlbabBiaWRGNaozxU9XnfNwKpQ D4jn540HXz9efN8iIazaZ2lXhsO89sr4/xbxzSS84ao9e8M3xGjsCaSFBeeTPBjs x9BLlJqK4I9iQCmeeQn8zGTxmHWOuQV8wOa6YBys9f0ohByuusU= =3lJR -----END PGP SIGNATURE----- --u5w64ha2nlvvjth4-- 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 EE726C5B572 for ; Mon, 17 Aug 2026 16:20:59 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type:Cc: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=d+cWk3BPONk+S6EvXICymmmIdqCOgMHcn7/LeeoP7Ls=; b=3f9b2YUGqFN2eA0H1nNthfkJM6 JgRYI5ajQ2VbHdlVb3FNcubGyN61JidclvR7VJCA+JbzzVjAAHoBPdkmvg+Pc1y5tntqHUzfVnDlp tVoHa+3R2ZC7fa8ShYxwsN52yO8SQrQDcdVPb6nEMRdZgCeh7fwseGknUMdfBm55jnAsv96h93XZx wcw5Fm0Wdyk4Ai5We84MQOF+rM/FFRAOm7IihwJz4JqH+N+q8S03A7ouQCoBJEJLL0+h1yNL4HR9+ tr4dfqyjwj84upK3tYfp2jHM1mSnJYs2baV7Z9QqUJC44PkXv7bmPkAPezxpJFJjcFPFTpYzVFvKm nx31DFCg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1ww04y-00000006V1a-0lU1; Mon, 17 Aug 2026 16:20:52 +0000 Received: from sender4-op-o11.zoho.com ([136.143.188.11]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1ww04u-00000006V02-2IVV; Mon, 17 Aug 2026 16:20:50 +0000 ARC-Seal: i=1; a=rsa-sha256; t=1786983588; cv=none; d=zohomail.com; s=zohoarc; b=Xr8qyASLoDgQ2DjiqPrEPNFn9Up3WmtOy3y8eb3DEHDR65PciA/p0wJPP3OQ/o27Nwwp2tAVIDJr+IMA5+TBXFdXcJnUy2rSV0IbepDwbZz9Lq3lSINS/HRkfv9DlMsQRHDrtBB6sYHrc9ZvqiCH6pWR1XMGd2l4nCtaYlMuYyA= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1786983588; h=Content-Type:Cc:Cc:Date:Date:From:From:In-Reply-To:MIME-Version:Message-ID:Subject:Subject:To:To:Message-Id:Reply-To; bh=xzNR0kuhuJBFEV0f82GT0CErGpD2go5KMmOhRyLWgCI=; b=XXIIbKa2RLX/5PZfNWuexqcRzDnHHvw3u9tpLiY4CF6vAQicYvuhUkfBySZ3YY5MfY50qftNXBS3QslkmHQeECgHikjSgTvf8NdbeE7jWmhl46VMs/hDaA2Dv/Ms5yR6Mh5gBOBI1wuKr5wP65e7UJLxqNn1ifI3DAQXv6W9Oug= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass header.i=collabora.com; spf=pass smtp.mailfrom=sebastian.reichel@collabora.com; dmarc=pass header.from= DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; t=1786983588; s=zohomail; d=collabora.com; i=sebastian.reichel@collabora.com; h=Date:Date:From:From:To:To:Cc:Cc:Subject:Subject:Message-ID:MIME-Version:Content-Type:In-Reply-To:Message-Id:Reply-To; bh=xzNR0kuhuJBFEV0f82GT0CErGpD2go5KMmOhRyLWgCI=; b=W2NS3n8HZey3KIRtc8L/1VaoVfRELxoyzSIg0FUREnTIz3aD24iyMRBHdnl+bo9G vNde7evp7rssgxpHWcVT7FwPJri3RzgqELGWqiQM1DvQdjPPZmqoyLbfaFp6Y9TrFKW EpHLsTjzhxn5JSeroAMBWwhK0LePxsgOfMjPvzQg= Received: by mx.zohomail.com with SMTPS id 1786983584671747.0867418837129; Mon, 17 Aug 2026 09:19:44 -0700 (PDT) Received: by venus (Postfix, from userid 1000) id 401A7182033; Mon, 17 Aug 2026 18:19:37 +0200 (CEST) Date: Mon, 17 Aug 2026 18:19:37 +0200 From: Sebastian Reichel To: Andy Yan Subject: Re: [PATCH v11 01/21] drm/bridge: synopsys: dw-dp: Register DP AUX on bridge attach Message-ID: References: <20260806-synopsys-dw-dp-improvements-v11-0-0d508505f383@collabora.com> <20260806-synopsys-dw-dp-improvements-v11-1-0d508505f383@collabora.com> <39274bba.18a3.1a00a939caf.Coremail.andyshrk@163.com> MIME-Version: 1.0 In-Reply-To: <39274bba.18a3.1a00a939caf.Coremail.andyshrk@163.com> X-Zoho-Virus-Status: 1 X-Zoho-AV-Stamp: zmail-av-0.2.10.1.5.2/286.975.6 X-ZohoMailClient: External X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.9.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20260817_092048_640468_579150CD X-CRM114-Status: GOOD ( 23.23 ) X-BeenThere: linux-rockchip@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Upstream kernel work for Rockchip platforms List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Heiko =?utf-8?Q?St=C3=BCbner?= , Sashiko , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, Laurent Pinchart , Andrzej Hajda , kernel@collabora.com, David Airlie , Simona Vetter , Rob Herring , Robert Foss , Jernej Skrabec , linux-rockchip@lists.infradead.org, Luca Ceresoli , devicetree@vger.kernel.org, Conor Dooley , Thomas Zimmermann , Jonas Karlman , Maarten Lankhorst , Maxime Ripard , Alexey Charkov , Damon Ding , linux-arm-kernel@lists.infradead.org, Dmitry Baryshkov , Neil Armstrong , Dmitry Baryshkov , Sandy Huang , Philipp Zabel , Andy Yan , Krzysztof Kozlowski Content-Type: multipart/mixed; boundary="===============9026248788359913300==" Sender: "Linux-rockchip" Errors-To: linux-rockchip-bounces+linux-rockchip=archiver.kernel.org@lists.infradead.org --===============9026248788359913300== Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="u5w64ha2nlvvjth4" Content-Disposition: inline --u5w64ha2nlvvjth4 Content-Type: text/plain; protected-headers=v1; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Subject: Re: [PATCH v11 01/21] drm/bridge: synopsys: dw-dp: Register DP AUX on bridge attach MIME-Version: 1.0 Hello Andy, On Sun, Aug 16, 2026 at 08:37:21PM +0800, Andy Yan wrote: > Hello Sebastian=EF=BC=8C >=20 > At 2026-08-07 01:02:07, "Sebastian Reichel" wrote: > >Unregister the DP AUX device at the right spot as documented in the > >drm_dp_aux_register() function description. This helps that it is > >only accessed when the DRM device is ready and the bridge is powered > >and initialized (further fixes are required for that). > > > >Fixes: 86eecc3a9c2e ("drm/bridge: synopsys: Add DW DPTX Controller suppo= rt library") > >Reported-by: Sashiko > >Signed-off-by: Sebastian Reichel > >--- > > drivers/gpu/drm/bridge/synopsys/dw-dp.c | 55 +++++++++++++++++++++-----= ------- > > 1 file changed, 35 insertions(+), 20 deletions(-) > > > >diff --git a/drivers/gpu/drm/bridge/synopsys/dw-dp.c b/drivers/gpu/drm/b= ridge/synopsys/dw-dp.c > >index 8b8529e1723c..60feb3d1e14b 100644 > >--- a/drivers/gpu/drm/bridge/synopsys/dw-dp.c > >+++ b/drivers/gpu/drm/bridge/synopsys/dw-dp.c > >@@ -1813,7 +1813,36 @@ static struct drm_bridge_state *dw_dp_bridge_atom= ic_duplicate_state(struct drm_b > > return &state->base; > > } > >=20 > >+static int dw_dp_bridge_attach(struct drm_bridge *bridge, > >+ struct drm_encoder *encoder, > >+ enum drm_bridge_attach_flags flags) > >+{ > >+ struct dw_dp *dp =3D bridge_to_dp(bridge); > >+ struct device *dev =3D dp->dev; > >+ int ret; > >+ > >+ dp->aux.dev =3D dev; > >+ dp->aux.drm_dev =3D encoder->dev; > >+ dp->aux.name =3D dev_name(dev); > >+ dp->aux.transfer =3D dw_dp_aux_transfer; > >+ > >+ ret =3D drm_dp_aux_register(&dp->aux); > >+ if (ret) > >+ dev_err(dev, "Aux register failed: %d\n", ret); > >+ > >+ return ret; > >+} > >+ > >+static void dw_dp_bridge_detach(struct drm_bridge *bridge) > >+{ > >+ struct dw_dp *dp =3D bridge_to_dp(bridge); > >+ > >+ drm_dp_aux_unregister(&dp->aux); > >+} > >+ > > static const struct drm_bridge_funcs dw_dp_bridge_funcs =3D { > >+ .attach =3D dw_dp_bridge_attach, > >+ .detach =3D dw_dp_bridge_detach, > > .atomic_duplicate_state =3D dw_dp_bridge_atomic_duplicate_state, > > .atomic_destroy_state =3D drm_atomic_helper_bridge_destroy_state, > > .atomic_create_state =3D drm_atomic_helper_bridge_create_state, > >@@ -2043,20 +2072,10 @@ struct dw_dp *dw_dp_bind(struct device *dev, str= uct drm_encoder *encoder, > > if (ret) > > return ERR_PTR(ret); > >=20 > >- dp->aux.dev =3D dev; > >- dp->aux.drm_dev =3D encoder->dev; > >- dp->aux.name =3D dev_name(dev); > >- dp->aux.transfer =3D dw_dp_aux_transfer; > >- ret =3D drm_dp_aux_register(&dp->aux); > >- if (ret) { > >- dev_err_probe(dev, ret, "Aux register failed\n"); > >- return ERR_PTR(ret); > >- } > >- > > ret =3D drm_bridge_attach(encoder, bridge, NULL, DRM_BRIDGE_ATTACH_NO_= CONNECTOR); > > if (ret) { > > dev_err_probe(dev, ret, "Failed to attach bridge\n"); > >- goto unregister_aux; > >+ return ERR_PTR(ret); > > } > >=20 > > dw_dp_init_hw(dp); > >@@ -2064,37 +2083,33 @@ struct dw_dp *dw_dp_bind(struct device *dev, str= uct drm_encoder *encoder, > > ret =3D phy_init(dp->phy); > > if (ret) { > > dev_err_probe(dev, ret, "phy init failed\n"); > >- goto unregister_aux; > >+ return ERR_PTR(ret); > > } > >=20 > > ret =3D devm_add_action_or_reset(dev, dw_dp_phy_exit, dp); > > if (ret) > >- goto unregister_aux; > >+ return ERR_PTR(ret); > >=20 > > dp->irq =3D platform_get_irq(pdev, 0); > > if (dp->irq < 0) { > > ret =3D dp->irq; > >- goto unregister_aux; > >+ return ERR_PTR(ret); > > } > >=20 > > ret =3D devm_request_threaded_irq(dev, dp->irq, NULL, dw_dp_irq, > > IRQF_ONESHOT, dev_name(dev), dp); > > if (ret) { > > dev_err_probe(dev, ret, "failed to request irq\n"); >=20 >=20 > After this patch[0] is applied, there will be a conflict here. >=20 > [0]https://lore.kernel.org/all/20260723131649.134127-6-panchuang@vivo.co= m/#t This got applied after I send this. I do rebase to latest drm-misc-next on every submission, so it will be fixed with the next version. Greetings, -- Sebastian >=20 >=20 > >- goto unregister_aux; > >+ return ERR_PTR(ret); > > } >=20 >=20 >=20 > >=20 > > return dp; > >- > >-unregister_aux: > >- drm_dp_aux_unregister(&dp->aux); > >- return ERR_PTR(ret); > > } > > EXPORT_SYMBOL_GPL(dw_dp_bind); > >=20 > > void dw_dp_unbind(struct dw_dp *dp) > > { > >- drm_dp_aux_unregister(&dp->aux); > >+ /* nothing to do */ > > } > > EXPORT_SYMBOL_GPL(dw_dp_unbind); > >=20 > > > >--=20 > >2.53.0 > > --u5w64ha2nlvvjth4 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCgAdFiEE72YNB0Y/i3JqeVQT2O7X88g7+poFAmqDNJUACgkQ2O7X88g7 +poGaxAAgeNiIBmqZCzTK0GPvq5iKwuyIr8+xL44ArVV7Lqm9EAor3E8wc7d7rZz /uXUPWw76k/rt+tba7ZwpwtjdXIYIc7uvq9x6HyCIxUeAjPemr5q4iLF/eK5o4XF flNISlAmFhzvmRL9doDKsZri1TclNHl+JzJUieSQEOlkZtFF6Bw/elJ2WUV7dFdS gs+t5Rig5fOpfdOgDyWWs3NBhUWe818KXfHtUMoX7FsHAGb8Al5p9WN3ARCc4rYp mRggpOE/EVgD+KMsUXSC/k3iVdNOKAHBwMbJ0burCj69G4iS+TaVtINHR0tzmivj 9B+1/9Kh29D+sLT3gVB4QjPvGnrtft1/xmB3Y9PLuUePZrpjGmSGmvrE9Me8FDsg J9hEBp+xl0LdcQDAj3ifbJNFqUl2f3VWF47snIkMahGty6WB2/zIgO/UIC2mbMB0 GjUHEqJxiPNrHBi/nIEx+XYpky7Xa2T9mRmWea+Md6gRybg/bdGlF0ji+QVxH8tu 5kURLI73p0w9h3Xn6SRREfZqMc2JmbaE+S++dJXlbabBiaWRGNaozxU9XnfNwKpQ D4jn540HXz9efN8iIazaZ2lXhsO89sr4/xbxzSS84ao9e8M3xGjsCaSFBeeTPBjs x9BLlJqK4I9iQCmeeQn8zGTxmHWOuQV8wOa6YBys9f0ohByuusU= =3lJR -----END PGP SIGNATURE----- --u5w64ha2nlvvjth4-- --===============9026248788359913300== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip --===============9026248788359913300==--