[RFC weston 02/14] compositor-drm: Refactor initial mode out of create_output

[Bryce Harrington]

On Thu, May 21, 2015 at 08:28:59AM +0100, Daniel Stone wrote:
> From: Pekka Paalanen <pekka.paalanen at collabora.co.uk>
> 
> Refactor the code for choosing the initial mode for an output from
> create_output_for_connector() to drm_output_choose_initial_mode().
> 
> This makes create_output_for_connector() slightly easier to read.
> 
> Signed-off-by: Pekka Paalanen <pekka.paalanen at collabora.co.uk>

Why not doxygen the new routine while you're at it?

While it's internal, this modesetting code will likely get a lot of
eyeballs in the future. 

> ---
>  src/compositor-drm.c | 119 ++++++++++++++++++++++++++++++---------------------
>  1 file changed, 70 insertions(+), 49 deletions(-)
> 
> diff --git a/src/compositor-drm.c b/src/compositor-drm.c
> index bf921be..516693b 100644
> --- a/src/compositor-drm.c
> +++ b/src/compositor-drm.c
> @@ -1460,7 +1460,7 @@ init_pixman(struct drm_compositor *ec)
>  }
>  
>  static struct drm_mode *
> -drm_output_add_mode(struct drm_output *output, drmModeModeInfo *info)
> +drm_output_add_mode(struct drm_output *output, const drmModeModeInfo *info)
>  {
>  	struct drm_mode *mode;
>  	uint64_t refresh;
> @@ -1973,6 +1973,69 @@ get_gbm_format_from_section(struct weston_config_section *section,
>  	return ret;
>  }
>  
> +static struct drm_mode *
> +drm_output_choose_initial_mode(struct drm_output *output,
> +			       enum output_config kind,
> +			       int width, int height,
> +			       const drmModeModeInfo *current_mode,
> +			       const drmModeModeInfo *modeline)
> +{
> +	struct drm_mode *preferred = NULL;
> +	struct drm_mode *current = NULL;
> +	struct drm_mode *configured = NULL;
> +	struct drm_mode *best = NULL;
> +	struct drm_mode *drm_mode;
> +
> +	wl_list_for_each_reverse(drm_mode, &output->base.mode_list, base.link) {
> +		if (kind == OUTPUT_CONFIG_MODE &&
> +		    width == drm_mode->base.width &&
> +		    height == drm_mode->base.height)
> +			configured = drm_mode;
> +
> +		if (memcmp(&current_mode, &drm_mode->mode_info,
> +			   sizeof *current_mode) == 0)
> +			current = drm_mode;
> +
> +		if (drm_mode->base.flags & WL_OUTPUT_MODE_PREFERRED)
> +			preferred = drm_mode;
> +
> +		best = drm_mode;
> +	}
> +
> +	if (kind == OUTPUT_CONFIG_MODELINE) {
> +		configured = drm_output_add_mode(output, modeline);
> +		if (!configured)
> +			return NULL;
> +	}
> +
> +	if (current == NULL && current_mode->clock != 0) {
> +		current = drm_output_add_mode(output, current_mode);
> +		if (!current)
> +			return NULL;
> +	}
> +
> +	if (kind == OUTPUT_CONFIG_CURRENT)
> +		configured = current;
> +
> +	if (option_current_mode && current)
> +		return current;
> +
> +	if (configured)
> +		return configured;
> +
> +	if (preferred)
> +		return preferred;
> +
> +	if (current)
> +		return current;
> +
> +	if (best)
> +		return best;
> +
> +	weston_log("no available modes for %s\n", output->base.name);
> +	return NULL;
> +}
> +
>  static int
>  create_output_for_connector(struct drm_compositor *ec,
>  			    drmModeRes *resources,
> @@ -1980,7 +2043,7 @@ create_output_for_connector(struct drm_compositor *ec,
>  			    int x, int y, struct udev_device *drm_device)
>  {
>  	struct drm_output *output;
> -	struct drm_mode *drm_mode, *next, *preferred, *current, *configured, *best;
> +	struct drm_mode *drm_mode, *next, *current;
>  	struct weston_mode *m;
>  	struct weston_config_section *section;
>  	drmModeEncoder *encoder;
> @@ -2088,54 +2151,12 @@ create_output_for_connector(struct drm_compositor *ec,
>  		goto err_free;
>  	}
>  
> -	preferred = NULL;
> -	current = NULL;
> -	configured = NULL;
> -	best = NULL;
> -
> -	wl_list_for_each_reverse(drm_mode, &output->base.mode_list, base.link) {
> -		if (config == OUTPUT_CONFIG_MODE &&
> -		    width == drm_mode->base.width &&
> -		    height == drm_mode->base.height)
> -			configured = drm_mode;
> -		if (!memcmp(&crtc_mode, &drm_mode->mode_info, sizeof crtc_mode))
> -			current = drm_mode;
> -		if (drm_mode->base.flags & WL_OUTPUT_MODE_PREFERRED)
> -			preferred = drm_mode;
> -		best = drm_mode;
> -	}
> -
> -	if (config == OUTPUT_CONFIG_MODELINE) {
> -		configured = drm_output_add_mode(output, &modeline);
> -		if (!configured)
> -			goto err_free;
> -	}
> -
> -	if (current == NULL && crtc_mode.clock != 0) {
> -		current = drm_output_add_mode(output, &crtc_mode);
> -		if (!current)
> -			goto err_free;
> -	}
> -
> -	if (config == OUTPUT_CONFIG_CURRENT)
> -		configured = current;
> -
> -	if (option_current_mode && current)
> -		output->base.current_mode = &current->base;
> -	else if (configured)
> -		output->base.current_mode = &configured->base;
> -	else if (preferred)
> -		output->base.current_mode = &preferred->base;
> -	else if (current)
> -		output->base.current_mode = &current->base;
> -	else if (best)
> -		output->base.current_mode = &best->base;
> -
> -	if (output->base.current_mode == NULL) {
> -		weston_log("no available modes for %s\n", output->base.name);
> +	current = drm_output_choose_initial_mode(output, config,
> +						 width, height,
> +						 &crtc_mode, &modeline);
> +	if (!current)
>  		goto err_free;
> -	}
> -
> +	output->base.current_mode = &current->base;
>  	output->base.current_mode->flags |= WL_OUTPUT_MODE_CURRENT;
>  
>  	weston_output_init(&output->base, &ec->base, x, y,
> -- 
> 2.4.1
> 
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel