phoenix-rtos · Darchiv · Sep 27, 2024 · Sep 19, 2024 · Sep 17, 2024
diff --git a/devices/pipe-rtt/rtt.c b/devices/pipe-rtt/rtt.c
@@ -55,8 +55,9 @@ struct rtt_desc {
 
 static struct {
 	/* NOTE: each buffer must be aligned to cache line */
-	unsigned char bytes[1024] __attribute__((aligned(32)));
-} rttBufferPool[RTT_TXCHANNELS + RTT_RXCHANNELS];
+	unsigned char tx[RTT_TXCHANNELS][1024] __attribute__((aligned(32)));
+	unsigned char rx[RTT_RXCHANNELS][1024] __attribute__((aligned(32)));
+} rttBuffers __attribute__((section(".rttmem")));
 
 
 static const char rtt_tagBackward[] = RTT_TAG_BACKWARD;
@@ -96,10 +97,10 @@ ssize_t rtt_read(int chan, void *buf, size_t count)
 		todo--;
 	}
 
-	rtt->rxChannel[chan].rd = rd;
-
 	hal_cpuDataMemoryBarrier();
 
+	rtt->rxChannel[chan].rd = rd;
+
 	return count - todo;
 }
 
@@ -125,10 +126,10 @@ ssize_t rtt_write(int chan, const void *buf, size_t count)
 		todo--;
 	}
 
-	rtt->txChannel[chan].wr = wr;
-
 	hal_cpuDataMemoryBarrier();
 
+	rtt->txChannel[chan].wr = wr;
+
 	return count - todo;
 }
 
@@ -171,19 +172,20 @@ void rtt_init(void *addr)
 	rtt->txChannels = RTT_TXCHANNELS;
 	rtt->rxChannels = RTT_RXCHANNELS;
 
-	m = 0;
 	for (n = 0; n < rtt->txChannels; n++) {
 		rtt->txChannel[n].name = rtt_txName[n];
-		rtt->txChannel[n].ptr = rttBufferPool[m].bytes;
-		rtt->txChannel[n].sz = sizeof(rttBufferPool[m].bytes);
-		m++;
+		rtt->txChannel[n].ptr = rttBuffers.tx[n];
+		rtt->txChannel[n].sz = sizeof(rttBuffers.tx[n]);
+		rtt->txChannel[n].wr = 0;
+		rtt->txChannel[n].rd = 0;
 	}
 
 	for (n = 0; n < rtt->rxChannels; n++) {
 		rtt->rxChannel[n].name = rtt_rxName[n];
-		rtt->rxChannel[n].ptr = rttBufferPool[m].bytes;
-		rtt->rxChannel[n].sz = sizeof(rttBufferPool[m].bytes);
-		m++;
+		rtt->rxChannel[n].ptr = rttBuffers.rx[n];
+		rtt->rxChannel[n].sz = sizeof(rttBuffers.rx[n]);
+		rtt->rxChannel[n].wr = 0;
+		rtt->rxChannel[n].rd = 0;
 	}
 
 	n = 0;

diff --git a/hal/armv7m/imxrt/10xx/105x/Makefile b/hal/armv7m/imxrt/10xx/105x/Makefile
@@ -15,7 +15,7 @@ CFLAGS+= -mfloat-abi=soft
 PLO_COMMANDS ?= alias app blob bridge call console copy devices dump echo erase go help kernel \
   kernelimg map mem mpu otp phfs ptable reboot script stop wait
 
-# pipe-rtt is disabled due to small amount of space in DTCM; RTT_ADDR is not defined for this architecture
+# pipe-rtt is disabled due to small amount of space in DTCM; RTT_ENABLED_PLO is not defined for this architecture
 PLO_ALLDEVICES := usbc-cdc uart-imxrt106x flash-imxrt
 
 OBJS += $(addprefix $(PREFIX_O)hal/$(TARGET_SUFF)/imxrt/10xx/105x/, _init.o imxrt.o)
diff --git a/hal/armv7m/imxrt/10xx/106x/peripherals.h b/hal/armv7m/imxrt/10xx/106x/peripherals.h
@@ -26,9 +26,8 @@
 
 
 /* DEBUG - RTT PIPE */
-#ifndef RTT_ADDR
-/* RTT descriptors location, last 256 bytes of DTCM */
-#define RTT_ADDR (0x20058000 - 0x100)
+#ifndef RTT_ENABLED_PLO
+#define RTT_ENABLED_PLO 1
 #endif
 
 

diff --git a/hal/armv7m/imxrt/117x/peripherals.h b/hal/armv7m/imxrt/117x/peripherals.h
@@ -27,9 +27,8 @@
 
 /* DEBUG - RTT PIPE */
 
-#ifndef RTT_ADDR
-/* RTT descriptors location, last 256 bytes of DTCM */
-#define RTT_ADDR (0x20040000 - 0x100)
+#ifndef RTT_ENABLED_PLO
+#define RTT_ENABLED_PLO 1
 #endif
 
 

diff --git a/hal/armv7m/imxrt/hal.c b/hal/armv7m/imxrt/hal.c
@@ -30,6 +30,9 @@ extern char __data_start[], __data_end[];
 extern char __bss_start[], __bss_end[];
 extern char __heap_base[], __heap_limit[];
 extern char __stack_top[], __stack_limit[];
+#if RTT_ENABLED_PLO
+extern char __rttmem_rttcb[];
+#endif
 
 /* Timer */
 extern void timer_init(void);
@@ -58,8 +61,8 @@ void hal_init(void)
 	timer_init();
 	otp_init();
 
-#ifdef RTT_ADDR
-	rtt_init((void *)RTT_ADDR);
+#if RTT_ENABLED_PLO
+	rtt_init(__rttmem_rttcb);
 #endif
 
 	console_init();

diff --git a/hal/armv7m/stm32/hal.c b/hal/armv7m/stm32/hal.c
@@ -33,6 +33,9 @@ extern char __data_start[], __data_end[];
 extern char __bss_start[], __bss_end[];
 extern char __heap_base[], __heap_limit[];
 extern char __stack_top[], __stack_limit[];
+#if RTT_ENABLED_PLO
+extern char __rttmem_rttcb[];
+#endif
 
 /* Timer */
 extern void timer_init(void);
@@ -60,8 +63,8 @@ void hal_init(void)
 	mpu_init();
 	timer_init();
 
-#ifdef RTT_ADDR
-	rtt_init((void *)RTT_ADDR);
+#if RTT_ENABLED_PLO
+	rtt_init(__rttmem_rttcb);
 #endif
 
 	console_init();

diff --git a/ld/armv7m4-stm32l4x6.ldt b/ld/armv7m4-stm32l4x6.ldt
@@ -33,13 +33,17 @@
 /* Space reserved for kernel data */
 #define AREA_KERNEL 0x10000
 
+/* Space reserved for RTT control block and buffers */
+#define SIZE_RTTMEM (2 * 2 * 0x400 + 256)
+
 
 #if defined(__LINKER__)
 
 /* Memory map setup */
 MEMORY
 {
-	m_sram    (rwx) : ORIGIN = RAM_ADDR + AREA_KERNEL, LENGTH = (256k + 64k) - AREA_KERNEL
+	m_sram    (rwx) : ORIGIN = RAM_ADDR + AREA_KERNEL, LENGTH = (256k + 64k) - AREA_KERNEL - SIZE_RTTMEM
+	m_rttmem  (rw)  : ORIGIN = RAM_ADDR + (256k + 64k) - SIZE_RTTMEM, LENGTH = SIZE_RTTMEM
 	m_flash2  (rx)  : ORIGIN = FLASH_PROGRAM_2_ADDR, LENGTH = 512k
 	m_flash1  (rx)  : ORIGIN = FLASH_PROGRAM_1_ADDR, LENGTH = 512k
 }
@@ -51,8 +55,10 @@ REGION_ALIAS("DATA", m_sram);
 REGION_ALIAS("BSS", m_sram);
 REGION_ALIAS("HEAP", m_sram);
 REGION_ALIAS("STACK", m_sram);
+REGION_ALIAS("RTTMEM", m_rttmem);
 
 #include "common/plo-arm.lds"
+#include "common/plo-rtt.lds"
 
 #endif /* end of __LINKER__ */
 

diff --git a/ld/armv7m7-imxrt105x.ldt b/ld/armv7m7-imxrt105x.ldt
@@ -23,6 +23,10 @@
 #define SIZE_STACK (4 * SIZE_PAGE)
 #define SIZE_HEAP  (8 * SIZE_PAGE)
 
+/* Space reserved for kernel data */
+#define AREA_KERNEL 0x2000
+
+
 #if defined(__LINKER__)
 
 /* FlexRAM configuration */
@@ -48,7 +52,7 @@ MEMORY
 {
 	m_itcm    (rwx) : ORIGIN = 0x00000000, LENGTH = FLEXRAM_ITCM_AREA
 	m_itext   (rwx) : ORIGIN = FLEXRAM_ITEXT_ADDR, LENGTH = FLEXRAM_ITEXT_AREA
-	m_dtcm    (rw)  : ORIGIN = 0x20002000, LENGTH = FLEXRAM_DTCM_AREA
+	m_dtcm    (rw)  : ORIGIN = 0x20000000 + AREA_KERNEL, LENGTH = FLEXRAM_DTCM_AREA - AREA_KERNEL
 	m_ocram   (rwx) : ORIGIN = 0x20200000, LENGTH = FLEXRAM_OCRAM_AREA
 	m_flash   (rx)  : ORIGIN = 0x60000000, LENGTH = 128k /* Not actual flash size. Initial flash size to be put into FCB block for imxrt BootROM init procedure only */
 }

diff --git a/ld/armv7m7-imxrt106x.ldt b/ld/armv7m7-imxrt106x.ldt
@@ -23,6 +23,13 @@
 #define SIZE_STACK (4 * SIZE_PAGE)
 #define SIZE_HEAP  (8 * SIZE_PAGE)
 
