diff --git a/openfpga/src/base/openfpga_build_fabric_template.h b/openfpga/src/base/openfpga_build_fabric_template.h
index 848742933..830e730d4 100644
--- a/openfpga/src/base/openfpga_build_fabric_template.h
+++ b/openfpga/src/base/openfpga_build_fabric_template.h
@@ -444,7 +444,7 @@ int write_fabric_pin_physical_location_template(
   std::string module_name("*"); /* Use a wildcard for everything */
   CommandOptionId opt_module = cmd.option("module");
   if (true == cmd_context.option_enable(cmd, opt_module)) {
-    module_name = cmd_context.option_value(cmd, opt_module).empty();
+    module_name = cmd_context.option_value(cmd, opt_module);
   }
 
   /* Write hierarchy to a file */
diff --git a/openfpga/src/fabric/write_xml_fabric_pin_physical_location.cpp b/openfpga/src/fabric/write_xml_fabric_pin_physical_location.cpp
index ccc9fd613..a725fafed 100644
--- a/openfpga/src/fabric/write_xml_fabric_pin_physical_location.cpp
+++ b/openfpga/src/fabric/write_xml_fabric_pin_physical_location.cpp
@@ -102,7 +102,7 @@ static int write_xml_fabric_module_pin_phy_loc(
     }
     cnt++;
   }
-  VTR_LOGV(verbose, "Output '%lu' ports with physical sides for module '%s'\n",
+  VTR_LOGV(verbose, "Output %lu ports with physical sides for module '%s'\n",
            cnt, module_manager.module_name(curr_module).c_str());
 
   /* Print a tail */
@@ -154,7 +154,7 @@ int write_xml_fabric_pin_physical_location(const char* fname,
     if (!std::regex_match(curr_module_name, wildcard_regex)) {
       continue;
     }
-    VTR_LOGV(verbose, "Output pin physical location of module '%s'.\n", curr_module_name.c_str());
+    VTR_LOGV(verbose, "Outputted pin physical location of module '%s'.\n", curr_module_name.c_str());
     /* Write the pin physical location for this module */
     int err_code = write_xml_fabric_module_pin_phy_loc(
       fp, module_manager, curr_module, show_invalid_side, verbose);