[PATCH 5/5] drm/amd/display: Rename atomic_commit parameter.

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

 



> -----Original Message-----
> From: amd-gfx [mailto:amd-gfx-bounces at lists.freedesktop.org] On Behalf
> Of Harry Wentland
> Sent: Wednesday, January 25, 2017 3:34 PM
> To: amd-gfx at lists.freedesktop.org
> Cc: Grodzovsky, Andrey
> Subject: [PATCH 5/5] drm/amd/display: Rename atomic_commit parameter.
> 
> From: Andrey Grodzovsky <Andrey.Grodzovsky at amd.com>
> 
> This parameter name is misleading. It's previous
> meaning confuses with ASYNC (immidiate flip)
> which is totatlly different thing then nonblocking
> commit.
> 
> Change-Id: Ia8ed1fad025b9b1344a470eff100da9cb1c46d31
> Signed-off-by: Andrey Grodzovsky <Andrey.Grodzovsky at amd.com>
> Reviewed-by: Harry Wentland <Harry.Wentland at amd.com>

Reviewed-by: Alex Deucher <alexander.deucher at amd.com>

> ---
>  drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_types.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_types.c
> b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_types.c
> index a26749854ec7..865ee1ecce4f 100644
> --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_types.c
> +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_types.c
> @@ -2481,7 +2481,7 @@ static void remove_stream(struct amdgpu_device
> *adev, struct amdgpu_crtc *acrtc)
>  int amdgpu_dm_atomic_commit(
>  	struct drm_device *dev,
>  	struct drm_atomic_state *state,
> -	bool async)
> +	bool nonblock)
>  {
>  	struct amdgpu_device *adev = dev->dev_private;
>  	struct amdgpu_display_manager *dm = &adev->dm;
> @@ -2510,7 +2510,7 @@ int amdgpu_dm_atomic_commit(
>  	 * we should not pin/unpin the fb here, it should be done in
>  	 * amdgpu_crtc_flip and from the vblank irq handler.
>  	 */
> -	if (!async) {
> +	if (!nonblock) {
>  		ret = drm_atomic_helper_prepare_planes(dev, state);
>  		if (ret)
>  			return ret;
> @@ -2849,7 +2849,7 @@ int amdgpu_dm_atomic_commit(
>  	/* In this state all old framebuffers would be unpinned */
> 
>  	/* TODO: Revisit when we support true asynchronous commit.*/
> -	if (!async)
> +	if (!nonblock)
>  		drm_atomic_helper_cleanup_planes(dev, state);
> 
>  	drm_atomic_state_free(state);
> --
> 2.9.3
> 
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx


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

  Powered by Linux