+/* Space reserved for kernel data */
+#define AREA_KERNEL 0x2000
+
+/* Space reserved for RTT control block and buffers */
+#define SIZE_RTTMEM (2 * 2 * 0x400 + 256)
+
+
 #if defined(__LINKER__)
 
 /* FlexRAM configuration */
@@ -47,7 +54,8 @@ MEMORY
 	/* TODO: use FLEXRAM_CONFIG value to setup ocram/itcm/dtcm partitioning (*32k) */
 	m_itcm    (rwx) : ORIGIN = 0x00000000, LENGTH = FLEXRAM_ITCM_AREA
 	m_itext   (rwx) : ORIGIN = FLEXRAM_ITEXT_ADDR, LENGTH = FLEXRAM_ITEXT_AREA
-	m_dtcm    (rw)  : ORIGIN = 0x20002000, LENGTH = FLEXRAM_DTCM_AREA
+	m_dtcm    (rw)  : ORIGIN = 0x20000000 + AREA_KERNEL, LENGTH = FLEXRAM_DTCM_AREA - AREA_KERNEL - SIZE_RTTMEM
+	m_rttmem  (rw)  : ORIGIN = 0x20000000 + FLEXRAM_DTCM_AREA - SIZE_RTTMEM, LENGTH = SIZE_RTTMEM
 	m_ocram   (rwx) : ORIGIN = 0x20200000, LENGTH = 0 * 32k
 	m_flash   (rx)  : ORIGIN = 0x70000000, LENGTH = 128k /* Not actual flash size. Initial flash size to be put into FCB block for imxrt BootROM init procedure only */
 }
@@ -74,8 +82,10 @@ REGION_ALIAS("TCM_TEXT", m_itext);
 REGION_ALIAS("BSS", m_dtcm);
 REGION_ALIAS("HEAP", m_dtcm);
 REGION_ALIAS("STACK", m_dtcm);
+REGION_ALIAS("RTTMEM", m_rttmem);
 
 #include "common/plo-arm.lds"
+#include "common/plo-rtt.lds"
 
 #endif /* end of __LINKER__ */
 

diff --git a/ld/armv7m7-imxrt117x.ldt b/ld/armv7m7-imxrt117x.ldt
@@ -29,6 +29,9 @@
 /* Space reserved for bootloader */
 #define AREA_BOOTLOADER 0x10000
 
+/* Space reserved for RTT control block and buffers */
+#define SIZE_RTTMEM (2 * 2 * 0x400 + 256)
+
 
 #if defined(__LINKER__)
 
@@ -53,7 +56,8 @@ MEMORY
 {
 	m_itcm    (rwx) : ORIGIN = 0x00000000, LENGTH = FLEXRAM_ITCM_AREA
 	m_itext   (rwx) : ORIGIN = 0x00000000 + FLEXRAM_ITEXT_ADDR, LENGTH = FLEXRAM_ITEXT_AREA
-	m_dtcm    (rw)  : ORIGIN = 0x20002000 + AREA_KERNEL, LENGTH = FLEXRAM_DTCM_AREA - AREA_KERNEL
+	m_dtcm    (rw)  : ORIGIN = 0x20000000 + AREA_KERNEL, LENGTH = FLEXRAM_DTCM_AREA - AREA_KERNEL - SIZE_RTTMEM
+	m_rttmem  (rw)  : ORIGIN = 0x20000000 + FLEXRAM_DTCM_AREA - SIZE_RTTMEM, LENGTH = SIZE_RTTMEM
 	m_ocram1  (rwx) : ORIGIN = 0x20240000 + AREA_BOOTLOADER, LENGTH = (8 * 32k) - AREA_BOOTLOADER
 	m_ocram2  (rwx) : ORIGIN = 0x202c0000, LENGTH = 512k
 	m_flash   (rx)  : ORIGIN = 0x30000000, LENGTH = 128k /* Not actual flash size. Initial flash size to be put into FCB block for imxrt BootROM init procedure only */
@@ -81,8 +85,10 @@ REGION_ALIAS("TCM_TEXT", m_itext);
 REGION_ALIAS("BSS", m_dtcm);
 REGION_ALIAS("HEAP", m_dtcm);
 REGION_ALIAS("STACK", m_dtcm);
+REGION_ALIAS("RTTMEM", m_rttmem);
 
 #include "common/plo-arm.lds"
+#include "common/plo-rtt.lds"
 
 #endif /* end of __LINKER__ */
 

diff --git a/ld/common/plo-rtt.lds b/ld/common/plo-rtt.lds
@@ -0,0 +1,12 @@
+/* Section for RTT control block and buffers */
+SECTIONS
+{
+    .rttmem ORIGIN(RTTMEM) (NOLOAD):
+	{
+		KEEP(*(.rttmem))
+		. = ABSOLUTE(ORIGIN(RTTMEM) + LENGTH(RTTMEM) - 256);
+		__rttmem_rttcb = .;
+		. = ALIGN(4);
+	} > RTTMEM
+
+}