>-----Original Message-----
>From: dev <dev-boun...@dpdk.org> On Behalf Of Fan Zhang
>Sent: Friday 22 October 2021 18:04
>To: dev@dpdk.org
>Cc: gak...@marvell.com; Zhang, Roy Fan <roy.fan.zh...@intel.com>; Adam
>Dybkowski <adamx.dybkow...@intel.com>; Kusztal, ArkadiuszX
><arkadiuszx.kusz...@intel.com>; Ji, Kai <kai...@intel.com>
>Subject: [dpdk-dev] [dpdk-dev v4 6/9] compress/qat: add gen specific
>implementation
>
>This patch replaces the mixed QAT compression support implementation by
>separate files with shared or individual implementation for specific QAT
>generation.
>
>Signed-off-by: Adam Dybkowski <adamx.dybkow...@intel.com>
>Signed-off-by: Arek Kusztal <arkadiuszx.kusz...@intel.com>
>Signed-off-by: Fan Zhang <roy.fan.zh...@intel.com>
>Signed-off-by: Kai Ji <kai...@intel.com>
>---
<snip>

>--- /dev/null
>+++ b/drivers/compress/qat/dev/qat_comp_pmd_gens.h
>@@ -0,0 +1,30 @@
>+/* SPDX-License-Identifier: BSD-3-Clause
>+ * Copyright(c) 2021 Intel Corporation
>+ */
>+
>+#ifndef _QAT_COMP_PMD_GEN1_H_
>+#define _QAT_COMP_PMD_GEN1_H_
>+

Maybe this should match the file name.

>+#include <rte_compressdev.h>
>+#include <rte_compressdev_pmd.h>
>+#include <stdint.h>
>+
>+#include "qat_comp_pmd.h"
>+
>+extern const struct rte_compressdev_capabilities
>+qat_gen1_comp_capabilities[];
>+
>+struct qat_comp_capabilities_info
>+qat_comp_cap_get_gen1(struct qat_pci_device *qat_dev);
>+
>+uint16_t qat_comp_get_ram_bank_flags_gen1(void);
>+
>+int qat_comp_set_slice_cfg_word_gen1(struct qat_comp_xform
>*qat_xform,
>+              const struct rte_comp_xform *xform,
>+              enum rte_comp_op_type op_type,
>+              uint32_t *comp_slice_cfg_word);
>+
>+uint64_t qat_comp_get_features_gen1(void);
>+
>+extern struct rte_compressdev_ops qat_comp_ops_gen1;
>+
>+#endif /* _QAT_COMP_PMD_GEN1_H_ */
>--
>2.25.1

Asides from that small comment,
Acked-by: Ciara Power <ciara.po...@intel.com>

Reply via email to