msm: camera: icp: Add support for synx signaling

Add support to map global sync and hw mutex memory for ICP.
Share the region information with FW over GP registers.

CRs-Fixed: 3351015
Change-Id: Ie7a6af40ffd922ae89d64205787e3c656a007c49
Signed-off-by: Karthik Anantha Ram <quic_kartanan@quicinc.com>

drivers/cam_icp/fw_inc/hfi_intf.h

@@ -38,6 +38,8 @@ struct hfi_mem {
  * @io_mem: io memory info
  * @io_mem2: 2nd io memory info
  * @fw_uncached: FW uncached region info
+ * @global_sync: Global sync mem for IPC
+ * @hwmutex: HWMutex memory
  */
 struct hfi_mem_info {
 	struct hfi_mem qtbl;
@@ -51,6 +53,8 @@ struct hfi_mem_info {
 	struct hfi_mem io_mem;
 	struct hfi_mem io_mem2;
 	struct hfi_mem fw_uncached;
+	struct hfi_mem global_sync;
+	struct hfi_mem hwmutex;
 };
 
 /**

drivers/cam_icp/fw_inc/hfi_reg.h

@@ -13,23 +13,27 @@
 /* general purpose registers */
 #define GEN_PURPOSE_REG(n)              (n * 4)
 
-#define HFI_REG_FW_VERSION              GEN_PURPOSE_REG(1)
+#define HFI_REG_FW_VERSION                   GEN_PURPOSE_REG(1)
-#define HFI_REG_HOST_ICP_INIT_REQUEST   GEN_PURPOSE_REG(2)
+#define HFI_REG_HOST_ICP_INIT_REQUEST        GEN_PURPOSE_REG(2)
-#define HFI_REG_ICP_HOST_INIT_RESPONSE  GEN_PURPOSE_REG(3)
+#define HFI_REG_ICP_HOST_INIT_RESPONSE       GEN_PURPOSE_REG(3)
-#define HFI_REG_SHARED_MEM_PTR          GEN_PURPOSE_REG(4)
+#define HFI_REG_SHARED_MEM_PTR               GEN_PURPOSE_REG(4)
-#define HFI_REG_SHARED_MEM_SIZE         GEN_PURPOSE_REG(5)
+#define HFI_REG_SHARED_MEM_SIZE              GEN_PURPOSE_REG(5)
-#define HFI_REG_QTBL_PTR                GEN_PURPOSE_REG(6)
+#define HFI_REG_QTBL_PTR                     GEN_PURPOSE_REG(6)
-#define HFI_REG_SECONDARY_HEAP_PTR      GEN_PURPOSE_REG(7)
+#define HFI_REG_SECONDARY_HEAP_PTR           GEN_PURPOSE_REG(7)
-#define HFI_REG_SECONDARY_HEAP_SIZE     GEN_PURPOSE_REG(8)
+#define HFI_REG_SECONDARY_HEAP_SIZE          GEN_PURPOSE_REG(8)
-#define HFI_REG_SFR_PTR                 GEN_PURPOSE_REG(10)
+#define HFI_REG_SFR_PTR                      GEN_PURPOSE_REG(10)
-#define HFI_REG_QDSS_IOVA               GEN_PURPOSE_REG(11)
+#define HFI_REG_QDSS_IOVA                    GEN_PURPOSE_REG(11)
-#define HFI_REG_QDSS_IOVA_SIZE          GEN_PURPOSE_REG(12)
+#define HFI_REG_QDSS_IOVA_SIZE               GEN_PURPOSE_REG(12)
-#define HFI_REG_IO_REGION_IOVA          GEN_PURPOSE_REG(13)
+#define HFI_REG_IO_REGION_IOVA               GEN_PURPOSE_REG(13)
-#define HFI_REG_IO_REGION_SIZE          GEN_PURPOSE_REG(14)
+#define HFI_REG_IO_REGION_SIZE               GEN_PURPOSE_REG(14)
-#define HFI_REG_IO2_REGION_IOVA         GEN_PURPOSE_REG(15)
+#define HFI_REG_IO2_REGION_IOVA              GEN_PURPOSE_REG(15)
-#define HFI_REG_IO2_REGION_SIZE         GEN_PURPOSE_REG(16)
+#define HFI_REG_IO2_REGION_SIZE              GEN_PURPOSE_REG(16)
-#define HFI_REG_FWUNCACHED_REGION_IOVA  GEN_PURPOSE_REG(17)
+#define HFI_REG_FWUNCACHED_REGION_IOVA       GEN_PURPOSE_REG(17)
-#define HFI_REG_FWUNCACHED_REGION_SIZE  GEN_PURPOSE_REG(18)
+#define HFI_REG_FWUNCACHED_REGION_SIZE       GEN_PURPOSE_REG(18)
+#define HFI_REG_FWUNCACHED_GLOBAL_SYNC_IOVA  GEN_PURPOSE_REG(19)
+#define HFI_REG_FWUNCACHED_GLOBAL_SYNC_LEN   GEN_PURPOSE_REG(20)
+#define HFI_REG_DEVICE_HWMUTEX_IOVA          GEN_PURPOSE_REG(21)
+#define HFI_REG_DEVICE_HWMUTEX_SIZE          GEN_PURPOSE_REG(22)
 
 /* start of Queue table and queues */
 #define MAX_ICP_HFI_QUEUES                      4

drivers/cam_icp/hfi.c

@@ -730,6 +730,16 @@ int cam_hfi_resume(struct hfi_mem_info *hfi_mem)
 	cam_io_w_mb((uint32_t)hfi_mem->fw_uncached.len,
 		icp_base + HFI_REG_FWUNCACHED_REGION_SIZE);
 
+	cam_io_w_mb((uint32_t)hfi_mem->global_sync.iova,
+		icp_base + HFI_REG_FWUNCACHED_GLOBAL_SYNC_IOVA);
+	cam_io_w_mb((uint32_t)hfi_mem->global_sync.len,
+		icp_base + HFI_REG_FWUNCACHED_GLOBAL_SYNC_LEN);
+
+	cam_io_w_mb((uint32_t)hfi_mem->hwmutex.iova,
+		icp_base + HFI_REG_DEVICE_HWMUTEX_IOVA);
+	cam_io_w_mb((uint32_t)hfi_mem->hwmutex.len,
+		icp_base + HFI_REG_DEVICE_HWMUTEX_SIZE);
+
 	CAM_DBG(CAM_HFI, "IO1 : [0x%x 0x%x] IO2 [0x%x 0x%x]",
 		hfi_mem->io_mem.iova, hfi_mem->io_mem.len,
 		hfi_mem->io_mem2.iova, hfi_mem->io_mem2.len);
@@ -746,6 +756,10 @@ int cam_hfi_resume(struct hfi_mem_info *hfi_mem)
 		hfi_mem->qtbl.iova, hfi_mem->qtbl.len,
 		hfi_mem->sfr_buf.iova, hfi_mem->sfr_buf.len);
 
+	CAM_DBG(CAM_HFI, "global sync : [0x%x 0x%x] hwmutex [0x%x 0x%x]",
+		hfi_mem->global_sync.iova, hfi_mem->global_sync.len,
+		hfi_mem->hwmutex.iova, hfi_mem->hwmutex.len);
+
 	return rc;
 }
 
@@ -939,6 +953,14 @@ int cam_hfi_init(struct hfi_mem_info *hfi_mem, const struct hfi_ops *hfi_ops,
 		icp_base + HFI_REG_FWUNCACHED_REGION_IOVA);
 	cam_io_w_mb((uint32_t)hfi_mem->fw_uncached.len,
 		icp_base + HFI_REG_FWUNCACHED_REGION_SIZE);
+	cam_io_w_mb((uint32_t)hfi_mem->global_sync.iova,
+		icp_base + HFI_REG_FWUNCACHED_GLOBAL_SYNC_IOVA);
+	cam_io_w_mb((uint32_t)hfi_mem->global_sync.len,
+		icp_base + HFI_REG_FWUNCACHED_GLOBAL_SYNC_LEN);
+	cam_io_w_mb((uint32_t)hfi_mem->hwmutex.iova,
+		icp_base + HFI_REG_DEVICE_HWMUTEX_IOVA);
+	cam_io_w_mb((uint32_t)hfi_mem->hwmutex.len,
+		icp_base + HFI_REG_DEVICE_HWMUTEX_SIZE);
 
 	CAM_DBG(CAM_HFI, "IO1 : [0x%x 0x%x] IO2 [0x%x 0x%x]",
 		hfi_mem->io_mem.iova, hfi_mem->io_mem.len,
@@ -956,6 +978,10 @@ int cam_hfi_init(struct hfi_mem_info *hfi_mem, const struct hfi_ops *hfi_ops,
 		hfi_mem->qtbl.iova, hfi_mem->qtbl.len,
 		hfi_mem->sfr_buf.iova, hfi_mem->sfr_buf.len);
 
+	CAM_DBG(CAM_HFI, "global sync : [0x%x 0x%x] hwmutex [0x%x 0x%x]",
+		hfi_mem->global_sync.iova, hfi_mem->global_sync.len,
+		hfi_mem->hwmutex.iova, hfi_mem->hwmutex.len);
+
 	if (cam_presil_mode_enabled())
 		cam_hfi_presil_setup(hfi_mem);
 

drivers/cam_icp/icp_hw/icp_hw_mgr/cam_icp_hw_mgr.c

@@ -55,6 +55,9 @@
 
 #define ICP_DEVICE_IDLE_TIMEOUT 400
 
+/* Memory required to setup all HFI queues and sec heap */
+#define ICP_HFI_QUEUES_MEM_SIZE 0x700000
+
 DECLARE_RWSEM(frame_in_process_sem);
 
 static struct cam_icp_hw_mgr icp_hw_mgr;
@@ -3073,23 +3076,16 @@ static void cam_icp_free_fw_mem(void)
 static void cam_icp_free_hfi_mem(void)
 {
 	int rc;
-	struct cam_smmu_region_info fwuncached_region_info;
-	bool fwuncached_region_exists = false;
 
 	cam_icp_free_fw_mem();
 
-	rc = cam_smmu_get_region_info(icp_hw_mgr.iommu_hdl,
+	if (icp_hw_mgr.hfi_mem.fw_uncached_region) {
-		CAM_SMMU_REGION_FWUNCACHED,
-		&fwuncached_region_info);
-	if (!rc)
-		fwuncached_region_exists = true;
-
-	if (fwuncached_region_exists) {
 		rc = cam_mem_mgr_free_memory_region(
-			&icp_hw_mgr.hfi_mem.fw_uncached);
+			&icp_hw_mgr.hfi_mem.fw_uncached_generic);
 		if (rc)
 			CAM_ERR(CAM_ICP,
 				"failed to unreserve fwuncached region");
+		icp_hw_mgr.hfi_mem.fw_uncached_region = false;
 	} else {
 		rc = cam_mem_mgr_free_memory_region(
 			&icp_hw_mgr.hfi_mem.sec_heap);
@@ -3104,6 +3100,14 @@ static void cam_icp_free_hfi_mem(void)
 	}
 
 	cam_smmu_unmap_phy_mem_region(icp_hw_mgr.iommu_hdl, CAM_SMMU_REGION_QDSS, 0);
+
+	/* Skip freeing if not allocated */
+	if (icp_hw_mgr.synx_signaling_en) {
+		cam_smmu_unmap_phy_mem_region(icp_hw_mgr.iommu_hdl, CAM_SMMU_REGION_FWUNCACHED,
+			CAM_SMMU_SUBREGION_GLOBAL_SYNC_MEM);
+		cam_smmu_unmap_phy_mem_region(icp_hw_mgr.iommu_hdl, CAM_SMMU_REGION_DEVICE,
+			CAM_SMMU_SUBREGION_HWMUTEX);
+	}
 }
 
 static int cam_icp_alloc_secheap_mem(struct cam_mem_mgr_memory_desc *secheap)
@@ -3114,8 +3118,7 @@ static int cam_icp_alloc_secheap_mem(struct cam_mem_mgr_memory_desc *secheap)
 	struct cam_smmu_region_info secheap_info;
 
 	rc = cam_smmu_get_region_info(icp_hw_mgr.iommu_hdl,
-		CAM_SMMU_REGION_SECHEAP,
+		CAM_SMMU_REGION_SECHEAP, &secheap_info);
-		&secheap_info);
 	if (rc) {
 		CAM_ERR(CAM_ICP, "Unable to get secheap memory info");
 		return rc;
@@ -3237,6 +3240,74 @@ static int cam_icp_allocate_qdss_mem(void)
 	return rc;
 }
 
+static int cam_icp_allocate_global_sync_mem(void)
+{
+	int rc;
+	size_t len;
+	dma_addr_t iova;
+
+	rc = cam_smmu_map_phy_mem_region(icp_hw_mgr.iommu_hdl,
+		CAM_SMMU_REGION_FWUNCACHED,
+		CAM_SMMU_SUBREGION_GLOBAL_SYNC_MEM, &iova, &len);
+	if (rc) {
+		CAM_ERR(CAM_ICP,
+			"Failed in allocating global sync ipc mem rc %d", rc);
+		return rc;
+	}
+
+	icp_hw_mgr.hfi_mem.fw_uncached_global_sync.len = len;
+	icp_hw_mgr.hfi_mem.fw_uncached_global_sync.iova = iova;
+	icp_hw_mgr.hfi_mem.fw_uncached_global_sync.smmu_hdl = icp_hw_mgr.iommu_hdl;
+
+	CAM_DBG(CAM_ICP, "iova: %llx, len: %zu", iova, len);
+
+	return rc;
+}
+
+static int cam_icp_allocate_device_hwmutex_mem(void)
+{
+	int rc;
+	size_t len;
+	dma_addr_t iova;
+
+	rc = cam_smmu_map_phy_mem_region(icp_hw_mgr.iommu_hdl,
+		CAM_SMMU_REGION_DEVICE, CAM_SMMU_SUBREGION_HWMUTEX, &iova, &len);
+	if (rc) {
+		CAM_ERR(CAM_ICP,
+			"Failed in allocating hwmutex mem rc %d", rc);
+		return rc;
+	}
+
+	icp_hw_mgr.hfi_mem.hwmutex.len = len;
+	icp_hw_mgr.hfi_mem.hwmutex.iova = iova;
+	icp_hw_mgr.hfi_mem.hwmutex.smmu_hdl = icp_hw_mgr.iommu_hdl;
+
+	CAM_DBG(CAM_ICP, "iova: %llx, len: %zu", iova, len);
+
+	return rc;
+}
+
+static int cam_icp_allocate_mem_for_fence_signaling(void)
+{
+	int rc;
+
+	rc = cam_icp_allocate_global_sync_mem();
+	if (rc)
+		return rc;
+
+	rc = cam_icp_allocate_device_hwmutex_mem();
+	if (rc)
+		goto unmap_global_sync;
+
+	return 0;
+
+unmap_global_sync:
+	cam_smmu_unmap_phy_mem_region(icp_hw_mgr.iommu_hdl,
+		CAM_SMMU_REGION_FWUNCACHED,
+		CAM_SMMU_SUBREGION_GLOBAL_SYNC_MEM);
+	return rc;
+}
+
 static int cam_icp_get_io_mem_info(void)
 {
 	int rc;
@@ -3266,8 +3337,7 @@ static int cam_icp_allocate_hfi_mem(void)
 	bool fwuncached_region_exists = false;
 
 	rc = cam_smmu_get_region_info(icp_hw_mgr.iommu_hdl,
-		CAM_SMMU_REGION_SHARED,
+		CAM_SMMU_REGION_SHARED, &icp_hw_mgr.hfi_mem.shmem);
-		&icp_hw_mgr.hfi_mem.shmem);
 	if (rc) {
 		CAM_ERR(CAM_ICP, "Unable to get shared memory info rc %d", rc);
 		return rc;
@@ -3286,8 +3356,7 @@ static int cam_icp_allocate_hfi_mem(void)
 	}
 
 	rc = cam_smmu_get_region_info(icp_hw_mgr.iommu_hdl,
-		CAM_SMMU_REGION_FWUNCACHED,
+		CAM_SMMU_REGION_FWUNCACHED, &fwuncached_region_info);
-		&fwuncached_region_info);
 	if (!rc)
 		fwuncached_region_exists = true;
 
@@ -3297,19 +3366,27 @@ static int cam_icp_allocate_hfi_mem(void)
 		uint32_t offset;
 		uint64_t size;
 
-		alloc.size = fwuncached_region_info.iova_len;
+		alloc.size = ICP_HFI_QUEUES_MEM_SIZE;
 		alloc.align = 0;
 		alloc.flags = CAM_MEM_FLAG_KMD_ACCESS;
 		alloc.smmu_hdl = icp_hw_mgr.iommu_hdl;
 		rc = cam_mem_mgr_reserve_memory_region(&alloc,
-			CAM_SMMU_REGION_FWUNCACHED,
+			CAM_SMMU_REGION_FWUNCACHED, &out);
-			&out);
 		if (rc) {
-			CAM_ERR(CAM_ICP, "Unable to reserve secheap memory rc %d", rc);
+			CAM_ERR(CAM_ICP, "Unable to reserve fw uncached memory rc %d", rc);
 			goto qtbl_alloc_failed;
 		}
 
-		icp_hw_mgr.hfi_mem.fw_uncached = out;
+		icp_hw_mgr.hfi_mem.fw_uncached_region = true;
+		/*
+		 * If the FW uncached region is carved into different subregions
+		 * send the whole region to FW, resulting in one ICP MMU entry.
+		 * If reserving for FW uncached the SMMU will map the FW uncached
+		 * generic, and return the appropriate va
+		 *
+		 */
+		icp_hw_mgr.hfi_mem.fw_uncached = fwuncached_region_info;
+		icp_hw_mgr.hfi_mem.fw_uncached_generic = out;
 
 		offset = 0;
 
@@ -3371,8 +3448,7 @@ static int cam_icp_allocate_hfi_mem(void)
 			CAM_ERR(CAM_ICP,
 				"FW uncached region size %lld not enough, required %lld",
 				offset, out.len);
-			cam_mem_mgr_free_memory_region(
+			cam_mem_mgr_free_memory_region(&icp_hw_mgr.hfi_mem.fw_uncached_generic);
-				&icp_hw_mgr.hfi_mem.fw_uncached);
 			goto qtbl_alloc_failed;
 		}
 	} else {
@@ -3413,7 +3489,7 @@ static int cam_icp_allocate_hfi_mem(void)
 		}
 	}
 
-	CAM_DBG(CAM_ICP, "Shared Region [0x%x %lld] FW Uncached Region [0x%x %lld]",
+	CAM_DBG(CAM_ICP, "Shared Region [0x%x %lld] FW Uncached nested Region [0x%x %lld]",
 		icp_hw_mgr.hfi_mem.shmem.iova_start,
 		icp_hw_mgr.hfi_mem.shmem.iova_len,
 		fwuncached_region_info.iova_start,
@@ -3421,9 +3497,9 @@ static int cam_icp_allocate_hfi_mem(void)
 
 	CAM_DBG(CAM_ICP,
 		"FwUncached[0x%x %p %lld] QTbl[0x%x %p %lld] CmdQ[0x%x %p %lld] MsgQ[0x%x %p %lld]",
-		icp_hw_mgr.hfi_mem.fw_uncached.iova,
+		icp_hw_mgr.hfi_mem.fw_uncached_generic.iova,
-		icp_hw_mgr.hfi_mem.fw_uncached.kva,
+		icp_hw_mgr.hfi_mem.fw_uncached_generic.kva,
-		icp_hw_mgr.hfi_mem.fw_uncached.len,
+		icp_hw_mgr.hfi_mem.fw_uncached_generic.len,
 		icp_hw_mgr.hfi_mem.qtbl.iova,
 		icp_hw_mgr.hfi_mem.qtbl.kva,
 		icp_hw_mgr.hfi_mem.qtbl.len,
@@ -3451,13 +3527,27 @@ static int cam_icp_allocate_hfi_mem(void)
 		CAM_ERR(CAM_ICP, "Unable to get I/O region info rc %d", rc);
 		if (fwuncached_region_exists) {
 			cam_mem_mgr_free_memory_region(
-				&icp_hw_mgr.hfi_mem.fw_uncached);
+				&icp_hw_mgr.hfi_mem.fw_uncached_generic);
 			goto qtbl_alloc_failed;
 		} else {
 			goto get_io_mem_failed;
 		}
 	}
 
+	/* Allocate sync global mem & hwmutex for IPC */
+	if (icp_hw_mgr.synx_signaling_en) {
+		rc = cam_icp_allocate_mem_for_fence_signaling();
+		if (rc) {
+			if (fwuncached_region_exists) {
+				cam_mem_mgr_free_memory_region(
+					&icp_hw_mgr.hfi_mem.fw_uncached_generic);
+				goto qtbl_alloc_failed;
+			} else {
+				goto get_io_mem_failed;
+			}
+		}
+	}
+
 	return rc;
 get_io_mem_failed:
 	cam_mem_mgr_free_memory_region(&icp_hw_mgr.hfi_mem.sec_heap);
@@ -3744,7 +3834,7 @@ static void cam_icp_mgr_proc_shutdown(struct cam_icp_hw_mgr *hw_mgr)
 
 static int cam_icp_mgr_hfi_resume(struct cam_icp_hw_mgr *hw_mgr)
 {
-	struct hfi_mem_info hfi_mem;
+	struct hfi_mem_info hfi_mem = {0};
 
 	hfi_mem.qtbl.kva = icp_hw_mgr.hfi_mem.qtbl.kva;
 	hfi_mem.qtbl.iova = icp_hw_mgr.hfi_mem.qtbl.iova;
@@ -3787,6 +3877,18 @@ static int cam_icp_mgr_hfi_resume(struct cam_icp_hw_mgr *hw_mgr)
 	hfi_mem.qdss.iova = icp_hw_mgr.hfi_mem.qdss_buf.iova;
 	hfi_mem.qdss.len = icp_hw_mgr.hfi_mem.qdss_buf.len;
 
+	if (icp_hw_mgr.synx_signaling_en) {
+		hfi_mem.global_sync.iova = icp_hw_mgr.hfi_mem.fw_uncached_global_sync.iova;
+		hfi_mem.global_sync.len = icp_hw_mgr.hfi_mem.fw_uncached_global_sync.len;
+
+		hfi_mem.hwmutex.iova = icp_hw_mgr.hfi_mem.hwmutex.iova;
+		hfi_mem.hwmutex.len = icp_hw_mgr.hfi_mem.hwmutex.len;
+		CAM_DBG(CAM_ICP,
+			"global sync [iova = 0x%llx len = 0x%llx] hwmutex [iova = 0x%llx len = 0x%llx]",
+			hfi_mem.global_sync.iova, hfi_mem.global_sync.len,
+			hfi_mem.hwmutex.iova, hfi_mem.hwmutex.len);
+	}
+
 	if (icp_hw_mgr.hfi_mem.io_mem.discard_iova_start &&
 		icp_hw_mgr.hfi_mem.io_mem.discard_iova_len) {
 		/* IO Region 1 */
@@ -3813,8 +3915,8 @@ static int cam_icp_mgr_hfi_resume(struct cam_icp_hw_mgr *hw_mgr)
 		hfi_mem.io_mem2.len = 0x0;
 	}
 
-	hfi_mem.fw_uncached.iova = icp_hw_mgr.hfi_mem.fw_uncached.iova;
+	hfi_mem.fw_uncached.iova = icp_hw_mgr.hfi_mem.fw_uncached.iova_start;
-	hfi_mem.fw_uncached.len = icp_hw_mgr.hfi_mem.fw_uncached.len;
+	hfi_mem.fw_uncached.len = icp_hw_mgr.hfi_mem.fw_uncached.iova_len;
 
 	CAM_DBG(CAM_ICP,
 		"IO region1 IOVA = %X length = %lld, IO region2 IOVA = %X length = %lld",
@@ -4293,7 +4395,7 @@ static int cam_icp_mgr_hfi_init(struct cam_icp_hw_mgr *hw_mgr)
 {
 	struct cam_hw_intf *icp_dev_intf = NULL;
 	struct cam_hw_info *icp_dev = NULL;
-	struct hfi_mem_info hfi_mem;
+	struct hfi_mem_info hfi_mem = {0};
 	const struct hfi_ops *hfi_ops;
 	int rc;
 
@@ -4334,6 +4436,19 @@ static int cam_icp_mgr_hfi_init(struct cam_icp_hw_mgr *hw_mgr)
 	hfi_mem.qdss.iova = icp_hw_mgr.hfi_mem.qdss_buf.iova;
 	hfi_mem.qdss.len = icp_hw_mgr.hfi_mem.qdss_buf.len;
 
+	if (icp_hw_mgr.synx_signaling_en) {
+		hfi_mem.global_sync.iova = icp_hw_mgr.hfi_mem.fw_uncached_global_sync.iova;
+		hfi_mem.global_sync.len = icp_hw_mgr.hfi_mem.fw_uncached_global_sync.len;
+
+		hfi_mem.hwmutex.iova = icp_hw_mgr.hfi_mem.hwmutex.iova;
+		hfi_mem.hwmutex.len = icp_hw_mgr.hfi_mem.hwmutex.len;
+
+		CAM_DBG(CAM_ICP,
+			"global sync [iova = 0x%llx len = 0x%llx] hwmutex [iova = 0x%llx len = 0x%llx]",
+			hfi_mem.global_sync.iova, hfi_mem.global_sync.len,
+			hfi_mem.hwmutex.iova, hfi_mem.hwmutex.len);
+	}
+
 	if (icp_hw_mgr.hfi_mem.io_mem.discard_iova_start &&
 		icp_hw_mgr.hfi_mem.io_mem.discard_iova_len) {
 		/* IO Region 1 */
@@ -4360,8 +4475,8 @@ static int cam_icp_mgr_hfi_init(struct cam_icp_hw_mgr *hw_mgr)
 		hfi_mem.io_mem2.len = 0x0;
 	}
 
-	hfi_mem.fw_uncached.iova = icp_hw_mgr.hfi_mem.fw_uncached.iova;
+	hfi_mem.fw_uncached.iova = icp_hw_mgr.hfi_mem.fw_uncached.iova_start;
-	hfi_mem.fw_uncached.len = icp_hw_mgr.hfi_mem.fw_uncached.len;
+	hfi_mem.fw_uncached.len = icp_hw_mgr.hfi_mem.fw_uncached.iova_len;
 
 	rc = cam_icp_get_hfi_device_ops(icp_dev_intf->hw_type, &hfi_ops);
 	if (rc) {
@@ -6710,6 +6825,7 @@ static int cam_icp_mgr_alloc_devs(struct device_node *np)
 	icp_hw_mgr.dev_pc_flag = of_property_read_bool(np, "ipe_bps_pc_en");
 	icp_hw_mgr.icp_pc_flag = of_property_read_bool(np, "icp_pc_en");
 	icp_hw_mgr.icp_use_pil = of_property_read_bool(np, "icp_use_pil");
+	icp_hw_mgr.synx_signaling_en = of_property_read_bool(np, "synx_signaling_en");
 
 	return 0;
 

drivers/cam_icp/icp_hw/icp_hw_mgr/cam_icp_hw_mgr.h

@@ -83,9 +83,13 @@ struct hfi_mini_dump_info;
  * @fw_buf: Memory info of firmware
  * @qdss_buf: Memory info of qdss
  * @sfr_buf: Memory info for sfr buffer
- * @fw_uncached: Memory info for fw uncached region
+ * @fw_uncached_generic: Memory info for fw uncached region
+ * @fw_uncached_global_sync: Memory info for global sync, in fw uncached region
+ * @hwmutex: Memory info for hwmutex region mapped as device memory
  * @shmem: Memory info for shared region
  * @io_mem: Memory info for io region
+ * @fw_uncached: Memory info for fw uncached nested region
+ * @fw_uncached_region: region support for fw uncached
  */
 struct icp_hfi_mem_info {
 	struct cam_mem_mgr_memory_desc qtbl;
@@ -96,9 +100,13 @@ struct icp_hfi_mem_info {
 	struct cam_mem_mgr_memory_desc fw_buf;
 	struct cam_mem_mgr_memory_desc qdss_buf;
 	struct cam_mem_mgr_memory_desc sfr_buf;
-	struct cam_mem_mgr_memory_desc fw_uncached;
+	struct cam_mem_mgr_memory_desc fw_uncached_generic;
+	struct cam_mem_mgr_memory_desc fw_uncached_global_sync;
+	struct cam_mem_mgr_memory_desc hwmutex;
 	struct cam_smmu_region_info shmem;
 	struct cam_smmu_region_info io_mem;
+	struct cam_smmu_region_info fw_uncached;
+	bool fw_uncached_region;
 };
 
 /**
@@ -407,6 +415,8 @@ struct cam_icp_clk_info {
  *            re-downloaded for new camera session.
  * @frame_in_process: Counter for frames in process
  * @frame_in_process_ctx_id: Contxt id processing frame
+ * @synx_signaling_en: core to core fencing is enabled
+ *                     using synx
  */
 struct cam_icp_hw_mgr {
 	struct mutex hw_mgr_mutex;
@@ -461,6 +471,7 @@ struct cam_icp_hw_mgr {
 	uint64_t icp_svs_clk;
 	atomic_t frame_in_process;
 	int frame_in_process_ctx_id;
+	bool synx_signaling_en;
 };
 
 /**