diff options
author | Andrew Jeffery <andrew@aj.id.au> | 2018-02-27 14:56:07 +1030 |
---|---|---|
committer | Andrew Jeffery <andrew@aj.id.au> | 2018-03-24 13:59:32 +1030 |
commit | d976c9c9f93847063bdde53ae7024766885e0d0e (patch) | |
tree | 3e100f591f1e5d80cbafab74fdd47f1763ee5d4e | |
parent | d394a787595281838831182f534ff8faf95a15ba (diff) | |
download | phosphor-mboxd-d976c9c9f93847063bdde53ae7024766885e0d0e.tar.gz phosphor-mboxd-d976c9c9f93847063bdde53ae7024766885e0d0e.zip |
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>
-rw-r--r-- | mboxd_pnor_partition_table.cpp | 9 | ||||
-rw-r--r-- | pnor_partition_table.cpp | 5 | ||||
-rw-r--r-- | pnor_partition_table.hpp | 7 |
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; |