diff options
author | Vernon Mauery <vernon.mauery@linux.intel.com> | 2020-02-06 13:25:08 -0800 |
---|---|---|
committer | Vernon Mauery <vernon.mauery@linux.intel.com> | 2020-02-06 13:35:26 -0800 |
commit | ccf12d213c95201a14f7d7014076af6162c6e995 (patch) | |
tree | 906f8ed09c728e7157a21435f3f9db75458bdc0c /storagehandler.cpp | |
parent | e9f8857a66c5df61a4e795a7d71b035fd89b92ef (diff) | |
download | phosphor-host-ipmid-ccf12d213c95201a14f7d7014076af6162c6e995.tar.gz phosphor-host-ipmid-ccf12d213c95201a14f7d7014076af6162c6e995.zip |
Remove useless wildcard handlers
The default action is to return Command Not Found, which is exactly what
the wildcard handlers are doing. This just removes that extra code that
is not needed.
Change-Id: I0a25153395858efcdf56384ececaabc7eaf4e1af
Signed-off-by: Vernon Mauery <vernon.mauery@linux.intel.com>
Diffstat (limited to 'storagehandler.cpp')
-rw-r--r-- | storagehandler.cpp | 16 |
1 files changed, 0 insertions, 16 deletions
diff --git a/storagehandler.cpp b/storagehandler.cpp index d2bc8a5..5b9726a 100644 --- a/storagehandler.cpp +++ b/storagehandler.cpp @@ -76,18 +76,6 @@ enum class AccessMode words ///< Device is accessed by words }; -ipmi_ret_t ipmi_storage_wildcard(ipmi_netfn_t netfn, ipmi_cmd_t cmd, - ipmi_request_t request, - ipmi_response_t response, - ipmi_data_len_t data_len, - ipmi_context_t context) -{ - // Status code. - ipmi_ret_t rc = IPMI_CC_INVALID; - *data_len = 0; - return rc; -} - /** @brief implements the get SEL Info command * @returns IPMI completion code plus response data * - selVersion - SEL revision @@ -744,10 +732,6 @@ ipmi::RspType<uint8_t, // SDR version void register_netfn_storage_functions() { - // <Wildcard Command> - ipmi_register_callback(NETFUN_STORAGE, IPMI_CMD_WILDCARD, NULL, - ipmi_storage_wildcard, PRIVILEGE_USER); - // <Get SEL Info> ipmi::registerHandler(ipmi::prioOpenBmcBase, ipmi::netFnStorage, ipmi::storage::cmdGetSelInfo, ipmi::Privilege::User, |