Initial zephyr/west integration support

brondani · brondani · commit a3bb486919b3 · 2025-09-30T09:33:21.000+02:00
diff --git a/tools/projmgr/include/ProjMgrParser.h b/tools/projmgr/include/ProjMgrParser.h
@@ -209,6 +209,7 @@ struct TargetSetItem {
  *        misc compiler controls,
  *        platform processor
  *        context map
+ *        west defines
 */
 struct BuildType {
   std::string compiler;
@@ -228,6 +229,7 @@ struct BuildType {
   ProcessorItem processor;
   std::vector<std::pair<std::string, std::string>> variables;
   std::vector<ContextName> contextMap;
+  std::vector<std::string> westDefs;
 };
 
 /**
@@ -452,6 +454,18 @@ struct GroupNode {
   TypeFilter type;
 };
 
+/**
+ * @brief west descriptor
+*/
+struct WestDesc {
+  std::string projectId;
+  std::string app;
+  std::string board;
+  std::string device;
+  std::vector<std::string> westDefs;
+  std::vector<std::string> westOpt;
+};
+
 /**
  * @brief context descriptor containing
  *        cproject filename,
@@ -460,6 +474,7 @@ struct GroupNode {
 struct ContextDesc {
   std::string cproject;
   TypeFilter type;
+  WestDesc west;
 };
 
 /**
@@ -506,6 +521,7 @@ typedef std::vector<std::pair<std::string, TargetType>> TargetTypes;
  *        target types,
  *        target properties,
  *        list of cprojects,
+ *        list of west apps,
  *        list of contexts descriptors,
  *        list of packs,
  *        cdefault enable switch,
@@ -524,6 +540,7 @@ struct CsolutionItem {
   TargetTypes targetTypes;
   TargetType target;
   std::vector<std::string> cprojects;
+  std::vector<std::string> westApps;
   std::vector<ContextDesc> contexts;
   std::vector<PackItem> packs;
   bool enableCdefault;
diff --git a/tools/projmgr/include/ProjMgrUtils.h b/tools/projmgr/include/ProjMgrUtils.h
@@ -321,6 +321,13 @@ class ProjMgrUtils {
   */
   static const std::string NormalizeLineEndings(const std::string& in);
 
+  /**
+   * @brief convert board string to west board string
+   * @param board string
+   * @return output west board string
+  */
+  static const std::string GetWestBoard(const std::string& board);
+
 protected:
   /**
    * @brief get filtered list of contexts
diff --git a/tools/projmgr/include/ProjMgrWorker.h b/tools/projmgr/include/ProjMgrWorker.h
@@ -429,6 +429,7 @@ struct DebuggerType {
  *        load offset for generated binary
  *        elf load mode
  *        image only flag
+ *        west options
 */
 struct ContextItem {
   CdefaultItem* cdefault = nullptr;
@@ -498,6 +499,7 @@ struct ContextItem {
   std::string loadOffset;
   std::string elfLoadMode;
   bool imageOnly = false;
+  WestDesc west;
 };
 
 /**
@@ -509,6 +511,11 @@ static constexpr const char* PLM_STATUS_UPDATE_REQUIRED = "update required";
 static constexpr const char* PLM_STATUS_UPDATE_RECOMMENDED = "update recommended";
 static constexpr const char* PLM_STATUS_UPDATE_SUGGESTED = "update suggested";
 
+/**
+  * @brief west board variable
+*/
+static constexpr const char* WEST_BOARD = "west-board";
+
 /**
   * @brief set policy for packs loading [latest|all|required]
 */
diff --git a/tools/projmgr/include/ProjMgrYamlParser.h b/tools/projmgr/include/ProjMgrYamlParser.h
@@ -27,6 +27,7 @@ static constexpr const char* YAML_ARGUMENTS = "arguments";
 static constexpr const char* YAML_APID = "apid";
 static constexpr const char* YAML_APIS = "apis";
 static constexpr const char* YAML_API = "api";
+static constexpr const char* YAML_APP_PATH = "app-path";
 static constexpr const char* YAML_ATOMIC = "atomic";
 static constexpr const char* YAML_ATTR = "attr";
 static constexpr const char* YAML_AUTO = "auto";
@@ -189,6 +190,7 @@ static constexpr const char* YAML_PROCESSORS = "processors";
 static constexpr const char* YAML_PROCESSOR = "processor";
 static constexpr const char* YAML_PROGRAMMING = "programming";
 static constexpr const char* YAML_PROJECT = "project";
+static constexpr const char* YAML_PROJECT_ID = "project-id";
 static constexpr const char* YAML_PROJECTS = "projects";
 static constexpr const char* YAML_PROJECT_CONTEXT = "project-context";
 static constexpr const char* YAML_PROJECT_TYPE = "project-type";
@@ -243,6 +245,10 @@ static constexpr const char* YAML_VARS = "vars";
 static constexpr const char* YAML_VALUE = "value";
 static constexpr const char* YAML_VERSION = "version";
 static constexpr const char* YAML_WARNINGS = "warnings";
+static constexpr const char* YAML_WEST = "west";
+static constexpr const char* YAML_WEST_BOARD = "west-board";
+static constexpr const char* YAML_WEST_DEFS = "west-defs";
+static constexpr const char* YAML_WEST_OPT = "west-opt";
 static constexpr const char* YAML_WHILE = "while";
 static constexpr const char* YAML_WORKING_DIR = "working-dir";
 
diff --git a/tools/projmgr/schemas/common.schema.json b/tools/projmgr/schemas/common.schema.json
@@ -601,7 +601,8 @@
         "target-set":   { "$ref": "#/definitions/TargetSetsType" },
         "undefine":     { "$ref": "#/definitions/UndefinesType" },
         "variables":    { "$ref": "#/definitions/VariablesType" },
-        "warnings":     { "$ref": "#/definitions/WarningsType" }
+        "warnings":     { "$ref": "#/definitions/WarningsType" },
+        "west-defs":    { "$ref": "#/definitions/WestDefsType" }
       },
       "additionalProperties": false,
       "required" : [ "type"]
@@ -638,7 +639,8 @@
         "processor":    { "$ref": "#/definitions/ProcessorType" },
         "undefine":     { "$ref": "#/definitions/UndefinesType" },
         "variables":    { "$ref": "#/definitions/VariablesType" },
-        "warnings":     { "$ref": "#/definitions/WarningsType" }
+        "warnings":     { "$ref": "#/definitions/WarningsType" },
+        "west-defs":    { "$ref": "#/definitions/WestDefsType" }
       },
       "additionalProperties": false,
       "required" : [ "type"]
@@ -660,13 +662,13 @@
           "pattern": "^.*[/]([^.+]*\\.cproject\\.(yml|yaml))$|^([^.+]+)\\.cproject\\.(yml|yaml)$",
           "description": "Path to the valid project file."
         },
+        "west": { "$ref": "#/definitions/WestProjectType" },
         "for-context":     { "$ref": "#/definitions/ForContext" },
         "not-for-context": { "$ref": "#/definitions/NotForContext" }
       },
-      "additionalProperties": false,
-      "allOf": [
-        { "$ref": "#/definitions/TypeListMutualExclusion"},
-        { "required": [ "project" ] }
+      "additionalProperties": true,
+      "oneOf": [
+        { "$ref": "#/definitions/TypeListMutualExclusion"}
       ]
     },
     "ProcessorTrustzone": {
@@ -1274,6 +1276,7 @@
       "type": "object",
       "properties": {
         "cbuild": { "type": "string", "description": "Path to <context>.cbuild.yml file." },
+        "west": { "type": "boolean" },
         "project": { "type": "string", "description": "Project name." },
         "configuration": { "$ref": "#/definitions/BuildContext" },
         "depends-on": {
@@ -1427,7 +1430,8 @@
         "licenses": {
           "type": "array",
           "items": { "$ref": "#/definitions/LicenseInfoType" }
-        }
+        },
+        "west": { "$ref": "#/definitions/WestProjectType" }
       },
       "additionalProperties": false
     },
@@ -2398,6 +2402,15 @@
       "title": "optimize:\nDocumentation: https://open-cmsis-pack.github.io/cmsis-toolbox/YML-Input-Format/#load",
       "description": "Specifies the load mode for the project output or image file.",
       "enum": [ "image+symbols", "symbols", "image", "none" ]
