Commit d4035d10 authored by Marek Szyprowski's avatar Marek Szyprowski Committed by Inki Dae

drm/exynos: gem: Fix sparse warning

kvaddr element of the exynos_gem object points to a memory buffer, thus
it should not have a __iomem annotation. Then, to avoid a warning or
casting on assignment to fbi structure, the screen_buffer element of the
union should be used instead of the screen_base.
Reported-by: default avatarkernel test robot <lkp@intel.com>
Suggested-by: default avatarSam Ravnborg <sam@ravnborg.org>
Signed-off-by: default avatarMarek Szyprowski <m.szyprowski@samsung.com>
Reviewed-by: default avatarKrzysztof Kozlowski <krzk@kernel.org>
Reviewed-by: default avatarSam Ravnborg <sam@ravnborg.org>
Signed-off-by: default avatarInki Dae <inki.dae@samsung.com>
parent d012a719
...@@ -92,7 +92,7 @@ static int exynos_drm_fbdev_update(struct drm_fb_helper *helper, ...@@ -92,7 +92,7 @@ static int exynos_drm_fbdev_update(struct drm_fb_helper *helper,
offset = fbi->var.xoffset * fb->format->cpp[0]; offset = fbi->var.xoffset * fb->format->cpp[0];
offset += fbi->var.yoffset * fb->pitches[0]; offset += fbi->var.yoffset * fb->pitches[0];
fbi->screen_base = exynos_gem->kvaddr + offset; fbi->screen_buffer = exynos_gem->kvaddr + offset;
fbi->screen_size = size; fbi->screen_size = size;
fbi->fix.smem_len = size; fbi->fix.smem_len = size;
......
...@@ -40,7 +40,7 @@ struct exynos_drm_gem { ...@@ -40,7 +40,7 @@ struct exynos_drm_gem {
unsigned int flags; unsigned int flags;
unsigned long size; unsigned long size;
void *cookie; void *cookie;
void __iomem *kvaddr; void *kvaddr;
dma_addr_t dma_addr; dma_addr_t dma_addr;
unsigned long dma_attrs; unsigned long dma_attrs;
struct sg_table *sgt; struct sg_table *sgt;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment