From 70525731872b26b9c64c2ec6617b927045d41dae Mon Sep 17 00:00:00 2001
From: Marc Schink <dev@zapb.de>
Date: Fri, 11 Jul 2025 22:08:14 +0000
Subject: [PATCH] server/gdb: Add support for NVM banks without sectors

This is necessary for non-flash NVM which does not have the concept of
sectors, such as RRAM. Also small flash memory regions without sectors,
such as configuration areas, can now be exposed to GDB.

Tested with nRF54L15-DK, tested for regression with AT32F421C8T7.

Change-Id: Ifc96ddca7dcfcd4ab17683c0e59f1be912901941
Signed-off-by: Marc Schink <dev@zapb.de>
Reviewed-on: https://review.openocd.org/c/openocd/+/9290
Reviewed-by: Tomas Vanek <vanekt@fbl.cz>
Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com>
Tested-by: jenkins
---
 src/server/gdb_server.c | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/src/server/gdb_server.c b/src/server/gdb_server.c
index b966e8593..cf7a2bde0 100644
--- a/src/server/gdb_server.c
+++ b/src/server/gdb_server.c
@@ -2010,6 +2010,15 @@ static int gdb_memory_map(struct connection *connection,
 				"length=\"0x%x\"/>\n",
 				p->base, p->size);
 		} else {
+			if (p->num_sectors == 0) {
+				xml_printf(&retval, &xml, &pos, &size,
+					"<memory type=\"flash\" "
+						"start=\"" TARGET_ADDR_FMT "\" "
+						"length=\"0x%x\">"
+						"<property name=\"blocksize\">0x%x</property>\n"
+					"</memory>\n", p->base, p->size, p->size);
+			}
+
 			/* Report adjacent groups of same-size sectors.  So for
 			 * example top boot CFI flash will list an initial region
 			 * with several large sectors (maybe 128KB) and several