+    },
+    "WestProjectType": {
+      "type": "object",
+      "additionalProperties": true
+    },
+    "WestDefsType": {
+      "type": "array",
+      "minItems": 1,
+      "items": {}
     }
   }
 }
diff --git a/tools/projmgr/src/ProjMgr.cpp b/tools/projmgr/src/ProjMgr.cpp
@@ -487,7 +487,7 @@ bool ProjMgr::PopulateContexts(void) {
     }
     // Check cproject separate folders and unique names
     const StrVec& cprojects = m_parser.GetCsolution().cprojects;
-    if (!IsSolutionImageOnly() && cprojects.empty()) {
+    if (!IsSolutionImageOnly() && cprojects.empty() && m_parser.GetCsolution().westApps.empty()) {
       ProjMgrLogger::Get().Error("projects not found", "", m_csolutionFile);
       return false;
     }
diff --git a/tools/projmgr/src/ProjMgrCbuild.cpp b/tools/projmgr/src/ProjMgrCbuild.cpp
@@ -40,6 +40,7 @@ class ProjMgrCbuild : public ProjMgrCbuildBase {
   void SetBooksNode(YAML::Node node, const std::vector<BookItem>& books, const std::string& dir);
   void SetDebugConfigNode(YAML::Node node, const ContextItem* context);
   void SetPLMStatus(YAML::Node node, const ContextItem* context, const string& file);
+  void SetWestNode(YAML::Node node, const ContextItem* context);
   bool m_ignoreRteFileMissing;
 };
 
@@ -108,7 +109,14 @@ void ProjMgrCbuild::SetContextNode(YAML::Node contextNode, const ContextItem* co
       }
     }
     SetOutputDirsNode(contextNode[YAML_OUTPUTDIRS], context);
