Wrong assignment of Floorplan mode

efabless · Jan 24, 2024 · ad6dd18 · ad6dd18
1 parent e95ef63
commit ad6dd18
Showing 1 changed file with 8 additions and 7 deletions.
diff --git a/openlane/steps/openroad.py b/openlane/steps/openroad.py
@@ -194,6 +194,11 @@ class OpenROADStep(TclStep):
             Optional[Path],
             "Specifies the SDC file used during all implementation (PnR) steps",
         ),
+        Variable(
+            "FP_DEF_TEMPLATE",
+            Optional[Path],
+            "Points to the DEF file to be used as a template.",
+        ),
     ]
 
     @abstractmethod
@@ -653,11 +658,6 @@ class Floorplan(OpenROADStep):
     inputs = [DesignFormat.NETLIST]
 
     config_vars = OpenROADStep.config_vars + [
-        Variable(
-            "FP_DEF_TEMPLATE",
-            Optional[Path],
-            "Points to the DEF file to be used as a template.",
-        ),
         Variable(
             "FP_ASPECT_RATIO",
             Decimal,
@@ -734,11 +734,12 @@ def get_mode(Self, config: Config) -> Mode:
         if config.get("FP_DEF_TEMPLATE") and config.get("DIE_AREA"):
             warn("Specifing DIE_AREA with FP_DEF_TEMPLATE is redundant")
         if config.get("FP_DEF_TEMPLATE"):
-            mode = Self.Mode.RELATIVE
+            mode = Self.Mode.TEMPLATE
         elif config.get("DIE_AREA"):
             mode = Self.Mode.ABSOULTE
         else:
             mode = Self.Mode.RELATIVE
+        debug(f"Floorplan mode: {mode}")
         return mode
 
     def run(self, state_in: State, **kwargs) -> Tuple[ViewsUpdate, MetricsUpdate]:
@@ -801,7 +802,7 @@ def run(self, state_in: State, **kwargs) -> Tuple[ViewsUpdate, MetricsUpdate]:
             return {}, {}
         if Floorplan.get_mode(self.config) == Floorplan.Mode.TEMPLATE:
             info(
-                f"Flooplan was loaded from {self.config['FP_DEF_TEMPLATE']}. Skipping {self.id}"
+                f"Flooplan was loaded from {self.config['FP_DEF_TEMPLATE']} Skipping {self.id}"
             )
             return {}, {}