Re: [PATCH i-g-t 1/6] lib/igt_kms: fix clearing up connector/pipe state on atomic commit

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, May 12, 2016 at 01:58:07PM +0100, Lionel Landwerlin wrote:
> Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@xxxxxxxxx>
> ---
>  lib/igt_kms.c | 10 ++++------
>  1 file changed, 4 insertions(+), 6 deletions(-)
> 
> diff --git a/lib/igt_kms.c b/lib/igt_kms.c
> index 4da645a..795ecb0 100644
> --- a/lib/igt_kms.c
> +++ b/lib/igt_kms.c
> @@ -2026,9 +2026,8 @@ static void igt_atomic_prepare_crtc_commit(igt_output_t *output, drmModeAtomicRe
>  		igt_atomic_populate_crtc_req(req, output, IGT_CRTC_GAMMA_LUT, pipe_obj->gamma_blob);
>  	}
>  
> -	/*
> -	 *	TODO: Add all crtc level properties here
> -	 */
> +	pipe_obj->background_changed = false;
> +	pipe_obj->color_mgmt_changed = false;
>  }
Just noticed *_changed are defined as uint32_t *_changed : 1. Maybe
change to bool in igt_pipe_t.
>  
>  /*
> @@ -2044,10 +2043,9 @@ static void igt_atomic_prepare_connector_commit(igt_output_t *output, drmModeAto
>  
>  	if (config->connector_dpms_changed)
>  		igt_atomic_populate_connector_req(req, output, IGT_CONNECTOR_DPMS, config->connector_dpms);
> -	/*
> -	 *	TODO: Add all other connector level properties here
> -	 */
>  
> +	config->connector_scaling_mode_changed = false;
> +	config->connector_dpms_changed = false;
>  }
>  
>  /*
> -- 
> 2.8.1
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Attachment: signature.asc
Description: Digital signature

_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux