common/sfc_efx/base: separate target EvQ and IRQ config

[dpdk.git] / drivers / common / sfc_efx / base / efx_mcdi.c

diff --git a/drivers/common/sfc_efx/base/efx_mcdi.c b/drivers/common/sfc_efx/base/efx_mcdi.c
index 8c984d8..b68fc05 100644 (file)
--- a/drivers/common/sfc_efx/base/efx_mcdi.c
+++ b/drivers/common/sfc_efx/base/efx_mcdi.c
@@ -1,6 +1,6 @@
 /* SPDX-License-Identifier: BSD-3-Clause
  *
- * Copyright(c) 2019-2020 Xilinx, Inc.
+ * Copyright(c) 2019-2021 Xilinx, Inc.
  * Copyright(c) 2008-2019 Solarflare Communications Inc.
  */
 
@@ -516,6 +516,9 @@ efx_mcdi_finish_response(
        bytes = MIN(emrp->emr_out_length_used, emrp->emr_out_length);
        efx_mcdi_read_response(enp, emrp->emr_out_buf, resp_off, bytes);
 
+       /* Report bytes copied to caller (response message may be larger) */
+       emrp->emr_out_length_used = bytes;
+
 #if EFSYS_OPT_MCDI_LOGGING
        if (emtp->emt_logger != NULL) {
                emtp->emt_logger(emtp->emt_context,
@@ -964,15 +967,17 @@ efx_mcdi_ev_death(
        __checkReturn           efx_rc_t
 efx_mcdi_get_version(
        __in                    efx_nic_t *enp,
-       __in                    uint32_t flags_req,
+       __in                    uint32_t flags,
        __out                   efx_mcdi_version_t *verp)
 {
        efx_nic_board_info_t *board_infop = &verp->emv_board_info;
        EFX_MCDI_DECLARE_BUF(payload,
            MC_CMD_GET_VERSION_EXT_IN_LEN,
            MC_CMD_GET_VERSION_V2_OUT_LEN);
-       size_t min_resp_len_required;
+       efx_word_t *ver_words;
+       uint16_t version[4];
        efx_mcdi_req_t req;
+       uint32_t firmware;
        efx_rc_t rc;
 
        EFX_STATIC_ASSERT(sizeof (verp->emv_version) ==
@@ -996,18 +1001,14 @@ efx_mcdi_get_version(
        req.emr_in_buf = payload;
        req.emr_out_buf = payload;
 
-       if (flags_req != 0) {
+       if ((flags & EFX_MCDI_VERSION_BOARD_INFO) != 0) {
                /* Request basic + extended version information. */
                req.emr_in_length = MC_CMD_GET_VERSION_EXT_IN_LEN;
                req.emr_out_length = MC_CMD_GET_VERSION_V2_OUT_LEN;
-
-               min_resp_len_required = MC_CMD_GET_VERSION_V2_OUT_LEN;
        } else {
                /* Request only basic version information. */
                req.emr_in_length = MC_CMD_GET_VERSION_IN_LEN;
                req.emr_out_length = MC_CMD_GET_VERSION_OUT_LEN;
-
-               min_resp_len_required = MC_CMD_GET_VERSION_V0_OUT_LEN;
        }
 
        efx_mcdi_execute(enp, &req);
@@ -1017,33 +1018,36 @@ efx_mcdi_get_version(
                goto fail1;
        }
 
-       if (req.emr_out_length_used < min_resp_len_required) {
+       /* bootrom support */
+       if (req.emr_out_length_used == MC_CMD_GET_VERSION_V0_OUT_LEN) {
+               version[0] = version[1] = version[2] = version[3] = 0;
+               firmware = MCDI_OUT_DWORD(req, GET_VERSION_OUT_FIRMWARE);
+               goto out;
+       }
+
+       if (req.emr_out_length_used < req.emr_out_length) {
                rc = EMSGSIZE;
                goto fail2;
        }
 
-       memset(verp, 0, sizeof (*verp));
-
-       if (req.emr_out_length_used > min_resp_len_required) {
-               efx_word_t *ver_words;
+       ver_words = MCDI_OUT2(req, efx_word_t, GET_VERSION_OUT_VERSION);
+       version[0] = EFX_WORD_FIELD(ver_words[0], EFX_WORD_0);
+       version[1] = EFX_WORD_FIELD(ver_words[1], EFX_WORD_0);
+       version[2] = EFX_WORD_FIELD(ver_words[2], EFX_WORD_0);
+       version[3] = EFX_WORD_FIELD(ver_words[3], EFX_WORD_0);
+       firmware = MCDI_OUT_DWORD(req, GET_VERSION_OUT_FIRMWARE);
 
-               if (req.emr_out_length_used < MC_CMD_GET_VERSION_OUT_LEN) {
-                       rc = EMSGSIZE;
-                       goto fail3;
-               }
-
-               ver_words = MCDI_OUT2(req, efx_word_t, GET_VERSION_OUT_VERSION);
-
-               verp->emv_version[0] = EFX_WORD_FIELD(ver_words[0], EFX_WORD_0);
-               verp->emv_version[1] = EFX_WORD_FIELD(ver_words[1], EFX_WORD_0);
-               verp->emv_version[2] = EFX_WORD_FIELD(ver_words[2], EFX_WORD_0);
-               verp->emv_version[3] = EFX_WORD_FIELD(ver_words[3], EFX_WORD_0);
-       }
+out:
+       memset(verp, 0, sizeof (*verp));
 
-       verp->emv_firmware = MCDI_OUT_DWORD(req, GET_VERSION_OUT_FIRMWARE);
+       verp->emv_version[0] = version[0];
+       verp->emv_version[1] = version[1];
+       verp->emv_version[2] = version[2];
+       verp->emv_version[3] = version[3];
+       verp->emv_firmware = firmware;
 
        verp->emv_flags = MCDI_OUT_DWORD(req, GET_VERSION_V2_OUT_FLAGS);
-       verp->emv_flags &= flags_req;
+       verp->emv_flags &= flags;
 
        if ((verp->emv_flags & EFX_MCDI_VERSION_BOARD_INFO) != 0) {
                memcpy(board_infop->enbi_serial,
@@ -1058,8 +1062,6 @@ efx_mcdi_get_version(
 
        return (0);
 
-fail3:
-       EFSYS_PROBE(fail3);
 fail2:
        EFSYS_PROBE(fail2);
 fail1:
@@ -1089,6 +1091,12 @@ efx_mcdi_get_boot_status(
 
        efx_mcdi_execute_quiet(enp, &req);
 
+       /*
+        * NOTE: Unprivileged functions cannot access boot status,
+        *       so the MCDI request will return EACCES. This is
+        *       also checked in efx_mcdi_version.
+        */
+
        if (req.emr_rc != 0) {
                rc = req.emr_rc;
                goto fail1;
@@ -2286,6 +2294,11 @@ efx_mcdi_get_workarounds(
                goto fail1;
        }
 
+       if (req.emr_out_length_used < MC_CMD_GET_WORKAROUNDS_OUT_LEN) {
+               rc = EMSGSIZE;
+               goto fail2;
+       }
+
        if (implementedp != NULL) {
                *implementedp =
                    MCDI_OUT_DWORD(req, GET_WORKAROUNDS_OUT_IMPLEMENTED);
@@ -2297,6 +2310,8 @@ efx_mcdi_get_workarounds(
 
        return (0);
 
+fail2:
+       EFSYS_PROBE(fail2);
 fail1:
        EFSYS_PROBE1(fail1, efx_rc_t, rc);
 
@@ -2553,6 +2568,7 @@ efx_mcdi_init_evq(
        __in            efsys_mem_t *esmp,
        __in            size_t nevs,
        __in            uint32_t irq,
+       __in            uint32_t target_evq,
        __in            uint32_t us,
        __in            uint32_t flags,
        __in            boolean_t low_latency)
@@ -2587,11 +2603,15 @@ efx_mcdi_init_evq(
 
        MCDI_IN_SET_DWORD(req, INIT_EVQ_V2_IN_SIZE, nevs);
        MCDI_IN_SET_DWORD(req, INIT_EVQ_V2_IN_INSTANCE, instance);
-       MCDI_IN_SET_DWORD(req, INIT_EVQ_V2_IN_IRQ_NUM, irq);
 
        interrupting = ((flags & EFX_EVQ_FLAGS_NOTIFY_MASK) ==
            EFX_EVQ_FLAGS_NOTIFY_INTERRUPT);
 
+       if (interrupting)
+               MCDI_IN_SET_DWORD(req, INIT_EVQ_V2_IN_IRQ_NUM, irq);
+       else
+               MCDI_IN_SET_DWORD(req, INIT_EVQ_V2_IN_TARGET_EVQ, target_evq);
+
        if (encp->enc_init_evq_v2_supported) {
                /*
                 * On Medford the low latency license is required to enable RX