Re: [PATCH i-g-t v2 08/15] igt_kms: Remove pan members from igt_plane, v2.

[Maarten Lankhorst <maarten.lankhorst@linux.intel.com>]

Op 21-07-16 om 13:42 schreef Ander Conselvan De Oliveira:
> On Wed, 2016-07-06 at 11:55 +0200, Maarten Lankhorst wrote:
>> They're duplicates with src_x/y, so just use those.
>>
>> Changes since v1:
>> - Fix order of parameters in calls to igt_fb_set_position.
>>
>> Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
>> ---
>>  lib/igt_kms.c             | 24 ++++--------------------
>>  lib/igt_kms.h             |  4 ----
>>  tests/kms_panel_fitting.c |  2 +-
>>  tests/kms_plane.c         |  8 +++-----
>>  tests/kms_plane_scaling.c |  2 +-
>>  5 files changed, 9 insertions(+), 31 deletions(-)
>>
>> diff --git a/lib/igt_kms.c b/lib/igt_kms.c
>> index 8c7598a88a69..9fc5559a5df4 100644
>> --- a/lib/igt_kms.c
>> +++ b/lib/igt_kms.c
>> @@ -1915,7 +1915,7 @@ static int igt_primary_plane_commit_legacy(igt_plane_t
>> *primary,
>>  	igt_assert(!primary->rotation_changed);
>>  
>>  	if (!primary->fb_changed && !primary->position_changed &&
>> -		!primary->size_changed && !primary->panning_changed)
>> +		!primary->size_changed)
>>  		return 0;
>>  
>>  	crtc_id = pipe->crtc_id;
>> @@ -1927,18 +1927,18 @@ static int igt_primary_plane_commit_legacy(igt_plane_t
>> *primary,
>>  
>>  	if (fb_id) {
>>  		LOG(display,
>> -		    "%s: SetCrtc pipe %s, fb %u, panning (%d, %d), "
>> +		    "%s: SetCrtc pipe %s, fb %u, src (%d, %d), "
>>  		    "mode %dx%d\n",
>>  		    igt_output_name(output),
>>  		    kmstest_pipe_name(pipe->pipe),
>>  		    fb_id,
>> -		    primary->pan_x, primary->pan_y,
>> +		    primary->src_x, primary->src_x,
>>  		    mode->hdisplay, mode->vdisplay);
>>  
>>  		ret = drmModeSetCrtc(display->drm_fd,
>>  				     crtc_id,
>>  				     fb_id,
>> -				     primary->pan_x, primary->pan_y,
>> +				     primary->src_x, primary->src_x,
>>  				     &output->id,
>>  				     1,
>>  				     mode);
>> @@ -1962,7 +1962,6 @@ static int igt_primary_plane_commit_legacy(igt_plane_t
>> *primary,
>>  	primary->fb_changed = false;
>>  	primary->position_changed = false;
>>  	primary->size_changed = false;
>> -	primary->panning_changed = false;
>>  
>>  	return 0;
>>  }
>> @@ -2408,21 +2407,6 @@ void igt_fb_set_size(struct igt_fb *fb, igt_plane_t
>> *plane,
>>  	plane->fb_changed = true;
>>  }
>>  
>> -void igt_plane_set_panning(igt_plane_t *plane, int x, int y)
>> -{
>> -	igt_pipe_t *pipe = plane->pipe;
>> -	igt_display_t *display = pipe->display;
>> -
>> -	LOG(display, "%s.%d: plane_set_panning(%d,%d)\n",
>> -	    kmstest_pipe_name(pipe->pipe),
>> -	    plane->index, x, y);
>> -
>> -	plane->pan_x = x;
>> -	plane->pan_y = y;
>> -
>> -	plane->panning_changed = true;
>> -}
>> -
>>  static const char *rotation_name(igt_rotation_t rotation)
>>  {
>>  	switch (rotation) {
>> diff --git a/lib/igt_kms.h b/lib/igt_kms.h
>> index b9e8d344ed8c..4d18c01042c9 100644
>> --- a/lib/igt_kms.h
>> +++ b/lib/igt_kms.h
>> @@ -220,7 +220,6 @@ typedef struct {
>>  	/* state tracking */
>>  	unsigned int fb_changed       : 1;
>>  	unsigned int position_changed : 1;
>> -	unsigned int panning_changed  : 1;
>>  	unsigned int rotation_changed : 1;
>>  	unsigned int size_changed     : 1;
>>  	/*
>> @@ -244,8 +243,6 @@ typedef struct {
>>  	uint32_t src_w;
>>  	uint32_t src_h;
>>  
>> -	/* panning offset within the fb */
>> -	unsigned int pan_x, pan_y;
>>  	igt_rotation_t rotation;
>>  	uint32_t atomic_props_plane[IGT_NUM_PLANE_PROPS];
>>  } igt_plane_t;
>> @@ -326,7 +323,6 @@ void igt_pipe_set_gamma_lut(igt_pipe_t *pipe, void *ptr,
>> size_t length);
>>  void igt_plane_set_fb(igt_plane_t *plane, struct igt_fb *fb);
>>  void igt_plane_set_position(igt_plane_t *plane, int x, int y);
>>  void igt_plane_set_size(igt_plane_t *plane, int w, int h);
>> -void igt_plane_set_panning(igt_plane_t *plane, int x, int y);
>>  void igt_plane_set_rotation(igt_plane_t *plane, igt_rotation_t rotation);
>>  void igt_crtc_set_background(igt_pipe_t *pipe, uint64_t background);
>>  void igt_fb_set_position(struct igt_fb *fb, igt_plane_t *plane,
>> diff --git a/tests/kms_panel_fitting.c b/tests/kms_panel_fitting.c
>> index 7c501fcdd3fb..da09da32d704 100644
>> --- a/tests/kms_panel_fitting.c
>> +++ b/tests/kms_panel_fitting.c
>> @@ -89,7 +89,7 @@ static void prepare_crtc(data_t *data, igt_output_t *output,
>> enum pipe pipe,
>>  		ret = drmModeSetCrtc(data->drm_fd,
>>  				plane->pipe->crtc_id,
>>  				data->fb_id1,
>> -				plane->pan_x, plane->pan_y,
>> +				plane->src_x, plane->src_y,
>>  				&output->id,
>>  				1,
>>  				mode);
>> diff --git a/tests/kms_plane.c b/tests/kms_plane.c
>> index 60b8d71d52d4..7c01fe91b263 100644
>> --- a/tests/kms_plane.c
>> +++ b/tests/kms_plane.c
>> @@ -321,11 +321,9 @@ test_plane_panning_with_output(data_t *data,
>>  	igt_plane_set_fb(primary, &primary_fb);
>>  
>>  	if (flags & TEST_PANNING_TOP_LEFT)
>> -		igt_plane_set_panning(primary, 0, 0);
>> +		igt_fb_set_position(&primary_fb, primary, 0, 0);
> While at it you might want to rename igt_fb_set_position() to
> igt_plane_set_position() or something. The name is very misleading as it doens't
> depend on the fb and only sets properties on the plane.
That one already exists for dst, but yeah it should be renamed to igt_plane_set_src_position,
the fb argument is unused anyway..

Will probably just commit some patches to do so later on.
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx