From nobody Wed Feb 5 20:13:17 2025 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id B37FF217F54 for ; Tue, 4 Feb 2025 15:00:28 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1738681228; cv=none; b=MFTL0HoUgZKqE24778Q4Aaxq4UdlnGicAtUw7+3ipwbtXWE1vJKjU1Gg09Ctg+HelSuLldhewxYWJUK++5xpBdixNS/32zxbGlX4fdSfiQV+s/bPMptWmQNjLi97USfdjJZdZjZ2t4rW7mUhYmAq3lnfA2AbWXYIYEp8FUTspYQ= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1738681228; c=relaxed/simple; bh=rGu5wfrKCHRDk0KK3pf76S3/bNRBlBDqwgY0ngcP5gc=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=CgD4NQEu1VBhnmuzH/MKNFWGtR8xeDYYh8NPwVqIeGOf3M84BRUv8NubiRdcVkauVk8lcZMY/+CMVGAfR1cJx1YIvc/qYtR9jLUv7tjXyMhE+Zm/R70abGTOrg566EfGEIouRMsVW/2FKhxaKQ8c2YiD+t6anwE1sAehbpcK5Dg= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=gcJGAKwU; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="gcJGAKwU" Received: by smtp.kernel.org (Postfix) with ESMTPSA id EBC94C4CEDF; Tue, 4 Feb 2025 15:00:27 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1738681228; bh=rGu5wfrKCHRDk0KK3pf76S3/bNRBlBDqwgY0ngcP5gc=; h=From:Date:Subject:References:In-Reply-To:To:Cc:From; b=gcJGAKwUH1UH+fI6ZbGkPuOfu29V2zJJumvRqOsqbNlTyIbKSHptLWuWcJIXrhBjr Pd90HynmyoVGY2ljKEw6VIa6fx/6MyUzor2kJIeTCRwB6LLgsxrmbWmjiY1OSjPQOl 31UCfKaCprYE4a7IXhVtPAHHocLsKMe1zRzNxePN5TSWFUQsNu3TEGiWVBiMrN1Di+ pC/kIcDN/NpymoZWISRw4UMhNfNxltV2Ofm8qjOSlOkQOSGz+oceTCYtNGcK4DayYO /noWlcMiPfiNF9jfv/1QWhj9UcSnS1jlCae8hXkzGtTuP+D/gg+nGYj7QbvA1MatBq qf5QhjCpSL/Jw== From: Maxime Ripard Date: Tue, 04 Feb 2025 15:57:55 +0100 Subject: [PATCH v2 27/35] drm/bridge: Add encoder parameter to drm_bridge_funcs.attach Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20250204-bridge-connector-v2-27-35dd6c834e08@kernel.org> References: <20250204-bridge-connector-v2-0-35dd6c834e08@kernel.org> In-Reply-To: <20250204-bridge-connector-v2-0-35dd6c834e08@kernel.org> To: Maarten Lankhorst , Thomas Zimmermann , David Airlie , Simona Vetter , Andrzej Hajda , Neil Armstrong , Robert Foss , Laurent Pinchart , Jonas Karlman , Jernej Skrabec , Douglas Anderson Cc: dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, Maxime Ripard X-Mailer: b4 0.14.2 X-Developer-Signature: v=1; a=openpgp-sha256; l=87922; i=mripard@kernel.org; h=from:subject:message-id; bh=rGu5wfrKCHRDk0KK3pf76S3/bNRBlBDqwgY0ngcP5gc=; b=owGbwMvMwCmsHn9OcpHtvjLG02pJDOmLtL7uXVlzN+1sWGPt43RN4/9MqpqX3DfMyD+iq/RXJ mHdgWNuHVNZGIQ5GWTFFFmeyISdXt6+uMrBfuUPmDmsTCBDGLg4BWAi528w1of2MzVuvbR4XylX eeuN9TddW78ZGi8Ir08wmZC78QZXwZr/jNaBYk0851hfnVSqW/+/mbGh813RuT3zRbyNdiR90ty 74OZu39DGUzG/WyfNMv9aFf9ty6ecPW/+mFRFBu7W5fzVfH8iAA== X-Developer-Key: i=mripard@kernel.org; a=openpgp; fpr=BE5675C37E818C8B5764241C254BCFC56BF6CE8D The drm_bridge structure contains an encoder pointer that is widely used by bridge drivers. This pattern is largely documented as deprecated in other KMS entities for atomic drivers. However, one of the main use of that pointer is done in attach to just call drm_bridge_attach on the next bridge to add it to the bridge list. While this dereferences the bridge->encoder pointer, it's effectively the same encoder the bridge was being attached to. We can make it more explicit by adding the encoder the bridge is attached to to the list of attach parameters. This also removes the need to dereference bridge->encoder in most drivers. Signed-off-by: Maxime Ripard --- drivers/gpu/drm/bridge/adv7511/adv7511_drv.c | 3 ++- drivers/gpu/drm/bridge/analogix/analogix-anx6345.c | 3 ++- drivers/gpu/drm/bridge/analogix/analogix-anx78xx.c | 3 ++- drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 2 +- drivers/gpu/drm/bridge/analogix/anx7625.c | 3 ++- drivers/gpu/drm/bridge/aux-bridge.c | 3 ++- drivers/gpu/drm/bridge/aux-hpd-bridge.c | 1 + drivers/gpu/drm/bridge/cadence/cdns-dsi-core.c | 3 ++- drivers/gpu/drm/bridge/cadence/cdns-mhdp8546-core.c | 1 + drivers/gpu/drm/bridge/chipone-icn6211.c | 6 ++++-- drivers/gpu/drm/bridge/chrontel-ch7033.c | 5 +++-- drivers/gpu/drm/bridge/display-connector.c | 1 + drivers/gpu/drm/bridge/fsl-ldb.c | 3 ++- drivers/gpu/drm/bridge/imx/imx-ldb-helper.c | 7 +++---- drivers/gpu/drm/bridge/imx/imx-ldb-helper.h | 2 +- drivers/gpu/drm/bridge/imx/imx-legacy-bridge.c | 3 ++- drivers/gpu/drm/bridge/imx/imx8mp-hdmi-pvi.c | 3 ++- drivers/gpu/drm/bridge/imx/imx8qxp-pixel-combiner.c | 3 ++- drivers/gpu/drm/bridge/imx/imx8qxp-pixel-link.c | 3 ++- drivers/gpu/drm/bridge/imx/imx8qxp-pxl2dpi.c | 3 ++- drivers/gpu/drm/bridge/ite-it6263.c | 7 ++++--- drivers/gpu/drm/bridge/ite-it6505.c | 1 + drivers/gpu/drm/bridge/ite-it66121.c | 3 ++- drivers/gpu/drm/bridge/lontium-lt8912b.c | 3 ++- drivers/gpu/drm/bridge/lontium-lt9211.c | 3 ++- drivers/gpu/drm/bridge/lontium-lt9611.c | 3 ++- drivers/gpu/drm/bridge/lontium-lt9611uxc.c | 3 ++- drivers/gpu/drm/bridge/lvds-codec.c | 3 ++- drivers/gpu/drm/bridge/megachips-stdpxxxx-ge-b850v3-fw.c | 1 + drivers/gpu/drm/bridge/microchip-lvds.c | 3 ++- drivers/gpu/drm/bridge/nwl-dsi.c | 3 ++- drivers/gpu/drm/bridge/nxp-ptn3460.c | 5 +++-- drivers/gpu/drm/bridge/panel.c | 3 ++- drivers/gpu/drm/bridge/parade-ps8622.c | 1 + drivers/gpu/drm/bridge/parade-ps8640.c | 3 ++- drivers/gpu/drm/bridge/samsung-dsim.c | 3 ++- drivers/gpu/drm/bridge/sii902x.c | 5 +++-- drivers/gpu/drm/bridge/sil-sii8620.c | 1 + drivers/gpu/drm/bridge/simple-bridge.c | 5 +++-- drivers/gpu/drm/bridge/synopsys/dw-hdmi.c | 3 ++- drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c | 5 +++-- drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi2.c | 5 +++-- drivers/gpu/drm/bridge/tc358762.c | 3 ++- drivers/gpu/drm/bridge/tc358764.c | 3 ++- drivers/gpu/drm/bridge/tc358767.c | 2 ++ drivers/gpu/drm/bridge/tc358768.c | 3 ++- drivers/gpu/drm/bridge/tc358775.c | 3 ++- drivers/gpu/drm/bridge/thc63lvd1024.c | 3 ++- drivers/gpu/drm/bridge/ti-dlpc3433.c | 4 ++-- drivers/gpu/drm/bridge/ti-sn65dsi83.c | 3 ++- drivers/gpu/drm/bridge/ti-sn65dsi86.c | 3 ++- drivers/gpu/drm/bridge/ti-tdp158.c | 6 ++++-- drivers/gpu/drm/bridge/ti-tfp410.c | 5 +++-- drivers/gpu/drm/bridge/ti-tpd12s015.c | 3 ++- drivers/gpu/drm/drm_bridge.c | 2 +- drivers/gpu/drm/i2c/tda998x_drv.c | 1 + drivers/gpu/drm/imx/ipuv3/parallel-display.c | 3 ++- drivers/gpu/drm/ingenic/ingenic-drm-drv.c | 5 +++-- drivers/gpu/drm/mcde/mcde_dsi.c | 3 ++- drivers/gpu/drm/mediatek/mtk_dp.c | 3 ++- drivers/gpu/drm/mediatek/mtk_dpi.c | 3 ++- drivers/gpu/drm/mediatek/mtk_dsi.c | 3 ++- drivers/gpu/drm/mediatek/mtk_hdmi.c | 3 ++- drivers/gpu/drm/meson/meson_encoder_cvbs.c | 3 ++- drivers/gpu/drm/meson/meson_encoder_dsi.c | 3 ++- drivers/gpu/drm/meson/meson_encoder_hdmi.c | 3 ++- drivers/gpu/drm/msm/dsi/dsi_manager.c | 3 ++- drivers/gpu/drm/omapdrm/dss/dpi.c | 3 ++- drivers/gpu/drm/omapdrm/dss/dsi.c | 3 ++- drivers/gpu/drm/omapdrm/dss/hdmi4.c | 3 ++- drivers/gpu/drm/omapdrm/dss/hdmi5.c | 3 ++- drivers/gpu/drm/omapdrm/dss/sdi.c | 3 ++- drivers/gpu/drm/omapdrm/dss/venc.c | 3 ++- drivers/gpu/drm/renesas/rcar-du/rcar_lvds.c | 3 ++- drivers/gpu/drm/renesas/rcar-du/rcar_mipi_dsi.c | 3 ++- drivers/gpu/drm/renesas/rz-du/rzg2l_mipi_dsi.c | 3 ++- drivers/gpu/drm/stm/lvds.c | 11 +++++------ drivers/gpu/drm/tidss/tidss_encoder.c | 3 ++- drivers/gpu/drm/vc4/vc4_dsi.c | 3 ++- drivers/gpu/drm/xlnx/zynqmp_dp.c | 3 ++- drivers/platform/arm64/acer-aspire1-ec.c | 3 ++- include/drm/drm_bridge.h | 2 +- 82 files changed, 170 insertions(+), 94 deletions(-) diff --git a/drivers/gpu/drm/bridge/adv7511/adv7511_drv.c b/drivers/gpu/drm= /bridge/adv7511/adv7511_drv.c index fda66e7876b86f6dfc1ed0f781cd9c184e380090..e0567b8b0fb7723e5953d7134ed= 879a169e4533e 100644 --- a/drivers/gpu/drm/bridge/adv7511/adv7511_drv.c +++ b/drivers/gpu/drm/bridge/adv7511/adv7511_drv.c @@ -944,17 +944,18 @@ static enum drm_mode_status adv7511_bridge_mode_valid= (struct drm_bridge *bridge, else return adv7511_mode_valid(adv, mode); } =20 static int adv7511_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct adv7511 *adv =3D bridge_to_adv7511(bridge); int ret =3D 0; =20 if (adv->next_bridge) { - ret =3D drm_bridge_attach(bridge->encoder, adv->next_bridge, bridge, + ret =3D drm_bridge_attach(encoder, adv->next_bridge, bridge, flags | DRM_BRIDGE_ATTACH_NO_CONNECTOR); if (ret) return ret; } =20 diff --git a/drivers/gpu/drm/bridge/analogix/analogix-anx6345.c b/drivers/g= pu/drm/bridge/analogix/analogix-anx6345.c index 83d711ee3a2eb8b7a491c115171b5cc12a9df066..a88a33eb5d972cb47e82cfccde8= 7ecf43d051637 100644 --- a/drivers/gpu/drm/bridge/analogix/analogix-anx6345.c +++ b/drivers/gpu/drm/bridge/analogix/analogix-anx6345.c @@ -515,10 +515,11 @@ static const struct drm_connector_funcs anx6345_conne= ctor_funcs =3D { .atomic_duplicate_state =3D drm_atomic_helper_connector_duplicate_state, .atomic_destroy_state =3D drm_atomic_helper_connector_destroy_state, }; =20 static int anx6345_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct anx6345 *anx6345 =3D bridge_to_anx6345(bridge); int err; =20 @@ -551,11 +552,11 @@ static int anx6345_bridge_attach(struct drm_bridge *b= ridge, &anx6345_connector_helper_funcs); =20 anx6345->connector.polled =3D DRM_CONNECTOR_POLL_HPD; =20 err =3D drm_connector_attach_encoder(&anx6345->connector, - bridge->encoder); + encoder); if (err) { DRM_ERROR("Failed to link up connector to encoder: %d\n", err); goto connector_cleanup; } =20 diff --git a/drivers/gpu/drm/bridge/analogix/analogix-anx78xx.c b/drivers/g= pu/drm/bridge/analogix/analogix-anx78xx.c index f74694bb9c50a2f71d51d27e08c08bb854cf29ea..8b4597885614d6278f5782fdce5= 065044d5c2966 100644 --- a/drivers/gpu/drm/bridge/analogix/analogix-anx78xx.c +++ b/drivers/gpu/drm/bridge/analogix/analogix-anx78xx.c @@ -886,10 +886,11 @@ static const struct drm_connector_funcs anx78xx_conne= ctor_funcs =3D { .atomic_duplicate_state =3D drm_atomic_helper_connector_duplicate_state, .atomic_destroy_state =3D drm_atomic_helper_connector_destroy_state, }; =20 static int anx78xx_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct anx78xx *anx78xx =3D bridge_to_anx78xx(bridge); int err; =20 @@ -922,11 +923,11 @@ static int anx78xx_bridge_attach(struct drm_bridge *b= ridge, &anx78xx_connector_helper_funcs); =20 anx78xx->connector.polled =3D DRM_CONNECTOR_POLL_HPD; =20 err =3D drm_connector_attach_encoder(&anx78xx->connector, - bridge->encoder); + encoder); if (err) { DRM_ERROR("Failed to link up connector to encoder: %d\n", err); goto connector_cleanup; } =20 diff --git a/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c b/drivers/g= pu/drm/bridge/analogix/analogix_dp_core.c index 60ce33eadd97548c39201332e26b56e71245253a..bc87055ab82607e42cee638cbab= 5d2c136232bd7 100644 --- a/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c +++ b/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c @@ -1111,14 +1111,14 @@ static const struct drm_connector_funcs analogix_dp= _connector_funcs =3D { .atomic_duplicate_state =3D drm_atomic_helper_connector_duplicate_state, .atomic_destroy_state =3D drm_atomic_helper_connector_destroy_state, }; =20 static int analogix_dp_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct analogix_dp_device *dp =3D bridge->driver_private; - struct drm_encoder *encoder =3D dp->encoder; struct drm_connector *connector =3D NULL; int ret =3D 0; =20 if (flags & DRM_BRIDGE_ATTACH_NO_CONNECTOR) { DRM_ERROR("Fix bridge driver to make connector optional!"); diff --git a/drivers/gpu/drm/bridge/analogix/anx7625.c b/drivers/gpu/drm/br= idge/analogix/anx7625.c index 0b97b66de577421b0e68e748d0075f674c71fbf1..0b61e77c0398bbce3ecf55753dd= e71a27eda8fe5 100644 --- a/drivers/gpu/drm/bridge/analogix/anx7625.c +++ b/drivers/gpu/drm/bridge/analogix/anx7625.c @@ -2139,10 +2139,11 @@ static void hdcp_check_work_func(struct work_struct= *work) mutex_unlock(&ctx->hdcp_wq_lock); drm_modeset_unlock(&drm_dev->mode_config.connection_mutex); } =20 static int anx7625_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct anx7625_data *ctx =3D bridge_to_anx7625(bridge); int err; struct device *dev =3D ctx->dev; @@ -2157,11 +2158,11 @@ static int anx7625_bridge_attach(struct drm_bridge = *bridge, dev_err(dev, "failed to register aux channel: %d\n", err); return err; } =20 if (ctx->pdata.panel_bridge) { - err =3D drm_bridge_attach(bridge->encoder, + err =3D drm_bridge_attach(encoder, ctx->pdata.panel_bridge, &ctx->bridge, flags); if (err) return err; } diff --git a/drivers/gpu/drm/bridge/aux-bridge.c b/drivers/gpu/drm/bridge/a= ux-bridge.c index 015983c015e508f75435bf97ee1dba785edcd23e..c179b86d208f70d95b41e6f2157= b78f97bac4d8d 100644 --- a/drivers/gpu/drm/bridge/aux-bridge.c +++ b/drivers/gpu/drm/bridge/aux-bridge.c @@ -84,20 +84,21 @@ struct drm_aux_bridge_data { struct drm_bridge *next_bridge; struct device *dev; }; =20 static int drm_aux_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct drm_aux_bridge_data *data; =20 if (!(flags & DRM_BRIDGE_ATTACH_NO_CONNECTOR)) return -EINVAL; =20 data =3D container_of(bridge, struct drm_aux_bridge_data, bridge); =20 - return drm_bridge_attach(bridge->encoder, data->next_bridge, bridge, + return drm_bridge_attach(encoder, data->next_bridge, bridge, DRM_BRIDGE_ATTACH_NO_CONNECTOR); } =20 static const struct drm_bridge_funcs drm_aux_bridge_funcs =3D { .attach =3D drm_aux_bridge_attach, diff --git a/drivers/gpu/drm/bridge/aux-hpd-bridge.c b/drivers/gpu/drm/brid= ge/aux-hpd-bridge.c index 48f297c78ee67c8f9034869f6824a6dc7f88bdd7..b3f588b71a7d7ad5c2ee7b07c39= 079bc5ba34cee 100644 --- a/drivers/gpu/drm/bridge/aux-hpd-bridge.c +++ b/drivers/gpu/drm/bridge/aux-hpd-bridge.c @@ -154,10 +154,11 @@ void drm_aux_hpd_bridge_notify(struct device *dev, en= um drm_connector_status sta drm_bridge_hpd_notify(&data->bridge, status); } EXPORT_SYMBOL_GPL(drm_aux_hpd_bridge_notify); =20 static int drm_aux_hpd_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { return flags & DRM_BRIDGE_ATTACH_NO_CONNECTOR ? 0 : -EINVAL; } =20 diff --git a/drivers/gpu/drm/bridge/cadence/cdns-dsi-core.c b/drivers/gpu/d= rm/bridge/cadence/cdns-dsi-core.c index c7a0247e06adf3e532f830363b3422a8058513ef..8f54c034ac4f3e82c38607a0e52= d4745654b571f 100644 --- a/drivers/gpu/drm/bridge/cadence/cdns-dsi-core.c +++ b/drivers/gpu/drm/bridge/cadence/cdns-dsi-core.c @@ -603,10 +603,11 @@ static int cdns_dsi_check_conf(struct cdns_dsi *dsi, =20 return 0; } =20 static int cdns_dsi_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct cdns_dsi_input *input =3D bridge_to_cdns_dsi_input(bridge); struct cdns_dsi *dsi =3D input_to_dsi(input); struct cdns_dsi_output *output =3D &dsi->output; @@ -615,11 +616,11 @@ static int cdns_dsi_bridge_attach(struct drm_bridge *= bridge, dev_err(dsi->base.dev, "cdns-dsi driver is only compatible with DRM devices supporting atomic = updates"); return -ENOTSUPP; } =20 - return drm_bridge_attach(bridge->encoder, output->bridge, bridge, + return drm_bridge_attach(encoder, output->bridge, bridge, flags); } =20 static enum drm_mode_status cdns_dsi_bridge_mode_valid(struct drm_bridge *bridge, diff --git a/drivers/gpu/drm/bridge/cadence/cdns-mhdp8546-core.c b/drivers/= gpu/drm/bridge/cadence/cdns-mhdp8546-core.c index ef586e7df83b96f39ab552171d2ef665971a81ef..69347799913aed257a4a9447028= 83753df9c34bb 100644 --- a/drivers/gpu/drm/bridge/cadence/cdns-mhdp8546-core.c +++ b/drivers/gpu/drm/bridge/cadence/cdns-mhdp8546-core.c @@ -1724,10 +1724,11 @@ static int cdns_mhdp_connector_init(struct cdns_mhd= p_device *mhdp) =20 return ret; } =20 static int cdns_mhdp_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct cdns_mhdp_device *mhdp =3D bridge_to_mhdp(bridge); bool hw_ready; int ret; diff --git a/drivers/gpu/drm/bridge/chipone-icn6211.c b/drivers/gpu/drm/bri= dge/chipone-icn6211.c index 81f7c701961fddadbced6e4ad1bf9f12d3325fa7..634c5b0306679d2e68798c2b901= 3aae4491dd44c 100644 --- a/drivers/gpu/drm/bridge/chipone-icn6211.c +++ b/drivers/gpu/drm/bridge/chipone-icn6211.c @@ -578,15 +578,17 @@ static int chipone_dsi_host_attach(struct chipone *ic= n) mipi_dsi_device_unregister(dsi); =20 return ret; } =20 -static int chipone_attach(struct drm_bridge *bridge, enum drm_bridge_attac= h_flags flags) +static int chipone_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, + enum drm_bridge_attach_flags flags) { struct chipone *icn =3D bridge_to_chipone(bridge); =20 - return drm_bridge_attach(bridge->encoder, icn->panel_bridge, bridge, flag= s); + return drm_bridge_attach(encoder, icn->panel_bridge, bridge, flags); } =20 #define MAX_INPUT_SEL_FORMATS 1 =20 static u32 * diff --git a/drivers/gpu/drm/bridge/chrontel-ch7033.c b/drivers/gpu/drm/bri= dge/chrontel-ch7033.c index da17f0978a791fe9bffab3fa0527e3a355d65c6b..210c45c1efd48f5b541bf73da66= a169c27e110b2 100644 --- a/drivers/gpu/drm/bridge/chrontel-ch7033.c +++ b/drivers/gpu/drm/bridge/chrontel-ch7033.c @@ -266,17 +266,18 @@ static void ch7033_hpd_event(void *arg, enum drm_conn= ector_status status) if (priv->bridge.dev) drm_helper_hpd_irq_event(priv->connector.dev); } =20 static int ch7033_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct ch7033_priv *priv =3D bridge_to_ch7033_priv(bridge); struct drm_connector *connector =3D &priv->connector; int ret; =20 - ret =3D drm_bridge_attach(bridge->encoder, priv->next_bridge, bridge, + ret =3D drm_bridge_attach(encoder, priv->next_bridge, bridge, DRM_BRIDGE_ATTACH_NO_CONNECTOR); if (ret) return ret; =20 if (flags & DRM_BRIDGE_ATTACH_NO_CONNECTOR) @@ -303,11 +304,11 @@ static int ch7033_bridge_attach(struct drm_bridge *br= idge, if (ret) { DRM_ERROR("Failed to initialize connector\n"); return ret; } =20 - return drm_connector_attach_encoder(&priv->connector, bridge->encoder); + return drm_connector_attach_encoder(&priv->connector, encoder); } =20 static void ch7033_bridge_detach(struct drm_bridge *bridge) { struct ch7033_priv *priv =3D bridge_to_ch7033_priv(bridge); diff --git a/drivers/gpu/drm/bridge/display-connector.c b/drivers/gpu/drm/b= ridge/display-connector.c index 72bc508d4e6e2bf5be535c8856b46465d1ff835e..09c08a53d5bdc5c48f5d520472f= 5a311289d4862 100644 --- a/drivers/gpu/drm/bridge/display-connector.c +++ b/drivers/gpu/drm/bridge/display-connector.c @@ -32,10 +32,11 @@ to_display_connector(struct drm_bridge *bridge) { return container_of(bridge, struct display_connector, bridge); } =20 static int display_connector_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { return flags & DRM_BRIDGE_ATTACH_NO_CONNECTOR ? 0 : -EINVAL; } =20 diff --git a/drivers/gpu/drm/bridge/fsl-ldb.c b/drivers/gpu/drm/bridge/fsl-= ldb.c index 26ae1ab5237f81ebd6c56075a18136f4a5b56ec9..72d8f32d48faf6726ea67d8770e= d83f77f3a0862 100644 --- a/drivers/gpu/drm/bridge/fsl-ldb.c +++ b/drivers/gpu/drm/bridge/fsl-ldb.c @@ -111,15 +111,16 @@ static unsigned long fsl_ldb_link_frequency(struct fs= l_ldb *fsl_ldb, int clock) else return clock * 7000; } =20 static int fsl_ldb_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct fsl_ldb *fsl_ldb =3D to_fsl_ldb(bridge); =20 - return drm_bridge_attach(bridge->encoder, fsl_ldb->panel_bridge, + return drm_bridge_attach(encoder, fsl_ldb->panel_bridge, bridge, flags); } =20 static void fsl_ldb_atomic_enable(struct drm_bridge *bridge, struct drm_atomic_state *state) diff --git a/drivers/gpu/drm/bridge/imx/imx-ldb-helper.c b/drivers/gpu/drm/= bridge/imx/imx-ldb-helper.c index 9b5bebbe357dd4905a792abc5cecdbb9ce10ffb8..61347f6ec33d906264f7e06902b= 0d915d263f3f8 100644 --- a/drivers/gpu/drm/bridge/imx/imx-ldb-helper.c +++ b/drivers/gpu/drm/bridge/imx/imx-ldb-helper.c @@ -102,11 +102,11 @@ void ldb_bridge_disable_helper(struct drm_bridge *bri= dge) =20 regmap_write(ldb->regmap, ldb->ctrl_reg, ldb->ldb_ctrl); } EXPORT_SYMBOL_GPL(ldb_bridge_disable_helper); =20 -int ldb_bridge_attach_helper(struct drm_bridge *bridge, +int ldb_bridge_attach_helper(struct drm_bridge *bridge, struct drm_encoder= *encoder, enum drm_bridge_attach_flags flags) { struct ldb_channel *ldb_ch =3D bridge->driver_private; struct ldb *ldb =3D ldb_ch->ldb; =20 @@ -114,13 +114,12 @@ int ldb_bridge_attach_helper(struct drm_bridge *bridg= e, DRM_DEV_ERROR(ldb->dev, "do not support creating a drm_connector\n"); return -EINVAL; } =20 - return drm_bridge_attach(bridge->encoder, - ldb_ch->next_bridge, bridge, - DRM_BRIDGE_ATTACH_NO_CONNECTOR); + return drm_bridge_attach(encoder, ldb_ch->next_bridge, bridge, + DRM_BRIDGE_ATTACH_NO_CONNECTOR); } EXPORT_SYMBOL_GPL(ldb_bridge_attach_helper); =20 int ldb_init_helper(struct ldb *ldb) { diff --git a/drivers/gpu/drm/bridge/imx/imx-ldb-helper.h b/drivers/gpu/drm/= bridge/imx/imx-ldb-helper.h index a0a5cde27fbc6eb0d3932381f78d6803542b9bac..38a8a54b37a60e1be942aaa60b1= d1bc375a7a131 100644 --- a/drivers/gpu/drm/bridge/imx/imx-ldb-helper.h +++ b/drivers/gpu/drm/bridge/imx/imx-ldb-helper.h @@ -79,11 +79,11 @@ void ldb_bridge_mode_set_helper(struct drm_bridge *brid= ge, =20 void ldb_bridge_enable_helper(struct drm_bridge *bridge); =20 void ldb_bridge_disable_helper(struct drm_bridge *bridge); =20 -int ldb_bridge_attach_helper(struct drm_bridge *bridge, +int ldb_bridge_attach_helper(struct drm_bridge *bridge, struct drm_encoder= *encoder, enum drm_bridge_attach_flags flags); =20 int ldb_init_helper(struct ldb *ldb); =20 int ldb_find_next_bridge_helper(struct ldb *ldb); diff --git a/drivers/gpu/drm/bridge/imx/imx-legacy-bridge.c b/drivers/gpu/d= rm/bridge/imx/imx-legacy-bridge.c index 3ebf0b9866de13b12fab2b3c1e4db5f3d446089a..7d9cb25595b17c3f008ebf0bf4b= 6190020a793d7 100644 --- a/drivers/gpu/drm/bridge/imx/imx-legacy-bridge.c +++ b/drivers/gpu/drm/bridge/imx/imx-legacy-bridge.c @@ -21,11 +21,12 @@ struct imx_legacy_bridge { }; =20 #define to_imx_legacy_bridge(bridge) container_of(bridge, struct imx_legac= y_bridge, base) =20 static int imx_legacy_bridge_attach(struct drm_bridge *bridge, - enum drm_bridge_attach_flags flags) + struct drm_encoder *encoder, + enum drm_bridge_attach_flags flags) { if (!(flags & DRM_BRIDGE_ATTACH_NO_CONNECTOR)) return -EINVAL; =20 return 0; diff --git a/drivers/gpu/drm/bridge/imx/imx8mp-hdmi-pvi.c b/drivers/gpu/drm= /bridge/imx/imx8mp-hdmi-pvi.c index a17433a7c75530668662ed8dd1526b44f9a32f32..8a4fd7d77a8d516b3b46f41cf07= d2633d23bde12 100644 --- a/drivers/gpu/drm/bridge/imx/imx8mp-hdmi-pvi.c +++ b/drivers/gpu/drm/bridge/imx/imx8mp-hdmi-pvi.c @@ -38,15 +38,16 @@ to_imx8mp_hdmi_pvi(struct drm_bridge *bridge) { return container_of(bridge, struct imx8mp_hdmi_pvi, bridge); } =20 static int imx8mp_hdmi_pvi_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct imx8mp_hdmi_pvi *pvi =3D to_imx8mp_hdmi_pvi(bridge); =20 - return drm_bridge_attach(bridge->encoder, pvi->next_bridge, + return drm_bridge_attach(encoder, pvi->next_bridge, bridge, flags); } =20 static void imx8mp_hdmi_pvi_bridge_enable(struct drm_bridge *bridge, struct drm_atomic_state *state) diff --git a/drivers/gpu/drm/bridge/imx/imx8qxp-pixel-combiner.c b/drivers/= gpu/drm/bridge/imx/imx8qxp-pixel-combiner.c index 1d9529dc7f2a93e7547ae1f4c6193598a5574135..1f6fd488e7039e943351006d337= 3009f0c15cb08 100644 --- a/drivers/gpu/drm/bridge/imx/imx8qxp-pixel-combiner.c +++ b/drivers/gpu/drm/bridge/imx/imx8qxp-pixel-combiner.c @@ -106,10 +106,11 @@ imx8qxp_pc_bridge_mode_valid(struct drm_bridge *bridg= e, =20 return MODE_OK; } =20 static int imx8qxp_pc_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct imx8qxp_pc_channel *ch =3D bridge->driver_private; struct imx8qxp_pc *pc =3D ch->pc; =20 @@ -117,11 +118,11 @@ static int imx8qxp_pc_bridge_attach(struct drm_bridge= *bridge, DRM_DEV_ERROR(pc->dev, "do not support creating a drm_connector\n"); return -EINVAL; } =20 - return drm_bridge_attach(bridge->encoder, + return drm_bridge_attach(encoder, ch->next_bridge, bridge, DRM_BRIDGE_ATTACH_NO_CONNECTOR); } =20 static void diff --git a/drivers/gpu/drm/bridge/imx/imx8qxp-pixel-link.c b/drivers/gpu/= drm/bridge/imx/imx8qxp-pixel-link.c index cd6818db0fd332cac03241917bbb107b7261097c..e092c9ea99b0224802919ff84b4= 48acb53508951 100644 --- a/drivers/gpu/drm/bridge/imx/imx8qxp-pixel-link.c +++ b/drivers/gpu/drm/bridge/imx/imx8qxp-pixel-link.c @@ -126,21 +126,22 @@ static void imx8qxp_pixel_link_set_mst_addr(struct im= x8qxp_pixel_link *pl) "failed to set DC%u stream%u pixel link mst addr(%u): %d\n", pl->dc_id, pl->stream_id, pl->mst_addr, ret); } =20 static int imx8qxp_pixel_link_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct imx8qxp_pixel_link *pl =3D bridge->driver_private; =20 if (!(flags & DRM_BRIDGE_ATTACH_NO_CONNECTOR)) { DRM_DEV_ERROR(pl->dev, "do not support creating a drm_connector\n"); return -EINVAL; } =20 - return drm_bridge_attach(bridge->encoder, + return drm_bridge_attach(encoder, pl->next_bridge, bridge, DRM_BRIDGE_ATTACH_NO_CONNECTOR); } =20 static void diff --git a/drivers/gpu/drm/bridge/imx/imx8qxp-pxl2dpi.c b/drivers/gpu/drm= /bridge/imx/imx8qxp-pxl2dpi.c index 49dd4f96d52c966d9a038b0b04007631d0837f4d..da138ab51b3bd2c6ebd3780c098= 18891d5320092 100644 --- a/drivers/gpu/drm/bridge/imx/imx8qxp-pxl2dpi.c +++ b/drivers/gpu/drm/bridge/imx/imx8qxp-pxl2dpi.c @@ -46,21 +46,22 @@ struct imx8qxp_pxl2dpi { }; =20 #define bridge_to_p2d(b) container_of(b, struct imx8qxp_pxl2dpi, bridge) =20 static int imx8qxp_pxl2dpi_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct imx8qxp_pxl2dpi *p2d =3D bridge->driver_private; =20 if (!(flags & DRM_BRIDGE_ATTACH_NO_CONNECTOR)) { DRM_DEV_ERROR(p2d->dev, "do not support creating a drm_connector\n"); return -EINVAL; } =20 - return drm_bridge_attach(bridge->encoder, + return drm_bridge_attach(encoder, p2d->next_bridge, bridge, DRM_BRIDGE_ATTACH_NO_CONNECTOR); } =20 static int diff --git a/drivers/gpu/drm/bridge/ite-it6263.c b/drivers/gpu/drm/bridge/i= te-it6263.c index 12dcd569972a6eea8283a3b9b9ad4981f1133a5b..c28e0629e63afa183a2f285f92a= bb03fed9b8849 100644 --- a/drivers/gpu/drm/bridge/ite-it6263.c +++ b/drivers/gpu/drm/bridge/ite-it6263.c @@ -663,33 +663,34 @@ it6263_bridge_mode_valid(struct drm_bridge *bridge, =20 return bridge->funcs->hdmi_tmds_char_rate_valid(bridge, mode, rate); } =20 static int it6263_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct it6263 *it =3D bridge_to_it6263(bridge); struct drm_connector *connector; int ret; =20 - ret =3D drm_bridge_attach(bridge->encoder, it->next_bridge, bridge, + ret =3D drm_bridge_attach(encoder, it->next_bridge, bridge, flags | DRM_BRIDGE_ATTACH_NO_CONNECTOR); if (ret < 0) return ret; =20 if (flags & DRM_BRIDGE_ATTACH_NO_CONNECTOR) return 0; =20 - connector =3D drm_bridge_connector_init(bridge->dev, bridge->encoder); + connector =3D drm_bridge_connector_init(bridge->dev, encoder); if (IS_ERR(connector)) { ret =3D PTR_ERR(connector); dev_err(it->dev, "failed to initialize bridge connector: %d\n", ret); return ret; } =20 - drm_connector_attach_encoder(connector, bridge->encoder); + drm_connector_attach_encoder(connector, encoder); =20 return 0; } =20 static enum drm_connector_status it6263_bridge_detect(struct drm_bridge *b= ridge) diff --git a/drivers/gpu/drm/bridge/ite-it6505.c b/drivers/gpu/drm/bridge/i= te-it6505.c index 8573b33f823730cacea2dae11ce82c0268999aa4..a0f8c5c037aa259c6edba67e2c2= 5a2ddc55a3b4c 100644 --- a/drivers/gpu/drm/bridge/ite-it6505.c +++ b/drivers/gpu/drm/bridge/ite-it6505.c @@ -3121,10 +3121,11 @@ static inline struct it6505 *bridge_to_it6505(struc= t drm_bridge *bridge) { return container_of(bridge, struct it6505, bridge); } =20 static int it6505_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct it6505 *it6505 =3D bridge_to_it6505(bridge); struct device *dev =3D it6505->dev; int ret; diff --git a/drivers/gpu/drm/bridge/ite-it66121.c b/drivers/gpu/drm/bridge/= ite-it66121.c index b9f90f32145de0f17eadfdf0efbe95a2b5ca026d..7b110ae532918d2d6f91ebc5f74= 7c38e7e77dc07 100644 --- a/drivers/gpu/drm/bridge/ite-it66121.c +++ b/drivers/gpu/drm/bridge/ite-it66121.c @@ -584,19 +584,20 @@ static bool it66121_is_hpd_detect(struct it66121_ctx = *ctx) =20 return val & IT66121_SYS_STATUS_HPDETECT; } =20 static int it66121_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct it66121_ctx *ctx =3D container_of(bridge, struct it66121_ctx, brid= ge); int ret; =20 if (!(flags & DRM_BRIDGE_ATTACH_NO_CONNECTOR)) return -EINVAL; =20 - ret =3D drm_bridge_attach(bridge->encoder, ctx->next_bridge, bridge, flag= s); + ret =3D drm_bridge_attach(encoder, ctx->next_bridge, bridge, flags); if (ret) return ret; =20 if (ctx->info->id =3D=3D ID_IT66121) { ret =3D regmap_write_bits(ctx->regmap, IT66121_CLK_BANK_REG, diff --git a/drivers/gpu/drm/bridge/lontium-lt8912b.c b/drivers/gpu/drm/bri= dge/lontium-lt8912b.c index 52da204f57404e63c7eadb4318b55082ae732105..3e49d855b3648880cea9bce5f3f= 04fbb6f838a45 100644 --- a/drivers/gpu/drm/bridge/lontium-lt8912b.c +++ b/drivers/gpu/drm/bridge/lontium-lt8912b.c @@ -541,16 +541,17 @@ static int lt8912_bridge_connector_init(struct drm_br= idge *bridge) exit: return ret; } =20 static int lt8912_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct lt8912 *lt =3D bridge_to_lt8912(bridge); int ret; =20 - ret =3D drm_bridge_attach(bridge->encoder, lt->hdmi_port, bridge, + ret =3D drm_bridge_attach(encoder, lt->hdmi_port, bridge, DRM_BRIDGE_ATTACH_NO_CONNECTOR); if (ret < 0) { dev_err(lt->dev, "Failed to attach next bridge (%d)\n", ret); return ret; } diff --git a/drivers/gpu/drm/bridge/lontium-lt9211.c b/drivers/gpu/drm/brid= ge/lontium-lt9211.c index 0fc5ea18fe6a8d0c1578dcae960d78cc33f4fb6d..9b2dac9bd63c5afd4ffbafafdbb= b1230549bc36f 100644 --- a/drivers/gpu/drm/bridge/lontium-lt9211.c +++ b/drivers/gpu/drm/bridge/lontium-lt9211.c @@ -97,15 +97,16 @@ static struct lt9211 *bridge_to_lt9211(struct drm_bridg= e *bridge) { return container_of(bridge, struct lt9211, bridge); } =20 static int lt9211_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct lt9211 *ctx =3D bridge_to_lt9211(bridge); =20 - return drm_bridge_attach(bridge->encoder, ctx->panel_bridge, + return drm_bridge_attach(encoder, ctx->panel_bridge, &ctx->bridge, flags); } =20 static int lt9211_read_chipid(struct lt9211 *ctx) { diff --git a/drivers/gpu/drm/bridge/lontium-lt9611.c b/drivers/gpu/drm/brid= ge/lontium-lt9611.c index 026803034231f78c17f619dc04119bdd9b2b6679..53987e826ccd3368e94a1d75633= 54d359ebcbb17 100644 --- a/drivers/gpu/drm/bridge/lontium-lt9611.c +++ b/drivers/gpu/drm/bridge/lontium-lt9611.c @@ -738,15 +738,16 @@ static struct mipi_dsi_device *lt9611_attach_dsi(stru= ct lt9611 *lt9611, =20 return dsi; } =20 static int lt9611_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct lt9611 *lt9611 =3D bridge_to_lt9611(bridge); =20 - return drm_bridge_attach(bridge->encoder, lt9611->next_bridge, + return drm_bridge_attach(encoder, lt9611->next_bridge, bridge, flags); } =20 static enum drm_mode_status lt9611_bridge_mode_valid(struct drm_bridge *br= idge, const struct drm_display_info *info, diff --git a/drivers/gpu/drm/bridge/lontium-lt9611uxc.c b/drivers/gpu/drm/b= ridge/lontium-lt9611uxc.c index f4c3ff1fdc6923eb7a8c0d8f7f92e7649c797d77..20bf1a3c786d8ecc10ac58225f2= 8e5c519fd1bf9 100644 --- a/drivers/gpu/drm/bridge/lontium-lt9611uxc.c +++ b/drivers/gpu/drm/bridge/lontium-lt9611uxc.c @@ -278,15 +278,16 @@ static struct mipi_dsi_device *lt9611uxc_attach_dsi(s= truct lt9611uxc *lt9611uxc, =20 return dsi; } =20 static int lt9611uxc_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct lt9611uxc *lt9611uxc =3D bridge_to_lt9611uxc(bridge); =20 - return drm_bridge_attach(bridge->encoder, lt9611uxc->next_bridge, + return drm_bridge_attach(encoder, lt9611uxc->next_bridge, bridge, flags); } =20 static enum drm_mode_status lt9611uxc_bridge_mode_valid(struct drm_bridge *bridge, diff --git a/drivers/gpu/drm/bridge/lvds-codec.c b/drivers/gpu/drm/bridge/l= vds-codec.c index 389af0233fcdec9af7a4fbfe8d267170fbc4759c..1646e454e0b0b558d00f9421f15= bb7084e2aa45a 100644 --- a/drivers/gpu/drm/bridge/lvds-codec.c +++ b/drivers/gpu/drm/bridge/lvds-codec.c @@ -32,15 +32,16 @@ static inline struct lvds_codec *to_lvds_codec(struct d= rm_bridge *bridge) { return container_of(bridge, struct lvds_codec, bridge); } =20 static int lvds_codec_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct lvds_codec *lvds_codec =3D to_lvds_codec(bridge); =20 - return drm_bridge_attach(bridge->encoder, lvds_codec->panel_bridge, + return drm_bridge_attach(encoder, lvds_codec->panel_bridge, bridge, flags); } =20 static void lvds_codec_enable(struct drm_bridge *bridge) { diff --git a/drivers/gpu/drm/bridge/megachips-stdpxxxx-ge-b850v3-fw.c b/dri= vers/gpu/drm/bridge/megachips-stdpxxxx-ge-b850v3-fw.c index a47aabf134fd5c78448eae50b14a5ec322e6d6d0..15a5a1f644fc10182c55bc9e489= ccb81d4f924f9 100644 --- a/drivers/gpu/drm/bridge/megachips-stdpxxxx-ge-b850v3-fw.c +++ b/drivers/gpu/drm/bridge/megachips-stdpxxxx-ge-b850v3-fw.c @@ -188,10 +188,11 @@ static irqreturn_t ge_b850v3_lvds_irq_handler(int irq= , void *dev_id) =20 return IRQ_HANDLED; } =20 static int ge_b850v3_lvds_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct i2c_client *stdp4028_i2c =3D ge_b850v3_lvds_ptr->stdp4028_i2c; =20 diff --git a/drivers/gpu/drm/bridge/microchip-lvds.c b/drivers/gpu/drm/brid= ge/microchip-lvds.c index b8313dad60725b3e64519d1131d33c8485f01b77..55e89ba54b132922f42b62aef9c= 97b8a4d2a5cf0 100644 --- a/drivers/gpu/drm/bridge/microchip-lvds.c +++ b/drivers/gpu/drm/bridge/microchip-lvds.c @@ -102,15 +102,16 @@ static void lvds_serialiser_on(struct mchp_lvds *lvds) /* Enable the LVDS serializer */ lvds_writel(lvds, LVDSC_CR, LVDSC_CR_SER_EN); } =20 static int mchp_lvds_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct mchp_lvds *lvds =3D bridge_to_lvds(bridge); =20 - return drm_bridge_attach(bridge->encoder, lvds->panel_bridge, + return drm_bridge_attach(encoder, lvds->panel_bridge, bridge, flags); } =20 static void mchp_lvds_enable(struct drm_bridge *bridge) { diff --git a/drivers/gpu/drm/bridge/nwl-dsi.c b/drivers/gpu/drm/bridge/nwl-= dsi.c index 09b8b2e7d9fb358b5fb11d9646cf9c2f13221f3c..2a536c61c92b89ee2f332daa944= dfff974ba18d2 100644 --- a/drivers/gpu/drm/bridge/nwl-dsi.c +++ b/drivers/gpu/drm/bridge/nwl-dsi.c @@ -908,20 +908,21 @@ static void nwl_dsi_bridge_atomic_enable(struct drm_b= ridge *bridge, if (ret < 0) DRM_DEV_ERROR(dsi->dev, "Failed to deassert DPI: %d\n", ret); } =20 static int nwl_dsi_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct nwl_dsi *dsi =3D bridge_to_dsi(bridge); struct drm_bridge *panel_bridge; =20 panel_bridge =3D devm_drm_of_get_bridge(dsi->dev, dsi->dev->of_node, 1, 0= ); if (IS_ERR(panel_bridge)) return PTR_ERR(panel_bridge); =20 - return drm_bridge_attach(bridge->encoder, panel_bridge, bridge, flags); + return drm_bridge_attach(encoder, panel_bridge, bridge, flags); } =20 static u32 *nwl_bridge_atomic_get_input_bus_fmts(struct drm_bridge *bridge, struct drm_bridge_state *bridge_state, struct drm_crtc_state *crtc_state, diff --git a/drivers/gpu/drm/bridge/nxp-ptn3460.c b/drivers/gpu/drm/bridge/= nxp-ptn3460.c index 44e36ae66db487c871712432eecfcb0d68a28da4..fa1edeee159a5c92f2f607539b4= 24bda7b7120c5 100644 --- a/drivers/gpu/drm/bridge/nxp-ptn3460.c +++ b/drivers/gpu/drm/bridge/nxp-ptn3460.c @@ -213,17 +213,18 @@ static const struct drm_connector_funcs ptn3460_conne= ctor_funcs =3D { .atomic_duplicate_state =3D drm_atomic_helper_connector_duplicate_state, .atomic_destroy_state =3D drm_atomic_helper_connector_destroy_state, }; =20 static int ptn3460_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct ptn3460_bridge *ptn_bridge =3D bridge_to_ptn3460(bridge); int ret; =20 /* Let this driver create connector if requested */ - ret =3D drm_bridge_attach(bridge->encoder, ptn_bridge->panel_bridge, + ret =3D drm_bridge_attach(encoder, ptn_bridge->panel_bridge, bridge, flags | DRM_BRIDGE_ATTACH_NO_CONNECTOR); if (ret < 0) return ret; =20 if (flags & DRM_BRIDGE_ATTACH_NO_CONNECTOR) @@ -238,11 +239,11 @@ static int ptn3460_bridge_attach(struct drm_bridge *b= ridge, } drm_connector_helper_add(&ptn_bridge->connector, &ptn3460_connector_helper_funcs); drm_connector_register(&ptn_bridge->connector); drm_connector_attach_encoder(&ptn_bridge->connector, - bridge->encoder); + encoder); =20 drm_helper_hpd_irq_event(ptn_bridge->connector.dev); =20 return ret; } diff --git a/drivers/gpu/drm/bridge/panel.c b/drivers/gpu/drm/bridge/panel.c index 890bd45c46eda8366e44974e3ec5891aa473fd00..41eac98c29d7592110a1e2f4a60= b447586fbffd6 100644 --- a/drivers/gpu/drm/bridge/panel.c +++ b/drivers/gpu/drm/bridge/panel.c @@ -56,10 +56,11 @@ static const struct drm_connector_funcs panel_bridge_co= nnector_funcs =3D { .atomic_duplicate_state =3D drm_atomic_helper_connector_duplicate_state, .atomic_destroy_state =3D drm_atomic_helper_connector_destroy_state, }; =20 static int panel_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct panel_bridge *panel_bridge =3D drm_bridge_to_panel_bridge(bridge); struct drm_connector *connector =3D &panel_bridge->connector; int ret; @@ -79,11 +80,11 @@ static int panel_bridge_attach(struct drm_bridge *bridg= e, } =20 drm_panel_bridge_set_orientation(connector, bridge); =20 drm_connector_attach_encoder(&panel_bridge->connector, - bridge->encoder); + encoder); =20 if (bridge->dev->registered) { if (connector->funcs->reset) connector->funcs->reset(connector); drm_connector_register(connector); diff --git a/drivers/gpu/drm/bridge/parade-ps8622.c b/drivers/gpu/drm/bridg= e/parade-ps8622.c index ae3ab9262ef1f184873929c7892d648a32c7cf37..68a3a324f7f8eee05a2d0c0a52c= 82258bca1e118 100644 --- a/drivers/gpu/drm/bridge/parade-ps8622.c +++ b/drivers/gpu/drm/bridge/parade-ps8622.c @@ -417,10 +417,11 @@ static void ps8622_post_disable(struct drm_bridge *br= idge) =20 msleep(PS8622_POWER_OFF_T17_MS); } =20 static int ps8622_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct ps8622_bridge *ps8622 =3D bridge_to_ps8622(bridge); =20 return drm_bridge_attach(ps8622->bridge.encoder, ps8622->panel_bridge, diff --git a/drivers/gpu/drm/bridge/parade-ps8640.c b/drivers/gpu/drm/bridg= e/parade-ps8640.c index 0f9a0639550bd25bba0a7175e1a86e839a7687fe..82263ca10aeed7fb3ca894de4e4= dc57c75cb66f9 100644 --- a/drivers/gpu/drm/bridge/parade-ps8640.c +++ b/drivers/gpu/drm/bridge/parade-ps8640.c @@ -493,10 +493,11 @@ static void ps8640_atomic_post_disable(struct drm_bri= dge *bridge, =20 mutex_unlock(&ps_bridge->aux_lock); } =20 static int ps8640_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct ps8640 *ps_bridge =3D bridge_to_ps8640(bridge); struct device *dev =3D &ps_bridge->page[0]->dev; int ret; @@ -517,11 +518,11 @@ static int ps8640_bridge_attach(struct drm_bridge *br= idge, ret =3D -EINVAL; goto err_devlink; } =20 /* Attach the panel-bridge to the dsi bridge */ - ret =3D drm_bridge_attach(bridge->encoder, ps_bridge->panel_bridge, + ret =3D drm_bridge_attach(encoder, ps_bridge->panel_bridge, &ps_bridge->bridge, flags); if (ret) goto err_bridge_attach; =20 return 0; diff --git a/drivers/gpu/drm/bridge/samsung-dsim.c b/drivers/gpu/drm/bridge= /samsung-dsim.c index 54de6ed2fae81bc13301a6b1ee8f38183a3118b6..55ac6bd5da08c037aa7974df679= d51e72bc54faf 100644 --- a/drivers/gpu/drm/bridge/samsung-dsim.c +++ b/drivers/gpu/drm/bridge/samsung-dsim.c @@ -1638,15 +1638,16 @@ static void samsung_dsim_mode_set(struct drm_bridge= *bridge, =20 drm_mode_copy(&dsi->mode, adjusted_mode); } =20 static int samsung_dsim_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct samsung_dsim *dsi =3D bridge_to_dsi(bridge); =20 - return drm_bridge_attach(bridge->encoder, dsi->out_bridge, bridge, + return drm_bridge_attach(encoder, dsi->out_bridge, bridge, flags); } =20 static const struct drm_bridge_funcs samsung_dsim_bridge_funcs =3D { .atomic_duplicate_state =3D drm_atomic_helper_bridge_duplicate_state, diff --git a/drivers/gpu/drm/bridge/sii902x.c b/drivers/gpu/drm/bridge/sii9= 02x.c index 2100a687096e50c83a7c554e6d21b3c638ef713b..4d2fca64c50b4d7f7bc7f6f0edc= af47aa5e3d1f3 100644 --- a/drivers/gpu/drm/bridge/sii902x.c +++ b/drivers/gpu/drm/bridge/sii902x.c @@ -414,19 +414,20 @@ static void sii902x_bridge_mode_set(struct drm_bridge= *bridge, out: mutex_unlock(&sii902x->mutex); } =20 static int sii902x_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct sii902x *sii902x =3D bridge_to_sii902x(bridge); u32 bus_format =3D MEDIA_BUS_FMT_RGB888_1X24; struct drm_device *drm =3D bridge->dev; int ret; =20 if (flags & DRM_BRIDGE_ATTACH_NO_CONNECTOR) - return drm_bridge_attach(bridge->encoder, sii902x->next_bridge, + return drm_bridge_attach(encoder, sii902x->next_bridge, bridge, flags); =20 drm_connector_helper_add(&sii902x->connector, &sii902x_connector_helper_funcs); =20 @@ -450,11 +451,11 @@ static int sii902x_bridge_attach(struct drm_bridge *b= ridge, ret =3D drm_display_info_set_bus_formats(&sii902x->connector.display_info, &bus_format, 1); if (ret) return ret; =20 - drm_connector_attach_encoder(&sii902x->connector, bridge->encoder); + drm_connector_attach_encoder(&sii902x->connector, encoder); =20 return 0; } =20 static enum drm_connector_status sii902x_bridge_detect(struct drm_bridge *= bridge) diff --git a/drivers/gpu/drm/bridge/sil-sii8620.c b/drivers/gpu/drm/bridge/= sil-sii8620.c index 28a2e1ee04b2828364c6e633399b821562fc8728..3af650dc92a1612ca88fe378319= 519546b79901f 100644 --- a/drivers/gpu/drm/bridge/sil-sii8620.c +++ b/drivers/gpu/drm/bridge/sil-sii8620.c @@ -2201,10 +2201,11 @@ static inline struct sii8620 *bridge_to_sii8620(str= uct drm_bridge *bridge) { return container_of(bridge, struct sii8620, bridge); } =20 static int sii8620_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct sii8620 *ctx =3D bridge_to_sii8620(bridge); =20 sii8620_init_rcp_input_dev(ctx); diff --git a/drivers/gpu/drm/bridge/simple-bridge.c b/drivers/gpu/drm/bridg= e/simple-bridge.c index ab0b0e36e97ac244427ac316acd9230c9490f2ce..70db5b99e5bb84f099ec54cf62a= bbda53475311d 100644 --- a/drivers/gpu/drm/bridge/simple-bridge.c +++ b/drivers/gpu/drm/bridge/simple-bridge.c @@ -101,16 +101,17 @@ static const struct drm_connector_funcs simple_bridge= _con_funcs =3D { .atomic_duplicate_state =3D drm_atomic_helper_connector_duplicate_state, .atomic_destroy_state =3D drm_atomic_helper_connector_destroy_state, }; =20 static int simple_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct simple_bridge *sbridge =3D drm_bridge_to_simple_bridge(bridge); int ret; =20 - ret =3D drm_bridge_attach(bridge->encoder, sbridge->next_bridge, bridge, + ret =3D drm_bridge_attach(encoder, sbridge->next_bridge, bridge, DRM_BRIDGE_ATTACH_NO_CONNECTOR); if (ret < 0) return ret; =20 if (flags & DRM_BRIDGE_ATTACH_NO_CONNECTOR) @@ -125,11 +126,11 @@ static int simple_bridge_attach(struct drm_bridge *br= idge, if (ret) { DRM_ERROR("Failed to initialize connector\n"); return ret; } =20 - drm_connector_attach_encoder(&sbridge->connector, bridge->encoder); + drm_connector_attach_encoder(&sbridge->connector, encoder); =20 return 0; } =20 static void simple_bridge_enable(struct drm_bridge *bridge) diff --git a/drivers/gpu/drm/bridge/synopsys/dw-hdmi.c b/drivers/gpu/drm/br= idge/synopsys/dw-hdmi.c index 612b81c79f4a1ffe5bc4adf2502c20adea45d922..39db1284344cfb6f2567a226d19= f93982f80ff2a 100644 --- a/drivers/gpu/drm/bridge/synopsys/dw-hdmi.c +++ b/drivers/gpu/drm/bridge/synopsys/dw-hdmi.c @@ -2887,16 +2887,17 @@ static int dw_hdmi_bridge_atomic_check(struct drm_b= ridge *bridge, =20 return 0; } =20 static int dw_hdmi_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct dw_hdmi *hdmi =3D bridge->driver_private; =20 if (flags & DRM_BRIDGE_ATTACH_NO_CONNECTOR) - return drm_bridge_attach(bridge->encoder, hdmi->next_bridge, + return drm_bridge_attach(encoder, hdmi->next_bridge, bridge, flags); =20 return dw_hdmi_connector_create(hdmi); } =20 diff --git a/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c b/drivers/gpu/dr= m/bridge/synopsys/dw-mipi-dsi.c index 2b6e70a49f438c475bc60d9595bf246a9e54995b..b08ada920a501d6a62f39581944= a87019f5e5c15 100644 --- a/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c +++ b/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c @@ -1070,19 +1070,20 @@ dw_mipi_dsi_bridge_mode_valid(struct drm_bridge *br= idge, =20 return mode_status; } =20 static int dw_mipi_dsi_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct dw_mipi_dsi *dsi =3D bridge_to_dsi(bridge); =20 /* Set the encoder type as caller does not know it */ - bridge->encoder->encoder_type =3D DRM_MODE_ENCODER_DSI; + encoder->encoder_type =3D DRM_MODE_ENCODER_DSI; =20 /* Attach the panel-bridge to the dsi bridge */ - return drm_bridge_attach(bridge->encoder, dsi->panel_bridge, bridge, + return drm_bridge_attach(encoder, dsi->panel_bridge, bridge, flags); } =20 static const struct drm_bridge_funcs dw_mipi_dsi_bridge_funcs =3D { .atomic_duplicate_state =3D drm_atomic_helper_bridge_duplicate_state, diff --git a/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi2.c b/drivers/gpu/d= rm/bridge/synopsys/dw-mipi-dsi2.c index 5fd7a459efdd4fc4884cdf77ffedc19c36d1ca13..c76f5f2e74d14bd372f969c6c78= 32aa57f80772b 100644 --- a/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi2.c +++ b/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi2.c @@ -868,19 +868,20 @@ dw_mipi_dsi2_bridge_mode_valid(struct drm_bridge *bri= dge, =20 return mode_status; } =20 static int dw_mipi_dsi2_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct dw_mipi_dsi2 *dsi2 =3D bridge_to_dsi2(bridge); =20 /* Set the encoder type as caller does not know it */ - bridge->encoder->encoder_type =3D DRM_MODE_ENCODER_DSI; + encoder->encoder_type =3D DRM_MODE_ENCODER_DSI; =20 /* Attach the panel-bridge to the dsi bridge */ - return drm_bridge_attach(bridge->encoder, dsi2->panel_bridge, bridge, + return drm_bridge_attach(encoder, dsi2->panel_bridge, bridge, flags); } =20 static const struct drm_bridge_funcs dw_mipi_dsi2_bridge_funcs =3D { .atomic_duplicate_state =3D drm_atomic_helper_bridge_duplicate_state, diff --git a/drivers/gpu/drm/bridge/tc358762.c b/drivers/gpu/drm/bridge/tc3= 58762.c index 6526c9685758ac57bd53c16551127074fb22b5b7..c7e7c3e40f9e0df070742097df5= 1cabbc3a590b5 100644 --- a/drivers/gpu/drm/bridge/tc358762.c +++ b/drivers/gpu/drm/bridge/tc358762.c @@ -200,15 +200,16 @@ static void tc358762_enable(struct drm_bridge *bridge, if (ret < 0) dev_err(ctx->dev, "error initializing bridge (%d)\n", ret); } =20 static int tc358762_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct tc358762 *ctx =3D bridge_to_tc358762(bridge); =20 - return drm_bridge_attach(bridge->encoder, ctx->panel_bridge, + return drm_bridge_attach(encoder, ctx->panel_bridge, bridge, flags); } =20 static void tc358762_bridge_mode_set(struct drm_bridge *bridge, const struct drm_display_mode *mode, diff --git a/drivers/gpu/drm/bridge/tc358764.c b/drivers/gpu/drm/bridge/tc3= 58764.c index 3d3d135b43487aaf5a1908c21c1c988e87d24d48..3f76c890fad9ffa50dc02f289a3= 7378332830516 100644 --- a/drivers/gpu/drm/bridge/tc358764.c +++ b/drivers/gpu/drm/bridge/tc358764.c @@ -293,15 +293,16 @@ static void tc358764_pre_enable(struct drm_bridge *br= idge) if (ret < 0) dev_err(ctx->dev, "error initializing bridge (%d)\n", ret); } =20 static int tc358764_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct tc358764 *ctx =3D bridge_to_tc358764(bridge); =20 - return drm_bridge_attach(bridge->encoder, ctx->next_bridge, bridge, flags= ); + return drm_bridge_attach(encoder, ctx->next_bridge, bridge, flags); } =20 static const struct drm_bridge_funcs tc358764_bridge_funcs =3D { .post_disable =3D tc358764_post_disable, .pre_enable =3D tc358764_pre_enable, diff --git a/drivers/gpu/drm/bridge/tc358767.c b/drivers/gpu/drm/bridge/tc3= 58767.c index 39e2d3a7a27d2603bb847d55c4812fb170389ba0..7e5449fb86a3fcdae8255bc490d= 12c543ef3f8ae 100644 --- a/drivers/gpu/drm/bridge/tc358767.c +++ b/drivers/gpu/drm/bridge/tc358767.c @@ -1793,10 +1793,11 @@ static const struct drm_connector_funcs tc_connecto= r_funcs =3D { .atomic_duplicate_state =3D drm_atomic_helper_connector_duplicate_state, .atomic_destroy_state =3D drm_atomic_helper_connector_destroy_state, }; =20 static int tc_dpi_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct tc_data *tc =3D bridge_to_tc(bridge); =20 if (!tc->panel_bridge) @@ -1805,10 +1806,11 @@ static int tc_dpi_bridge_attach(struct drm_bridge *= bridge, return drm_bridge_attach(tc->bridge.encoder, tc->panel_bridge, &tc->bridge, flags); } =20 static int tc_edp_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { u32 bus_format =3D MEDIA_BUS_FMT_RGB888_1X24; struct tc_data *tc =3D bridge_to_tc(bridge); struct drm_device *drm =3D bridge->dev; diff --git a/drivers/gpu/drm/bridge/tc358768.c b/drivers/gpu/drm/bridge/tc3= 58768.c index ec79b0dd0e2cd6dfbdd8aee54d6a8d9cf8bfb060..6db18d1e8824dd7d387211d6d1e= 668645cf88bbe 100644 --- a/drivers/gpu/drm/bridge/tc358768.c +++ b/drivers/gpu/drm/bridge/tc358768.c @@ -552,20 +552,21 @@ static const struct mipi_dsi_host_ops tc358768_dsi_ho= st_ops =3D { .detach =3D tc358768_dsi_host_detach, .transfer =3D tc358768_dsi_host_transfer, }; =20 static int tc358768_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct tc358768_priv *priv =3D bridge_to_tc358768(bridge); =20 if (!drm_core_check_feature(bridge->dev, DRIVER_ATOMIC)) { dev_err(priv->dev, "needs atomic updates support\n"); return -ENOTSUPP; } =20 - return drm_bridge_attach(bridge->encoder, priv->output.bridge, bridge, + return drm_bridge_attach(encoder, priv->output.bridge, bridge, flags); } =20 static enum drm_mode_status tc358768_bridge_mode_valid(struct drm_bridge *bridge, diff --git a/drivers/gpu/drm/bridge/tc358775.c b/drivers/gpu/drm/bridge/tc3= 58775.c index 0b4efaca6d682320b76ce09ed41824ae7f84ca2d..4963b25ff1b0b97d08124fdba8e= f5f94c3d7f401 100644 --- a/drivers/gpu/drm/bridge/tc358775.c +++ b/drivers/gpu/drm/bridge/tc358775.c @@ -588,16 +588,17 @@ static int tc358775_parse_dt(struct device_node *np, = struct tc_data *tc) =20 return 0; } =20 static int tc_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct tc_data *tc =3D bridge_to_tc(bridge); =20 /* Attach the panel-bridge to the dsi bridge */ - return drm_bridge_attach(bridge->encoder, tc->panel_bridge, + return drm_bridge_attach(encoder, tc->panel_bridge, &tc->bridge, flags); } =20 static const struct drm_bridge_funcs tc_bridge_funcs =3D { .attach =3D tc_bridge_attach, diff --git a/drivers/gpu/drm/bridge/thc63lvd1024.c b/drivers/gpu/drm/bridge= /thc63lvd1024.c index bba10cf9b4f96aef69f82403a7642d08c2b9f4b7..e2fc78adebcf22f0d8cdb484078= e37b748b776c6 100644 --- a/drivers/gpu/drm/bridge/thc63lvd1024.c +++ b/drivers/gpu/drm/bridge/thc63lvd1024.c @@ -41,15 +41,16 @@ static inline struct thc63_dev *to_thc63(struct drm_bri= dge *bridge) { return container_of(bridge, struct thc63_dev, bridge); } =20 static int thc63_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct thc63_dev *thc63 =3D to_thc63(bridge); =20 - return drm_bridge_attach(bridge->encoder, thc63->next, bridge, flags); + return drm_bridge_attach(encoder, thc63->next, bridge, flags); } =20 static enum drm_mode_status thc63_mode_valid(struct drm_bridge *bridge, const struct drm_display_info *info, const struct drm_display_mode *mode) diff --git a/drivers/gpu/drm/bridge/ti-dlpc3433.c b/drivers/gpu/drm/bridge/= ti-dlpc3433.c index 85f2a0e74a1c8aaa272ce828ad1c82660503b7a8..47638d1c96ec5ad999604c8c7e8= 839ff85936d98 100644 --- a/drivers/gpu/drm/bridge/ti-dlpc3433.c +++ b/drivers/gpu/drm/bridge/ti-dlpc3433.c @@ -240,16 +240,16 @@ static void dlpc_mode_set(struct drm_bridge *bridge, struct dlpc *dlpc =3D bridge_to_dlpc(bridge); =20 drm_mode_copy(&dlpc->mode, adjusted_mode); } =20 -static int dlpc_attach(struct drm_bridge *bridge, +static int dlpc_attach(struct drm_bridge *bridge, struct drm_encoder *enco= der, enum drm_bridge_attach_flags flags) { struct dlpc *dlpc =3D bridge_to_dlpc(bridge); =20 - return drm_bridge_attach(bridge->encoder, dlpc->next_bridge, bridge, flag= s); + return drm_bridge_attach(encoder, dlpc->next_bridge, bridge, flags); } =20 static const struct drm_bridge_funcs dlpc_bridge_funcs =3D { .atomic_duplicate_state =3D drm_atomic_helper_bridge_duplicate_state, .atomic_destroy_state =3D drm_atomic_helper_bridge_destroy_state, diff --git a/drivers/gpu/drm/bridge/ti-sn65dsi83.c b/drivers/gpu/drm/bridge= /ti-sn65dsi83.c index 4d9cbe966a9e728b35fc4a00c2898bd5eb9f455a..6da192c8675bb56a0d4741c9720= ece0eeb3df3e8 100644 --- a/drivers/gpu/drm/bridge/ti-sn65dsi83.c +++ b/drivers/gpu/drm/bridge/ti-sn65dsi83.c @@ -283,15 +283,16 @@ static struct sn65dsi83 *bridge_to_sn65dsi83(struct d= rm_bridge *bridge) { return container_of(bridge, struct sn65dsi83, bridge); } =20 static int sn65dsi83_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct sn65dsi83 *ctx =3D bridge_to_sn65dsi83(bridge); =20 - return drm_bridge_attach(bridge->encoder, ctx->panel_bridge, + return drm_bridge_attach(encoder, ctx->panel_bridge, &ctx->bridge, flags); } =20 static void sn65dsi83_detach(struct drm_bridge *bridge) { diff --git a/drivers/gpu/drm/bridge/ti-sn65dsi86.c b/drivers/gpu/drm/bridge= /ti-sn65dsi86.c index a78f80bbc65b8c5aed7aeae0cd15cc7d5210c5e5..497e7212d7719a03045d055b706= 9f9a10c7f3877 100644 --- a/drivers/gpu/drm/bridge/ti-sn65dsi86.c +++ b/drivers/gpu/drm/bridge/ti-sn65dsi86.c @@ -731,10 +731,11 @@ static int ti_sn_attach_host(struct auxiliary_device = *adev, struct ti_sn65dsi86 =20 return devm_mipi_dsi_attach(&adev->dev, dsi); } =20 static int ti_sn_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct ti_sn65dsi86 *pdata =3D bridge_to_ti_sn65dsi86(bridge); int ret; =20 @@ -747,11 +748,11 @@ static int ti_sn_bridge_attach(struct drm_bridge *bri= dge, =20 /* * Attach the next bridge. * We never want the next bridge to *also* create a connector. */ - ret =3D drm_bridge_attach(bridge->encoder, pdata->next_bridge, + ret =3D drm_bridge_attach(encoder, pdata->next_bridge, &pdata->bridge, flags | DRM_BRIDGE_ATTACH_NO_CONNECTOR); if (ret < 0) goto err_initted_aux; =20 if (flags & DRM_BRIDGE_ATTACH_NO_CONNECTOR) diff --git a/drivers/gpu/drm/bridge/ti-tdp158.c b/drivers/gpu/drm/bridge/ti= -tdp158.c index 22316382451fa3a55e1b3977982bd306b52b5888..cca75443f0121e39527a6cecf71= 1eff2c0d507b5 100644 --- a/drivers/gpu/drm/bridge/ti-tdp158.c +++ b/drivers/gpu/drm/bridge/ti-tdp158.c @@ -43,15 +43,17 @@ static void tdp158_disable(struct drm_bridge *bridge, gpiod_set_value_cansleep(tdp158->enable, 0); regulator_disable(tdp158->vdd); regulator_disable(tdp158->vcc); } =20 -static int tdp158_attach(struct drm_bridge *bridge, enum drm_bridge_attach= _flags flags) +static int tdp158_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, + enum drm_bridge_attach_flags flags) { struct tdp158 *tdp158 =3D bridge->driver_private; =20 - return drm_bridge_attach(bridge->encoder, tdp158->next, bridge, flags); + return drm_bridge_attach(encoder, tdp158->next, bridge, flags); } =20 static const struct drm_bridge_funcs tdp158_bridge_funcs =3D { .attach =3D tdp158_attach, .atomic_enable =3D tdp158_enable, diff --git a/drivers/gpu/drm/bridge/ti-tfp410.c b/drivers/gpu/drm/bridge/ti= -tfp410.c index 79ab5da827e1fd3850435a02daedad87186a4ab8..e15d232ddbac55c6f5f966471a6= c63f9c29a06c1 100644 --- a/drivers/gpu/drm/bridge/ti-tfp410.c +++ b/drivers/gpu/drm/bridge/ti-tfp410.c @@ -118,16 +118,17 @@ static void tfp410_hpd_callback(void *arg, enum drm_c= onnector_status status) mod_delayed_work(system_wq, &dvi->hpd_work, msecs_to_jiffies(HOTPLUG_DEBOUNCE_MS)); } =20 static int tfp410_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct tfp410 *dvi =3D drm_bridge_to_tfp410(bridge); int ret; =20 - ret =3D drm_bridge_attach(bridge->encoder, dvi->next_bridge, bridge, + ret =3D drm_bridge_attach(encoder, dvi->next_bridge, bridge, DRM_BRIDGE_ATTACH_NO_CONNECTOR); if (ret < 0) return ret; =20 if (flags & DRM_BRIDGE_ATTACH_NO_CONNECTOR) @@ -157,11 +158,11 @@ static int tfp410_attach(struct drm_bridge *bridge, } =20 drm_display_info_set_bus_formats(&dvi->connector.display_info, &dvi->bus_format, 1); =20 - drm_connector_attach_encoder(&dvi->connector, bridge->encoder); + drm_connector_attach_encoder(&dvi->connector, encoder); =20 return 0; } =20 static void tfp410_detach(struct drm_bridge *bridge) diff --git a/drivers/gpu/drm/bridge/ti-tpd12s015.c b/drivers/gpu/drm/bridge= /ti-tpd12s015.c index 47b74cb25b14fcf4a09137d8b5b7f4ff739eae59..1c289051a5987e0aec4c286ef4c= 01ee1a2f9421f 100644 --- a/drivers/gpu/drm/bridge/ti-tpd12s015.c +++ b/drivers/gpu/drm/bridge/ti-tpd12s015.c @@ -36,19 +36,20 @@ static inline struct tpd12s015_device *to_tpd12s015(str= uct drm_bridge *bridge) { return container_of(bridge, struct tpd12s015_device, bridge); } =20 static int tpd12s015_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct tpd12s015_device *tpd =3D to_tpd12s015(bridge); int ret; =20 if (!(flags & DRM_BRIDGE_ATTACH_NO_CONNECTOR)) return -EINVAL; =20 - ret =3D drm_bridge_attach(bridge->encoder, tpd->next_bridge, + ret =3D drm_bridge_attach(encoder, tpd->next_bridge, bridge, flags); if (ret < 0) return ret; =20 gpiod_set_value_cansleep(tpd->ls_oe_gpio, 1); diff --git a/drivers/gpu/drm/drm_bridge.c b/drivers/gpu/drm/drm_bridge.c index 567400074d25aecd3174b24bae2f70ebea347071..d7c17a26e86fd9a3e5a39f335ed= ff4ffce30dd79 100644 --- a/drivers/gpu/drm/drm_bridge.c +++ b/drivers/gpu/drm/drm_bridge.c @@ -321,11 +321,11 @@ int drm_bridge_attach(struct drm_encoder *encoder, st= ruct drm_bridge *bridge, list_add(&bridge->chain_node, &previous->chain_node); else list_add(&bridge->chain_node, &encoder->bridge_chain); =20 if (bridge->funcs->attach) { - ret =3D bridge->funcs->attach(bridge, flags); + ret =3D bridge->funcs->attach(bridge, encoder, flags); if (ret < 0) goto err_reset_bridge; } =20 if (bridge->funcs->atomic_reset) { diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c b/drivers/gpu/drm/i2c/tda998= x_drv.c index 82d4a4e206a584e045a8fcd593fc12b7c1257bfe..a8669b27e916389da6f5ecc6e5a= d608fddb594a2 100644 --- a/drivers/gpu/drm/i2c/tda998x_drv.c +++ b/drivers/gpu/drm/i2c/tda998x_drv.c @@ -1362,10 +1362,11 @@ static int tda998x_connector_init(struct tda998x_pr= iv *priv, } =20 /* DRM bridge functions */ =20 static int tda998x_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct tda998x_priv *priv =3D bridge_to_tda998x_priv(bridge); =20 if (flags & DRM_BRIDGE_ATTACH_NO_CONNECTOR) { diff --git a/drivers/gpu/drm/imx/ipuv3/parallel-display.c b/drivers/gpu/drm= /imx/ipuv3/parallel-display.c index 9e66eb77b1ebe853599a60df69ce8db94fdffd49..6d8325c766979aa3ba98970f008= 06e99c139d3c3 100644 --- a/drivers/gpu/drm/imx/ipuv3/parallel-display.c +++ b/drivers/gpu/drm/imx/ipuv3/parallel-display.c @@ -160,15 +160,16 @@ static int imx_pd_bridge_atomic_check(struct drm_brid= ge *bridge, =20 return 0; } =20 static int imx_pd_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct imx_parallel_display *imxpd =3D bridge_to_imxpd(bridge); =20 - return drm_bridge_attach(bridge->encoder, imxpd->next_bridge, bridge, fla= gs); + return drm_bridge_attach(encoder, imxpd->next_bridge, bridge, flags); } =20 static const struct drm_bridge_funcs imx_pd_bridge_funcs =3D { .attach =3D imx_pd_bridge_attach, .atomic_reset =3D drm_atomic_helper_bridge_reset, diff --git a/drivers/gpu/drm/ingenic/ingenic-drm-drv.c b/drivers/gpu/drm/in= genic/ingenic-drm-drv.c index 20b93fff023995062096941ae571756185688387..f851e9ffdb280388452c9ad0883= 932f5b990d6f0 100644 --- a/drivers/gpu/drm/ingenic/ingenic-drm-drv.c +++ b/drivers/gpu/drm/ingenic/ingenic-drm-drv.c @@ -789,15 +789,16 @@ static void ingenic_drm_encoder_atomic_mode_set(struc= t drm_encoder *encoder, regmap_write(priv->map, JZ_REG_LCD_CFG, cfg); regmap_write(priv->map, JZ_REG_LCD_RGBC, rgbcfg); } =20 static int ingenic_drm_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { - struct ingenic_drm_bridge *ib =3D to_ingenic_drm_bridge(bridge->encoder); + struct ingenic_drm_bridge *ib =3D to_ingenic_drm_bridge(encoder); =20 - return drm_bridge_attach(bridge->encoder, ib->next_bridge, + return drm_bridge_attach(encoder, ib->next_bridge, &ib->bridge, flags); } =20 static int ingenic_drm_bridge_atomic_check(struct drm_bridge *bridge, struct drm_bridge_state *bridge_state, diff --git a/drivers/gpu/drm/mcde/mcde_dsi.c b/drivers/gpu/drm/mcde/mcde_ds= i.c index 395449a72f0a1dc0d8ee97ecd245ca52f0282402..b302d8ec3ad0e82ace9c10194ae= 37948ebb8e753 100644 --- a/drivers/gpu/drm/mcde/mcde_dsi.c +++ b/drivers/gpu/drm/mcde/mcde_dsi.c @@ -1046,10 +1046,11 @@ void mcde_dsi_disable(struct drm_bridge *bridge) clk_disable_unprepare(d->hs_clk); clk_disable_unprepare(d->lp_clk); } =20 static int mcde_dsi_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct mcde_dsi *d =3D bridge_to_mcde_dsi(bridge); struct drm_device *drm =3D bridge->dev; =20 @@ -1057,11 +1058,11 @@ static int mcde_dsi_bridge_attach(struct drm_bridge= *bridge, dev_err(d->dev, "we need atomic updates\n"); return -ENOTSUPP; } =20 /* Attach the DSI bridge to the output (panel etc) bridge */ - return drm_bridge_attach(bridge->encoder, d->bridge_out, bridge, flags); + return drm_bridge_attach(encoder, d->bridge_out, bridge, flags); } =20 static const struct drm_bridge_funcs mcde_dsi_bridge_funcs =3D { .attach =3D mcde_dsi_bridge_attach, .mode_set =3D mcde_dsi_bridge_mode_set, diff --git a/drivers/gpu/drm/mediatek/mtk_dp.c b/drivers/gpu/drm/mediatek/m= tk_dp.c index d3ffdbca48955a488e2fa5a9435deee970ab12a9..74286e0a26c9b6a763828312047= aedffb1d3b666 100644 --- a/drivers/gpu/drm/mediatek/mtk_dp.c +++ b/drivers/gpu/drm/mediatek/mtk_dp.c @@ -2263,10 +2263,11 @@ static void mtk_dp_poweroff(struct mtk_dp *mtk_dp) mtk_dp_power_disable(mtk_dp); phy_exit(mtk_dp->phy); } =20 static int mtk_dp_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct mtk_dp *mtk_dp =3D mtk_dp_from_bridge(bridge); int ret; =20 @@ -2286,11 +2287,11 @@ static int mtk_dp_bridge_attach(struct drm_bridge *= bridge, ret =3D mtk_dp_poweron(mtk_dp); if (ret) goto err_aux_register; =20 if (mtk_dp->next_bridge) { - ret =3D drm_bridge_attach(bridge->encoder, mtk_dp->next_bridge, + ret =3D drm_bridge_attach(encoder, mtk_dp->next_bridge, &mtk_dp->bridge, flags); if (ret) { drm_warn(mtk_dp->drm_dev, "Failed to attach external bridge: %d\n", ret); goto err_bridge_attach; diff --git a/drivers/gpu/drm/mediatek/mtk_dpi.c b/drivers/gpu/drm/mediatek/= mtk_dpi.c index 1864eb02dbf50af09c65064834a8b27ca56c3528..6b96ed4fc8614569e53f76fe3e5= 2507c7dc1249c 100644 --- a/drivers/gpu/drm/mediatek/mtk_dpi.c +++ b/drivers/gpu/drm/mediatek/mtk_dpi.c @@ -699,10 +699,11 @@ static int mtk_dpi_bridge_atomic_check(struct drm_bri= dge *bridge, =20 return 0; } =20 static int mtk_dpi_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct mtk_dpi *dpi =3D bridge_to_dpi(bridge); int ret; =20 @@ -717,11 +718,11 @@ static int mtk_dpi_bridge_attach(struct drm_bridge *b= ridge, if (IS_ERR(dpi->next_bridge)) return dev_err_probe(dpi->dev, PTR_ERR(dpi->next_bridge), "Failed to get bridge\n"); } =20 - return drm_bridge_attach(bridge->encoder, dpi->next_bridge, + return drm_bridge_attach(encoder, dpi->next_bridge, &dpi->bridge, flags); } =20 static void mtk_dpi_bridge_mode_set(struct drm_bridge *bridge, const struct drm_display_mode *mode, diff --git a/drivers/gpu/drm/mediatek/mtk_dsi.c b/drivers/gpu/drm/mediatek/= mtk_dsi.c index 40f2258a42dab0ededf09196423d95a88f21fc39..1086356a20e53c47b879e77eaf6= 2577e569143ed 100644 --- a/drivers/gpu/drm/mediatek/mtk_dsi.c +++ b/drivers/gpu/drm/mediatek/mtk_dsi.c @@ -804,16 +804,17 @@ static void mtk_output_dsi_disable(struct mtk_dsi *ds= i) =20 dsi->enabled =3D false; } =20 static int mtk_dsi_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct mtk_dsi *dsi =3D bridge_to_dsi(bridge); =20 /* Attach the panel or bridge to the dsi bridge */ - return drm_bridge_attach(bridge->encoder, dsi->next_bridge, + return drm_bridge_attach(encoder, dsi->next_bridge, &dsi->bridge, flags); } =20 static void mtk_dsi_bridge_mode_set(struct drm_bridge *bridge, const struct drm_display_mode *mode, diff --git a/drivers/gpu/drm/mediatek/mtk_hdmi.c b/drivers/gpu/drm/mediatek= /mtk_hdmi.c index ac5e40c2761710dfbe722e1ba569d76e4cd5b8fb..b24ebce8163ae6f0a876b53f1ec= 21348d5ac5286 100644 --- a/drivers/gpu/drm/mediatek/mtk_hdmi.c +++ b/drivers/gpu/drm/mediatek/mtk_hdmi.c @@ -1276,10 +1276,11 @@ static const struct drm_edid *mtk_hdmi_bridge_edid_= read(struct drm_bridge *bridg =20 return drm_edid; } =20 static int mtk_hdmi_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct mtk_hdmi *hdmi =3D hdmi_ctx_from_bridge(bridge); int ret; =20 @@ -1288,11 +1289,11 @@ static int mtk_hdmi_bridge_attach(struct drm_bridge= *bridge, __func__); return -EINVAL; } =20 if (hdmi->next_bridge) { - ret =3D drm_bridge_attach(bridge->encoder, hdmi->next_bridge, + ret =3D drm_bridge_attach(encoder, hdmi->next_bridge, bridge, flags); if (ret) return ret; } =20 diff --git a/drivers/gpu/drm/meson/meson_encoder_cvbs.c b/drivers/gpu/drm/m= eson/meson_encoder_cvbs.c index e79f7c3ce32efa8e5fe7afe274fe991ac363b734..c9678dc68fa142882e2beb24fe8= 1185fbdef733b 100644 --- a/drivers/gpu/drm/meson/meson_encoder_cvbs.c +++ b/drivers/gpu/drm/meson/meson_encoder_cvbs.c @@ -81,16 +81,17 @@ meson_cvbs_get_mode(const struct drm_display_mode *req_= mode) =20 return NULL; } =20 static int meson_encoder_cvbs_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct meson_encoder_cvbs *meson_encoder_cvbs =3D bridge_to_meson_encoder_cvbs(bridge); =20 - return drm_bridge_attach(bridge->encoder, meson_encoder_cvbs->next_bridge, + return drm_bridge_attach(encoder, meson_encoder_cvbs->next_bridge, &meson_encoder_cvbs->bridge, flags); } =20 static int meson_encoder_cvbs_get_modes(struct drm_bridge *bridge, struct drm_connector *connector) diff --git a/drivers/gpu/drm/meson/meson_encoder_dsi.c b/drivers/gpu/drm/me= son/meson_encoder_dsi.c index fe204437bd6576a258dc11319e2b905f170a24a1..3db518e5f95d324c218b730e094= 8c3dc845382bd 100644 --- a/drivers/gpu/drm/meson/meson_encoder_dsi.c +++ b/drivers/gpu/drm/meson/meson_encoder_dsi.c @@ -31,15 +31,16 @@ struct meson_encoder_dsi { =20 #define bridge_to_meson_encoder_dsi(x) \ container_of(x, struct meson_encoder_dsi, bridge) =20 static int meson_encoder_dsi_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct meson_encoder_dsi *encoder_dsi =3D bridge_to_meson_encoder_dsi(bri= dge); =20 - return drm_bridge_attach(bridge->encoder, encoder_dsi->next_bridge, + return drm_bridge_attach(encoder, encoder_dsi->next_bridge, &encoder_dsi->bridge, flags); } =20 static void meson_encoder_dsi_atomic_enable(struct drm_bridge *bridge, struct drm_atomic_state *state) diff --git a/drivers/gpu/drm/meson/meson_encoder_hdmi.c b/drivers/gpu/drm/m= eson/meson_encoder_hdmi.c index 6d1c9262a2cfb7b08a68efb86821bf5ecf8c0941..5f02695aafd1aa8444cd936a36e= 8f3a8010881a0 100644 --- a/drivers/gpu/drm/meson/meson_encoder_hdmi.c +++ b/drivers/gpu/drm/meson/meson_encoder_hdmi.c @@ -47,15 +47,16 @@ struct meson_encoder_hdmi { =20 #define bridge_to_meson_encoder_hdmi(x) \ container_of(x, struct meson_encoder_hdmi, bridge) =20 static int meson_encoder_hdmi_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct meson_encoder_hdmi *encoder_hdmi =3D bridge_to_meson_encoder_hdmi(= bridge); =20 - return drm_bridge_attach(bridge->encoder, encoder_hdmi->next_bridge, + return drm_bridge_attach(encoder, encoder_hdmi->next_bridge, &encoder_hdmi->bridge, flags); } =20 static void meson_encoder_hdmi_detach(struct drm_bridge *bridge) { diff --git a/drivers/gpu/drm/msm/dsi/dsi_manager.c b/drivers/gpu/drm/msm/ds= i/dsi_manager.c index a210b7c9e5ca281a46fbdb226e25832719a684ea..895ba9815a652f00f7ace9cb712= 5ef8cef555d35 100644 --- a/drivers/gpu/drm/msm/dsi/dsi_manager.c +++ b/drivers/gpu/drm/msm/dsi/dsi_manager.c @@ -422,16 +422,17 @@ static enum drm_mode_status dsi_mgr_bridge_mode_valid= (struct drm_bridge *bridge, =20 return msm_dsi_host_check_dsc(host, mode); } =20 static int dsi_mgr_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { int id =3D dsi_mgr_bridge_get_id(bridge); struct msm_dsi *msm_dsi =3D dsi_mgr_get_dsi(id); =20 - return drm_bridge_attach(bridge->encoder, msm_dsi->next_bridge, + return drm_bridge_attach(encoder, msm_dsi->next_bridge, bridge, flags); } =20 static const struct drm_bridge_funcs dsi_mgr_bridge_funcs =3D { .attach =3D dsi_mgr_bridge_attach, diff --git a/drivers/gpu/drm/omapdrm/dss/dpi.c b/drivers/gpu/drm/omapdrm/ds= s/dpi.c index b17e77f700ddda6c9972e5ff10d394fd72bf96b3..6eff97a091602f6d137095b3b7b= f54fce17e8d3e 100644 --- a/drivers/gpu/drm/omapdrm/dss/dpi.c +++ b/drivers/gpu/drm/omapdrm/dss/dpi.c @@ -418,20 +418,21 @@ static void dpi_init_pll(struct dpi_data *dpi) /* -----------------------------------------------------------------------= ------ * DRM Bridge Operations */ =20 static int dpi_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct dpi_data *dpi =3D drm_bridge_to_dpi(bridge); =20 if (!(flags & DRM_BRIDGE_ATTACH_NO_CONNECTOR)) return -EINVAL; =20 dpi_init_pll(dpi); =20 - return drm_bridge_attach(bridge->encoder, dpi->output.next_bridge, + return drm_bridge_attach(encoder, dpi->output.next_bridge, bridge, flags); } =20 static enum drm_mode_status dpi_bridge_mode_valid(struct drm_bridge *bridge, diff --git a/drivers/gpu/drm/omapdrm/dss/dsi.c b/drivers/gpu/drm/omapdrm/ds= s/dsi.c index 59d20eb8a7e0fdcc13060d00980452bc80c54eb2..35e3e332bdcf4cf69659de65dee= ea0f5dbd70358 100644 --- a/drivers/gpu/drm/omapdrm/dss/dsi.c +++ b/drivers/gpu/drm/omapdrm/dss/dsi.c @@ -4615,18 +4615,19 @@ static const struct component_ops dsi_component_ops= =3D { /* -----------------------------------------------------------------------= ------ * DRM Bridge Operations */ =20 static int dsi_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct dsi_data *dsi =3D drm_bridge_to_dsi(bridge); =20 if (!(flags & DRM_BRIDGE_ATTACH_NO_CONNECTOR)) return -EINVAL; =20 - return drm_bridge_attach(bridge->encoder, dsi->output.next_bridge, + return drm_bridge_attach(encoder, dsi->output.next_bridge, bridge, flags); } =20 static enum drm_mode_status dsi_bridge_mode_valid(struct drm_bridge *bridge, diff --git a/drivers/gpu/drm/omapdrm/dss/hdmi4.c b/drivers/gpu/drm/omapdrm/= dss/hdmi4.c index e1ac447221ee5ae5ee50637acfc8a30a8f111850..a3b22952fdc32b5899dae82d413= 108c5c0a1c3c8 100644 --- a/drivers/gpu/drm/omapdrm/dss/hdmi4.c +++ b/drivers/gpu/drm/omapdrm/dss/hdmi4.c @@ -312,18 +312,19 @@ void hdmi4_core_disable(struct hdmi_core_data *core) /* -----------------------------------------------------------------------= ------ * DRM Bridge Operations */ =20 static int hdmi4_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct omap_hdmi *hdmi =3D drm_bridge_to_hdmi(bridge); =20 if (!(flags & DRM_BRIDGE_ATTACH_NO_CONNECTOR)) return -EINVAL; =20 - return drm_bridge_attach(bridge->encoder, hdmi->output.next_bridge, + return drm_bridge_attach(encoder, hdmi->output.next_bridge, bridge, flags); } =20 static void hdmi4_bridge_mode_set(struct drm_bridge *bridge, const struct drm_display_mode *mode, diff --git a/drivers/gpu/drm/omapdrm/dss/hdmi5.c b/drivers/gpu/drm/omapdrm/= dss/hdmi5.c index fa9904e4c218235bafa87be600680b3e5d5e2ea5..0c98444d39a93d8336b4d8dbd45= aa4521181c3b4 100644 --- a/drivers/gpu/drm/omapdrm/dss/hdmi5.c +++ b/drivers/gpu/drm/omapdrm/dss/hdmi5.c @@ -310,18 +310,19 @@ static void hdmi_core_disable(struct omap_hdmi *hdmi) /* -----------------------------------------------------------------------= ------ * DRM Bridge Operations */ =20 static int hdmi5_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct omap_hdmi *hdmi =3D drm_bridge_to_hdmi(bridge); =20 if (!(flags & DRM_BRIDGE_ATTACH_NO_CONNECTOR)) return -EINVAL; =20 - return drm_bridge_attach(bridge->encoder, hdmi->output.next_bridge, + return drm_bridge_attach(encoder, hdmi->output.next_bridge, bridge, flags); } =20 static void hdmi5_bridge_mode_set(struct drm_bridge *bridge, const struct drm_display_mode *mode, diff --git a/drivers/gpu/drm/omapdrm/dss/sdi.c b/drivers/gpu/drm/omapdrm/ds= s/sdi.c index f9ae358e8e52145b3832b18286ba4834f3c2d8db..e78826e4b560a2b9af2d8a5a38e= 181bd3e44d250 100644 --- a/drivers/gpu/drm/omapdrm/dss/sdi.c +++ b/drivers/gpu/drm/omapdrm/dss/sdi.c @@ -126,18 +126,19 @@ static void sdi_config_lcd_manager(struct sdi_device = *sdi) /* -----------------------------------------------------------------------= ------ * DRM Bridge Operations */ =20 static int sdi_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct sdi_device *sdi =3D drm_bridge_to_sdi(bridge); =20 if (!(flags & DRM_BRIDGE_ATTACH_NO_CONNECTOR)) return -EINVAL; =20 - return drm_bridge_attach(bridge->encoder, sdi->output.next_bridge, + return drm_bridge_attach(encoder, sdi->output.next_bridge, bridge, flags); } =20 static enum drm_mode_status sdi_bridge_mode_valid(struct drm_bridge *bridge, diff --git a/drivers/gpu/drm/omapdrm/dss/venc.c b/drivers/gpu/drm/omapdrm/d= ss/venc.c index aaeef603682c25773f2f809d8ac04f64cba5fb21..50349518eda1630400529caf27c= a4469bb09fc82 100644 --- a/drivers/gpu/drm/omapdrm/dss/venc.c +++ b/drivers/gpu/drm/omapdrm/dss/venc.c @@ -536,18 +536,19 @@ static int venc_get_clocks(struct venc_device *venc) /* -----------------------------------------------------------------------= ------ * DRM Bridge Operations */ =20 static int venc_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct venc_device *venc =3D drm_bridge_to_venc(bridge); =20 if (!(flags & DRM_BRIDGE_ATTACH_NO_CONNECTOR)) return -EINVAL; =20 - return drm_bridge_attach(bridge->encoder, venc->output.next_bridge, + return drm_bridge_attach(encoder, venc->output.next_bridge, bridge, flags); } =20 static enum drm_mode_status venc_bridge_mode_valid(struct drm_bridge *bridge, diff --git a/drivers/gpu/drm/renesas/rcar-du/rcar_lvds.c b/drivers/gpu/drm/= renesas/rcar-du/rcar_lvds.c index 380a855b832af9c09652ce39a78f974b63d949bb..a9145253294fcaef6bae4e1406a= 781f6d710d357 100644 --- a/drivers/gpu/drm/renesas/rcar-du/rcar_lvds.c +++ b/drivers/gpu/drm/renesas/rcar-du/rcar_lvds.c @@ -632,18 +632,19 @@ static bool rcar_lvds_mode_fixup(struct drm_bridge *b= ridge, =20 return true; } =20 static int rcar_lvds_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct rcar_lvds *lvds =3D bridge_to_rcar_lvds(bridge); =20 if (!lvds->next_bridge) return 0; =20 - return drm_bridge_attach(bridge->encoder, lvds->next_bridge, bridge, + return drm_bridge_attach(encoder, lvds->next_bridge, bridge, flags); } =20 static const struct drm_bridge_funcs rcar_lvds_bridge_ops =3D { .attach =3D rcar_lvds_attach, diff --git a/drivers/gpu/drm/renesas/rcar-du/rcar_mipi_dsi.c b/drivers/gpu/= drm/renesas/rcar-du/rcar_mipi_dsi.c index d1e626068065c559a708772d3bbf16efe166ff59..7ab8be46c7f6547f29b4d45af7a= c704283da9dcd 100644 --- a/drivers/gpu/drm/renesas/rcar-du/rcar_mipi_dsi.c +++ b/drivers/gpu/drm/renesas/rcar-du/rcar_mipi_dsi.c @@ -797,15 +797,16 @@ static void rcar_mipi_dsi_stop_video(struct rcar_mipi= _dsi *dsi) /* -----------------------------------------------------------------------= ------ * Bridge */ =20 static int rcar_mipi_dsi_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct rcar_mipi_dsi *dsi =3D bridge_to_rcar_mipi_dsi(bridge); =20 - return drm_bridge_attach(bridge->encoder, dsi->next_bridge, bridge, + return drm_bridge_attach(encoder, dsi->next_bridge, bridge, flags); } =20 static void rcar_mipi_dsi_atomic_enable(struct drm_bridge *bridge, struct drm_atomic_state *state) diff --git a/drivers/gpu/drm/renesas/rz-du/rzg2l_mipi_dsi.c b/drivers/gpu/d= rm/renesas/rz-du/rzg2l_mipi_dsi.c index 4550c6d847962f06886f2b7645bae73646f1ffb6..96c014449547b4042d7568573fd= e876b5b39bd04 100644 --- a/drivers/gpu/drm/renesas/rz-du/rzg2l_mipi_dsi.c +++ b/drivers/gpu/drm/renesas/rz-du/rzg2l_mipi_dsi.c @@ -521,15 +521,16 @@ static int rzg2l_mipi_dsi_stop_video(struct rzg2l_mip= i_dsi *dsi) /* -----------------------------------------------------------------------= ------ * Bridge */ =20 static int rzg2l_mipi_dsi_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct rzg2l_mipi_dsi *dsi =3D bridge_to_rzg2l_mipi_dsi(bridge); =20 - return drm_bridge_attach(bridge->encoder, dsi->next_bridge, bridge, + return drm_bridge_attach(encoder, dsi->next_bridge, bridge, flags); } =20 static void rzg2l_mipi_dsi_atomic_enable(struct drm_bridge *bridge, struct drm_atomic_state *state) diff --git a/drivers/gpu/drm/stm/lvds.c b/drivers/gpu/drm/stm/lvds.c index 4613e8e3b8fddc4455be0e3f81e0e811418746a0..a3ae9a93ce6670eb2c4dd36b3e5= 72fcbca791a1c 100644 --- a/drivers/gpu/drm/stm/lvds.c +++ b/drivers/gpu/drm/stm/lvds.c @@ -932,32 +932,31 @@ static const struct drm_connector_funcs lvds_conn_fun= cs =3D { .destroy =3D drm_connector_cleanup, .atomic_duplicate_state =3D drm_atomic_helper_connector_duplicate_state, .atomic_destroy_state =3D drm_atomic_helper_connector_destroy_state, }; =20 -static int lvds_attach(struct drm_bridge *bridge, +static int lvds_attach(struct drm_bridge *bridge, struct drm_encoder *enco= der, enum drm_bridge_attach_flags flags) { struct stm_lvds *lvds =3D bridge_to_stm_lvds(bridge); struct drm_connector *connector =3D &lvds->connector; - struct drm_encoder *encoder =3D bridge->encoder; int ret; =20 - if (!bridge->encoder) { + if (!encoder) { drm_err(bridge->dev, "Parent encoder object not found\n"); return -ENODEV; } =20 /* Set the encoder type as caller does not know it */ - bridge->encoder->encoder_type =3D DRM_MODE_ENCODER_LVDS; + encoder->encoder_type =3D DRM_MODE_ENCODER_LVDS; =20 /* No cloning support */ - bridge->encoder->possible_clones =3D 0; + encoder->possible_clones =3D 0; =20 /* If we have a next bridge just attach it. */ if (lvds->next_bridge) - return drm_bridge_attach(bridge->encoder, lvds->next_bridge, + return drm_bridge_attach(encoder, lvds->next_bridge, bridge, flags); =20 if (flags & DRM_BRIDGE_ATTACH_NO_CONNECTOR) { drm_err(bridge->dev, "Fix bridge driver to make connector optional!"); return -EINVAL; diff --git a/drivers/gpu/drm/tidss/tidss_encoder.c b/drivers/gpu/drm/tidss/= tidss_encoder.c index 17a86bed805481c6cf054f6cb3176d60faa0351c..95b4aeff277505d64254177093d= 9bcc4de9f4a86 100644 --- a/drivers/gpu/drm/tidss/tidss_encoder.c +++ b/drivers/gpu/drm/tidss/tidss_encoder.c @@ -32,15 +32,16 @@ static inline struct tidss_encoder { return container_of(b, struct tidss_encoder, bridge); } =20 static int tidss_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct tidss_encoder *t_enc =3D bridge_to_tidss_encoder(bridge); =20 - return drm_bridge_attach(bridge->encoder, t_enc->next_bridge, + return drm_bridge_attach(encoder, t_enc->next_bridge, bridge, flags); } =20 static int tidss_bridge_atomic_check(struct drm_bridge *bridge, struct drm_bridge_state *bridge_state, diff --git a/drivers/gpu/drm/vc4/vc4_dsi.c b/drivers/gpu/drm/vc4/vc4_dsi.c index 779b22efe27bffc586a760fdc49012e02ac23d0d..efc6f6078b026764c59cfb2a33b= 28a88b7018c3a 100644 --- a/drivers/gpu/drm/vc4/vc4_dsi.c +++ b/drivers/gpu/drm/vc4/vc4_dsi.c @@ -1158,16 +1158,17 @@ static void vc4_dsi_bridge_enable(struct drm_bridge= *bridge, drm_print_regset32(&p, &dsi->regset); } } =20 static int vc4_dsi_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct vc4_dsi *dsi =3D bridge_to_vc4_dsi(bridge); =20 /* Attach the panel or bridge to the dsi bridge */ - return drm_bridge_attach(bridge->encoder, dsi->out_bridge, + return drm_bridge_attach(encoder, dsi->out_bridge, &dsi->bridge, flags); } =20 static ssize_t vc4_dsi_host_transfer(struct mipi_dsi_host *host, const struct mipi_dsi_msg *msg) diff --git a/drivers/gpu/drm/xlnx/zynqmp_dp.c b/drivers/gpu/drm/xlnx/zynqmp= _dp.c index 82d10891d9cca2f5f6c26b54eeefcb75462339ba..242a3a3fadf4b62e3eaf1cb14c7= 522f67af5b44c 100644 --- a/drivers/gpu/drm/xlnx/zynqmp_dp.c +++ b/drivers/gpu/drm/xlnx/zynqmp_dp.c @@ -1479,10 +1479,11 @@ static void zynqmp_dp_disp_disable(struct zynqmp_dp= *dp, /* -----------------------------------------------------------------------= ------ * DRM Bridge */ =20 static int zynqmp_dp_bridge_attach(struct drm_bridge *bridge, + struct drm_encoder *encoder, enum drm_bridge_attach_flags flags) { struct zynqmp_dp *dp =3D bridge_to_dp(bridge); int ret; =20 @@ -1492,11 +1493,11 @@ static int zynqmp_dp_bridge_attach(struct drm_bridg= e *bridge, dev_err(dp->dev, "failed to initialize DP aux\n"); return ret; } =20 if (dp->next_bridge) { - ret =3D drm_bridge_attach(bridge->encoder, dp->next_bridge, + ret =3D drm_bridge_attach(encoder, dp->next_bridge, bridge, flags); if (ret < 0) goto error; } =20 diff --git a/drivers/platform/arm64/acer-aspire1-ec.c b/drivers/platform/ar= m64/acer-aspire1-ec.c index 2df42406430db7bd9ae5dcca6aace88c3bf3baa0..958fe1bf5f85bb69ac7962f217d= e9f0b40cde9a1 100644 --- a/drivers/platform/arm64/acer-aspire1-ec.c +++ b/drivers/platform/arm64/acer-aspire1-ec.c @@ -364,11 +364,12 @@ static const struct power_supply_desc aspire_ec_adp_p= sy_desc =3D { =20 /* * USB-C DP Alt mode HPD. */ =20 -static int aspire_ec_bridge_attach(struct drm_bridge *bridge, enum drm_bri= dge_attach_flags flags) +static int aspire_ec_bridge_attach(struct drm_bridge *bridge, struct drm_e= ncoder *encoder, + enum drm_bridge_attach_flags flags) { return flags & DRM_BRIDGE_ATTACH_NO_CONNECTOR ? 0 : -EINVAL; } =20 static void aspire_ec_bridge_update_hpd_work(struct work_struct *work) diff --git a/include/drm/drm_bridge.h b/include/drm/drm_bridge.h index df62b9391d5613933ad68b4d8f69a732966b3a0b..cf7ab89e652a982510ae9df1f59= 72b9c6eb13cf2 100644 --- a/include/drm/drm_bridge.h +++ b/include/drm/drm_bridge.h @@ -71,11 +71,11 @@ struct drm_bridge_funcs { * * RETURNS: * * Zero on success, error code on failure. */ - int (*attach)(struct drm_bridge *bridge, + int (*attach)(struct drm_bridge *bridge, struct drm_encoder *encoder, enum drm_bridge_attach_flags flags); =20 /** * @detach: * --=20 2.48.0