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

Harry Wentland harry.wentland at amd.com
Tue Jun 26 15:45:59 UTC 2018


On 2018-06-26 11:01 AM, Harry Wentland wrote:
> On 2018-06-15 03:52 PM, 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.
>>
>> 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>
> 
> amdgpu parts are
> Acked-by: Harry Wentland <harry.wentland at amd.com>
> 

Upgrading after proper review of the entire patch

Reviewed-by: Harry Wentland <harry.wentland at amd.com>

Harry

> Harry
> 
>> ---
>>  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:
>>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
> 


More information about the Intel-gfx mailing list