Commit b3b0ddd0 authored by Michael Chan's avatar Michael Chan Committed by David S. Miller

bnxt_en: Use a common function to print the same ethtool -f error message.

The same message is printed 3 times in the code, so use a common function
to do that.
Signed-off-by: default avatarMichael Chan <michael.chan@broadcom.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 19828bd3
...@@ -1699,6 +1699,11 @@ static u32 bnxt_get_link(struct net_device *dev) ...@@ -1699,6 +1699,11 @@ static u32 bnxt_get_link(struct net_device *dev)
return bp->link_info.link_up; return bp->link_info.link_up;
} }
static void bnxt_print_admin_err(struct bnxt *bp)
{
netdev_info(bp->dev, "PF does not have admin privileges to flash or reset the device\n");
}
static int bnxt_find_nvram_item(struct net_device *dev, u16 type, u16 ordinal, static int bnxt_find_nvram_item(struct net_device *dev, u16 type, u16 ordinal,
u16 ext, u16 *index, u32 *item_length, u16 ext, u16 *index, u32 *item_length,
u32 *data_length); u32 *data_length);
...@@ -1739,8 +1744,7 @@ static int bnxt_flash_nvram(struct net_device *dev, ...@@ -1739,8 +1744,7 @@ static int bnxt_flash_nvram(struct net_device *dev,
dma_free_coherent(&bp->pdev->dev, data_len, kmem, dma_handle); dma_free_coherent(&bp->pdev->dev, data_len, kmem, dma_handle);
if (rc == HWRM_ERR_CODE_RESOURCE_ACCESS_DENIED) { if (rc == HWRM_ERR_CODE_RESOURCE_ACCESS_DENIED) {
netdev_info(dev, bnxt_print_admin_err(bp);
"PF does not have admin privileges to flash the device\n");
rc = -EACCES; rc = -EACCES;
} else if (rc) { } else if (rc) {
rc = -EIO; rc = -EIO;
...@@ -1795,8 +1799,7 @@ static int bnxt_firmware_reset(struct net_device *dev, ...@@ -1795,8 +1799,7 @@ static int bnxt_firmware_reset(struct net_device *dev,
rc = hwrm_send_message(bp, &req, sizeof(req), HWRM_CMD_TIMEOUT); rc = hwrm_send_message(bp, &req, sizeof(req), HWRM_CMD_TIMEOUT);
if (rc == HWRM_ERR_CODE_RESOURCE_ACCESS_DENIED) { if (rc == HWRM_ERR_CODE_RESOURCE_ACCESS_DENIED) {
netdev_info(dev, bnxt_print_admin_err(bp);
"PF does not have admin privileges to reset the device\n");
rc = -EACCES; rc = -EACCES;
} else if (rc) { } else if (rc) {
rc = -EIO; rc = -EIO;
...@@ -2096,8 +2099,7 @@ static int bnxt_flash_package_from_file(struct net_device *dev, ...@@ -2096,8 +2099,7 @@ static int bnxt_flash_package_from_file(struct net_device *dev,
mutex_unlock(&bp->hwrm_cmd_lock); mutex_unlock(&bp->hwrm_cmd_lock);
err_exit: err_exit:
if (hwrm_err == HWRM_ERR_CODE_RESOURCE_ACCESS_DENIED) { if (hwrm_err == HWRM_ERR_CODE_RESOURCE_ACCESS_DENIED) {
netdev_info(dev, bnxt_print_admin_err(bp);
"PF does not have admin privileges to flash the device\n");
rc = -EACCES; rc = -EACCES;
} else if (hwrm_err) { } else if (hwrm_err) {
rc = -EOPNOTSUPP; rc = -EOPNOTSUPP;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment