Re: [PATCH v48 5/7] firmware: imx: adds miscdev

diff --git a/drivers/firmware/imx/ele_base_msg.c b/drivers/firmware/imx/ele_base_msg.c
--- a/drivers/firmware/imx/ele_base_msg.c
+++ b/drivers/firmware/imx/ele_base_msg.c

@@ -121,9 +121,9 @@ int ele_get_info(struct se_if_priv *priv, struct ele_dev_info *s_info)
 	get_info_len = ELE_GET_INFO_BUFF_SZ;
 	if (priv->mem_pool) {
 		ret = se_get_mem_pool_buf(priv->priv_dev_ctx, &get_info_data,
 					  &get_info_addr, get_info_len);

diff --git a/drivers/firmware/imx/ele_common.c b/drivers/firmware/imx/ele_common.c
--- a/drivers/firmware/imx/ele_common.c
+++ b/drivers/firmware/imx/ele_common.c

@@ -342,6 +342,7 @@ int ele_msg_rcv(struct se_if_device_ctx *dev_ctx, struct se_clbk_handle *se_clbk
 				if (se_clbk_hdl->rx_msg) {
 					/*
 					 * The enclave may still DMA into this
 					 * buffer (either a normal timeout or
 					 * a teardown complete_all() wakeup).
 					 * Quarantine the buffer and arm the
 					 * circuit breaker unconditionally.
 					 */
 					se_clbk_hdl->rx_msg = NULL;
 					se_mark_fw_busy(dev_ctx);
 				}
 				spin_unlock_irqrestore(&se_clbk_hdl->clbk_rx_lock, flags);
 			}
 			break;

diff --git a/drivers/firmware/imx/se_ctrl.c b/drivers/firmware/imx/se_ctrl.c
--- a/drivers/firmware/imx/se_ctrl.c
+++ b/drivers/firmware/imx/se_ctrl.c

@@ -406,6 +406,12 @@ static void cleanup_se_shared_mem(struct se_if_device_ctx *dev_ctx, bool recla
 	if (free_dma_buf) {
 		dma_free_coherent(priv->dev, MAX_DATA_SIZE_PER_USER,
 				  se_shared_mem_mgmt->non_secure_mem.ptr,
 				  se_shared_mem_mgmt->non_secure_mem.dma_addr);
 	}
 
 	/*
 	 * Drop the host-side tracking unconditionally. On the reclaim path the
 	 * buffer has been freed. On the deliberate-leak path the buffer is
 	 * abandoned on purpose, so clearing the pointer here guarantees a later
 	 * cleanup pass (e.g. se_if_priv_release()) cannot double-free it.
 	 */
 	se_shared_mem_mgmt->non_secure_mem.ptr = NULL;
 	se_shared_mem_mgmt->non_secure_mem.dma_addr = 0;
 	se_shared_mem_mgmt->non_secure_mem.size = 0;

@@ -1149,6 +1149,7 @@ static void se_clear_fw_busy(struct se_if_priv *priv)
 			/*
 			 * Late response arrived after going_away is clear.
 			 * fw_busy has already been cleared atomically above;
 			 * reclaim the shared-memory slot now. For priv_dev_ctx
 			 * this is the only reclaim site (no close() path). For
 			 * userspace contexts cleanup_se_shared_mem() is a
 			 * safe no-op if the close() path already reset pos.
 			 */
 			cleanup_se_shared_mem(dev_ctx, true);

@@ -1331,6 +1331,10 @@ static void cleanup_dev_ctx(struct se_if_device_ctx *dev_ctx, bool is_fclose)
 				struct se_if_device_ctx *tx_ctx = is_fclose ? dev_ctx :
 							dev_ctx->priv->priv_dev_ctx;
 
 				if (dev_ctx->strg_hdl && se_close_storage(tx_ctx,
 									  dev_ctx->strg_hdl))
 					dev_err(dev_ctx->priv->dev, "failed to close storage.\n");
 				if (dev_ctx->sess_hdl && se_close_session(tx_ctx,
 									  dev_ctx->sess_hdl))
 					dev_err(dev_ctx->priv->dev, "failed to close session.\n");

@@ -2145,6 +2145,8 @@ int se_get_mem_pool_buf(struct se_if_device_ctx *dev_ctx, void **buf,
 	if (se_is_fw_busy_ctx(dev_ctx))
 		return -EBUSY;
 
 	b_desc = kzalloc_obj(*b_desc);
 	if (!b_desc)
 		return -ENOMEM;