Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 4f261955 authored by Arun KS's avatar Arun KS Committed by Gerrit - the friendly Code Review server
Browse files

soc: qcom: pil-q6v5-mss: Do not free memory allocated to modem pil 



Modem now uses new memory feature(fixup-reserve-region). Instead of
fixed carve-out, region can adapt and shrink to actual memory
requirement after first allocation. This memory is not considered
as carve out and is exposed to kernel. When memory is freed, kernel
maps the region as normal cacheable memory. This results in XPU
violation errors because the region is still marked as XPU protected
by MBA. Hence modem pil is avoiding freeing of memory and reusing
the allocated region.

Change-Id: I507f4a09e6f86dc8584fb23877fe9eab51ef9bb6
Signed-off-by: default avatarArun KS <arunks@codeaurora.org>
parent 122fe9b1

drivers/soc/qcom/pil-q6v5-mss.c

+0 −8
Original line number Diff line number Diff line
@@ -154,13 +154,6 @@ static void modem_crash_shutdown(const struct subsys_desc *subsys) 
	}

}



static void modem_free_memory(const struct subsys_desc *subsys)

{

	struct modem_data *drv = subsys_to_drv(subsys);



	pil_free_memory(&drv->q6->desc);

}



static int modem_ramdump(int enable, const struct subsys_desc *subsys)

{

	struct modem_data *drv = subsys_to_drv(subsys);
@@ -216,7 +209,6 @@ static int pil_subsys_init(struct modem_data *drv, 
	drv->subsys_desc.shutdown = modem_shutdown;

	drv->subsys_desc.powerup = modem_powerup;

	drv->subsys_desc.ramdump = modem_ramdump;

	drv->subsys_desc.free_memory = modem_free_memory;

	drv->subsys_desc.crash_shutdown = modem_crash_shutdown;

	drv->subsys_desc.err_fatal_handler = modem_err_fatal_intr_handler;

	drv->subsys_desc.stop_ack_handler = modem_stop_ack_intr_handler;