From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tomasz Jozwiak Subject: [PATCH v3 34/38] crypto/qat: remove unused arguments Date: Wed, 13 Jun 2018 14:14:18 +0200 Message-ID: <1528892062-4997-35-git-send-email-tomaszx.jozwiak@intel.com> References: <1523040732-3290-1-git-send-email-fiona.trahe@intel.com> <1528892062-4997-1-git-send-email-tomaszx.jozwiak@intel.com> To: fiona.trahe@intel.com, tomaszx.jozwiak@intel.com, dev@dpdk.org Return-path: Received: from mga12.intel.com (mga12.intel.com [192.55.52.136]) by dpdk.org (Postfix) with ESMTP id 474801EF96 for ; Wed, 13 Jun 2018 14:15:11 +0200 (CEST) In-Reply-To: <1528892062-4997-1-git-send-email-tomaszx.jozwiak@intel.com> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" From: Fiona Trahe Process response function is only implemented for crypto symmetric operations, which do not require some of the arguments. Therefore, these arguments can be removed from the function prototype. Signed-off-by: Tomasz Jozwiak Signed-off-by: Fiona Trahe --- drivers/crypto/qat/qat_qp.c | 4 +--- drivers/crypto/qat/qat_qp.h | 4 +--- drivers/crypto/qat/qat_sym.c | 4 +--- drivers/crypto/qat/qat_sym.h | 4 +--- 4 files changed, 4 insertions(+), 12 deletions(-) diff --git a/drivers/crypto/qat/qat_qp.c b/drivers/crypto/qat/qat_qp.c index b84ba643c..0fdec0da0 100644 --- a/drivers/crypto/qat/qat_qp.c +++ b/drivers/crypto/qat/qat_qp.c @@ -612,9 +612,7 @@ qat_dequeue_op_burst(void *qp, void **ops, uint16_t nb_ops) while (*(uint32_t *)resp_msg != ADF_RING_EMPTY_SIG && resp_counter != nb_ops) { - tmp_qp->process_response(ops, resp_msg, - tmp_qp->op_cookies[head / rx_queue->msg_size], - tmp_qp->qat_dev_gen); + tmp_qp->process_response(ops, resp_msg); head = adf_modulo(head + rx_queue->msg_size, rx_queue->modulo_mask); diff --git a/drivers/crypto/qat/qat_qp.h b/drivers/crypto/qat/qat_qp.h index 764125d59..eb9188410 100644 --- a/drivers/crypto/qat/qat_qp.h +++ b/drivers/crypto/qat/qat_qp.h @@ -21,9 +21,7 @@ typedef int (*build_request_t)(void *op, enum qat_device_gen qat_dev_gen); /**< Build a request from an op. */ -typedef int (*process_response_t)(void **ops, - uint8_t *resp, void *op_cookie, - enum qat_device_gen qat_dev_gen); +typedef int (*process_response_t)(void **ops, uint8_t *resp); /**< Process a response descriptor and return the associated op. */ /** diff --git a/drivers/crypto/qat/qat_sym.c b/drivers/crypto/qat/qat_sym.c index 15244d113..f613adce6 100644 --- a/drivers/crypto/qat/qat_sym.c +++ b/drivers/crypto/qat/qat_sym.c @@ -661,9 +661,7 @@ qat_sym_build_request(void *in_op, uint8_t *out_msg, } int -qat_sym_process_response(void **op, uint8_t *resp, - __rte_unused void *op_cookie, - __rte_unused enum qat_device_gen qat_dev_gen) +qat_sym_process_response(void **op, uint8_t *resp) { struct icp_qat_fw_comn_resp *resp_msg = diff --git a/drivers/crypto/qat/qat_sym.h b/drivers/crypto/qat/qat_sym.h index bdb672be4..dffd5f369 100644 --- a/drivers/crypto/qat/qat_sym.h +++ b/drivers/crypto/qat/qat_sym.h @@ -22,8 +22,6 @@ int qat_sym_build_request(void *in_op, uint8_t *out_msg, void *op_cookie, enum qat_device_gen qat_dev_gen); int -qat_sym_process_response(void **op, uint8_t *resp, - __rte_unused void *op_cookie, - __rte_unused enum qat_device_gen qat_dev_gen); +qat_sym_process_response(void **op, uint8_t *resp); #endif /* _QAT_SYM_H_ */ -- 2.17.0