|
 |
f95c89 |
From 2c4d992b57a4b6709fcda6fd12b1f53c19a719cc Mon Sep 17 00:00:00 2001
|
|
|
f95c89 |
From: Petr Oros <poros@redhat.com>
|
|
|
f95c89 |
Date: Thu, 5 Sep 2019 10:00:58 -0400
|
|
|
f95c89 |
Subject: [PATCH 03/96] [netdrv] revert "devlink: Add a generic wake_on_lan
|
|
|
f95c89 |
port parameter"
|
|
|
f95c89 |
|
|
|
f95c89 |
Message-id: <e21412cfa07f102b8c9b95a10d802d6abdd9b409.1567595345.git.poros@redhat.com>
|
|
|
f95c89 |
Patchwork-id: 270934
|
|
|
f95c89 |
O-Subject: [RHEL8.2 net PATCH 02/15] Revert "devlink: Add a generic wake_on_lan port parameter"
|
|
|
f95c89 |
Bugzilla: 1737133
|
|
|
f95c89 |
RH-Acked-by: Jonathan Toppins <jtoppins@redhat.com>
|
|
|
f95c89 |
RH-Acked-by: Neil Horman <nhorman@redhat.com>
|
|
|
f95c89 |
RH-Acked-by: John Linville <linville@redhat.com>
|
|
|
f95c89 |
RH-Acked-by: Ivan Vecera <ivecera@redhat.com>
|
|
|
f95c89 |
|
|
|
f95c89 |
Bugzilla: http://bugzilla.redhat.com/show_bug.cgi?id=1737133
|
|
|
f95c89 |
|
|
|
f95c89 |
Upstream commit(s):
|
|
|
f95c89 |
commit da203dfa89ce83c55b6623f73560ef7ec742aca4
|
|
|
f95c89 |
Author: Vasundhara Volam <vasundhara-v.volam@broadcom.com>
|
|
|
f95c89 |
Date: Mon Feb 11 14:46:17 2019 +0530
|
|
|
f95c89 |
|
|
|
f95c89 |
Revert "devlink: Add a generic wake_on_lan port parameter"
|
|
|
f95c89 |
|
|
|
f95c89 |
This reverts commit b639583f9e36d044ac1b13090ae812266992cbac.
|
|
|
f95c89 |
|
|
|
f95c89 |
As per discussion with Jakub Kicinski and Michal Kubecek,
|
|
|
f95c89 |
this will be better addressed by soon-too-come ethtool netlink
|
|
|
f95c89 |
API with additional indication that given configuration request
|
|
|
f95c89 |
is supposed to be persisted.
|
|
|
f95c89 |
|
|
|
f95c89 |
Also, remove the parameter support from bnxt_en driver.
|
|
|
f95c89 |
|
|
|
f95c89 |
Cc: Jiri Pirko <jiri@mellanox.com>
|
|
|
f95c89 |
Cc: Michael Chan <michael.chan@broadcom.com>
|
|
|
f95c89 |
Cc: Michal Kubecek <mkubecek@suse.cz>
|
|
|
f95c89 |
Suggested-by: Jakub Kicinski <jakub.kicinski@netronome.com>
|
|
|
f95c89 |
Signed-off-by: Vasundhara Volam <vasundhara-v.volam@broadcom.com>
|
|
|
f95c89 |
Signed-off-by: David S. Miller <davem@davemloft.net>
|
|
|
f95c89 |
|
|
|
f95c89 |
Signed-off-by: Petr Oros <poros@redhat.com>
|
|
|
f95c89 |
Signed-off-by: Timothy Redaelli <tredaelli@redhat.com>
|
|
|
f95c89 |
Signed-off-by: Bruno Meneguele <bmeneg@redhat.com>
|
|
|
f95c89 |
---
|
|
|
f95c89 |
drivers/net/ethernet/broadcom/bnxt/bnxt_devlink.c | 19 +------------------
|
|
|
f95c89 |
drivers/net/ethernet/broadcom/bnxt/bnxt_devlink.h | 1 -
|
|
|
f95c89 |
2 files changed, 1 insertion(+), 19 deletions(-)
|
|
|
f95c89 |
|
|
|
f95c89 |
Index: src/drivers/net/ethernet/broadcom/bnxt/bnxt_devlink.c
|
|
|
f95c89 |
===================================================================
|
|
|
f95c89 |
--- src.orig/drivers/net/ethernet/broadcom/bnxt/bnxt_devlink.c 2020-02-06 16:23:11.134555156 +0100
|
|
|
f95c89 |
+++ src/drivers/net/ethernet/broadcom/bnxt/bnxt_devlink.c 2020-02-06 16:23:11.278553834 +0100
|
|
|
f95c89 |
@@ -37,8 +37,6 @@
|
|
|
f95c89 |
NVM_OFF_MSIX_VEC_PER_PF_MIN, BNXT_NVM_SHARED_CFG, 7},
|
|
|
f95c89 |
{BNXT_DEVLINK_PARAM_ID_GRE_VER_CHECK, NVM_OFF_DIS_GRE_VER_CHECK,
|
|
|
f95c89 |
BNXT_NVM_SHARED_CFG, 1},
|
|
|
f95c89 |
-
|
|
|
f95c89 |
- {DEVLINK_PARAM_GENERIC_ID_WOL, NVM_OFF_WOL, BNXT_NVM_PORT_CFG, 1},
|
|
|
f95c89 |
};
|
|
|
f95c89 |
|
|
|
f95c89 |
static int bnxt_hwrm_nvm_req(struct bnxt *bp, u32 param_id, void *msg,
|
|
|
f95c89 |
@@ -72,8 +70,7 @@
|
|
|
f95c89 |
bytesize = roundup(nvm_param.num_bits, BITS_PER_BYTE) / BITS_PER_BYTE;
|
|
|
f95c89 |
switch (bytesize) {
|
|
|
f95c89 |
case 1:
|
|
|
f95c89 |
- if (nvm_param.num_bits == 1 &&
|
|
|
f95c89 |
- nvm_param.id != DEVLINK_PARAM_GENERIC_ID_WOL)
|
|
|
f95c89 |
+ if (nvm_param.num_bits == 1)
|
|
|
f95c89 |
buf = &val->vbool;
|
|
|
f95c89 |
else
|
|
|
f95c89 |
buf = &val->vu8;
|
|
|
f95c89 |
@@ -167,17 +164,6 @@
|
|
|
f95c89 |
return 0;
|
|
|
f95c89 |
}
|
|
|
f95c89 |
|
|
|
f95c89 |
-static int bnxt_dl_wol_validate(struct devlink *dl, u32 id,
|
|
|
f95c89 |
- union devlink_param_value val,
|
|
|
f95c89 |
- struct netlink_ext_ack *extack)
|
|
|
f95c89 |
-{
|
|
|
f95c89 |
- if (val.vu8 && val.vu8 != DEVLINK_PARAM_WAKE_MAGIC) {
|
|
|
f95c89 |
- NL_SET_ERR_MSG_MOD(extack, "WOL type is not supported");
|
|
|
f95c89 |
- return -EINVAL;
|
|
|
f95c89 |
- }
|
|
|
f95c89 |
- return 0;
|
|
|
f95c89 |
-}
|
|
|
f95c89 |
-
|
|
|
f95c89 |
static const struct devlink_param bnxt_dl_params[] = {
|
|
|
f95c89 |
DEVLINK_PARAM_GENERIC(ENABLE_SRIOV,
|
|
|
f95c89 |
BIT(DEVLINK_PARAM_CMODE_PERMANENT),
|
|
|
f95c89 |
@@ -203,9 +189,6 @@
|
|
|
f95c89 |
};
|
|
|
f95c89 |
|
|
|
f95c89 |
static const struct devlink_param bnxt_dl_port_params[] = {
|
|
|
f95c89 |
- DEVLINK_PARAM_GENERIC(WOL, BIT(DEVLINK_PARAM_CMODE_PERMANENT),
|
|
|
f95c89 |
- bnxt_dl_nvm_param_get, bnxt_dl_nvm_param_set,
|
|
|
f95c89 |
- bnxt_dl_wol_validate),
|
|
|
f95c89 |
};
|
|
|
f95c89 |
|
|
|
f95c89 |
int bnxt_dl_register(struct bnxt *bp)
|
|
|
f95c89 |
Index: src/drivers/net/ethernet/broadcom/bnxt/bnxt_devlink.h
|
|
|
f95c89 |
===================================================================
|
|
|
f95c89 |
--- src.orig/drivers/net/ethernet/broadcom/bnxt/bnxt_devlink.h 2020-02-06 16:23:11.134555156 +0100
|
|
|
f95c89 |
+++ src/drivers/net/ethernet/broadcom/bnxt/bnxt_devlink.h 2020-02-06 16:23:11.278553834 +0100
|
|
|
f95c89 |
@@ -35,7 +35,6 @@
|
|
|
f95c89 |
|
|
|
f95c89 |
#define NVM_OFF_MSIX_VEC_PER_PF_MAX 108
|
|
|
f95c89 |
#define NVM_OFF_MSIX_VEC_PER_PF_MIN 114
|
|
|
f95c89 |
-#define NVM_OFF_WOL 152
|
|
|
f95c89 |
#define NVM_OFF_IGNORE_ARI 164
|
|
|
f95c89 |
#define NVM_OFF_DIS_GRE_VER_CHECK 171
|
|
|
f95c89 |
#define NVM_OFF_ENABLE_SRIOV 401
|