Commit b931f0e4 authored Feb 02, 2017 by Puja Gupta Committed by Gerrit - the friendly Code Review server Oct 04, 2017

soc: qcom: pil: Fix error case scenario in subsystem ramdump



During ramdump collection we assign memory to HLOS from subsystem for
non-secure pil. Whether ramdump collection is successful or not, we
should assign memory back to subsystem. This is to avoid access
violations in powerup path which happens after ramdump.

CRs-Fixed: 2002073
Change-Id: I7f1d42aebb44464fe077ca544ce91c2d7a8eefbb
Signed-off-by: Puja Gupta <pujag@codeaurora.org>

parent 047ccea1

drivers/soc/qcom/peripheral-loader.c

+5 −1

Original line number	Diff line number	Diff line
		@@ -172,7 +172,11 @@ int pil_do_ramdump(struct pil_desc desc, void ramdump_dev)
		ret = do_elf_ramdump(ramdump_dev, ramdump_segs, count);
		kfree(ramdump_segs);

		if (!ret && desc->subsys_vmid > 0)
		if (ret)
		pil_err(desc, "%s: Ramdump collection failed for subsys %s rc:%d\n",
		__func__, desc->name, ret);

		if (desc->subsys_vmid > 0)
		ret = pil_assign_mem_to_subsys(desc, priv->region_start,
		(priv->region_end - priv->region_start));