[Intel-gfx] [PATCH i-g-t 2/5] lib/igt_kms: Neaten up pipe->planes[] assignment

Robert Foss robert.foss at collabora.com
Sun Feb 19 20:41:45 UTC 2017



On 2017-02-17 12:54 PM, Brian Starkey wrote:
> Remove a bunch of branches, functionally equivalent.
>
> Signed-off-by: Brian Starkey <brian.starkey at arm.com>
> ---
>  lib/igt_kms.c |   34 ++++++++++------------------------
>  1 file changed, 10 insertions(+), 24 deletions(-)
>
> diff --git a/lib/igt_kms.c b/lib/igt_kms.c
> index 4ca9145726e2..783c891aebf1 100644
> --- a/lib/igt_kms.c
> +++ b/lib/igt_kms.c
> @@ -1785,32 +1785,18 @@ void igt_display_init(igt_display_t *display, int drm_fd)
>
>  			type = get_drm_plane_type(display->drm_fd,
>  						  plane_resources->planes[j]);
> -			switch (type) {
> -			case DRM_PLANE_TYPE_PRIMARY:
> -				if (pipe->plane_primary == -1) {
> -					plane = &pipe->planes[0];
> -					plane->index = 0;
> -					pipe->plane_primary = 0;
> -				} else {
> -					plane = &pipe->planes[p];
> -					plane->index = p++;
> -				}
> -				break;
> -			case DRM_PLANE_TYPE_CURSOR:
> -				if (pipe->plane_cursor == -1) {
> -					plane = &pipe->planes[last_plane];
> -					plane->index = last_plane;
> -					pipe->plane_cursor = last_plane;
> -				} else {
> -					plane = &pipe->planes[p];
> -					plane->index = p++;
> -				}
> -				display->has_cursor_plane = true;

The display->has_cursor_plane assignment does not seem to appear in the 
refactored code.

With that fixed this patch is:
Reviewed-by: Robert Foss <robert.foss at collabora.com>


Rob.

> -				break;
> -			default:
> +
> +			if (type == DRM_PLANE_TYPE_PRIMARY && pipe->plane_primary == -1) {
> +				plane = &pipe->planes[0];
> +				plane->index = 0;
> +				pipe->plane_primary = 0;
> +			} else if (type == DRM_PLANE_TYPE_CURSOR && pipe->plane_cursor == -1) {
> +				plane = &pipe->planes[last_plane];
> +				plane->index = last_plane;
> +				pipe->plane_cursor = last_plane;
> +			} else {
>  				plane = &pipe->planes[p];
>  				plane->index = p++;
> -				break;
>  			}
>
>  			igt_assert_f(plane->index < n_planes, "n_planes < plane->index failed\n");
>


More information about the Intel-gfx mailing list