xref: /openbmc/linux/drivers/media/platform/qcom/venus/firmware.c (revision c900529f3d9161bfde5cca0754f83b4d3c3e0220)
197fb5e8dSThomas Gleixner // SPDX-License-Identifier: GPL-2.0-only
2af2c3834SStanimir Varbanov /*
3af2c3834SStanimir Varbanov  * Copyright (C) 2017 Linaro Ltd.
4af2c3834SStanimir Varbanov  */
5af2c3834SStanimir Varbanov 
6377a22d3SStanimir Varbanov #include <linux/device.h>
7af2c3834SStanimir Varbanov #include <linux/firmware.h>
8af2c3834SStanimir Varbanov #include <linux/kernel.h>
9df381dc8SVikash Garodia #include <linux/iommu.h>
10377a22d3SStanimir Varbanov #include <linux/io.h>
11af2c3834SStanimir Varbanov #include <linux/of.h>
12377a22d3SStanimir Varbanov #include <linux/of_address.h>
13*32136e28SStephan Gerhold #include <linux/of_reserved_mem.h>
14f9799fccSStanimir Varbanov #include <linux/platform_device.h>
15f9799fccSStanimir Varbanov #include <linux/of_device.h>
163bf90ecaSElliot Berman #include <linux/firmware/qcom/qcom_scm.h>
17377a22d3SStanimir Varbanov #include <linux/sizes.h>
18af2c3834SStanimir Varbanov #include <linux/soc/qcom/mdt_loader.h>
19af2c3834SStanimir Varbanov 
205df317c8SVikash Garodia #include "core.h"
21af2c3834SStanimir Varbanov #include "firmware.h"
225df317c8SVikash Garodia #include "hfi_venus_io.h"
23af2c3834SStanimir Varbanov 
24af2c3834SStanimir Varbanov #define VENUS_PAS_ID			9
25377a22d3SStanimir Varbanov #define VENUS_FW_MEM_SIZE		(6 * SZ_1M)
265df317c8SVikash Garodia #define VENUS_FW_START_ADDR		0x0
275df317c8SVikash Garodia 
venus_reset_cpu(struct venus_core * core)285df317c8SVikash Garodia static void venus_reset_cpu(struct venus_core *core)
295df317c8SVikash Garodia {
305792ae7cSStanimir Varbanov 	u32 fw_size = core->fw.mapped_mem_size;
31afeae6efSDikshita Agarwal 	void __iomem *wrapper_base;
32afeae6efSDikshita Agarwal 
33c38610f8SKonrad Dybcio 	if (IS_IRIS2_1(core))
34afeae6efSDikshita Agarwal 		wrapper_base = core->wrapper_tz_base;
35afeae6efSDikshita Agarwal 	else
36afeae6efSDikshita Agarwal 		wrapper_base = core->wrapper_base;
375df317c8SVikash Garodia 
38ff2a7013SBryan O'Donoghue 	writel(0, wrapper_base + WRAPPER_FW_START_ADDR);
39ff2a7013SBryan O'Donoghue 	writel(fw_size, wrapper_base + WRAPPER_FW_END_ADDR);
40ff2a7013SBryan O'Donoghue 	writel(0, wrapper_base + WRAPPER_CPA_START_ADDR);
41ff2a7013SBryan O'Donoghue 	writel(fw_size, wrapper_base + WRAPPER_CPA_END_ADDR);
42f95b8ea7SJavier Martinez Canillas 	writel(fw_size, wrapper_base + WRAPPER_NONPIX_START_ADDR);
43f95b8ea7SJavier Martinez Canillas 	writel(fw_size, wrapper_base + WRAPPER_NONPIX_END_ADDR);
44afeae6efSDikshita Agarwal 
45c38610f8SKonrad Dybcio 	if (IS_IRIS2_1(core)) {
46afeae6efSDikshita Agarwal 		/* Bring XTSS out of reset */
47afeae6efSDikshita Agarwal 		writel(0, wrapper_base + WRAPPER_TZ_XTSS_SW_RESET);
48afeae6efSDikshita Agarwal 	} else {
49ff2a7013SBryan O'Donoghue 		writel(0x0, wrapper_base + WRAPPER_CPU_CGC_DIS);
50ff2a7013SBryan O'Donoghue 		writel(0x0, wrapper_base + WRAPPER_CPU_CLOCK_CONFIG);
515df317c8SVikash Garodia 
525df317c8SVikash Garodia 		/* Bring ARM9 out of reset */
53ff2a7013SBryan O'Donoghue 		writel(0, wrapper_base + WRAPPER_A9SS_SW_RESET);
545df317c8SVikash Garodia 	}
55afeae6efSDikshita Agarwal }
565df317c8SVikash Garodia 
venus_set_hw_state(struct venus_core * core,bool resume)575df317c8SVikash Garodia int venus_set_hw_state(struct venus_core *core, bool resume)
585df317c8SVikash Garodia {
592632e7b6SStanimir Varbanov 	int ret;
602632e7b6SStanimir Varbanov 
612632e7b6SStanimir Varbanov 	if (core->use_tz) {
622632e7b6SStanimir Varbanov 		ret = qcom_scm_set_remote_state(resume, 0);
632632e7b6SStanimir Varbanov 		if (resume && ret == -EINVAL)
642632e7b6SStanimir Varbanov 			ret = 0;
652632e7b6SStanimir Varbanov 		return ret;
662632e7b6SStanimir Varbanov 	}
675df317c8SVikash Garodia 
68ec7ad117SDikshita Agarwal 	if (resume) {
695df317c8SVikash Garodia 		venus_reset_cpu(core);
70ec7ad117SDikshita Agarwal 	} else {
71c38610f8SKonrad Dybcio 		if (IS_IRIS2_1(core))
72bd32d085SStanimir Varbanov 			writel(WRAPPER_XTSS_SW_RESET_BIT,
73bd32d085SStanimir Varbanov 			       core->wrapper_tz_base + WRAPPER_TZ_XTSS_SW_RESET);
74afeae6efSDikshita Agarwal 		else
75bd32d085SStanimir Varbanov 			writel(WRAPPER_A9SS_SW_RESET_BIT,
76bd32d085SStanimir Varbanov 			       core->wrapper_base + WRAPPER_A9SS_SW_RESET);
77ec7ad117SDikshita Agarwal 	}
785df317c8SVikash Garodia 
795df317c8SVikash Garodia 	return 0;
805df317c8SVikash Garodia }
81af2c3834SStanimir Varbanov 
venus_load_fw(struct venus_core * core,const char * fwname,phys_addr_t * mem_phys,size_t * mem_size)82a4cf7e3cSVikash Garodia static int venus_load_fw(struct venus_core *core, const char *fwname,
83a4cf7e3cSVikash Garodia 			 phys_addr_t *mem_phys, size_t *mem_size)
84af2c3834SStanimir Varbanov {
85af2c3834SStanimir Varbanov 	const struct firmware *mdt;
86*32136e28SStephan Gerhold 	struct reserved_mem *rmem;
87377a22d3SStanimir Varbanov 	struct device_node *node;
88a4cf7e3cSVikash Garodia 	struct device *dev;
89af2c3834SStanimir Varbanov 	ssize_t fw_size;
90af2c3834SStanimir Varbanov 	void *mem_va;
91af2c3834SStanimir Varbanov 	int ret;
92af2c3834SStanimir Varbanov 
935792ae7cSStanimir Varbanov 	*mem_phys = 0;
945792ae7cSStanimir Varbanov 	*mem_size = 0;
955792ae7cSStanimir Varbanov 
96a4cf7e3cSVikash Garodia 	dev = core->dev;
97377a22d3SStanimir Varbanov 	node = of_parse_phandle(dev->of_node, "memory-region", 0);
98377a22d3SStanimir Varbanov 	if (!node) {
99377a22d3SStanimir Varbanov 		dev_err(dev, "no memory-region specified\n");
100377a22d3SStanimir Varbanov 		return -EINVAL;
101af2c3834SStanimir Varbanov 	}
102af2c3834SStanimir Varbanov 
103*32136e28SStephan Gerhold 	rmem = of_reserved_mem_lookup(node);
104*32136e28SStephan Gerhold 	of_node_put(node);
105*32136e28SStephan Gerhold 	if (!rmem) {
106*32136e28SStephan Gerhold 		dev_err(dev, "failed to lookup reserved memory-region\n");
107*32136e28SStephan Gerhold 		return -EINVAL;
108*32136e28SStephan Gerhold 	}
109377a22d3SStanimir Varbanov 
1105792ae7cSStanimir Varbanov 	ret = request_firmware(&mdt, fwname, dev);
1115792ae7cSStanimir Varbanov 	if (ret < 0)
112*32136e28SStephan Gerhold 		return ret;
1135792ae7cSStanimir Varbanov 
1145792ae7cSStanimir Varbanov 	fw_size = qcom_mdt_get_size(mdt);
1155792ae7cSStanimir Varbanov 	if (fw_size < 0) {
1165792ae7cSStanimir Varbanov 		ret = fw_size;
1175792ae7cSStanimir Varbanov 		goto err_release_fw;
1185792ae7cSStanimir Varbanov 	}
1195792ae7cSStanimir Varbanov 
120*32136e28SStephan Gerhold 	*mem_phys = rmem->base;
121*32136e28SStephan Gerhold 	*mem_size = rmem->size;
122377a22d3SStanimir Varbanov 
1235792ae7cSStanimir Varbanov 	if (*mem_size < fw_size || fw_size > VENUS_FW_MEM_SIZE) {
1245792ae7cSStanimir Varbanov 		ret = -EINVAL;
1255792ae7cSStanimir Varbanov 		goto err_release_fw;
1265792ae7cSStanimir Varbanov 	}
127377a22d3SStanimir Varbanov 
128*32136e28SStephan Gerhold 	mem_va = memremap(*mem_phys, *mem_size, MEMREMAP_WC);
129377a22d3SStanimir Varbanov 	if (!mem_va) {
130*32136e28SStephan Gerhold 		dev_err(dev, "unable to map memory region %pa size %#zx\n", mem_phys, *mem_size);
1315792ae7cSStanimir Varbanov 		ret = -ENOMEM;
1325792ae7cSStanimir Varbanov 		goto err_release_fw;
133af2c3834SStanimir Varbanov 	}
134af2c3834SStanimir Varbanov 
135a4cf7e3cSVikash Garodia 	if (core->use_tz)
136a4cf7e3cSVikash Garodia 		ret = qcom_mdt_load(dev, mdt, fwname, VENUS_PAS_ID,
137a4cf7e3cSVikash Garodia 				    mem_va, *mem_phys, *mem_size, NULL);
138a4cf7e3cSVikash Garodia 	else
139a4cf7e3cSVikash Garodia 		ret = qcom_mdt_load_no_init(dev, mdt, fwname, VENUS_PAS_ID,
140a4cf7e3cSVikash Garodia 					    mem_va, *mem_phys, *mem_size, NULL);
141af2c3834SStanimir Varbanov 
142377a22d3SStanimir Varbanov 	memunmap(mem_va);
1435792ae7cSStanimir Varbanov err_release_fw:
1445792ae7cSStanimir Varbanov 	release_firmware(mdt);
145af2c3834SStanimir Varbanov 	return ret;
146af2c3834SStanimir Varbanov }
147af2c3834SStanimir Varbanov 
venus_boot_no_tz(struct venus_core * core,phys_addr_t mem_phys,size_t mem_size)148df381dc8SVikash Garodia static int venus_boot_no_tz(struct venus_core *core, phys_addr_t mem_phys,
149df381dc8SVikash Garodia 			    size_t mem_size)
150df381dc8SVikash Garodia {
151df381dc8SVikash Garodia 	struct iommu_domain *iommu;
152df381dc8SVikash Garodia 	struct device *dev;
153df381dc8SVikash Garodia 	int ret;
154df381dc8SVikash Garodia 
155df381dc8SVikash Garodia 	dev = core->fw.dev;
156df381dc8SVikash Garodia 	if (!dev)
157df381dc8SVikash Garodia 		return -EPROBE_DEFER;
158df381dc8SVikash Garodia 
159df381dc8SVikash Garodia 	iommu = core->fw.iommu_domain;
1605792ae7cSStanimir Varbanov 	core->fw.mapped_mem_size = mem_size;
161df381dc8SVikash Garodia 
162df381dc8SVikash Garodia 	ret = iommu_map(iommu, VENUS_FW_START_ADDR, mem_phys, mem_size,
1631369459bSJason Gunthorpe 			IOMMU_READ | IOMMU_WRITE | IOMMU_PRIV, GFP_KERNEL);
164df381dc8SVikash Garodia 	if (ret) {
165df381dc8SVikash Garodia 		dev_err(dev, "could not map video firmware region\n");
166df381dc8SVikash Garodia 		return ret;
167df381dc8SVikash Garodia 	}
168df381dc8SVikash Garodia 
169df381dc8SVikash Garodia 	venus_reset_cpu(core);
170df381dc8SVikash Garodia 
171df381dc8SVikash Garodia 	return 0;
172df381dc8SVikash Garodia }
173df381dc8SVikash Garodia 
venus_shutdown_no_tz(struct venus_core * core)174df381dc8SVikash Garodia static int venus_shutdown_no_tz(struct venus_core *core)
175df381dc8SVikash Garodia {
1765792ae7cSStanimir Varbanov 	const size_t mapped = core->fw.mapped_mem_size;
177df381dc8SVikash Garodia 	struct iommu_domain *iommu;
178df381dc8SVikash Garodia 	size_t unmapped;
179df381dc8SVikash Garodia 	u32 reg;
180df381dc8SVikash Garodia 	struct device *dev = core->fw.dev;
181ff2a7013SBryan O'Donoghue 	void __iomem *wrapper_base = core->wrapper_base;
182afeae6efSDikshita Agarwal 	void __iomem *wrapper_tz_base = core->wrapper_tz_base;
183df381dc8SVikash Garodia 
184c38610f8SKonrad Dybcio 	if (IS_IRIS2_1(core)) {
185afeae6efSDikshita Agarwal 		/* Assert the reset to XTSS */
1861eee6bb9SStanimir Varbanov 		reg = readl(wrapper_tz_base + WRAPPER_TZ_XTSS_SW_RESET);
187afeae6efSDikshita Agarwal 		reg |= WRAPPER_XTSS_SW_RESET_BIT;
1881eee6bb9SStanimir Varbanov 		writel(reg, wrapper_tz_base + WRAPPER_TZ_XTSS_SW_RESET);
189afeae6efSDikshita Agarwal 	} else {
190df381dc8SVikash Garodia 		/* Assert the reset to ARM9 */
1911eee6bb9SStanimir Varbanov 		reg = readl(wrapper_base + WRAPPER_A9SS_SW_RESET);
192df381dc8SVikash Garodia 		reg |= WRAPPER_A9SS_SW_RESET_BIT;
1931eee6bb9SStanimir Varbanov 		writel(reg, wrapper_base + WRAPPER_A9SS_SW_RESET);
194afeae6efSDikshita Agarwal 	}
195df381dc8SVikash Garodia 
196df381dc8SVikash Garodia 	iommu = core->fw.iommu_domain;
197df381dc8SVikash Garodia 
198de15e623SMansur Alisha Shaik 	if (core->fw.mapped_mem_size && iommu) {
1995792ae7cSStanimir Varbanov 		unmapped = iommu_unmap(iommu, VENUS_FW_START_ADDR, mapped);
200de15e623SMansur Alisha Shaik 
2015792ae7cSStanimir Varbanov 		if (unmapped != mapped)
202df381dc8SVikash Garodia 			dev_err(dev, "failed to unmap firmware\n");
203de15e623SMansur Alisha Shaik 		else
204de15e623SMansur Alisha Shaik 			core->fw.mapped_mem_size = 0;
205de15e623SMansur Alisha Shaik 	}
206df381dc8SVikash Garodia 
207df381dc8SVikash Garodia 	return 0;
208df381dc8SVikash Garodia }
209df381dc8SVikash Garodia 
venus_boot(struct venus_core * core)210a4cf7e3cSVikash Garodia int venus_boot(struct venus_core *core)
211a4cf7e3cSVikash Garodia {
212a4cf7e3cSVikash Garodia 	struct device *dev = core->dev;
213530ad317SStanimir Varbanov 	const struct venus_resources *res = core->res;
21418a6262bSStanimir Varbanov 	const char *fwpath = NULL;
215a4cf7e3cSVikash Garodia 	phys_addr_t mem_phys;
216a4cf7e3cSVikash Garodia 	size_t mem_size;
217a4cf7e3cSVikash Garodia 	int ret;
218a4cf7e3cSVikash Garodia 
219a4cf7e3cSVikash Garodia 	if (!IS_ENABLED(CONFIG_QCOM_MDT_LOADER) ||
220a4cf7e3cSVikash Garodia 	    (core->use_tz && !qcom_scm_is_available()))
221a4cf7e3cSVikash Garodia 		return -EPROBE_DEFER;
222a4cf7e3cSVikash Garodia 
22318a6262bSStanimir Varbanov 	ret = of_property_read_string_index(dev->of_node, "firmware-name", 0,
22418a6262bSStanimir Varbanov 					    &fwpath);
22518a6262bSStanimir Varbanov 	if (ret)
22618a6262bSStanimir Varbanov 		fwpath = core->res->fwname;
22718a6262bSStanimir Varbanov 
22818a6262bSStanimir Varbanov 	ret = venus_load_fw(core, fwpath, &mem_phys, &mem_size);
229a4cf7e3cSVikash Garodia 	if (ret) {
230a4cf7e3cSVikash Garodia 		dev_err(dev, "fail to load video firmware\n");
231a4cf7e3cSVikash Garodia 		return -EINVAL;
232a4cf7e3cSVikash Garodia 	}
233a4cf7e3cSVikash Garodia 
2340ca0ca98SDikshita Agarwal 	core->fw.mem_size = mem_size;
2350ca0ca98SDikshita Agarwal 	core->fw.mem_phys = mem_phys;
2360ca0ca98SDikshita Agarwal 
237df381dc8SVikash Garodia 	if (core->use_tz)
238df381dc8SVikash Garodia 		ret = qcom_scm_pas_auth_and_reset(VENUS_PAS_ID);
239df381dc8SVikash Garodia 	else
240df381dc8SVikash Garodia 		ret = venus_boot_no_tz(core, mem_phys, mem_size);
241df381dc8SVikash Garodia 
242530ad317SStanimir Varbanov 	if (ret)
243df381dc8SVikash Garodia 		return ret;
244530ad317SStanimir Varbanov 
245530ad317SStanimir Varbanov 	if (core->use_tz && res->cp_size) {
246375b48f4SKonrad Dybcio 		/*
247375b48f4SKonrad Dybcio 		 * Clues for porting using downstream data:
248375b48f4SKonrad Dybcio 		 * cp_start = 0
249375b48f4SKonrad Dybcio 		 * cp_size = venus_ns/virtual-addr-pool[0] - yes, address and not size!
250375b48f4SKonrad Dybcio 		 *   This works, as the non-secure context bank is placed
251375b48f4SKonrad Dybcio 		 *   contiguously right after the Content Protection region.
252375b48f4SKonrad Dybcio 		 *
253375b48f4SKonrad Dybcio 		 * cp_nonpixel_start = venus_sec_non_pixel/virtual-addr-pool[0]
254375b48f4SKonrad Dybcio 		 * cp_nonpixel_size = venus_sec_non_pixel/virtual-addr-pool[1]
255375b48f4SKonrad Dybcio 		 */
256530ad317SStanimir Varbanov 		ret = qcom_scm_mem_protect_video_var(res->cp_start,
257530ad317SStanimir Varbanov 						     res->cp_size,
258530ad317SStanimir Varbanov 						     res->cp_nonpixel_start,
259530ad317SStanimir Varbanov 						     res->cp_nonpixel_size);
260530ad317SStanimir Varbanov 		if (ret) {
261530ad317SStanimir Varbanov 			qcom_scm_pas_shutdown(VENUS_PAS_ID);
262530ad317SStanimir Varbanov 			dev_err(dev, "set virtual address ranges fail (%d)\n",
263530ad317SStanimir Varbanov 				ret);
264530ad317SStanimir Varbanov 			return ret;
265530ad317SStanimir Varbanov 		}
266530ad317SStanimir Varbanov 	}
267530ad317SStanimir Varbanov 
268530ad317SStanimir Varbanov 	return 0;
269a4cf7e3cSVikash Garodia }
270a4cf7e3cSVikash Garodia 
venus_shutdown(struct venus_core * core)271df381dc8SVikash Garodia int venus_shutdown(struct venus_core *core)
272af2c3834SStanimir Varbanov {
273df381dc8SVikash Garodia 	int ret;
274df381dc8SVikash Garodia 
275df381dc8SVikash Garodia 	if (core->use_tz)
276df381dc8SVikash Garodia 		ret = qcom_scm_pas_shutdown(VENUS_PAS_ID);
277df381dc8SVikash Garodia 	else
278df381dc8SVikash Garodia 		ret = venus_shutdown_no_tz(core);
279df381dc8SVikash Garodia 
280df381dc8SVikash Garodia 	return ret;
281af2c3834SStanimir Varbanov }
282f9799fccSStanimir Varbanov 
venus_firmware_init(struct venus_core * core)283f9799fccSStanimir Varbanov int venus_firmware_init(struct venus_core *core)
284f9799fccSStanimir Varbanov {
285f9799fccSStanimir Varbanov 	struct platform_device_info info;
286df381dc8SVikash Garodia 	struct iommu_domain *iommu_dom;
287f9799fccSStanimir Varbanov 	struct platform_device *pdev;
288f9799fccSStanimir Varbanov 	struct device_node *np;
289f9799fccSStanimir Varbanov 	int ret;
290f9799fccSStanimir Varbanov 
291f9799fccSStanimir Varbanov 	np = of_get_child_by_name(core->dev->of_node, "video-firmware");
292f9799fccSStanimir Varbanov 	if (!np) {
293f9799fccSStanimir Varbanov 		core->use_tz = true;
294f9799fccSStanimir Varbanov 		return 0;
295f9799fccSStanimir Varbanov 	}
296f9799fccSStanimir Varbanov 
297f9799fccSStanimir Varbanov 	memset(&info, 0, sizeof(info));
298f9799fccSStanimir Varbanov 	info.fwnode = &np->fwnode;
299f9799fccSStanimir Varbanov 	info.parent = core->dev;
300f9799fccSStanimir Varbanov 	info.name = np->name;
301f9799fccSStanimir Varbanov 	info.dma_mask = DMA_BIT_MASK(32);
302f9799fccSStanimir Varbanov 
303f9799fccSStanimir Varbanov 	pdev = platform_device_register_full(&info);
304f9799fccSStanimir Varbanov 	if (IS_ERR(pdev)) {
305f9799fccSStanimir Varbanov 		of_node_put(np);
306f9799fccSStanimir Varbanov 		return PTR_ERR(pdev);
307f9799fccSStanimir Varbanov 	}
308f9799fccSStanimir Varbanov 
309f9799fccSStanimir Varbanov 	pdev->dev.of_node = np;
310f9799fccSStanimir Varbanov 
311f9799fccSStanimir Varbanov 	ret = of_dma_configure(&pdev->dev, np, true);
312f9799fccSStanimir Varbanov 	if (ret) {
313f9799fccSStanimir Varbanov 		dev_err(core->dev, "dma configure fail\n");
314f9799fccSStanimir Varbanov 		goto err_unregister;
315f9799fccSStanimir Varbanov 	}
316f9799fccSStanimir Varbanov 
317f9799fccSStanimir Varbanov 	core->fw.dev = &pdev->dev;
318f9799fccSStanimir Varbanov 
319df381dc8SVikash Garodia 	iommu_dom = iommu_domain_alloc(&platform_bus_type);
320df381dc8SVikash Garodia 	if (!iommu_dom) {
321df381dc8SVikash Garodia 		dev_err(core->fw.dev, "Failed to allocate iommu domain\n");
322df381dc8SVikash Garodia 		ret = -ENOMEM;
323df381dc8SVikash Garodia 		goto err_unregister;
324df381dc8SVikash Garodia 	}
325df381dc8SVikash Garodia 
326df381dc8SVikash Garodia 	ret = iommu_attach_device(iommu_dom, core->fw.dev);
327df381dc8SVikash Garodia 	if (ret) {
328df381dc8SVikash Garodia 		dev_err(core->fw.dev, "could not attach device\n");
329df381dc8SVikash Garodia 		goto err_iommu_free;
330df381dc8SVikash Garodia 	}
331df381dc8SVikash Garodia 
332df381dc8SVikash Garodia 	core->fw.iommu_domain = iommu_dom;
333df381dc8SVikash Garodia 
334f9799fccSStanimir Varbanov 	of_node_put(np);
335f9799fccSStanimir Varbanov 
336f9799fccSStanimir Varbanov 	return 0;
337f9799fccSStanimir Varbanov 
338df381dc8SVikash Garodia err_iommu_free:
339df381dc8SVikash Garodia 	iommu_domain_free(iommu_dom);
340f9799fccSStanimir Varbanov err_unregister:
341f9799fccSStanimir Varbanov 	platform_device_unregister(pdev);
342f9799fccSStanimir Varbanov 	of_node_put(np);
343f9799fccSStanimir Varbanov 	return ret;
344f9799fccSStanimir Varbanov }
345f9799fccSStanimir Varbanov 
venus_firmware_deinit(struct venus_core * core)346f9799fccSStanimir Varbanov void venus_firmware_deinit(struct venus_core *core)
347f9799fccSStanimir Varbanov {
348df381dc8SVikash Garodia 	struct iommu_domain *iommu;
349df381dc8SVikash Garodia 
350f9799fccSStanimir Varbanov 	if (!core->fw.dev)
351f9799fccSStanimir Varbanov 		return;
352f9799fccSStanimir Varbanov 
353df381dc8SVikash Garodia 	iommu = core->fw.iommu_domain;
354df381dc8SVikash Garodia 
355df381dc8SVikash Garodia 	iommu_detach_device(iommu, core->fw.dev);
356de15e623SMansur Alisha Shaik 
357de15e623SMansur Alisha Shaik 	if (core->fw.iommu_domain) {
358df381dc8SVikash Garodia 		iommu_domain_free(iommu);
359de15e623SMansur Alisha Shaik 		core->fw.iommu_domain = NULL;
360de15e623SMansur Alisha Shaik 	}
361df381dc8SVikash Garodia 
362f9799fccSStanimir Varbanov 	platform_device_unregister(to_platform_device(core->fw.dev));
363f9799fccSStanimir Varbanov }
364