Remove the redundancy define NFP6000_CPPTGT_ in nfp_resid.h,
the same define NFP_CPP_TARGET_ are exist in nfp6000.h, we
choose to keep them to sync with the kernel driver.

Signed-off-by: Chaoyong He <chaoyong...@corigine.com>
Reviewed-by: Niklas Söderlund <niklas.soderl...@corigine.com>
---
 .../net/nfp/nfpcore/nfp-common/nfp_resid.h    | 11 -----
 drivers/net/nfp/nfpcore/nfp_target.c          | 44 +++++++++----------
 2 files changed, 22 insertions(+), 33 deletions(-)

diff --git a/drivers/net/nfp/nfpcore/nfp-common/nfp_resid.h 
b/drivers/net/nfp/nfpcore/nfp-common/nfp_resid.h
index 394a7628e0..ac2bf0335d 100644
--- a/drivers/net/nfp/nfpcore/nfp-common/nfp_resid.h
+++ b/drivers/net/nfp/nfpcore/nfp-common/nfp_resid.h
@@ -48,17 +48,6 @@
 #define NFP_CHIP_REVISION_C0 0x20
 #define NFP_CHIP_REVISION_PF 0xff /* Maximum possible revision */
 
-/* CPP Targets for each chip architecture */
-#define NFP6000_CPPTGT_NBI 1
-#define NFP6000_CPPTGT_VQDR 2
-#define NFP6000_CPPTGT_ILA 6
-#define NFP6000_CPPTGT_MU 7
-#define NFP6000_CPPTGT_PCIE 9
-#define NFP6000_CPPTGT_ARM 10
-#define NFP6000_CPPTGT_CRYPTO 12
-#define NFP6000_CPPTGT_CTXPB 14
-#define NFP6000_CPPTGT_CLS 15
-
 /*
  * Wildcard indicating a CPP read or write action
  *
diff --git a/drivers/net/nfp/nfpcore/nfp_target.c 
b/drivers/net/nfp/nfpcore/nfp_target.c
index 3f7ddfb5e9..611848e233 100644
--- a/drivers/net/nfp/nfpcore/nfp_target.c
+++ b/drivers/net/nfp/nfpcore/nfp_target.c
@@ -489,26 +489,26 @@ nfp_target_pushpull(uint32_t cpp_id,
                uint64_t address)
 {
        switch (NFP_CPP_ID_TARGET_of(cpp_id)) {
-       case NFP6000_CPPTGT_NBI:
+       case NFP_CPP_TARGET_NBI:
                return nfp6000_nbi(cpp_id, address);
-       case NFP6000_CPPTGT_VQDR:
+       case NFP_CPP_TARGET_QDR:
                return target_rw(cpp_id, P32, 24, 4);
-       case NFP6000_CPPTGT_ILA:
+       case NFP_CPP_TARGET_ILA:
                return nfp6000_ila(cpp_id);
-       case NFP6000_CPPTGT_MU:
+       case NFP_CPP_TARGET_MU:
                return nfp6000_mu(cpp_id, address);
-       case NFP6000_CPPTGT_PCIE:
+       case NFP_CPP_TARGET_PCIE:
                return nfp6000_pci(cpp_id);
-       case NFP6000_CPPTGT_ARM:
+       case NFP_CPP_TARGET_ARM:
                if (address < 0x10000)
                        return target_rw(cpp_id, P64, 1, 1);
                else
                        return target_rw(cpp_id, P32, 1, 1);
-       case NFP6000_CPPTGT_CRYPTO:
+       case NFP_CPP_TARGET_CRYPTO:
                return nfp6000_crypto(cpp_id);
-       case NFP6000_CPPTGT_CTXPB:
+       case NFP_CPP_TARGET_CT_XPB:
                return nfp6000_cap_xpb(cpp_id);
-       case NFP6000_CPPTGT_CLS:
+       case NFP_CPP_TARGET_CLS:
                return nfp6000_cls(cpp_id);
        case 0:
                return target_rw(cpp_id, P32, 4, 4);
@@ -719,7 +719,7 @@ nfp_encode_basic(uint64_t *addr,
 
        switch (mode) {
        case 0:
-               if (cpp_tgt == NFP6000_CPPTGT_VQDR && addr40 == 0) {
+               if (cpp_tgt == NFP_CPP_TARGET_QDR && addr40 == 0) {
                        /*
                         * In this specific mode we'd rather not modify the
                         * address but we can verify if the existing contents
@@ -737,7 +737,7 @@ nfp_encode_basic(uint64_t *addr,
                *addr |= (((uint64_t)dest_island) << iid_lsb) & value;
                return 0;
        case 1:
-               if (cpp_tgt == NFP6000_CPPTGT_VQDR && addr40 == 0) {
+               if (cpp_tgt == NFP_CPP_TARGET_QDR && addr40 == 0) {
                        return nfp_encode_basic_qdr(*addr, cpp_tgt, dest_island,
                                        mode, addr40, isld1, isld0);
                }
@@ -757,7 +757,7 @@ nfp_encode_basic(uint64_t *addr,
 
                return -ENODEV;
        case 2:
-               if (cpp_tgt == NFP6000_CPPTGT_VQDR && addr40 == 0) {
+               if (cpp_tgt == NFP_CPP_TARGET_QDR && addr40 == 0) {
                        /* iid<0> = addr<30> = channel<0> */
                        /* channel<1> = addr<31> = Index */
                        return nfp_encode_basic_qdr(*addr, cpp_tgt, dest_island,
@@ -777,7 +777,7 @@ nfp_encode_basic(uint64_t *addr,
                return nfp_encode_basic_search(addr, dest_island, isld,
                                iid_lsb, idx_lsb, 2);
        case 3:
-               if (cpp_tgt == NFP6000_CPPTGT_VQDR && addr40 == 0) {
+               if (cpp_tgt == NFP_CPP_TARGET_QDR && addr40 == 0) {
                        /*
                         * iid<0> = addr<29> = data
                         * iid<1> = addr<30> = channel<0>
@@ -924,19 +924,19 @@ nfp_cppat_addr_encode(uint64_t *addr,
        uint64_t value;
 
        switch (cpp_tgt) {
-       case NFP6000_CPPTGT_NBI:
-       case NFP6000_CPPTGT_VQDR:
-       case NFP6000_CPPTGT_ILA:
-       case NFP6000_CPPTGT_PCIE:
-       case NFP6000_CPPTGT_ARM:
-       case NFP6000_CPPTGT_CRYPTO:
-       case NFP6000_CPPTGT_CLS:
+       case NFP_CPP_TARGET_NBI:
+       case NFP_CPP_TARGET_QDR:
+       case NFP_CPP_TARGET_ILA:
+       case NFP_CPP_TARGET_PCIE:
+       case NFP_CPP_TARGET_ARM:
+       case NFP_CPP_TARGET_CRYPTO:
+       case NFP_CPP_TARGET_CLS:
                return nfp_encode_basic(addr, dest_island, cpp_tgt, mode,
                                addr40, isld1, isld0);
-       case NFP6000_CPPTGT_MU:
+       case NFP_CPP_TARGET_MU:
                return nfp_encode_mu(addr, dest_island, mode, addr40,
                                isld1, isld0);
-       case NFP6000_CPPTGT_CTXPB:
+       case NFP_CPP_TARGET_CT_XPB:
                if (mode != 1 || addr40 != 0)
                        return -EINVAL;
 
-- 
2.39.1

Reply via email to