Working icmsg sample for nRF53

Author: doki-nordic

samples/subsys/ipc/ipc_service/icmsg/boards/nrf5340dk_nrf5340_cpuapp.overlay

@@ -32,6 +32,7 @@
 			mboxes = <&mbox 0>, <&mbox 1>;
 			mbox-names = "tx", "rx";
 			status = "okay";
+			dcache-alignment = <8>;
 
 			bt_hci_ipc0: bt_hci_ipc0 {
 				compatible = "zephyr,bt-hci-ipc";

samples/subsys/ipc/ipc_service/icmsg/remote/boards/nrf5340dk_nrf5340_cpunet.overlay

@@ -32,6 +32,7 @@
 			mboxes = <&mbox 0>, <&mbox 1>;
 			mbox-names = "rx", "tx";
 			status = "okay";
+			dcache-alignment = <8>;
 		};
 	};
 };

subsys/ipc/ipc_service/lib/pbuf.c

@@ -32,6 +32,7 @@ static int validate_cfg(const struct pbuf_cfg *cfg)
 {
 	/* Validate pointers. */
 	if (!cfg || !cfg->rd_idx_loc || !cfg->wr_idx_loc || !cfg->data_loc) {
+		printk("Invalid pointers\n");
 		return -EINVAL;
 	}
 
@@ -40,11 +41,13 @@ static int validate_cfg(const struct pbuf_cfg *cfg)
 	    !IS_PTR_ALIGNED_BYTES(cfg->wr_idx_loc, MAX(cfg->dcache_alignment, _PBUF_IDX_SIZE)) ||
 	    !IS_PTR_ALIGNED_BYTES(cfg->handshake_loc, _PBUF_IDX_SIZE) ||
 	    !IS_PTR_ALIGNED_BYTES(cfg->data_loc, _PBUF_IDX_SIZE)) {
+		printk("Invalid alignment\n");
 		return -EINVAL;
 	}
 
 	/* Validate len. */
 	if (cfg->len < _PBUF_MIN_DATA_LEN || !IS_PTR_ALIGNED_BYTES(cfg->len, _PBUF_IDX_SIZE)) {
+		printk("Invalid length\n");
 		return -EINVAL;
 	}
 
@@ -55,6 +58,12 @@ static int validate_cfg(const struct pbuf_cfg *cfg)
 	    !((uint8_t *)cfg->wr_idx_loc < cfg->data_loc) ||
 	    !(((uint8_t *)cfg->rd_idx_loc + MAX(_PBUF_IDX_SIZE, cfg->dcache_alignment)) ==
 	    (uint8_t *)cfg->wr_idx_loc)) {
+		printk("Invalid pointer values\n");
+		printk("rd_idx_loc: 0x%08X\n", (uintptr_t)cfg->rd_idx_loc);
+		printk("wr_idx_loc: 0x%08X\n", (uintptr_t)cfg->wr_idx_loc);
+		printk("handshake_loc: 0x%08X\n", (uintptr_t)cfg->handshake_loc);
+		printk("data_loc: 0x%08X\n", (uintptr_t)cfg->data_loc);
+		printk("dcache_alignment: 0x%08X\n", (uintptr_t)cfg->dcache_alignment);
 		return -EINVAL;
 	}