[igt-dev] [PATCH i-g-t 5/5] lib/igt_fb: s/tiling/modifier/ where appropriate

Dhinakaran Pandiyan dhinakaran.pandiyan at intel.com
Wed Oct 31 22:48:09 UTC 2018


On Tuesday, October 30, 2018 9:20:38 AM PDT Ville Syrjala wrote:
> From: Ville Syrjälä <ville.syrjala at linux.intel.com>
> 
> Rename the igt_fb.tiling to igt_fb.modifier to better reflect
> what information it carries. Now it's clear whether we're talking
> about a modifier or a i915 tiling thing.
> 

Makes sense,
Reviewed-by: Dhinakaran Pandiyan <dhinakaran.pandiyan at intel.com>

We should probably rename the locals in the callers for more clarity. kms_flip_tiling.c, 
kms_rotation_crc.c, pm_rpm.c and kms_available_modes_crc.c store the modifier as
"tiling". Is this something that can be scripted easily?

-DK

> Signed-off-by: Ville Syrjälä <ville.syrjala at linux.intel.com>
> ---
>  lib/igt_fb.c    | 102 ++++++++++++++++++++++++------------------------
>  lib/igt_fb.h    |  22 +++++------
>  tests/kms_ccs.c |   2 +-









More information about the igt-dev mailing list