pnor_partition_table: Drop Table ctor with default location

Make the caller be explicit about what they want. Also, rearrange calls
in mboxd_pnor_partition_table.cpp to remove noise there as well.

Change-Id: I8e36101b84b1b24cf0edbedb744f8edca15a834a
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>

3 files changed, 5 insertions, 16 deletions

diff --git a/mboxd_pnor_partition_table.cpp b/mboxd_pnor_partition_table.cpp
index 3a09c92..84a0314 100644
--- a/mboxd_pnor_partition_table.cpp
+++ b/mboxd_pnor_partition_table.cpp
@@ -22,9 +22,7 @@ void init_vpnor(struct mbox_context *context)
         strcpy(context->paths.prsv_loc, PARTITION_FILES_PRSV_LOC);
         strcpy(context->paths.patch_loc, PARTITION_FILES_PATCH_LOC);
 
-        context->vpnor = new vpnor_partition_table;
-        context->vpnor->table = new openpower::virtual_pnor::partition::Table(
-            1 << context->erase_size_shift, context->flash_size);
+        vpnor_create_partition_table_from_path(context, PARTITION_FILES_RO_LOC);
     }
 }
 
@@ -78,8 +76,11 @@ void vpnor_copy_bootloader_partition(const struct mbox_context *context)
     constexpr size_t tocMaxSize = 0x8000;
     constexpr size_t tocStart = pnorSize - tocMaxSize - pageSize;
     constexpr auto blPartitionName = "HBB";
+    namespace fs = std::experimental::filesystem;
+
+    openpower::virtual_pnor::partition::Table blTable(
+        fs::path{PARTITION_FILES_RO_LOC}, eraseSize, pnorSize);
 
-    openpower::virtual_pnor::partition::Table blTable(eraseSize, pnorSize);
     vpnor_partition_table vtbl{};
     vtbl.table = &blTable;
     struct mbox_context local
diff --git a/pnor_partition_table.cpp b/pnor_partition_table.cpp
index d4c569c..b0d002e 100644
--- a/pnor_partition_table.cpp
+++ b/pnor_partition_table.cpp
@@ -20,11 +20,6 @@ using namespace sdbusplus::xyz::openbmc_project::Common::Error;
 namespace partition
 {
 
-Table::Table(size_t blockSize, size_t pnorSize) :
-    Table(fs::path(PARTITION_FILES_RO_LOC), blockSize, pnorSize)
-{
-}
-
 Table::Table(fs::path&& directory, size_t blockSize, size_t pnorSize) :
     szBlocks(0), directory(std::move(directory)), numParts(0),
     blockSize(blockSize), pnorSize(pnorSize)
diff --git a/pnor_partition_table.hpp b/pnor_partition_table.hpp
index d470822..1105134 100644
--- a/pnor_partition_table.hpp
+++ b/pnor_partition_table.hpp
@@ -94,13 +94,6 @@ class Table
      */
     Table(fs::path&& directory, size_t blockSize, size_t pnorSize);
 
-    /** @brief Constructor - creates partition table
-     *
-     *  @param[in] blockSize - PNOR block size, in bytes
-     *  @param[in] pnorSize - PNOR size, in bytes
-     */
-    Table(size_t blockSize, size_t pnorSize);
-
     Table(const Table&) = delete;
     Table& operator=(const Table&) = delete;
     Table(Table&&) = delete;