[Intel-gfx] [PATCH] drm: Pass crtc to .best_encoder()

Daniel Vetter daniel at ffwll.ch
Tue Jun 26 15:23:25 UTC 2018


On Fri, Jun 15, 2018 at 10:52:21PM +0300, Ville Syrjala wrote:
> From: Ville Syrjälä <ville.syrjala at linux.intel.com>
> 
> To pick the correct MST encoder i915 wants to know which crtc is going
> to be feeding us. To that end let's pass the crtc to the .best_encoder()
> hook. The atomic variant already knows the crtc via the connector state,
> but the non-atomic hooks is still being used by the fb_helper even on
> atomic drivers.

Can't we instead fix the fb helpers to use atomic_best_encoder? Or just
drop that code, userspace doesn't really know any better either and just
needs to figure this out without the help of ->best_encoder, so it should
be possible.
-Daniel

> 
> This allows us to fix up the possible_crtcs bitmask for the i915 MST
> encoders. We have one encoder for each crtc+port combination, and thus
> we have to know both the connector and the crtc to pick the right one.
> This has only worked so far because every MST encoder lied in its
> possible_crtcs bitmask that they can be driven by any crtc.
> 
> I took the easy way out and passed NULL as the crtc for all the driver
> internal uses of .best_encoder() in the amdgpu/radeon drivers. None of
> the other drivers have such internal uses. The other callers
> (crtc_helper, atomic_helper, fb_helper) will pass in the proper crtc.
> but no one besides i915 will currently look at it.
> 
> Cc: Alex Deucher <alexander.deucher at amd.com>
> Cc: "Christian König" <christian.koenig at amd.com>
> Cc: "David (ChunMing) Zhou" <David1.Zhou at amd.com>
> Cc: Harry Wentland <harry.wentland at amd.com>
> Cc: amd-gfx at lists.freedesktop.org
> Signed-off-by: Ville Syrjälä <ville.syrjala at linux.intel.com>
> ---
>  drivers/gpu/drm/amd/amdgpu/amdgpu_connectors.c     | 33 +++++++++--------
>  drivers/gpu/drm/amd/amdgpu/dce_virtual.c           |  3 +-
>  drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c  |  7 ++--
>  .../amd/display/amdgpu_dm/amdgpu_dm_mst_types.c    |  5 +--
>  drivers/gpu/drm/ast/ast_mode.c                     |  3 +-
>  drivers/gpu/drm/bochs/bochs_kms.c                  |  3 +-
>  drivers/gpu/drm/bridge/analogix/analogix_dp_core.c |  3 +-
>  drivers/gpu/drm/bridge/tc358767.c                  |  3 +-
>  drivers/gpu/drm/cirrus/cirrus_mode.c               |  5 +--
>  drivers/gpu/drm/drm_atomic_helper.c                | 16 ++++++---
>  drivers/gpu/drm/drm_crtc_helper.c                  |  3 +-
>  drivers/gpu/drm/drm_fb_helper.c                    | 41 ++++++++++++----------
>  drivers/gpu/drm/gma500/gma_display.c               |  3 +-
>  drivers/gpu/drm/gma500/mdfld_dsi_output.c          |  5 +--
>  drivers/gpu/drm/gma500/psb_intel_drv.h             |  3 +-
>  drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_vdac.c   |  3 +-
>  drivers/gpu/drm/i2c/tda998x_drv.c                  |  3 +-
>  drivers/gpu/drm/i915/intel_dp_mst.c                |  9 +++--
>  drivers/gpu/drm/imx/imx-ldb.c                      |  5 +--
>  drivers/gpu/drm/imx/imx-tve.c                      |  5 +--
>  drivers/gpu/drm/imx/parallel-display.c             |  5 +--
>  drivers/gpu/drm/mediatek/mtk_hdmi.c                |  3 +-
>  drivers/gpu/drm/mgag200/mgag200_mode.c             |  5 +--
>  drivers/gpu/drm/msm/dsi/dsi_manager.c              |  3 +-
>  drivers/gpu/drm/nouveau/dispnv50/disp.c            |  3 +-
>  drivers/gpu/drm/nouveau/nouveau_connector.c        |  3 +-
>  drivers/gpu/drm/qxl/qxl_display.c                  |  3 +-
>  drivers/gpu/drm/radeon/radeon_connectors.c         | 40 +++++++++++----------
>  drivers/gpu/drm/radeon/radeon_dp_mst.c             |  5 +--
>  drivers/gpu/drm/shmobile/shmob_drm_crtc.c          |  3 +-
>  drivers/gpu/drm/tilcdc/tilcdc_panel.c              |  5 +--
>  drivers/gpu/drm/tilcdc/tilcdc_tfp410.c             |  5 +--
>  drivers/gpu/drm/udl/udl_connector.c                |  3 +-
>  drivers/staging/vboxvideo/vbox_mode.c              |  5 +--
>  include/drm/drm_atomic_helper.h                    |  3 +-
>  include/drm/drm_modeset_helper_vtables.h           |  6 ++--
>  36 files changed, 155 insertions(+), 106 deletions(-)
> 
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_connectors.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_connectors.c
> index 8e66851eb427..3dfa50ec2589 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_connectors.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_connectors.c
> @@ -135,7 +135,8 @@ int amdgpu_connector_get_monitor_bpc(struct drm_connector *connector)
>  		else {
>  			const struct drm_connector_helper_funcs *connector_funcs =
>  				connector->helper_private;
> -			struct drm_encoder *encoder = connector_funcs->best_encoder(connector);
> +			struct drm_encoder *encoder = connector_funcs->best_encoder(connector,
> +										    NULL);
>  			struct amdgpu_encoder *amdgpu_encoder = to_amdgpu_encoder(encoder);
>  			struct amdgpu_encoder_atom_dig *dig = amdgpu_encoder->enc_priv;
>  
> @@ -218,7 +219,7 @@ amdgpu_connector_update_scratch_regs(struct drm_connector *connector,
>  	bool connected;
>  	int i;
>  
> -	best_encoder = connector_funcs->best_encoder(connector);
> +	best_encoder = connector_funcs->best_encoder(connector, NULL);
>  
>  	for (i = 0; i < DRM_CONNECTOR_MAX_ENCODER; i++) {
>  		if (connector->encoder_ids[i] == 0)
> @@ -358,7 +359,8 @@ static int amdgpu_connector_ddc_get_modes(struct drm_connector *connector)
>  }
>  
>  static struct drm_encoder *
> -amdgpu_connector_best_single_encoder(struct drm_connector *connector)
> +amdgpu_connector_best_single_encoder(struct drm_connector *connector,
> +				     struct drm_crtc *crtc)
>  {
>  	int enc_id = connector->encoder_ids[0];
>  
> @@ -370,7 +372,7 @@ amdgpu_connector_best_single_encoder(struct drm_connector *connector)
>  
>  static void amdgpu_get_native_mode(struct drm_connector *connector)
>  {
> -	struct drm_encoder *encoder = amdgpu_connector_best_single_encoder(connector);
> +	struct drm_encoder *encoder = amdgpu_connector_best_single_encoder(connector, NULL);
>  	struct amdgpu_encoder *amdgpu_encoder;
>  
>  	if (encoder == NULL)
> @@ -593,7 +595,7 @@ static int amdgpu_connector_set_property(struct drm_connector *connector,
>  			amdgpu_encoder = to_amdgpu_encoder(connector->encoder);
>  		} else {
>  			const struct drm_connector_helper_funcs *connector_funcs = connector->helper_private;
> -			amdgpu_encoder = to_amdgpu_encoder(connector_funcs->best_encoder(connector));
> +			amdgpu_encoder = to_amdgpu_encoder(connector_funcs->best_encoder(connector, NULL));
>  		}
>  
>  		switch (val) {
> @@ -663,7 +665,7 @@ static int amdgpu_connector_lvds_get_modes(struct drm_connector *connector)
>  	amdgpu_connector_get_edid(connector);
>  	ret = amdgpu_connector_ddc_get_modes(connector);
>  	if (ret > 0) {
> -		encoder = amdgpu_connector_best_single_encoder(connector);
> +		encoder = amdgpu_connector_best_single_encoder(connector, NULL);
>  		if (encoder) {
>  			amdgpu_connector_fixup_lcd_native_mode(encoder, connector);
>  			/* add scaled modes */
> @@ -672,7 +674,7 @@ static int amdgpu_connector_lvds_get_modes(struct drm_connector *connector)
>  		return ret;
>  	}
>  
> -	encoder = amdgpu_connector_best_single_encoder(connector);
> +	encoder = amdgpu_connector_best_single_encoder(connector, NULL);
>  	if (!encoder)
>  		return 0;
>  
> @@ -694,7 +696,7 @@ static int amdgpu_connector_lvds_get_modes(struct drm_connector *connector)
>  static enum drm_mode_status amdgpu_connector_lvds_mode_valid(struct drm_connector *connector,
>  					     struct drm_display_mode *mode)
>  {
> -	struct drm_encoder *encoder = amdgpu_connector_best_single_encoder(connector);
> +	struct drm_encoder *encoder = amdgpu_connector_best_single_encoder(connector, NULL);
>  
>  	if ((mode->hdisplay < 320) || (mode->vdisplay < 240))
>  		return MODE_PANEL;
> @@ -725,7 +727,7 @@ static enum drm_connector_status
>  amdgpu_connector_lvds_detect(struct drm_connector *connector, bool force)
>  {
>  	struct amdgpu_connector *amdgpu_connector = to_amdgpu_connector(connector);
> -	struct drm_encoder *encoder = amdgpu_connector_best_single_encoder(connector);
> +	struct drm_encoder *encoder = amdgpu_connector_best_single_encoder(connector, NULL);
>  	enum drm_connector_status ret = connector_status_disconnected;
>  	int r;
>  
> @@ -798,7 +800,7 @@ static int amdgpu_connector_set_lcd_property(struct drm_connector *connector,
>  		amdgpu_encoder = to_amdgpu_encoder(connector->encoder);
>  	else {
>  		const struct drm_connector_helper_funcs *connector_funcs = connector->helper_private;
> -		amdgpu_encoder = to_amdgpu_encoder(connector_funcs->best_encoder(connector));
> +		amdgpu_encoder = to_amdgpu_encoder(connector_funcs->best_encoder(connector, NULL));
>  	}
>  
>  	switch (value) {
> @@ -873,7 +875,7 @@ amdgpu_connector_vga_detect(struct drm_connector *connector, bool force)
>  			return connector_status_disconnected;
>  	}
>  
> -	encoder = amdgpu_connector_best_single_encoder(connector);
> +	encoder = amdgpu_connector_best_single_encoder(connector, NULL);
>  	if (!encoder)
>  		ret = connector_status_disconnected;
>  
> @@ -1130,7 +1132,8 @@ amdgpu_connector_dvi_detect(struct drm_connector *connector, bool force)
>  
>  /* okay need to be smart in here about which encoder to pick */
>  static struct drm_encoder *
> -amdgpu_connector_dvi_encoder(struct drm_connector *connector)
> +amdgpu_connector_dvi_encoder(struct drm_connector *connector,
> +			     struct drm_crtc *crtc)
>  {
>  	int enc_id = connector->encoder_ids[0];
>  	struct amdgpu_connector *amdgpu_connector = to_amdgpu_connector(connector);
> @@ -1224,7 +1227,7 @@ static int amdgpu_connector_dp_get_modes(struct drm_connector *connector)
>  {
>  	struct amdgpu_connector *amdgpu_connector = to_amdgpu_connector(connector);
>  	struct amdgpu_connector_atom_dig *amdgpu_dig_connector = amdgpu_connector->con_priv;
> -	struct drm_encoder *encoder = amdgpu_connector_best_single_encoder(connector);
> +	struct drm_encoder *encoder = amdgpu_connector_best_single_encoder(connector, NULL);
>  	int ret;
>  
>  	if ((connector->connector_type == DRM_MODE_CONNECTOR_eDP) ||
> @@ -1363,7 +1366,7 @@ amdgpu_connector_dp_detect(struct drm_connector *connector, bool force)
>  	struct amdgpu_connector *amdgpu_connector = to_amdgpu_connector(connector);
>  	enum drm_connector_status ret = connector_status_disconnected;
>  	struct amdgpu_connector_atom_dig *amdgpu_dig_connector = amdgpu_connector->con_priv;
> -	struct drm_encoder *encoder = amdgpu_connector_best_single_encoder(connector);
> +	struct drm_encoder *encoder = amdgpu_connector_best_single_encoder(connector, NULL);
>  	int r;
>  
>  	if (!drm_kms_helper_is_poll_worker()) {
> @@ -1458,7 +1461,7 @@ static enum drm_mode_status amdgpu_connector_dp_mode_valid(struct drm_connector
>  
>  	if ((connector->connector_type == DRM_MODE_CONNECTOR_eDP) ||
>  	    (connector->connector_type == DRM_MODE_CONNECTOR_LVDS)) {
> -		struct drm_encoder *encoder = amdgpu_connector_best_single_encoder(connector);
> +		struct drm_encoder *encoder = amdgpu_connector_best_single_encoder(connector, NULL);
>  
>  		if ((mode->hdisplay < 320) || (mode->vdisplay < 240))
>  			return MODE_PANEL;
> diff --git a/drivers/gpu/drm/amd/amdgpu/dce_virtual.c b/drivers/gpu/drm/amd/amdgpu/dce_virtual.c
> index dbf2ccd0c744..b02ce39e65b7 100644
> --- a/drivers/gpu/drm/amd/amdgpu/dce_virtual.c
> +++ b/drivers/gpu/drm/amd/amdgpu/dce_virtual.c
> @@ -267,7 +267,8 @@ static int dce_virtual_early_init(void *handle)
>  }
>  
>  static struct drm_encoder *
> -dce_virtual_encoder(struct drm_connector *connector)
> +dce_virtual_encoder(struct drm_connector *connector,
> +		    struct drm_crtc *crtc)
>  {
>  	int enc_id = connector->encoder_ids[0];
>  	struct drm_encoder *encoder;
> diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> index 3e1c2a04d669..e56ed4e5818f 100644
> --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> @@ -2758,7 +2758,8 @@ static const struct drm_connector_funcs amdgpu_dm_connector_funcs = {
>  	.atomic_get_property = amdgpu_dm_connector_atomic_get_property
>  };
>  
> -static struct drm_encoder *best_encoder(struct drm_connector *connector)
> +static struct drm_encoder *best_encoder(struct drm_connector *connector,
> +					struct drm_crtc *crtc)
>  {
>  	int enc_id = connector->encoder_ids[0];
>  	struct drm_mode_object *obj;
> @@ -3304,7 +3305,7 @@ static void amdgpu_dm_get_native_mode(struct drm_connector *connector)
>  	struct drm_encoder *encoder;
>  	struct amdgpu_encoder *amdgpu_encoder;
>  
> -	encoder = helper->best_encoder(connector);
> +	encoder = helper->best_encoder(connector, NULL);
>  
>  	if (encoder == NULL)
>  		return;
> @@ -3438,7 +3439,7 @@ static int amdgpu_dm_connector_get_modes(struct drm_connector *connector)
>  	struct drm_encoder *encoder;
>  	struct edid *edid = amdgpu_dm_connector->edid;
>  
> -	encoder = helper->best_encoder(connector);
> +	encoder = helper->best_encoder(connector, NULL);
>  	amdgpu_dm_connector_ddc_get_modes(connector, edid);
>  	amdgpu_dm_connector_add_common_modes(encoder, connector);
>  
> diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c
> index 4304d9e408b8..4544a1401caf 100644
> --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c
> +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c
> @@ -269,7 +269,8 @@ static int dm_dp_mst_get_modes(struct drm_connector *connector)
>  	return ret;
>  }
>  
> -static struct drm_encoder *dm_mst_best_encoder(struct drm_connector *connector)
> +static struct drm_encoder *dm_mst_best_encoder(struct drm_connector *connector,
> +					       struct drm_crtc *crtc)
>  {
>  	struct amdgpu_dm_connector *amdgpu_dm_connector = to_amdgpu_dm_connector(connector);
>  
> @@ -302,7 +303,7 @@ dm_dp_create_fake_mst_encoder(struct amdgpu_dm_connector *connector)
>  	const struct drm_connector_helper_funcs *connector_funcs =
>  		connector->base.helper_private;
>  	struct drm_encoder *enc_master =
> -		connector_funcs->best_encoder(&connector->base);
> +		connector_funcs->best_encoder(&connector->base, NULL);
>  
>  	DRM_DEBUG_KMS("enc master is %p\n", enc_master);
>  	amdgpu_encoder = kzalloc(sizeof(*amdgpu_encoder), GFP_KERNEL);
> diff --git a/drivers/gpu/drm/ast/ast_mode.c b/drivers/gpu/drm/ast/ast_mode.c
> index 036dff8a1f33..d5899aa41d37 100644
> --- a/drivers/gpu/drm/ast/ast_mode.c
> +++ b/drivers/gpu/drm/ast/ast_mode.c
> @@ -709,7 +709,8 @@ static void ast_encoder_destroy(struct drm_encoder *encoder)
>  }
>  
>  
> -static struct drm_encoder *ast_best_single_encoder(struct drm_connector *connector)
> +static struct drm_encoder *ast_best_single_encoder(struct drm_connector *connector,
> +						   struct drm_crtc *crtc)
>  {
>  	int enc_id = connector->encoder_ids[0];
>  	/* pick the encoder ids */
> diff --git a/drivers/gpu/drm/bochs/bochs_kms.c b/drivers/gpu/drm/bochs/bochs_kms.c
> index 233980a78591..dc80a238feef 100644
> --- a/drivers/gpu/drm/bochs/bochs_kms.c
> +++ b/drivers/gpu/drm/bochs/bochs_kms.c
> @@ -208,7 +208,8 @@ static enum drm_mode_status bochs_connector_mode_valid(struct drm_connector *con
>  }
>  
>  static struct drm_encoder *
> -bochs_connector_best_encoder(struct drm_connector *connector)
> +bochs_connector_best_encoder(struct drm_connector *connector,
> +			     struct drm_crtc *crtc)
>  {
>  	int enc_id = connector->encoder_ids[0];
>  	/* pick the encoder ids */
> diff --git a/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c b/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c
> index 2bcbfadb6ac5..0706471c8c63 100644
> --- a/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c
> +++ b/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c
> @@ -1137,7 +1137,8 @@ static int analogix_dp_get_modes(struct drm_connector *connector)
>  }
>  
>  static struct drm_encoder *
> -analogix_dp_best_encoder(struct drm_connector *connector)
> +analogix_dp_best_encoder(struct drm_connector *connector,
> +			 struct drm_crtc *crtc)
>  {
>  	struct analogix_dp_device *dp = to_dp(connector);
>  
> diff --git a/drivers/gpu/drm/bridge/tc358767.c b/drivers/gpu/drm/bridge/tc358767.c
> index 0fd9cf27542c..7cd954226fe3 100644
> --- a/drivers/gpu/drm/bridge/tc358767.c
> +++ b/drivers/gpu/drm/bridge/tc358767.c
> @@ -1155,7 +1155,8 @@ static void tc_connector_set_polling(struct tc_data *tc,
>  }
>  
>  static struct drm_encoder *
> -tc_connector_best_encoder(struct drm_connector *connector)
> +tc_connector_best_encoder(struct drm_connector *connector,
> +			  struct drm_crtc *crtc)
>  {
>  	struct tc_data *tc = connector_to_tc(connector);
>  
> diff --git a/drivers/gpu/drm/cirrus/cirrus_mode.c b/drivers/gpu/drm/cirrus/cirrus_mode.c
> index b529f8c8e2a6..378ebc3f25e8 100644
> --- a/drivers/gpu/drm/cirrus/cirrus_mode.c
> +++ b/drivers/gpu/drm/cirrus/cirrus_mode.c
> @@ -451,8 +451,9 @@ static int cirrus_vga_get_modes(struct drm_connector *connector)
>  	return count;
>  }
>  
> -static struct drm_encoder *cirrus_connector_best_encoder(struct drm_connector
> -						  *connector)
> +static struct drm_encoder *
> +cirrus_connector_best_encoder(struct drm_connector *connector,
> +			      struct drm_crtc *crtc)
>  {
>  	int enc_id = connector->encoder_ids[0];
>  	/* pick the encoder ids */
> diff --git a/drivers/gpu/drm/drm_atomic_helper.c b/drivers/gpu/drm/drm_atomic_helper.c
> index 232fa11a5e31..bd3aebbfd5b4 100644
> --- a/drivers/gpu/drm/drm_atomic_helper.c
> +++ b/drivers/gpu/drm/drm_atomic_helper.c
> @@ -115,9 +115,11 @@ static int handle_conflicting_encoders(struct drm_atomic_state *state,
>  		if (funcs->atomic_best_encoder)
>  			new_encoder = funcs->atomic_best_encoder(connector, new_conn_state);
>  		else if (funcs->best_encoder)
> -			new_encoder = funcs->best_encoder(connector);
> +			new_encoder = funcs->best_encoder(connector,
> +							  new_conn_state->crtc);
>  		else
> -			new_encoder = drm_atomic_helper_best_encoder(connector);
> +			new_encoder = drm_atomic_helper_best_encoder(connector,
> +								     new_conn_state->crtc);
>  
>  		if (new_encoder) {
>  			if (encoder_mask & (1 << drm_encoder_index(new_encoder))) {
> @@ -312,9 +314,11 @@ update_connector_routing(struct drm_atomic_state *state,
>  		new_encoder = funcs->atomic_best_encoder(connector,
>  							 new_connector_state);
>  	else if (funcs->best_encoder)
> -		new_encoder = funcs->best_encoder(connector);
> +		new_encoder = funcs->best_encoder(connector,
> +						  new_connector_state->crtc);
>  	else
> -		new_encoder = drm_atomic_helper_best_encoder(connector);
> +		new_encoder = drm_atomic_helper_best_encoder(connector,
> +							     new_connector_state->crtc);
>  
>  	if (!new_encoder) {
>  		DRM_DEBUG_ATOMIC("No suitable encoder found for [CONNECTOR:%d:%s]\n",
> @@ -3318,13 +3322,15 @@ EXPORT_SYMBOL(drm_atomic_helper_page_flip_target);
>   * drm_atomic_helper_best_encoder - Helper for
>   * 	&drm_connector_helper_funcs.best_encoder callback
>   * @connector: Connector control structure
> + * @crtc: DRM crtc
>   *
>   * This is a &drm_connector_helper_funcs.best_encoder callback helper for
>   * connectors that support exactly 1 encoder, statically determined at driver
>   * init time.
>   */
>  struct drm_encoder *
> -drm_atomic_helper_best_encoder(struct drm_connector *connector)
> +drm_atomic_helper_best_encoder(struct drm_connector *connector,
> +			       struct drm_crtc *crtc)
>  {
>  	WARN_ON(connector->encoder_ids[1]);
>  	return drm_encoder_find(connector->dev, NULL, connector->encoder_ids[0]);
> diff --git a/drivers/gpu/drm/drm_crtc_helper.c b/drivers/gpu/drm/drm_crtc_helper.c
> index 5a84c3bc915d..f9318fc11fb1 100644
> --- a/drivers/gpu/drm/drm_crtc_helper.c
> +++ b/drivers/gpu/drm/drm_crtc_helper.c
> @@ -637,7 +637,8 @@ int drm_crtc_helper_set_config(struct drm_mode_set *set,
>  		new_encoder = connector->encoder;
>  		for (ro = 0; ro < set->num_connectors; ro++) {
>  			if (set->connectors[ro] == connector) {
> -				new_encoder = connector_funcs->best_encoder(connector);
> +				new_encoder = connector_funcs->best_encoder(connector,
> +									    set->crtc);
>  				/* if we can't get an encoder for a connector
>  				   we are setting now - then fail */
>  				if (new_encoder == NULL)
> diff --git a/drivers/gpu/drm/drm_fb_helper.c b/drivers/gpu/drm/drm_fb_helper.c
> index cab14f253384..ce91ae12cccb 100644
> --- a/drivers/gpu/drm/drm_fb_helper.c
> +++ b/drivers/gpu/drm/drm_fb_helper.c
> @@ -2331,9 +2331,8 @@ static int drm_pick_crtcs(struct drm_fb_helper *fb_helper,
>  	int c, o;
>  	struct drm_connector *connector;
>  	const struct drm_connector_helper_funcs *connector_funcs;
> -	struct drm_encoder *encoder;
>  	int my_score, best_score, score;
> -	struct drm_fb_helper_crtc **crtcs, *crtc;
> +	struct drm_fb_helper_crtc **crtcs;
>  	struct drm_fb_helper_connector *fb_helper_conn;
>  
>  	if (n == fb_helper->connector_count)
> @@ -2362,28 +2361,32 @@ static int drm_pick_crtcs(struct drm_fb_helper *fb_helper,
>  
>  	connector_funcs = connector->helper_private;
>  
> -	/*
> -	 * If the DRM device implements atomic hooks and ->best_encoder() is
> -	 * NULL we fallback to the default drm_atomic_helper_best_encoder()
> -	 * helper.
> -	 */
> -	if (drm_drv_uses_atomic_modeset(fb_helper->dev) &&
> -	    !connector_funcs->best_encoder)
> -		encoder = drm_atomic_helper_best_encoder(connector);
> -	else
> -		encoder = connector_funcs->best_encoder(connector);
> -
> -	if (!encoder)
> -		goto out;
> -
>  	/*
>  	 * select a crtc for this connector and then attempt to configure
>  	 * remaining connectors
>  	 */
>  	for (c = 0; c < fb_helper->crtc_count; c++) {
> -		crtc = &fb_helper->crtc_info[c];
> +		struct drm_encoder *encoder;
> +		struct drm_fb_helper_crtc *crtc = &fb_helper->crtc_info[c];
>  
> -		if ((encoder->possible_crtcs & (1 << c)) == 0)
> +		/*
> +		 * If the DRM device implements atomic hooks and ->best_encoder() is
> +		 * NULL we fallback to the default drm_atomic_helper_best_encoder()
> +		 * helper.
> +		 */
> +		if (drm_drv_uses_atomic_modeset(fb_helper->dev) &&
> +		    !connector_funcs->best_encoder)
> +			encoder = drm_atomic_helper_best_encoder(connector,
> +								 crtc->mode_set.crtc);
> +		else
> +			encoder = connector_funcs->best_encoder(connector,
> +								crtc->mode_set.crtc);
> +
> +		if (!encoder)
> +			continue;
> +
> +		if ((encoder->possible_crtcs &
> +		     drm_crtc_mask(crtc->mode_set.crtc)) == 0)
>  			continue;
>  
>  		for (o = 0; o < n; o++)
> @@ -2410,7 +2413,7 @@ static int drm_pick_crtcs(struct drm_fb_helper *fb_helper,
>  			       sizeof(struct drm_fb_helper_crtc *));
>  		}
>  	}
> -out:
> +
>  	kfree(crtcs);
>  	return best_score;
>  }
> diff --git a/drivers/gpu/drm/gma500/gma_display.c b/drivers/gpu/drm/gma500/gma_display.c
> index c8f071c47daf..251b64653dc8 100644
> --- a/drivers/gpu/drm/gma500/gma_display.c
> +++ b/drivers/gpu/drm/gma500/gma_display.c
> @@ -652,7 +652,8 @@ void gma_encoder_destroy(struct drm_encoder *encoder)
>  }
>  
>  /* Currently there is only a 1:1 mapping of encoders and connectors */
> -struct drm_encoder *gma_best_encoder(struct drm_connector *connector)
> +struct drm_encoder *gma_best_encoder(struct drm_connector *connector,
> +				     struct drm_crtc *crtc)
>  {
>  	struct gma_encoder *gma_encoder = gma_attached_encoder(connector);
>  
> diff --git a/drivers/gpu/drm/gma500/mdfld_dsi_output.c b/drivers/gpu/drm/gma500/mdfld_dsi_output.c
> index fe020926ea4f..9e5b059a9e5b 100644
> --- a/drivers/gpu/drm/gma500/mdfld_dsi_output.c
> +++ b/drivers/gpu/drm/gma500/mdfld_dsi_output.c
> @@ -377,8 +377,9 @@ static enum drm_mode_status mdfld_dsi_connector_mode_valid(struct drm_connector
>  	return MODE_OK;
>  }
>  
> -static struct drm_encoder *mdfld_dsi_connector_best_encoder(
> -				struct drm_connector *connector)
> +static struct drm_encoder *
> +mdfld_dsi_connector_best_encoder(struct drm_connector *connector,
> +				 struct drm_crtc *crtc)
>  {
>  	struct mdfld_dsi_connector *dsi_connector =
>  				mdfld_dsi_connector(connector);
> diff --git a/drivers/gpu/drm/gma500/psb_intel_drv.h b/drivers/gpu/drm/gma500/psb_intel_drv.h
> index e05e5399af2d..e903525b9d30 100644
> --- a/drivers/gpu/drm/gma500/psb_intel_drv.h
> +++ b/drivers/gpu/drm/gma500/psb_intel_drv.h
> @@ -230,7 +230,8 @@ extern void oaktrail_lvds_i2c_init(struct drm_encoder *encoder);
>  extern void mid_dsi_init(struct drm_device *dev,
>  		    struct psb_intel_mode_device *mode_dev, int dsi_num);
>  
> -extern struct drm_encoder *gma_best_encoder(struct drm_connector *connector);
> +extern struct drm_encoder *gma_best_encoder(struct drm_connector *connector,
> +					    struct drm_crtc *crtc);
>  extern void gma_connector_attach_encoder(struct gma_connector *connector,
>  					 struct gma_encoder *encoder);
>  
> diff --git a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_vdac.c b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_vdac.c
> index d2f4749ebf8d..89a0104f9855 100644
> --- a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_vdac.c
> +++ b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_vdac.c
> @@ -34,7 +34,8 @@ static enum drm_mode_status hibmc_connector_mode_valid(struct drm_connector *con
>  }
>  
>  static struct drm_encoder *
> -hibmc_connector_best_encoder(struct drm_connector *connector)
> +hibmc_connector_best_encoder(struct drm_connector *connector,
> +			     struct drm_crtc *crtc)
>  {
>  	return drm_encoder_find(connector->dev, NULL, connector->encoder_ids[0]);
>  }
> diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c b/drivers/gpu/drm/i2c/tda998x_drv.c
> index 0038c976536a..422347d977b4 100644
> --- a/drivers/gpu/drm/i2c/tda998x_drv.c
> +++ b/drivers/gpu/drm/i2c/tda998x_drv.c
> @@ -1267,7 +1267,8 @@ static enum drm_mode_status tda998x_connector_mode_valid(struct drm_connector *c
>  }
>  
>  static struct drm_encoder *
> -tda998x_connector_best_encoder(struct drm_connector *connector)
> +tda998x_connector_best_encoder(struct drm_connector *connector,
> +			       struct drm_crtc *crtc)
>  {
>  	struct tda998x_priv *priv = conn_to_tda998x_priv(connector);
>  
> diff --git a/drivers/gpu/drm/i915/intel_dp_mst.c b/drivers/gpu/drm/i915/intel_dp_mst.c
> index 5890500a3a8b..d9119d516579 100644
> --- a/drivers/gpu/drm/i915/intel_dp_mst.c
> +++ b/drivers/gpu/drm/i915/intel_dp_mst.c
> @@ -403,20 +403,23 @@ static struct drm_encoder *intel_mst_atomic_best_encoder(struct drm_connector *c
>  	return &intel_dp->mst_encoders[crtc->pipe]->base.base;
>  }
>  
> -static struct drm_encoder *intel_mst_best_encoder(struct drm_connector *connector)
> +static struct drm_encoder *intel_mst_best_encoder(struct drm_connector *connector,
> +						  struct drm_crtc *_crtc)
>  {
>  	struct intel_connector *intel_connector = to_intel_connector(connector);
>  	struct intel_dp *intel_dp = intel_connector->mst_port;
> +	struct intel_crtc *crtc = to_intel_crtc(_crtc);
> +
>  	if (!intel_dp)
>  		return NULL;
> -	return &intel_dp->mst_encoders[0]->base.base;
> +	return &intel_dp->mst_encoders[crtc->pipe]->base.base;
>  }
>  
>  static const struct drm_connector_helper_funcs intel_dp_mst_connector_helper_funcs = {
>  	.get_modes = intel_dp_mst_get_modes,
>  	.mode_valid = intel_dp_mst_mode_valid,
>  	.atomic_best_encoder = intel_mst_atomic_best_encoder,
> -	.best_encoder = intel_mst_best_encoder,
> +	.best_encoder = intel_mst_best_encoder, /* only for fb_helper */
>  	.atomic_check = intel_dp_mst_atomic_check,
>  };
>  
> diff --git a/drivers/gpu/drm/imx/imx-ldb.c b/drivers/gpu/drm/imx/imx-ldb.c
> index 56dd7a9a8e25..f1523cccc090 100644
> --- a/drivers/gpu/drm/imx/imx-ldb.c
> +++ b/drivers/gpu/drm/imx/imx-ldb.c
> @@ -163,8 +163,9 @@ static int imx_ldb_connector_get_modes(struct drm_connector *connector)
>  	return num_modes;
>  }
>  
> -static struct drm_encoder *imx_ldb_connector_best_encoder(
> -		struct drm_connector *connector)
> +static struct drm_encoder *
> +imx_ldb_connector_best_encoder(struct drm_connector *connector,
> +			       struct drm_crtc *crtc)
>  {
>  	struct imx_ldb_channel *imx_ldb_ch = con_to_imx_ldb_ch(connector);
>  
> diff --git a/drivers/gpu/drm/imx/imx-tve.c b/drivers/gpu/drm/imx/imx-tve.c
> index bc27c2699464..71f49ac835ec 100644
> --- a/drivers/gpu/drm/imx/imx-tve.c
> +++ b/drivers/gpu/drm/imx/imx-tve.c
> @@ -265,8 +265,9 @@ static int imx_tve_connector_mode_valid(struct drm_connector *connector,
>  	return MODE_BAD;
>  }
>  
> -static struct drm_encoder *imx_tve_connector_best_encoder(
> -		struct drm_connector *connector)
> +static struct drm_encoder *
> +imx_tve_connector_best_encoder(struct drm_connector *connector,
> +			       struct drm_crtc *crtc)
>  {
>  	struct imx_tve *tve = con_to_tve(connector);
>  
> diff --git a/drivers/gpu/drm/imx/parallel-display.c b/drivers/gpu/drm/imx/parallel-display.c
> index aedecda9728a..438f95dee73e 100644
> --- a/drivers/gpu/drm/imx/parallel-display.c
> +++ b/drivers/gpu/drm/imx/parallel-display.c
> @@ -89,8 +89,9 @@ static int imx_pd_connector_get_modes(struct drm_connector *connector)
>  	return num_modes;
>  }
>  
> -static struct drm_encoder *imx_pd_connector_best_encoder(
> -		struct drm_connector *connector)
> +static struct drm_encoder *
> +imx_pd_connector_best_encoder(struct drm_connector *connector,
> +			      struct drm_crtc *crtc)
>  {
>  	struct imx_parallel_display *imxpd = con_to_imxpd(connector);
>  
> diff --git a/drivers/gpu/drm/mediatek/mtk_hdmi.c b/drivers/gpu/drm/mediatek/mtk_hdmi.c
> index 59a11026dceb..c0578779fb19 100644
> --- a/drivers/gpu/drm/mediatek/mtk_hdmi.c
> +++ b/drivers/gpu/drm/mediatek/mtk_hdmi.c
> @@ -1253,7 +1253,8 @@ static int mtk_hdmi_conn_mode_valid(struct drm_connector *conn,
>  	return drm_mode_validate_size(mode, 0x1fff, 0x1fff);
>  }
>  
> -static struct drm_encoder *mtk_hdmi_conn_best_enc(struct drm_connector *conn)
> +static struct drm_encoder *mtk_hdmi_conn_best_enc(struct drm_connector *conn,
> +						  struct drm_crtc *crtc)
>  {
>  	struct mtk_hdmi *hdmi = hdmi_ctx_from_conn(conn);
>  
> diff --git a/drivers/gpu/drm/mgag200/mgag200_mode.c b/drivers/gpu/drm/mgag200/mgag200_mode.c
> index 8918539a19aa..4059f04030ee 100644
> --- a/drivers/gpu/drm/mgag200/mgag200_mode.c
> +++ b/drivers/gpu/drm/mgag200/mgag200_mode.c
> @@ -1664,8 +1664,9 @@ static enum drm_mode_status mga_vga_mode_valid(struct drm_connector *connector,
>  	return MODE_OK;
>  }
>  
> -static struct drm_encoder *mga_connector_best_encoder(struct drm_connector
> -						  *connector)
> +static struct drm_encoder *
> +mga_connector_best_encoder(struct drm_connector *connector,
> +			   struct drm_crtc *crtc)
>  {
>  	int enc_id = connector->encoder_ids[0];
>  	/* pick the encoder ids */
> diff --git a/drivers/gpu/drm/msm/dsi/dsi_manager.c b/drivers/gpu/drm/msm/dsi/dsi_manager.c
> index 4cb1cb68878b..8cbede64819e 100644
> --- a/drivers/gpu/drm/msm/dsi/dsi_manager.c
> +++ b/drivers/gpu/drm/msm/dsi/dsi_manager.c
> @@ -426,7 +426,8 @@ static int dsi_mgr_connector_mode_valid(struct drm_connector *connector,
>  }
>  
>  static struct drm_encoder *
> -dsi_mgr_connector_best_encoder(struct drm_connector *connector)
> +dsi_mgr_connector_best_encoder(struct drm_connector *connector,
> +			       struct drm_crtc *crtc)
>  {
>  	int id = dsi_mgr_connector_get_id(connector);
>  	struct msm_dsi *msm_dsi = dsi_mgr_get_dsi(id);
> diff --git a/drivers/gpu/drm/nouveau/dispnv50/disp.c b/drivers/gpu/drm/nouveau/dispnv50/disp.c
> index b83465ae7c1b..a9bacb0a1e16 100644
> --- a/drivers/gpu/drm/nouveau/dispnv50/disp.c
> +++ b/drivers/gpu/drm/nouveau/dispnv50/disp.c
> @@ -826,7 +826,8 @@ nv50_mstc_atomic_best_encoder(struct drm_connector *connector,
>  }
>  
>  static struct drm_encoder *
> -nv50_mstc_best_encoder(struct drm_connector *connector)
> +nv50_mstc_best_encoder(struct drm_connector *connector,
> +		       struct drm_crtc *crtc)
>  {
>  	struct nv50_mstc *mstc = nv50_mstc(connector);
>  	if (mstc->port) {
> diff --git a/drivers/gpu/drm/nouveau/nouveau_connector.c b/drivers/gpu/drm/nouveau/nouveau_connector.c
> index 7b557c354307..ee4bf532b932 100644
> --- a/drivers/gpu/drm/nouveau/nouveau_connector.c
> +++ b/drivers/gpu/drm/nouveau/nouveau_connector.c
> @@ -1063,7 +1063,8 @@ nouveau_connector_mode_valid(struct drm_connector *connector,
>  }
>  
>  static struct drm_encoder *
> -nouveau_connector_best_encoder(struct drm_connector *connector)
> +nouveau_connector_best_encoder(struct drm_connector *connector,
> +			       struct drm_crtc *crtc)
>  {
>  	struct nouveau_connector *nv_connector = nouveau_connector(connector);
>  
> diff --git a/drivers/gpu/drm/qxl/qxl_display.c b/drivers/gpu/drm/qxl/qxl_display.c
> index 768207fbbae3..fa5c84f35531 100644
> --- a/drivers/gpu/drm/qxl/qxl_display.c
> +++ b/drivers/gpu/drm/qxl/qxl_display.c
> @@ -978,7 +978,8 @@ static enum drm_mode_status qxl_conn_mode_valid(struct drm_connector *connector,
>  	return MODE_BAD;
>  }
>  
> -static struct drm_encoder *qxl_best_encoder(struct drm_connector *connector)
> +static struct drm_encoder *qxl_best_encoder(struct drm_connector *connector,
> +					    struct drm_crtc *crtc)
>  {
>  	struct qxl_output *qxl_output =
>  		drm_connector_to_qxl_output(connector);
> diff --git a/drivers/gpu/drm/radeon/radeon_connectors.c b/drivers/gpu/drm/radeon/radeon_connectors.c
> index 2aea2bdff99b..8bf7298dccb3 100644
> --- a/drivers/gpu/drm/radeon/radeon_connectors.c
> +++ b/drivers/gpu/drm/radeon/radeon_connectors.c
> @@ -158,7 +158,7 @@ int radeon_get_monitor_bpc(struct drm_connector *connector)
>  		else if (ASIC_IS_DCE41(rdev) || ASIC_IS_DCE5(rdev)) {
>  			const struct drm_connector_helper_funcs *connector_funcs =
>  				connector->helper_private;
> -			struct drm_encoder *encoder = connector_funcs->best_encoder(connector);
> +			struct drm_encoder *encoder = connector_funcs->best_encoder(connector, NULL);
>  			struct radeon_encoder *radeon_encoder = to_radeon_encoder(encoder);
>  			struct radeon_encoder_atom_dig *dig = radeon_encoder->enc_priv;
>  
> @@ -250,7 +250,7 @@ radeon_connector_update_scratch_regs(struct drm_connector *connector, enum drm_c
>  	bool connected;
>  	int i;
>  
> -	best_encoder = connector_funcs->best_encoder(connector);
> +	best_encoder = connector_funcs->best_encoder(connector, NULL);
>  
>  	for (i = 0; i < DRM_CONNECTOR_MAX_ENCODER; i++) {
>  		if (connector->encoder_ids[i] == 0)
> @@ -391,7 +391,8 @@ static int radeon_ddc_get_modes(struct drm_connector *connector)
>  	return 0;
>  }
>  
> -static struct drm_encoder *radeon_best_single_encoder(struct drm_connector *connector)
> +static struct drm_encoder *radeon_best_single_encoder(struct drm_connector *connector,
> +						      struct drm_crtc *crtc)
>  {
>  	int enc_id = connector->encoder_ids[0];
>  	/* pick the encoder ids */
> @@ -402,7 +403,7 @@ static struct drm_encoder *radeon_best_single_encoder(struct drm_connector *conn
>  
>  static void radeon_get_native_mode(struct drm_connector *connector)
>  {
> -	struct drm_encoder *encoder = radeon_best_single_encoder(connector);
> +	struct drm_encoder *encoder = radeon_best_single_encoder(connector, NULL);
>  	struct radeon_encoder *radeon_encoder;
>  
>  	if (encoder == NULL)
> @@ -728,7 +729,7 @@ static int radeon_connector_set_property(struct drm_connector *connector, struct
>  			radeon_encoder = to_radeon_encoder(connector->encoder);
>  		else {
>  			const struct drm_connector_helper_funcs *connector_funcs = connector->helper_private;
> -			radeon_encoder = to_radeon_encoder(connector_funcs->best_encoder(connector));
> +			radeon_encoder = to_radeon_encoder(connector_funcs->best_encoder(connector, NULL));
>  		}
>  
>  		switch (val) {
> @@ -755,7 +756,7 @@ static int radeon_connector_set_property(struct drm_connector *connector, struct
>  			radeon_encoder = to_radeon_encoder(connector->encoder);
>  		else {
>  			const struct drm_connector_helper_funcs *connector_funcs = connector->helper_private;
> -			radeon_encoder = to_radeon_encoder(connector_funcs->best_encoder(connector));
> +			radeon_encoder = to_radeon_encoder(connector_funcs->best_encoder(connector, NULL));
>  		}
>  
>  		if (radeon_encoder->output_csc == val)
> @@ -824,7 +825,7 @@ static int radeon_lvds_get_modes(struct drm_connector *connector)
>  	radeon_connector_get_edid(connector);
>  	ret = radeon_ddc_get_modes(connector);
>  	if (ret > 0) {
> -		encoder = radeon_best_single_encoder(connector);
> +		encoder = radeon_best_single_encoder(connector, NULL);
>  		if (encoder) {
>  			radeon_fixup_lvds_native_mode(encoder, connector);
>  			/* add scaled modes */
> @@ -833,7 +834,7 @@ static int radeon_lvds_get_modes(struct drm_connector *connector)
>  		return ret;
>  	}
>  
> -	encoder = radeon_best_single_encoder(connector);
> +	encoder = radeon_best_single_encoder(connector, NULL);
>  	if (!encoder)
>  		return 0;
>  
> @@ -855,7 +856,7 @@ static int radeon_lvds_get_modes(struct drm_connector *connector)
>  static enum drm_mode_status radeon_lvds_mode_valid(struct drm_connector *connector,
>  				  struct drm_display_mode *mode)
>  {
> -	struct drm_encoder *encoder = radeon_best_single_encoder(connector);
> +	struct drm_encoder *encoder = radeon_best_single_encoder(connector, NULL);
>  
>  	if ((mode->hdisplay < 320) || (mode->vdisplay < 240))
>  		return MODE_PANEL;
> @@ -888,7 +889,7 @@ radeon_lvds_detect(struct drm_connector *connector, bool force)
>  	struct drm_device *dev = connector->dev;
>  	struct radeon_device *rdev = dev->dev_private;
>  	struct radeon_connector *radeon_connector = to_radeon_connector(connector);
> -	struct drm_encoder *encoder = radeon_best_single_encoder(connector);
> +	struct drm_encoder *encoder = radeon_best_single_encoder(connector, NULL);
>  	enum drm_connector_status ret = connector_status_disconnected;
>  	int r;
>  
> @@ -965,7 +966,7 @@ static int radeon_lvds_set_property(struct drm_connector *connector,
>  		radeon_encoder = to_radeon_encoder(connector->encoder);
>  	else {
>  		const struct drm_connector_helper_funcs *connector_funcs = connector->helper_private;
> -		radeon_encoder = to_radeon_encoder(connector_funcs->best_encoder(connector));
> +		radeon_encoder = to_radeon_encoder(connector_funcs->best_encoder(connector, NULL));
>  	}
>  
>  	switch (value) {
> @@ -1044,7 +1045,7 @@ radeon_vga_detect(struct drm_connector *connector, bool force)
>  			return connector_status_disconnected;
>  	}
>  
> -	encoder = radeon_best_single_encoder(connector);
> +	encoder = radeon_best_single_encoder(connector, NULL);
>  	if (!encoder)
>  		ret = connector_status_disconnected;
>  
> @@ -1139,7 +1140,7 @@ static int radeon_tv_get_modes(struct drm_connector *connector)
>  	struct drm_display_mode *tv_mode;
>  	struct drm_encoder *encoder;
>  
> -	encoder = radeon_best_single_encoder(connector);
> +	encoder = radeon_best_single_encoder(connector, NULL);
>  	if (!encoder)
>  		return 0;
>  
> @@ -1182,7 +1183,7 @@ radeon_tv_detect(struct drm_connector *connector, bool force)
>  			return connector_status_disconnected;
>  	}
>  
> -	encoder = radeon_best_single_encoder(connector);
> +	encoder = radeon_best_single_encoder(connector, NULL);
>  	if (!encoder)
>  		ret = connector_status_disconnected;
>  	else {
> @@ -1439,7 +1440,7 @@ radeon_dvi_detect(struct drm_connector *connector, bool force)
>  		const struct drm_connector_helper_funcs *connector_funcs =
>  			connector->helper_private;
>  
> -		encoder = connector_funcs->best_encoder(connector);
> +		encoder = connector_funcs->best_encoder(connector, NULL);
>  		if (encoder && (encoder->encoder_type == DRM_MODE_ENCODER_TMDS)) {
>  			radeon_connector_get_edid(connector);
>  			radeon_audio_detect(connector, encoder, ret);
> @@ -1456,7 +1457,8 @@ radeon_dvi_detect(struct drm_connector *connector, bool force)
>  }
>  
>  /* okay need to be smart in here about which encoder to pick */
> -static struct drm_encoder *radeon_dvi_encoder(struct drm_connector *connector)
> +static struct drm_encoder *radeon_dvi_encoder(struct drm_connector *connector,
> +					      struct drm_crtc *crtc)
>  {
>  	int enc_id = connector->encoder_ids[0];
>  	struct radeon_connector *radeon_connector = to_radeon_connector(connector);
> @@ -1556,7 +1558,7 @@ static int radeon_dp_get_modes(struct drm_connector *connector)
>  {
>  	struct radeon_connector *radeon_connector = to_radeon_connector(connector);
>  	struct radeon_connector_atom_dig *radeon_dig_connector = radeon_connector->con_priv;
> -	struct drm_encoder *encoder = radeon_best_single_encoder(connector);
> +	struct drm_encoder *encoder = radeon_best_single_encoder(connector, NULL);
>  	int ret;
>  
>  	if ((connector->connector_type == DRM_MODE_CONNECTOR_eDP) ||
> @@ -1695,7 +1697,7 @@ radeon_dp_detect(struct drm_connector *connector, bool force)
>  	struct radeon_connector *radeon_connector = to_radeon_connector(connector);
>  	enum drm_connector_status ret = connector_status_disconnected;
>  	struct radeon_connector_atom_dig *radeon_dig_connector = radeon_connector->con_priv;
> -	struct drm_encoder *encoder = radeon_best_single_encoder(connector);
> +	struct drm_encoder *encoder = radeon_best_single_encoder(connector, NULL);
>  	int r;
>  
>  	if (radeon_dig_connector->is_mst)
> @@ -1812,7 +1814,7 @@ static enum drm_mode_status radeon_dp_mode_valid(struct drm_connector *connector
>  
>  	if ((connector->connector_type == DRM_MODE_CONNECTOR_eDP) ||
>  	    (connector->connector_type == DRM_MODE_CONNECTOR_LVDS)) {
> -		struct drm_encoder *encoder = radeon_best_single_encoder(connector);
> +		struct drm_encoder *encoder = radeon_best_single_encoder(connector, NULL);
>  
>  		if ((mode->hdisplay < 320) || (mode->vdisplay < 240))
>  			return MODE_PANEL;
> diff --git a/drivers/gpu/drm/radeon/radeon_dp_mst.c b/drivers/gpu/drm/radeon/radeon_dp_mst.c
> index cd8a3ee16649..fd5a0dae0e1f 100644
> --- a/drivers/gpu/drm/radeon/radeon_dp_mst.c
> +++ b/drivers/gpu/drm/radeon/radeon_dp_mst.c
> @@ -224,7 +224,8 @@ radeon_dp_mst_mode_valid(struct drm_connector *connector,
>  }
>  
>  static struct
> -drm_encoder *radeon_mst_best_encoder(struct drm_connector *connector)
> +drm_encoder *radeon_mst_best_encoder(struct drm_connector *connector,
> +				     struct drm_crtc *crtc)
>  {
>  	struct radeon_connector *radeon_connector = to_radeon_connector(connector);
>  
> @@ -613,7 +614,7 @@ radeon_dp_create_fake_mst_encoder(struct radeon_connector *connector)
>  	struct radeon_encoder_mst *mst_enc;
>  	struct drm_encoder *encoder;
>  	const struct drm_connector_helper_funcs *connector_funcs = connector->base.helper_private;
> -	struct drm_encoder *enc_master = connector_funcs->best_encoder(&connector->base);
> +	struct drm_encoder *enc_master = connector_funcs->best_encoder(&connector->base, NULL);
>  
>  	DRM_DEBUG_KMS("enc master is %p\n", enc_master);
>  	radeon_encoder = kzalloc(sizeof(*radeon_encoder), GFP_KERNEL);
> diff --git a/drivers/gpu/drm/shmobile/shmob_drm_crtc.c b/drivers/gpu/drm/shmobile/shmob_drm_crtc.c
> index e7738939a86d..cdc991d5c28e 100644
> --- a/drivers/gpu/drm/shmobile/shmob_drm_crtc.c
> +++ b/drivers/gpu/drm/shmobile/shmob_drm_crtc.c
> @@ -668,7 +668,8 @@ static int shmob_drm_connector_get_modes(struct drm_connector *connector)
>  }
>  
>  static struct drm_encoder *
> -shmob_drm_connector_best_encoder(struct drm_connector *connector)
> +shmob_drm_connector_best_encoder(struct drm_connector *connector,
> +				 struct drm_crtc *crtc)
>  {
>  	struct shmob_drm_connector *scon = to_shmob_connector(connector);
>  
> diff --git a/drivers/gpu/drm/tilcdc/tilcdc_panel.c b/drivers/gpu/drm/tilcdc/tilcdc_panel.c
> index d616d64a6725..197962eca1af 100644
> --- a/drivers/gpu/drm/tilcdc/tilcdc_panel.c
> +++ b/drivers/gpu/drm/tilcdc/tilcdc_panel.c
> @@ -178,8 +178,9 @@ static int panel_connector_mode_valid(struct drm_connector *connector,
>  	return tilcdc_crtc_mode_valid(priv->crtc, mode);
>  }
>  
> -static struct drm_encoder *panel_connector_best_encoder(
> -		struct drm_connector *connector)
> +static struct drm_encoder *
> +panel_connector_best_encoder(struct drm_connector *connector,
> +			     struct drm_crtc *crtc)
>  {
>  	struct panel_connector *panel_connector = to_panel_connector(connector);
>  	return panel_connector->encoder;
> diff --git a/drivers/gpu/drm/tilcdc/tilcdc_tfp410.c b/drivers/gpu/drm/tilcdc/tilcdc_tfp410.c
> index c45cabb38db0..0ef8221c5334 100644
> --- a/drivers/gpu/drm/tilcdc/tilcdc_tfp410.c
> +++ b/drivers/gpu/drm/tilcdc/tilcdc_tfp410.c
> @@ -191,8 +191,9 @@ static int tfp410_connector_mode_valid(struct drm_connector *connector,
>  	return tilcdc_crtc_mode_valid(priv->crtc, mode);
>  }
>  
> -static struct drm_encoder *tfp410_connector_best_encoder(
> -		struct drm_connector *connector)
> +static struct drm_encoder *
> +tfp410_connector_best_encoder(struct drm_connector *connector,
> +			      struct drm_crtc *crtc)
>  {
>  	struct tfp410_connector *tfp410_connector = to_tfp410_connector(connector);
>  	return tfp410_connector->encoder;
> diff --git a/drivers/gpu/drm/udl/udl_connector.c b/drivers/gpu/drm/udl/udl_connector.c
> index 09dc585aa46f..803ae6515892 100644
> --- a/drivers/gpu/drm/udl/udl_connector.c
> +++ b/drivers/gpu/drm/udl/udl_connector.c
> @@ -145,7 +145,8 @@ udl_detect(struct drm_connector *connector, bool force)
>  }
>  
>  static struct drm_encoder*
> -udl_best_single_encoder(struct drm_connector *connector)
> +udl_best_single_encoder(struct drm_connector *connector,
> +			struct drm_crtc *crtc)
>  {
>  	int enc_id = connector->encoder_ids[0];
>  	return drm_encoder_find(connector->dev, NULL, enc_id);
> diff --git a/drivers/staging/vboxvideo/vbox_mode.c b/drivers/staging/vboxvideo/vbox_mode.c
> index b265fe924556..bf0e93705e1c 100644
> --- a/drivers/staging/vboxvideo/vbox_mode.c
> +++ b/drivers/staging/vboxvideo/vbox_mode.c
> @@ -370,8 +370,9 @@ static void vbox_encoder_destroy(struct drm_encoder *encoder)
>  	kfree(encoder);
>  }
>  
> -static struct drm_encoder *vbox_best_single_encoder(struct drm_connector
> -						    *connector)
> +static struct drm_encoder *
> +vbox_best_single_encoder(struct drm_connector *connector,
> +			 struct drm_crtc *crtc)
>  {
>  	int enc_id = connector->encoder_ids[0];
>  
> diff --git a/include/drm/drm_atomic_helper.h b/include/drm/drm_atomic_helper.h
> index 26aaba58d6ce..e0f00d43ed63 100644
> --- a/include/drm/drm_atomic_helper.h
> +++ b/include/drm/drm_atomic_helper.h
> @@ -143,7 +143,8 @@ int drm_atomic_helper_page_flip_target(
>  				uint32_t target,
>  				struct drm_modeset_acquire_ctx *ctx);
>  struct drm_encoder *
> -drm_atomic_helper_best_encoder(struct drm_connector *connector);
> +drm_atomic_helper_best_encoder(struct drm_connector *connector,
> +			       struct drm_crtc *crtc);
>  
>  /* default implementations for state handling */
>  void drm_atomic_helper_crtc_reset(struct drm_crtc *crtc);
> diff --git a/include/drm/drm_modeset_helper_vtables.h b/include/drm/drm_modeset_helper_vtables.h
> index 35e2a3a79fc5..a9b3a2c50877 100644
> --- a/include/drm/drm_modeset_helper_vtables.h
> +++ b/include/drm/drm_modeset_helper_vtables.h
> @@ -885,7 +885,8 @@ struct drm_connector_helper_funcs {
>  	/**
>  	 * @best_encoder:
>  	 *
> -	 * This function should select the best encoder for the given connector.
> +	 * This function should select the best encoder for the given connector
> +	 * and crtc. For some driver internal use crtc may be NULL.
>  	 *
>  	 * This function is used by both the atomic helpers (in the
>  	 * drm_atomic_helper_check_modeset() function) and in the legacy CRTC
> @@ -911,7 +912,8 @@ struct drm_connector_helper_funcs {
>  	 * will ensure that encoders aren't used twice, drivers should not check
>  	 * for this.
>  	 */
> -	struct drm_encoder *(*best_encoder)(struct drm_connector *connector);
> +	struct drm_encoder *(*best_encoder)(struct drm_connector *connector,
> +					    struct drm_crtc *crtc);
>  
>  	/**
>  	 * @atomic_best_encoder:
> -- 
> 2.16.4
> 
> _______________________________________________
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch


More information about the Intel-gfx mailing list