[Intel-gfx] [PATCH] drm/i915/selftests: Markup __iomem for igt_gem_coherency

[Mika Kuoppala]

Chris Wilson <chris at chris-wilson.co.uk> writes:

> Silence sparse warnings by using __iomem markup and io accessors.
>
> Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>

Reviewed-by: Mika Kuoppala <mika.kuoppala at linux.intel.com>

> ---
>  drivers/gpu/drm/i915/selftests/i915_gem_coherency.c | 16 ++++++++--------
>  1 file changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/selftests/i915_gem_coherency.c b/drivers/gpu/drm/i915/selftests/i915_gem_coherency.c
> index 35d778d70626..7a0d1e17c1ad 100644
> --- a/drivers/gpu/drm/i915/selftests/i915_gem_coherency.c
> +++ b/drivers/gpu/drm/i915/selftests/i915_gem_coherency.c
> @@ -33,7 +33,7 @@ static int cpu_set(struct drm_i915_gem_object *obj,
>  {
>  	unsigned int needs_clflush;
>  	struct page *page;
> -	typeof(v) *map;
> +	u32 *map;
>  	int err;
>  
>  	err = i915_gem_obj_prepare_shmem_write(obj, &needs_clflush);
> @@ -59,7 +59,7 @@ static int cpu_get(struct drm_i915_gem_object *obj,
>  {
>  	unsigned int needs_clflush;
>  	struct page *page;
> -	typeof(v) map;
> +	u32 *map;
>  	int err;
>  
>  	err = i915_gem_obj_prepare_shmem_read(obj, &needs_clflush);
> @@ -82,7 +82,7 @@ static int gtt_set(struct drm_i915_gem_object *obj,
>  		   u32 v)
>  {
>  	struct i915_vma *vma;
> -	typeof(v) *map;
> +	u32 __iomem *map;
>  	int err;
>  
>  	err = i915_gem_object_set_to_gtt_domain(obj, true);
> @@ -98,7 +98,7 @@ static int gtt_set(struct drm_i915_gem_object *obj,
>  	if (IS_ERR(map))
>  		return PTR_ERR(map);
>  
> -	map[offset / sizeof(*map)] = v;
> +	iowrite32(v, &map[offset / sizeof(*map)]);
>  	i915_vma_unpin_iomap(vma);
>  
>  	return 0;
> @@ -109,7 +109,7 @@ static int gtt_get(struct drm_i915_gem_object *obj,
>  		   u32 *v)
>  {
>  	struct i915_vma *vma;
> -	typeof(v) map;
> +	u32 __iomem *map;
>  	int err;
>  
>  	err = i915_gem_object_set_to_gtt_domain(obj, false);
> @@ -125,7 +125,7 @@ static int gtt_get(struct drm_i915_gem_object *obj,
>  	if (IS_ERR(map))
>  		return PTR_ERR(map);
>  
> -	*v = map[offset / sizeof(*map)];
> +	*v = ioread32(&map[offset / sizeof(*map)]);
>  	i915_vma_unpin_iomap(vma);
>  
>  	return 0;
> @@ -135,7 +135,7 @@ static int wc_set(struct drm_i915_gem_object *obj,
>  		  unsigned long offset,
>  		  u32 v)
>  {
> -	typeof(v) *map;
> +	u32 *map;
>  	int err;
>  
>  	err = i915_gem_object_set_to_wc_domain(obj, true);
> @@ -156,7 +156,7 @@ static int wc_get(struct drm_i915_gem_object *obj,
>  		  unsigned long offset,
>  		  u32 *v)
>  {
> -	typeof(v) map;
> +	u32 *map;
>  	int err;
>  
>  	err = i915_gem_object_set_to_wc_domain(obj, false);
> -- 
> 2.15.0
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx