[Intel-gfx] [PATCH 12/22] drm/i915: Set i915_ggtt_view_normal type explicitly

Tvrtko Ursulin tvrtko.ursulin at linux.intel.com
Thu Oct 15 04:15:21 PDT 2015


Hi,

On 14/10/15 17:29, ville.syrjala at linux.intel.com wrote:
> From: Ville Syrjälä <ville.syrjala at linux.intel.com>
>
> Just for clarity set the type for i915_ggtt_view_normal explicitly.
>
> While at it fix the indentation fail for i915_ggtt_view_rotated.
>
> Signed-off-by: Ville Syrjälä <ville.syrjala at linux.intel.com>
> ---
>   drivers/gpu/drm/i915/i915_gem_gtt.c | 6 ++++--
>   1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/i915_gem_gtt.c b/drivers/gpu/drm/i915/i915_gem_gtt.c
> index 620d57e..71acc71 100644
> --- a/drivers/gpu/drm/i915/i915_gem_gtt.c
> +++ b/drivers/gpu/drm/i915/i915_gem_gtt.c
> @@ -95,9 +95,11 @@
>   static int
>   i915_get_ggtt_vma_pages(struct i915_vma *vma);
>
> -const struct i915_ggtt_view i915_ggtt_view_normal;
> +const struct i915_ggtt_view i915_ggtt_view_normal = {
> +	.type = I915_GGTT_VIEW_NORMAL,
> +};

AFAIR while this was developed Daniel was very keen on the normal view 
type being implicitly zero. Can't remember at this very moment if some 
of the current code actually depends on it though.

Regards,

Tvrtko


More information about the Intel-gfx mailing list