[dpdk-dev] [PATCH v5 15/51] net/bnxt: add HCAPI interface support

[Ferruh Yigit]

On 7/3/2020 10:01 PM, Ajit Khaparde wrote:
> From: Peter Spreadborough <peter.spreadborough at broadcom.com>
> 
> Add new hardware shim APIs to support multiple
> device generations
> 
> Signed-off-by: Peter Spreadborough <peter.spreadborough at broadcom.com>
> Signed-off-by: Venkat Duvvuru <venkatkumar.duvvuru at broadcom.com>
> Reviewed-by: Randy Schacher <stuart.schacher at broadcom.com>
> Reviewed-by: Ajit Khaparde <ajit.khaparde at broadcom.com>
> ---
>  drivers/net/bnxt/hcapi/Makefile           |  10 +
>  drivers/net/bnxt/hcapi/hcapi_cfa.h        | 271 +++++++++
>  drivers/net/bnxt/hcapi/hcapi_cfa_common.c |  92 +++
>  drivers/net/bnxt/hcapi/hcapi_cfa_defs.h   | 672 ++++++++++++++++++++++
>  drivers/net/bnxt/hcapi/hcapi_cfa_p4.c     | 399 +++++++++++++
>  drivers/net/bnxt/hcapi/hcapi_cfa_p4.h     | 451 +++++++++++++++
>  drivers/net/bnxt/meson.build              |   2 +
>  drivers/net/bnxt/tf_core/tf_em.c          |  28 +-
>  drivers/net/bnxt/tf_core/tf_tbl.c         |  94 +--
>  drivers/net/bnxt/tf_core/tf_tbl.h         |  24 +-
>  10 files changed, 1970 insertions(+), 73 deletions(-)
>  create mode 100644 drivers/net/bnxt/hcapi/Makefile
>  create mode 100644 drivers/net/bnxt/hcapi/hcapi_cfa.h
>  create mode 100644 drivers/net/bnxt/hcapi/hcapi_cfa_common.c
>  create mode 100644 drivers/net/bnxt/hcapi/hcapi_cfa_defs.h
>  create mode 100644 drivers/net/bnxt/hcapi/hcapi_cfa_p4.c
>  create mode 100644 drivers/net/bnxt/hcapi/hcapi_cfa_p4.h
> 
> diff --git a/drivers/net/bnxt/hcapi/Makefile b/drivers/net/bnxt/hcapi/Makefile
> new file mode 100644
> index 000000000..65cddd789
> --- /dev/null
> +++ b/drivers/net/bnxt/hcapi/Makefile
> @@ -0,0 +1,10 @@
> +# SPDX-License-Identifier: BSD-3-Clause
> +# Copyright(c) 2019-2020 Broadcom Limited.
> +# All rights reserved.
> +
> +SRCS-$(CONFIG_RTE_LIBRTE_BNXT_PMD) += hcapi/hcapi_cfa_p4.c
> +
> +SYMLINK-$(CONFIG_RTE_LIBRTE_BNXT_PMD)-include += hcapi/hcapi_cfa.h
> +SYMLINK-$(CONFIG_RTE_LIBRTE_BNXT_PMD)-include += hcapi/hcapi_cfa_defs.h
> +SYMLINK-$(CONFIG_RTE_LIBRTE_BNXT_PMD)-include += hcapi/hcapi_cfa_p4.h
> +SYMLINK-$(CONFIG_RTE_LIBRTE_BNXT_PMD)-include += hcapi/cfa_p40_hw.h

Why PMD header files added to install?
They should not accessed by the application, so they should not installed.

I will try to remove them.