From 9bea1e9bd38879f2a44f2cb3444c38ad55361f40 Mon Sep 17 00:00:00 2001
From: Thorsten de Buhr <thorstendb-ARM@users.noreply.github.com>
Date: Tue, 25 Jun 2024 09:21:00 +0200
Subject: [PATCH] added check num regions against configured regions

---
 tools/svdconv/SVDConv/src/SVDConv_Msgs.cpp | 1 +
 tools/svdconv/SVDModel/src/SvdCpu.cpp      | 3 +++
 2 files changed, 4 insertions(+)

diff --git a/tools/svdconv/SVDConv/src/SVDConv_Msgs.cpp b/tools/svdconv/SVDConv/src/SVDConv_Msgs.cpp
index a67a45462..06a50a755 100644
--- a/tools/svdconv/SVDConv/src/SVDConv_Msgs.cpp
+++ b/tools/svdconv/SVDConv/src/SVDConv_Msgs.cpp
@@ -208,6 +208,7 @@ const MsgTable SvdConv::msgTable = {
   { "M388", { MsgLevel::LEVEL_ERROR,    CRLF_B,   "MVE Floating-Point support is set but MVE is not set"                       } },
   { "M389", { MsgLevel::LEVEL_ERROR,    CRLF_B,   "Specified <deviceNumInterrupts>: '%NUM%' greater or equal '%NAME%': '%NUM2%'."} },   // 05.05.2020
   { "M390", { MsgLevel::LEVEL_WARNING3, CRLF_B,   "Checking IRQ '%NAME%': CPU unknown (see <cpu>). Assuming a maximum of %NUM% external Interrupts."} },    // 13.05.2020
+  { "M391", { MsgLevel::LEVEL_WARNING,  CRLF_B,   "CPU: configured num of SAU regions '%NUM%' greater than <sauNumRegions> value: %NUM2%"        } },
 
   // 500... SfrCC2 related Data modification Errors
   { "M500", { MsgLevel::LEVEL_ERROR,    CRLF_B,   ""                                                                            } },
diff --git a/tools/svdconv/SVDModel/src/SvdCpu.cpp b/tools/svdconv/SVDModel/src/SvdCpu.cpp
index a516570de..31105fcd8 100644
--- a/tools/svdconv/SVDModel/src/SvdCpu.cpp
+++ b/tools/svdconv/SVDModel/src/SvdCpu.cpp
@@ -350,6 +350,9 @@ bool SvdCpu::CheckItem()
         m_sauRegionsConfig->Invalidate();
       }
     }
+    else if(m_sauNumRegions < m_sauRegionsConfig->GetChildCount()) {
+      LogMsg("M391", NUM(m_sauRegionsConfig->GetChildCount()), NUM2(m_sauNumRegions),lineNo);
+    }
   }
 
   if(m_pmuPresent) {