-    SetOutputNode(contextNode[YAML_OUTPUT], context);
+  }
+  if (!context->west.app.empty()) {
+    string outDir = context->directories.outdir;
+    RteFsUtils::NormalizePath(outDir, context->directories.cprj);
+    SetNodeValue(contextNode[YAML_OUTPUTDIRS][YAML_OUTPUT_OUTDIR], FormatPath(outDir, context->directories.cbuild));
+  }  
+  SetOutputNode(contextNode[YAML_OUTPUT], context);
+  if (!context->imageOnly) {
     SetComponentsNode(contextNode[YAML_COMPONENTS], context);
     SetApisNode(contextNode[YAML_APIS], context);
     SetGeneratorsNode(contextNode[YAML_GENERATORS], context);
@@ -117,6 +125,9 @@ void ProjMgrCbuild::SetContextNode(YAML::Node contextNode, const ContextItem* co
     SetConstructedFilesNode(contextNode[YAML_CONSTRUCTEDFILES], context);
   }
   SetLicenseInfoNode(contextNode[YAML_LICENSES], context);
+  if (!context->west.app.empty()) {
+    SetWestNode(contextNode[YAML_WEST], context);
+  }
 }
 
 void ProjMgrCbuild::SetComponentsNode(YAML::Node node, const ContextItem* context) {
@@ -590,3 +601,12 @@ bool ProjMgrYamlEmitter::GenerateCbuild(ContextItem* context,
   context->needRebuild = NeedRebuild(filename, rootNode);
   return WriteFile(rootNode, filename, context->name);
 }
+
+void ProjMgrCbuild::SetWestNode(YAML::Node node, const ContextItem* context) {
+  SetNodeValue(node[YAML_PROJECT_ID], context->west.projectId);
+  SetNodeValue(node[YAML_APP_PATH], FormatPath(context->west.app, context->directories.cbuild));
+  SetNodeValue(node[YAML_BOARD], context->west.board);
+  SetNodeValue(node[YAML_DEVICE], context->west.device);
+  SetNodeValue(node[YAML_WEST_DEFS], context->west.westDefs);
+  SetNodeValue(node[YAML_WEST_OPT], context->west.westOpt);
+}
diff --git a/tools/projmgr/src/ProjMgrCbuildIdx.cpp b/tools/projmgr/src/ProjMgrCbuildIdx.cpp
@@ -44,7 +44,7 @@ ProjMgrCbuildIdx::ProjMgrCbuildIdx(YAML::Node node,
   SetNodeValue(node[YAML_OUTPUT_TMPDIR], FormatPath(parser->GetCsolution().directories.tmpdir, directory));
 
   // Image Only flag
-  if (!processedContexts.empty() && processedContexts.front()->imageOnly) {
+  if (!processedContexts.empty() && processedContexts.front()->imageOnly && parser->GetCsolution().westApps.empty()) {
     node[YAML_IMAGE_ONLY] = true;
   }
 
@@ -134,8 +134,11 @@ ProjMgrCbuildIdx::ProjMgrCbuildIdx(YAML::Node node,
       const string& filename = context->directories.cprj + "/" + context->name + ".cbuild.yml";
       const string& relativeFilename = fs::relative(filename, directory, ec).generic_string();
       SetNodeValue(cbuildNode[YAML_CBUILD], relativeFilename);
+      if (!context->west.app.empty()) {
+        cbuildNode[YAML_WEST] = true;
+      }
       if (context->cproject) {
-        if (!context->imageOnly) {
+        if (!context->imageOnly || !context->west.app.empty()) {
           SetNodeValue(cbuildNode[YAML_PROJECT], context->cproject->name);
         }
         SetNodeValue(cbuildNode[YAML_CONFIGURATION],
diff --git a/tools/projmgr/src/ProjMgrUtils.cpp b/tools/projmgr/src/ProjMgrUtils.cpp
@@ -498,3 +498,11 @@ const std::string ProjMgrUtils::NormalizeLineEndings(const std::string& in) {
   }
   return out;
 }
+
+const std::string ProjMgrUtils::GetWestBoard(const std::string& board) {
+  string westBoard = RteUtils::StripSuffix(RteUtils::StripPrefix(board, "::"));
+  std::transform(westBoard.begin(), westBoard.end(), westBoard.begin(),
+    [](unsigned char c) { return std::tolower(c); });
+  std::replace(westBoard.begin(), westBoard.end(), '-', '_');
+  return westBoard;
+}
diff --git a/tools/projmgr/src/ProjMgrWorker.cpp b/tools/projmgr/src/ProjMgrWorker.cpp
@@ -84,16 +84,26 @@ void ProjMgrWorker::AddImageOnlyContext() {
 }
 
 bool ProjMgrWorker::AddContexts(ProjMgrParser& parser, ContextDesc& descriptor, const string& cprojectFile) {
-  error_code ec;
   ContextItem context;
+  context.cdefault = &parser.GetCdefault();
+  context.csolution = &parser.GetCsolution();
   std::map<std::string, CprojectItem>& cprojects = parser.GetCprojects();
   if (cprojects.find(cprojectFile) == cprojects.end()) {
     ProjMgrLogger::Get().Error("cproject not parsed, adding context failed", "", cprojectFile);
     return false;
   }
   context.cproject = &cprojects.at(cprojectFile);
-  context.cdefault = &parser.GetCdefault();
-  context.csolution = &parser.GetCsolution();
+
+  // west app
+  if (!descriptor.west.app.empty()) {
+    context.west = descriptor.west;
+    RteFsUtils::NormalizePath(context.west.app, context.csolution->directory);
+    context.imageOnly = true;
+    //context.cproject = &cprojects[westApp];
+    context.cproject->output.baseName = "zephyr";
+    context.cproject->name = context.west.projectId;
+    context.cproject->output.type = { "elf", "hex" };
+  } 
 
   // No build/target-types
   if (context.csolution->buildTypes.empty() && context.csolution->targetTypes.empty()) {
@@ -2204,7 +2214,7 @@ bool ProjMgrWorker::ProcessConfigFiles(ContextItem& context) {
     }
   }
   // Linker script
-  if (context.outputTypes.elf.on) {
+  if (context.outputTypes.elf.on && !context.imageOnly) {
     if (context.linker.autoGen) {
       if (!context.linker.script.empty()) {
         ProjMgrLogger::Get().Warn("conflict: automatic linker script generation overrules specified script '" + context.linker.script + "'", context.name);
@@ -2863,6 +2873,10 @@ bool ProjMgrWorker::ProcessPrecedences(ContextItem& context, BoardOrDevice proce
     error |= true;
   }
 
+  if (!context.west.device.empty()) {
+    context.cproject->target.device = context.west.device;
+  }
+
   StringCollection device = {
     &context.device,
     {
@@ -2901,7 +2915,7 @@ bool ProjMgrWorker::ProcessPrecedences(ContextItem& context, BoardOrDevice proce
   if (!ProcessCompilerPrecedence(compiler, true)) {
     error |= true;
   }
-  if (!context.imageOnly && !ProcessToolchain(context)) {
+  if ((!context.imageOnly || !context.west.app.empty()) && !ProcessToolchain(context)) {
     error |= true;
   }
 
@@ -2914,6 +2928,9 @@ bool ProjMgrWorker::ProcessPrecedences(ContextItem& context, BoardOrDevice proce
   context.variables[RteConstants::AS_PNAME] = deviceItem.pname;
   context.variables[RteConstants::AS_BNAME] = boardItem.name;
   context.variables[RteConstants::AS_COMPILER] = context.toolchain.name;
+  context.variables.insert({ WEST_BOARD, ProjMgrUtils::GetWestBoard(boardItem.name) });
+  context.west.board = context.west.board.empty() ? context.variables.at(WEST_BOARD) :
+    RteUtils::ExpandAccessSequences(context.west.board, context.variables);
 
   // Add cdefault misc into csolution
   if (context.cdefault) {
@@ -3129,6 +3146,10 @@ bool ProjMgrWorker::ProcessPrecedences(ContextItem& context, BoardOrDevice proce
     CollectionUtils::AddStringItemsUniquely(definesAsmRef, setup.definesAsm);
   }
 
+  // Merge west defines
+  CollectionUtils::AddStringItemsUniquely(context.west.westDefs, context.controls.target.westDefs);
+  CollectionUtils::AddStringItemsUniquely(context.west.westDefs, context.controls.build.westDefs);
+
   // Includes
   vector<string> projectAddPaths, projectDelPaths;
   CollectionUtils::AddStringItemsUniquely(projectAddPaths, context.controls.cproject.addpaths);
diff --git a/tools/projmgr/src/ProjMgrYamlParser.cpp b/tools/projmgr/src/ProjMgrYamlParser.cpp

Original file line number	Diff line number	Diff line change
`@@ -487,7 +487,7 @@ bool ProjMgr::PopulateContexts(void) {`
`487`	`487`	`}`
`488`	`488`	`// Check cproject separate folders and unique names`
`489`	`489`	`const StrVec& cprojects = m_parser.GetCsolution().cprojects;`
`490`		`- if (!IsSolutionImageOnly() && cprojects.empty()) {`
	`490`	`+ if (!IsSolutionImageOnly() && cprojects.empty() && m_parser.GetCsolution().westApps.empty()) {`
`491`	`491`	`ProjMgrLogger::Get().Error("projects not found", "", m_csolutionFile);`
`492`	`492`	`return false;`
`493`	`493`	`}`
Original file line number	Diff line number	Diff line change
`@@ -498,3 +498,11 @@ const std::string ProjMgrUtils::NormalizeLineEndings(const std::string& in) {`
`498`	`498`	`}`
`499`	`499`	`return out;`
`500`	`500`	`}`
	`501`	`+`
	`502`	`+const std::string ProjMgrUtils::GetWestBoard(const std::string& board) {`
	`503`	`+ string westBoard = RteUtils::StripSuffix(RteUtils::StripPrefix(board, "::"));`
	`504`	`+ std::transform(westBoard.begin(), westBoard.end(), westBoard.begin(),`
	`505`	`+ [](unsigned char c) { return std::tolower(c); });`
	`506`	`+ std::replace(westBoard.begin(), westBoard.end(), '-', '_');`
	`507`	`+ return westBoard;`
	`508`	`+}`