Commit 7b9fd195 authored Sep 16, 2020 by Deepak Kumar Committed by Pranav Patel Oct 30, 2020

msm: kgsl: Don't skip gmufw preallocations during firmware read



In scenario where GMU request firmware succeeded but preallocation
failed, a6xx_gmu_load_firmware returns error but during next open
call it return success only based on whether request_firmware was
successful or not. This results in issue during GMU start while
copying firmware to respective GMU memory blocks as some of the
memory blocks were not allocated.

Update the check in a6xx_gmu_load_firmware to ensure only request
firmware is skipped if it passed earlier.

Change-Id: I92920215aa9f47726c4470968ca804e1cee13468
Signed-off-by: Deepak Kumar <dkumar@codeaurora.org>
Signed-off-by: Pranav Patel <pranavp@codeaurora.org>

parent c19cc3d7

drivers/gpu/msm/adreno_a6xx_gmu.c

+10 −10

Original line number	Diff line number	Diff line
		@@ -1529,8 +1529,7 @@ int a6xx_gmu_parse_fw(struct adreno_device *adreno_dev)
		int ret, offset = 0;

		/* GMU fw already saved and verified so do nothing new */
		if (gmu->fw_image)
		return 0;
		if (!gmu->fw_image) {

		if (a6xx_core->gmufw_name == NULL)
		return -EINVAL;
		@@ -1542,6 +1541,7 @@ int a6xx_gmu_parse_fw(struct adreno_device *adreno_dev)
		a6xx_core->gmufw_name, ret);
		return ret;
		}
		}

		/*
		* Zero payload fw blocks contain meta data and are