Commit 96c876f1 authored Oct 23, 2020 by Rob Clark

drm/msm: Fix a couple incorrect usages of get_vaddr_active()



The microcode bo's should never be madvise(WONTNEED), so these should
not be using msm_gem_get_vaddr_active().

Signed-off-by: Rob Clark <robdclark@chromium.org>
Reviewed-by: Kristian H. Kristensen <hoegsberg@google.com>
Signed-off-by: Rob Clark <robdclark@chromium.org>

parent 0e720ea8

drivers/gpu/drm/msm/adreno/a5xx_gpu.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -426,7 +426,7 @@ static int a5xx_preempt_start(struct msm_gpu *gpu)
		static void a5xx_ucode_check_version(struct a5xx_gpu *a5xx_gpu,
		struct drm_gem_object *obj)
		{
		u32 *buf = msm_gem_get_vaddr_active(obj);
		u32 *buf = msm_gem_get_vaddr(obj);

		if (IS_ERR(buf))
		return;

drivers/gpu/drm/msm/adreno/a6xx_gpu.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -522,7 +522,7 @@ static int a6xx_cp_init(struct msm_gpu *gpu)
		static void a6xx_ucode_check_version(struct a6xx_gpu *a6xx_gpu,
		struct drm_gem_object *obj)
		{
		u32 *buf = msm_gem_get_vaddr_active(obj);
		u32 *buf = msm_gem_get_vaddr(obj);

		if (IS_ERR(buf))
		return;